i2c: designware: Consolidate default functionality bits
authorAlexander Stein <alexander.stein@systec-electronic.com>
Mon, 21 Nov 2016 10:43:20 +0000 (11:43 +0100)
committerWolfram Sang <wsa@the-dreams.de>
Tue, 29 Nov 2016 19:19:56 +0000 (20:19 +0100)
Use a common place for default functionality bits for both platform
and pci driver.

Signed-off-by: Alexander Stein <alexander.stein@systec-electronic.com>
Acked-by: Jarkko Nikula <jarkko.nikula@linux.intel.com>
Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
drivers/i2c/busses/i2c-designware-core.h
drivers/i2c/busses/i2c-designware-pcidrv.c
drivers/i2c/busses/i2c-designware-platdrv.c

index 0d44d2ae7d4c2ea9d091f446bccc76efa98f926d..26250b425e2f1046fa0b0483f3e59fb03ed33f75 100644 (file)
  *
  */
 
+#include <linux/i2c.h>
+
+#define DW_IC_DEFAULT_FUNCTIONALITY (I2C_FUNC_I2C |                    \
+                                       I2C_FUNC_SMBUS_BYTE |           \
+                                       I2C_FUNC_SMBUS_BYTE_DATA |      \
+                                       I2C_FUNC_SMBUS_WORD_DATA |      \
+                                       I2C_FUNC_SMBUS_BLOCK_DATA |     \
+                                       I2C_FUNC_SMBUS_I2C_BLOCK)
 
 #define DW_IC_CON_MASTER               0x1
 #define DW_IC_CON_SPEED_STD            0x2
index 8ffe2da6ca2f36aee8cb5d98313f0cf503fc05f1..300802e75d2e3519bd86f76487cb8f48c5d4047f 100644 (file)
@@ -71,13 +71,6 @@ struct dw_pci_controller {
                                DW_IC_CON_SLAVE_DISABLE |       \
                                DW_IC_CON_RESTART_EN)
 
-#define DW_DEFAULT_FUNCTIONALITY (I2C_FUNC_I2C |                       \
-                                       I2C_FUNC_SMBUS_BYTE |           \
-                                       I2C_FUNC_SMBUS_BYTE_DATA |      \
-                                       I2C_FUNC_SMBUS_WORD_DATA |      \
-                                       I2C_FUNC_SMBUS_BLOCK_DATA |     \
-                                       I2C_FUNC_SMBUS_I2C_BLOCK)
-
 /* Merrifield HCNT/LCNT/SDA hold time */
 static struct dw_scl_sda_cfg mrfld_config = {
        .ss_hcnt = 0x2f8,
@@ -250,7 +243,7 @@ static int i2c_dw_pci_probe(struct pci_dev *pdev,
        }
 
        dev->functionality = controller->functionality |
-                               DW_DEFAULT_FUNCTIONALITY;
+                               DW_IC_DEFAULT_FUNCTIONALITY;
 
        dev->master_cfg = controller->bus_cfg;
        if (controller->scl_sda_cfg) {
index 7429dfa39440e2f10c387e42b9524d34504d330b..08153ea4d848097ef5659f40c6bb71964cbc99e2 100644 (file)
@@ -219,14 +219,7 @@ static int dw_i2c_plat_probe(struct platform_device *pdev)
        if (r)
                return r;
 
-       dev->functionality =
-               I2C_FUNC_I2C |
-               I2C_FUNC_10BIT_ADDR |
-               I2C_FUNC_SMBUS_BYTE |
-               I2C_FUNC_SMBUS_BYTE_DATA |
-               I2C_FUNC_SMBUS_WORD_DATA |
-               I2C_FUNC_SMBUS_BLOCK_DATA |
-               I2C_FUNC_SMBUS_I2C_BLOCK;
+       dev->functionality = I2C_FUNC_10BIT_ADDR | DW_IC_DEFAULT_FUNCTIONALITY;
 
        dev->master_cfg = DW_IC_CON_MASTER | DW_IC_CON_SLAVE_DISABLE |
                          DW_IC_CON_RESTART_EN;