From 046dab28b10e6e1766685653d9a170e4c9b1e5fc Mon Sep 17 00:00:00 2001 From: Krzysztof Kozlowski Date: Sun, 1 May 2022 12:34:46 +0200 Subject: [PATCH] iio: stmpe-adc: use of_device_id for OF matching MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The of_device_id was added to allow module autoloading, but it should be also used to allow driver matching via Devicetree. This also fixes W=1 warning: drivers/iio/adc/stmpe-adc.c:357:34: error: ‘stmpe_adc_ids’ defined but not used [-Werror=unused-const-variable=] Signed-off-by: Krzysztof Kozlowski Link: https://lore.kernel.org/r/20220501103447.111392-1-krzysztof.kozlowski@linaro.org Signed-off-by: Jonathan Cameron --- drivers/iio/adc/stmpe-adc.c | 19 ++++++++++--------- 1 file changed, 10 insertions(+), 9 deletions(-) diff --git a/drivers/iio/adc/stmpe-adc.c b/drivers/iio/adc/stmpe-adc.c index 83e0ac4467ca..000e5cfecb43 100644 --- a/drivers/iio/adc/stmpe-adc.c +++ b/drivers/iio/adc/stmpe-adc.c @@ -345,21 +345,22 @@ static int __maybe_unused stmpe_adc_resume(struct device *dev) static SIMPLE_DEV_PM_OPS(stmpe_adc_pm_ops, NULL, stmpe_adc_resume); -static struct platform_driver stmpe_adc_driver = { - .probe = stmpe_adc_probe, - .driver = { - .name = "stmpe-adc", - .pm = &stmpe_adc_pm_ops, - }, -}; -module_platform_driver(stmpe_adc_driver); - static const struct of_device_id stmpe_adc_ids[] = { { .compatible = "st,stmpe-adc", }, { }, }; MODULE_DEVICE_TABLE(of, stmpe_adc_ids); +static struct platform_driver stmpe_adc_driver = { + .probe = stmpe_adc_probe, + .driver = { + .name = "stmpe-adc", + .pm = &stmpe_adc_pm_ops, + .of_match_table = stmpe_adc_ids, + }, +}; +module_platform_driver(stmpe_adc_driver); + MODULE_AUTHOR("Stefan Agner "); MODULE_DESCRIPTION("STMPEXXX ADC driver"); MODULE_LICENSE("GPL v2");