On Sun, Jun 28, 2020 at 3:39 PM Jonathan Cameron <jic23@xxxxxxxxxx> wrote:
>
> From: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx>
>
> Whilst is unlikely anyone will be using this part with an ACPI PRP0001
> based binding any time soon, we are getting a lot of cut and pasting
> of this pattern so I am looking to remove it entirely from IIO.
>
> In this case CONFIG_OF protections also removed and
> mod_devicetable.h include added given direct use of struct of_device_id.
>
Yes, it also fixes indirect inclusion of of.h.
LGTM!
> Signed-off-by: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx>
> ---
> drivers/iio/adc/lpc32xx_adc.c | 5 ++---
> 1 file changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/iio/adc/lpc32xx_adc.c b/drivers/iio/adc/lpc32xx_adc.c
> index bc2712a48844..b56ce15255cf 100644
> --- a/drivers/iio/adc/lpc32xx_adc.c
> +++ b/drivers/iio/adc/lpc32xx_adc.c
> @@ -14,6 +14,7 @@
> #include <linux/interrupt.h>
> #include <linux/io.h>
> #include <linux/module.h>
> +#include <linux/mod_devicetable.h>
> #include <linux/platform_device.h>
> #include <linux/regulator/consumer.h>
>
> @@ -209,19 +210,17 @@ static int lpc32xx_adc_probe(struct platform_device *pdev)
> return 0;
> }
>
> -#ifdef CONFIG_OF
> static const struct of_device_id lpc32xx_adc_match[] = {
> { .compatible = "nxp,lpc3220-adc" },
> {},
> };
> MODULE_DEVICE_TABLE(of, lpc32xx_adc_match);
> -#endif
>
> static struct platform_driver lpc32xx_adc_driver = {
> .probe = lpc32xx_adc_probe,
> .driver = {
> .name = LPC32XXAD_NAME,
> - .of_match_table = of_match_ptr(lpc32xx_adc_match),
> + .of_match_table = lpc32xx_adc_match,
> },
> };
>
> --
> 2.27.0
>
--
With Best Regards,
Andy Shevchenko
