Message ID | 20210528151113.85943-2-andriy.shevchenko@linux.intel.com (mailing list archive) |
---|---|
State | Accepted |
Commit | 08729298c4eea5a9bbc0598d92ff47638a97a1ae |
Headers | show |
Series | [v2,1/2] Input: resistive-adc-touch - describe parameters in kernel doc | expand |
On Fri, May 28, 2021 at 06:11:13PM +0300, Andy Shevchenko wrote: > The driver is actually OF independent and doesn't need anything from OF (*). > Replace OF headers with mod_devicetable.h and property.h. > > *) use of of_match_ptr() is actually wrong here, since it may provoke > a compiler to warn about unused variable. > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> Applied, thank you.
diff --git a/drivers/input/touchscreen/resistive-adc-touch.c b/drivers/input/touchscreen/resistive-adc-touch.c index 0353400c559a..ea1884fb49a1 100644 --- a/drivers/input/touchscreen/resistive-adc-touch.c +++ b/drivers/input/touchscreen/resistive-adc-touch.c @@ -13,10 +13,10 @@ #include <linux/input/touchscreen.h> #include <linux/iio/consumer.h> #include <linux/iio/iio.h> +#include <linux/mod_devicetable.h> #include <linux/module.h> -#include <linux/of.h> -#include <linux/of_device.h> #include <linux/platform_device.h> +#include <linux/property.h> #define DRIVER_NAME "resistive-adc-touch" #define GRTS_DEFAULT_PRESSURE_MIN 50000 @@ -307,7 +307,7 @@ static struct platform_driver grts_driver = { .probe = grts_probe, .driver = { .name = DRIVER_NAME, - .of_match_table = of_match_ptr(grts_of_match), + .of_match_table = grts_of_match, }, };
The driver is actually OF independent and doesn't need anything from OF (*). Replace OF headers with mod_devicetable.h and property.h. *) use of of_match_ptr() is actually wrong here, since it may provoke a compiler to warn about unused variable. Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> --- drivers/input/touchscreen/resistive-adc-touch.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)