lkml.org 
[lkml]   [2017]   [Dec]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] iio: accel: bmc150: Add OF device ID table
From
Date
Hi,

On 01-12-17 12:10, Javier Martinez Canillas wrote:
> 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.
>
> The I2C device ID table entries have the .driver_data field set, but they
> are not used in the driver so weren't set in the OF device table entries.
>
> Signed-off-by: Javier Martinez Canillas <javierm@redhat.com>
> ---
>
> drivers/iio/accel/bmc150-accel-i2c.c | 12 ++++++++++++
> 1 file changed, 12 insertions(+)
>
> diff --git a/drivers/iio/accel/bmc150-accel-i2c.c b/drivers/iio/accel/bmc150-accel-i2c.c
> index f85014fbaa12..8ffc308d5fd0 100644
> --- a/drivers/iio/accel/bmc150-accel-i2c.c
> +++ b/drivers/iio/accel/bmc150-accel-i2c.c
> @@ -81,9 +81,21 @@ static const struct i2c_device_id bmc150_accel_id[] = {
>
> MODULE_DEVICE_TABLE(i2c, bmc150_accel_id);
>
> +static const struct of_device_id bmc150_accel_of_match[] = {
> + { .compatible = "bosch,bmc150_accel" },
> + { .compatible = "bosch,bmi055_accel" },

These look a bit weird, there is no reason to mirror the i2c_device_ids
here and typically for devicetree / of we only list
the chip model without some postfix like _accel.

Also if you're doing this you should probably add a:
Documentation/devicetree/bindings/iio/accel/bmc150.txt
file documenting the compatible strings, and Cc:
devicetree@vger.kernel.org for the next version,
so that the devicetree maintainers get a chance to review
this.

> + { .compatible = "bosch,bma255" },
> + { .compatible = "bosch,bma250e" },
> + { .compatible = "bosch,bma222e" },
> + { .compatible = "bosch,bma280" },
> + { },
> +};
> +MODULE_DEVICE_TABLE(of, bmc150_accel_of_match);
> +
> static struct i2c_driver bmc150_accel_driver = {
> .driver = {
> .name = "bmc150_accel_i2c",
> + .of_match_table = bmc150_accel_of_match,
> .acpi_match_table = ACPI_PTR(bmc150_accel_acpi_match),
> .pm = &bmc150_accel_pm_ops,
> },
>

Otherwise looks good to me,

Regards,

Hans

\
 
 \ /
  Last update: 2017-12-04 09:30    [W:0.060 / U:1.732 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site