mfd: intel_quark_i2c_gpio: Use dmi_system_id table for retrieving frequency
authorJan Kiszka <jan.kiszka@siemens.com>
Tue, 30 May 2017 08:17:40 +0000 (10:17 +0200)
committerLee Jones <lee.jones@linaro.org>
Thu, 6 Jul 2017 07:29:12 +0000 (08:29 +0100)
Avoids reimplementation of DMI matching in intel_quark_i2c_setup.

Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
drivers/mfd/intel_quark_i2c_gpio.c

index 7946d6e..0500613 100644 (file)
@@ -58,19 +58,18 @@ struct intel_quark_mfd {
        struct clk_lookup       *i2c_clk_lookup;
 };
 
-struct i2c_mode_info {
-       const char *name;
-       unsigned int i2c_scl_freq;
-};
-
-static const struct i2c_mode_info platform_i2c_mode_info[] = {
+static const struct dmi_system_id dmi_platform_info[] = {
        {
-               .name = "Galileo",
-               .i2c_scl_freq = 100000,
+               .matches = {
+                       DMI_EXACT_MATCH(DMI_BOARD_NAME, "Galileo"),
+               },
+               .driver_data = (void *)100000,
        },
        {
-               .name = "GalileoGen2",
-               .i2c_scl_freq = 400000,
+               .matches = {
+                       DMI_EXACT_MATCH(DMI_BOARD_NAME, "GalileoGen2"),
+               },
+               .driver_data = (void *)400000,
        },
        {}
 };
@@ -160,8 +159,7 @@ static void intel_quark_unregister_i2c_clk(struct device *dev)
 
 static int intel_quark_i2c_setup(struct pci_dev *pdev, struct mfd_cell *cell)
 {
-       const char *board_name = dmi_get_system_info(DMI_BOARD_NAME);
-       const struct i2c_mode_info *info;
+       const struct dmi_system_id *dmi_id;
        struct dw_i2c_platform_data *pdata;
        struct resource *res = (struct resource *)cell->resources;
        struct device *dev = &pdev->dev;
@@ -181,14 +179,9 @@ static int intel_quark_i2c_setup(struct pci_dev *pdev, struct mfd_cell *cell)
        /* Normal mode by default */
        pdata->i2c_scl_freq = 100000;
 
-       if (board_name) {
-               for (info = platform_i2c_mode_info; info->name; info++) {
-                       if (!strcmp(board_name, info->name)) {
-                               pdata->i2c_scl_freq = info->i2c_scl_freq;
-                               break;
-                       }
-               }
-       }
+       dmi_id = dmi_first_match(dmi_platform_info);
+       if (dmi_id)
+               pdata->i2c_scl_freq = (uintptr_t)dmi_id->driver_data;
 
        cell->platform_data = pdata;
        cell->pdata_size = sizeof(*pdata);