| Message ID | 20170413001457.3918-2-javier@osg.samsung.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
diff --git a/drivers/hwmon/lm87.c b/drivers/hwmon/lm87.c index c0766e7392d3..85290c9aa46f 100644 --- a/drivers/hwmon/lm87.c +++ b/drivers/hwmon/lm87.c @@ -985,10 +985,18 @@ static const struct i2c_device_id lm87_id[] = { }; MODULE_DEVICE_TABLE(i2c, lm87_id); +static const struct of_device_id lm87_of_match[] = { + { .compatible = "adi,adm1024" }, + { .compatible = "ti,lm87" }, + { }, +}; +MODULE_DEVICE_TABLE(of, lm87_of_match); + static struct i2c_driver lm87_driver = { .class = I2C_CLASS_HWMON, .driver = { .name = "lm87", + .of_match_table = lm87_of_match, }, .probe = lm87_probe, .id_table = lm87_id,
The driver doesn't have a struct of_device_id table but supported devices are registered via Device Trees. This is working on the assumption that a I2C device registered via OF will always match a legacy I2C device ID and that the MODALIAS reported will always be of the form i2c:<device>. But this could change in the future so the correct approach is to have an OF device ID table if the devices are registered via OF. Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com> --- Hello, I should had included this patch in series [0] but it seems that I didn't. [0]: https://lkml.org/lkml/2017/2/24/360 Best regards, Javier drivers/hwmon/lm87.c | 8 ++++++++ 1 file changed, 8 insertions(+)