Re: [PATCH v3 2/4] iio: gyro: add device tree support for fxas21002c

From: Andy Shevchenko
Date: Tue Sep 04 2018 - 15:13:42 EST


On Tue, Sep 4, 2018 at 10:06 PM Afonso Bordado <afonsobordado@xxxxxx> wrote:
>
> This patch adds device tree support for the fxas21002c driver, including
> bindings.
>
> Signed-off-by: Afonso Bordado <afonsobordado@xxxxxx>

> +#ifdef CONFIG_OF

> +#endif

Please, remove this ifdef...

> + .of_match_table = of_match_ptr(fxas21002c_of_ids),

...and do not use of_match_ptr() macro.

This prevents to use device on embedded ACPI enabled devices,

> },
> .probe = fxas21002c_probe,
> .id_table = fxas21002c_id,
> --
> 2.18.0
>
>


--
With Best Regards,
Andy Shevchenko