iio:adc:bcm_iproc: Drop of_match_ptr protection and switch to mod_devicetable.h
authorJonathan Cameron <Jonathan.Cameron@huawei.com>
Tue, 21 Jul 2020 17:14:44 +0000 (18:14 +0100)
committerJonathan Cameron <Jonathan.Cameron@huawei.com>
Thu, 3 Sep 2020 18:40:45 +0000 (19:40 +0100)
This driver cannot be instantiated from ACPI due to it's use of
syscon_regmap_lookup_by_phandle() but in the interests of clearing
this anti pattern out of IIO, let us switch to an explicit
check in Kconfig and remove the protections on the of_match_table
The switch of header is because we only use of_device_id
in here and that is defined in mod_devicetable.h not of.h.

Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Cc: Raveendra Padasalagi <raveendra.padasalagi@broadcom.com>
Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
drivers/iio/adc/Kconfig
drivers/iio/adc/bcm_iproc_adc.c

index 66d9cc0..f495d01 100644 (file)
@@ -340,7 +340,7 @@ config AXP288_ADC
 
 config BCM_IPROC_ADC
        tristate "Broadcom IPROC ADC driver"
-       depends on ARCH_BCM_IPROC || COMPILE_TEST
+       depends on (ARCH_BCM_IPROC && OF) || COMPILE_TEST
        depends on MFD_SYSCON
        default ARCH_BCM_CYGNUS
        help
index 936da32..44e1e53 100644 (file)
@@ -4,7 +4,7 @@
  */
 
 #include <linux/module.h>
-#include <linux/of.h>
+#include <linux/mod_devicetable.h>
 #include <linux/io.h>
 #include <linux/clk.h>
 #include <linux/mfd/syscon.h>
@@ -617,7 +617,7 @@ static struct platform_driver iproc_adc_driver = {
        .remove = iproc_adc_remove,
        .driver = {
                .name   = "iproc-static-adc",
-               .of_match_table = of_match_ptr(iproc_adc_of_match),
+               .of_match_table = iproc_adc_of_match,
        },
 };
 module_platform_driver(iproc_adc_driver);