Currently the I2C framework insists on devices supplying an I2C ID
table.  Many of the devices which do so unnecessarily adding quite a
few wasted lines to kernel code.  This patch allows drivers a means
to 'not' supply the aforementioned table and match on DT match tables
instead.

Acked-by: Grant Likely <grant.lik...@linaro.org>
Signed-off-by: Lee Jones <lee.jo...@linaro.org>
---
 drivers/i2c/i2c-core.c | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/drivers/i2c/i2c-core.c b/drivers/i2c/i2c-core.c
index 5728b2e..c6013cd 100644
--- a/drivers/i2c/i2c-core.c
+++ b/drivers/i2c/i2c-core.c
@@ -101,7 +101,7 @@ static int i2c_device_match(struct device *dev, struct 
device_driver *drv)
 
 
        /* Attempt an OF style match */
-       if (of_driver_match_device(dev, drv))
+       if (i2c_of_match_device(drv->of_match_table, client))
                return 1;
 
        /* Then ACPI style match */
@@ -269,7 +269,15 @@ static int i2c_device_probe(struct device *dev)
                return 0;
 
        driver = to_i2c_driver(dev->driver);
-       if (!driver->probe || !driver->id_table)
+       if (!driver->probe)
+               return -EINVAL;
+
+       /*
+        * An I2C ID table is not mandatory, if and only if, a suitable Device
+        * Tree match table entry is supplied for the probing device.
+        */
+       if (!driver->id_table &&
+           !i2c_of_match_device(dev->driver->of_match_table, client))
                return -ENODEV;
 
        if (!device_can_wakeup(&client->dev))
-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to