| Message ID | 20200617133523.58158-1-darius.berghe@analog.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [v1,1/3] iio:adc:ltc2471: add match table for existing devices | expand |
On Wed, 17 Jun 2020 16:35:21 +0300 Darius Berghe <darius.berghe@analog.com> wrote: > OF style match table is the proper way of matching device tree nodes > with drivers and such table was missing, this commit adds it. > > Signed-off-by: Darius Berghe <darius.berghe@analog.com> Hi Darius A few minor things inline. Thanks, Jonathan > --- > drivers/iio/adc/ltc2471.c | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/drivers/iio/adc/ltc2471.c b/drivers/iio/adc/ltc2471.c > index 55fab612843a..0e5cdb43a943 100644 > --- a/drivers/iio/adc/ltc2471.c > +++ b/drivers/iio/adc/ltc2471.c > @@ -143,9 +143,17 @@ static const struct i2c_device_id ltc2471_i2c_id[] = { > }; > MODULE_DEVICE_TABLE(i2c, ltc2471_i2c_id); > > +static const struct of_device_id ltc2471_of_match[] = { Should include mod_device_table.h as we are using of_device_id which is defined in that header. > + { .compatible = "adi,ltc2471" }, > + { .compatible = "adi,ltc2473" }, > + {} > +}; > +MODULE_DEVICE_TABLE(of, ltc2471_of_match); > + > static struct i2c_driver ltc2471_i2c_driver = { > .driver = { > .name = "ltc2471", > + .of_match_table = of_match_ptr(ltc2471_of_match) Drop the of_match_ptr protection. That prevents the use of this binding with ACPI (via the magic of PRP0001 which allows you to use DT bindings in an ACPI DSDT). We are slowly working our way through all the drivers removing the use of this macro (and converting to generic binding handling where relevant). It'll take a while yet though before there are no instances of this to copy! > }, > .probe = ltc2471_i2c_probe, > .id_table = ltc2471_i2c_id,
diff --git a/drivers/iio/adc/ltc2471.c b/drivers/iio/adc/ltc2471.c index 55fab612843a..0e5cdb43a943 100644 --- a/drivers/iio/adc/ltc2471.c +++ b/drivers/iio/adc/ltc2471.c @@ -143,9 +143,17 @@ static const struct i2c_device_id ltc2471_i2c_id[] = { }; MODULE_DEVICE_TABLE(i2c, ltc2471_i2c_id); +static const struct of_device_id ltc2471_of_match[] = { + { .compatible = "adi,ltc2471" }, + { .compatible = "adi,ltc2473" }, + {} +}; +MODULE_DEVICE_TABLE(of, ltc2471_of_match); + static struct i2c_driver ltc2471_i2c_driver = { .driver = { .name = "ltc2471", + .of_match_table = of_match_ptr(ltc2471_of_match) }, .probe = ltc2471_i2c_probe, .id_table = ltc2471_i2c_id,
OF style match table is the proper way of matching device tree nodes with drivers and such table was missing, this commit adds it. Signed-off-by: Darius Berghe <darius.berghe@analog.com> --- drivers/iio/adc/ltc2471.c | 8 ++++++++ 1 file changed, 8 insertions(+)