* [PATCH] IIO: ADC: stm32-dfsdm: avoid unused-variable warning
@ 2018-01-11 10:34 Arnd Bergmann
2018-01-11 14:27 ` Robin Murphy
0 siblings, 1 reply; 4+ messages in thread
From: Arnd Bergmann @ 2018-01-11 10:34 UTC (permalink / raw)
To: Mark Brown
Cc: Arnd Bergmann, Jonathan Cameron, Hartmut Knaack,
Lars-Peter Clausen, Peter Meerwald-Stadler, Maxime Coquelin,
Alexandre Torgue, Arnaud Pouliquen, linux-iio, linux-arm-kernel,
linux-kernel
Building with CONFIG_OF disabled produces a compiler warning:
drivers/iio/adc/stm32-dfsdm-core.c: In function 'stm32_dfsdm_probe':
drivers/iio/adc/stm32-dfsdm-core.c:245:22: error: unused variable 'pnode' [-Werror=unused-variable]
This removes the variable and open-codes it in the only place
it gets used to avoid that warning.
Fixes: bed73904e76f ("IIO: ADC: add stm32 DFSDM core support")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
drivers/iio/adc/stm32-dfsdm-core.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/iio/adc/stm32-dfsdm-core.c b/drivers/iio/adc/stm32-dfsdm-core.c
index 72427414db7f..6cd655f8239b 100644
--- a/drivers/iio/adc/stm32-dfsdm-core.c
+++ b/drivers/iio/adc/stm32-dfsdm-core.c
@@ -242,7 +242,6 @@ MODULE_DEVICE_TABLE(of, stm32_dfsdm_of_match);
static int stm32_dfsdm_probe(struct platform_device *pdev)
{
struct dfsdm_priv *priv;
- struct device_node *pnode = pdev->dev.of_node;
const struct of_device_id *of_id;
const struct stm32_dfsdm_dev_data *dev_data;
struct stm32_dfsdm *dfsdm;
@@ -254,7 +253,7 @@ static int stm32_dfsdm_probe(struct platform_device *pdev)
priv->pdev = pdev;
- of_id = of_match_node(stm32_dfsdm_of_match, pnode);
+ of_id = of_match_node(stm32_dfsdm_of_match, pdev->dev.of_node);
if (!of_id->data) {
dev_err(&pdev->dev, "Data associated to device is missing\n");
return -EINVAL;
--
2.9.0
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH] IIO: ADC: stm32-dfsdm: avoid unused-variable warning 2018-01-11 10:34 [PATCH] IIO: ADC: stm32-dfsdm: avoid unused-variable warning Arnd Bergmann @ 2018-01-11 14:27 ` Robin Murphy 2018-01-11 17:38 ` Arnaud Pouliquen 0 siblings, 1 reply; 4+ messages in thread From: Robin Murphy @ 2018-01-11 14:27 UTC (permalink / raw) To: Arnd Bergmann, Mark Brown Cc: Lars-Peter Clausen, Peter Meerwald-Stadler, linux-iio, Arnaud Pouliquen, linux-kernel, linux-arm-kernel, Maxime Coquelin, Hartmut Knaack, Jonathan Cameron, Alexandre Torgue On 11/01/18 10:34, Arnd Bergmann wrote: > Building with CONFIG_OF disabled produces a compiler warning: > > drivers/iio/adc/stm32-dfsdm-core.c: In function 'stm32_dfsdm_probe': > drivers/iio/adc/stm32-dfsdm-core.c:245:22: error: unused variable 'pnode' [-Werror=unused-variable] > > This removes the variable and open-codes it in the only place > it gets used to avoid that warning. > > Fixes: bed73904e76f ("IIO: ADC: add stm32 DFSDM core support") > Signed-off-by: Arnd Bergmann <arnd@arndb.de> > --- > drivers/iio/adc/stm32-dfsdm-core.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/iio/adc/stm32-dfsdm-core.c b/drivers/iio/adc/stm32-dfsdm-core.c > index 72427414db7f..6cd655f8239b 100644 > --- a/drivers/iio/adc/stm32-dfsdm-core.c > +++ b/drivers/iio/adc/stm32-dfsdm-core.c > @@ -242,7 +242,6 @@ MODULE_DEVICE_TABLE(of, stm32_dfsdm_of_match); > static int stm32_dfsdm_probe(struct platform_device *pdev) > { > struct dfsdm_priv *priv; > - struct device_node *pnode = pdev->dev.of_node; > const struct of_device_id *of_id; > const struct stm32_dfsdm_dev_data *dev_data; > struct stm32_dfsdm *dfsdm; > @@ -254,7 +253,7 @@ static int stm32_dfsdm_probe(struct platform_device *pdev) > > priv->pdev = pdev; > > - of_id = of_match_node(stm32_dfsdm_of_match, pnode); > + of_id = of_match_node(stm32_dfsdm_of_match, pdev->dev.of_node); > if (!of_id->data) { > dev_err(&pdev->dev, "Data associated to device is missing\n"); > return -EINVAL; FWIW, it looks like this whole lot could be cleaned up by using of_device_get_match_data(). Robin. ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] IIO: ADC: stm32-dfsdm: avoid unused-variable warning 2018-01-11 14:27 ` Robin Murphy @ 2018-01-11 17:38 ` Arnaud Pouliquen 2018-01-14 12:07 ` Jonathan Cameron 0 siblings, 1 reply; 4+ messages in thread From: Arnaud Pouliquen @ 2018-01-11 17:38 UTC (permalink / raw) To: Robin Murphy, Arnd Bergmann, Mark Brown Cc: Lars-Peter Clausen, Peter Meerwald-Stadler, linux-iio, linux-kernel, linux-arm-kernel, Maxime Coquelin, Hartmut Knaack, Jonathan Cameron, Alexandre Torgue On 01/11/2018 03:27 PM, Robin Murphy wrote: > On 11/01/18 10:34, Arnd Bergmann wrote: >> Building with CONFIG_OF disabled produces a compiler warning: >> >> drivers/iio/adc/stm32-dfsdm-core.c: In function 'stm32_dfsdm_probe': >> drivers/iio/adc/stm32-dfsdm-core.c:245:22: error: unused variable >> 'pnode' [-Werror=unused-variable] >> >> This removes the variable and open-codes it in the only place >> it gets used to avoid that warning. >> >> Fixes: bed73904e76f ("IIO: ADC: add stm32 DFSDM core support") >> Signed-off-by: Arnd Bergmann <arnd@arndb.de> >> --- >> drivers/iio/adc/stm32-dfsdm-core.c | 3 +-- >> 1 file changed, 1 insertion(+), 2 deletions(-) >> >> diff --git a/drivers/iio/adc/stm32-dfsdm-core.c >> b/drivers/iio/adc/stm32-dfsdm-core.c >> index 72427414db7f..6cd655f8239b 100644 >> --- a/drivers/iio/adc/stm32-dfsdm-core.c >> +++ b/drivers/iio/adc/stm32-dfsdm-core.c >> @@ -242,7 +242,6 @@ MODULE_DEVICE_TABLE(of, stm32_dfsdm_of_match); >> static int stm32_dfsdm_probe(struct platform_device *pdev) >> { >> struct dfsdm_priv *priv; >> - struct device_node *pnode = pdev->dev.of_node; >> const struct of_device_id *of_id; >> const struct stm32_dfsdm_dev_data *dev_data; >> struct stm32_dfsdm *dfsdm; >> @@ -254,7 +253,7 @@ static int stm32_dfsdm_probe(struct >> platform_device *pdev) >> priv->pdev = pdev; >> - of_id = of_match_node(stm32_dfsdm_of_match, pnode); >> + of_id = of_match_node(stm32_dfsdm_of_match, pdev->dev.of_node); >> if (!of_id->data) { >> dev_err(&pdev->dev, "Data associated to device is missing\n"); >> return -EINVAL; > > FWIW, it looks like this whole lot could be cleaned up by using > of_device_get_match_data(). > Right, and test of the return now seems to me an overprotection as data structure is defined in the driver... Same optimization could be applied to stm32_dfsdm_adc_probe function. Here is the patch I tested: --- drivers/iio/adc/stm32-dfsdm-adc.c | 9 +-------- drivers/iio/adc/stm32-dfsdm-core.c | 9 +-------- 2 files changed, 2 insertions(+), 16 deletions(-) diff --git a/drivers/iio/adc/stm32-dfsdm-adc.c b/drivers/iio/adc/stm32-dfsdm-adc.c index b03ca3f..01836c9 100644 --- a/drivers/iio/adc/stm32-dfsdm-adc.c +++ b/drivers/iio/adc/stm32-dfsdm-adc.c @@ -1087,18 +1087,11 @@ static int stm32_dfsdm_adc_probe(struct platform_device *pdev) struct device_node *np = dev->of_node; const struct stm32_dfsdm_dev_data *dev_data; struct iio_dev *iio; - const struct of_device_id *of_id; char *name; int ret, irq, val; - of_id = of_match_node(stm32_dfsdm_adc_match, np); - if (!of_id->data) { - dev_err(&pdev->dev, "Data associated to device is missing\n"); - return -EINVAL; - } - - dev_data = (const struct stm32_dfsdm_dev_data *)of_id->data; + dev_data = of_device_get_match_data(dev); iio = devm_iio_device_alloc(dev, sizeof(*adc)); if (IS_ERR(iio)) { dev_err(dev, "%s: Failed to allocate IIO\n", __func__); diff --git a/drivers/iio/adc/stm32-dfsdm-core.c b/drivers/iio/adc/stm32-dfsdm-core.c index 7242741..6290332 100644 --- a/drivers/iio/adc/stm32-dfsdm-core.c +++ b/drivers/iio/adc/stm32-dfsdm-core.c @@ -242,8 +242,6 @@ MODULE_DEVICE_TABLE(of, stm32_dfsdm_of_match); static int stm32_dfsdm_probe(struct platform_device *pdev) { struct dfsdm_priv *priv; - struct device_node *pnode = pdev->dev.of_node; - const struct of_device_id *of_id; const struct stm32_dfsdm_dev_data *dev_data; struct stm32_dfsdm *dfsdm; int ret; @@ -254,13 +252,8 @@ static int stm32_dfsdm_probe(struct platform_device *pdev) priv->pdev = pdev; - of_id = of_match_node(stm32_dfsdm_of_match, pnode); - if (!of_id->data) { - dev_err(&pdev->dev, "Data associated to device is missing\n"); - return -EINVAL; - } + dev_data = of_device_get_match_data(&pdev->dev); - dev_data = (const struct stm32_dfsdm_dev_data *)of_id->data; dfsdm = &priv->dfsdm; dfsdm->fl_list = devm_kcalloc(&pdev->dev, dev_data->num_filters, sizeof(*dfsdm->fl_list), GFP_KERNEL); -- Arnd, fell free to propose it (with my acked-by) or tell me if you prefer that i send it. Thanks, Arnaud ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] IIO: ADC: stm32-dfsdm: avoid unused-variable warning 2018-01-11 17:38 ` Arnaud Pouliquen @ 2018-01-14 12:07 ` Jonathan Cameron 0 siblings, 0 replies; 4+ messages in thread From: Jonathan Cameron @ 2018-01-14 12:07 UTC (permalink / raw) To: Arnaud Pouliquen Cc: Robin Murphy, Arnd Bergmann, Mark Brown, Lars-Peter Clausen, Peter Meerwald-Stadler, linux-iio, linux-kernel, linux-arm-kernel, Maxime Coquelin, Hartmut Knaack, Alexandre Torgue On Thu, 11 Jan 2018 18:38:49 +0100 Arnaud Pouliquen <arnaud.pouliquen@st.com> wrote: > On 01/11/2018 03:27 PM, Robin Murphy wrote: > > On 11/01/18 10:34, Arnd Bergmann wrote: > >> Building with CONFIG_OF disabled produces a compiler warning: > >> > >> drivers/iio/adc/stm32-dfsdm-core.c: In function 'stm32_dfsdm_probe': > >> drivers/iio/adc/stm32-dfsdm-core.c:245:22: error: unused variable > >> 'pnode' [-Werror=unused-variable] > >> > >> This removes the variable and open-codes it in the only place > >> it gets used to avoid that warning. > >> > >> Fixes: bed73904e76f ("IIO: ADC: add stm32 DFSDM core support") > >> Signed-off-by: Arnd Bergmann <arnd@arndb.de> > >> --- > >> drivers/iio/adc/stm32-dfsdm-core.c | 3 +-- > >> 1 file changed, 1 insertion(+), 2 deletions(-) > >> > >> diff --git a/drivers/iio/adc/stm32-dfsdm-core.c > >> b/drivers/iio/adc/stm32-dfsdm-core.c > >> index 72427414db7f..6cd655f8239b 100644 > >> --- a/drivers/iio/adc/stm32-dfsdm-core.c > >> +++ b/drivers/iio/adc/stm32-dfsdm-core.c > >> @@ -242,7 +242,6 @@ MODULE_DEVICE_TABLE(of, stm32_dfsdm_of_match); > >> static int stm32_dfsdm_probe(struct platform_device *pdev) > >> { > >> struct dfsdm_priv *priv; > >> - struct device_node *pnode = pdev->dev.of_node; > >> const struct of_device_id *of_id; > >> const struct stm32_dfsdm_dev_data *dev_data; > >> struct stm32_dfsdm *dfsdm; > >> @@ -254,7 +253,7 @@ static int stm32_dfsdm_probe(struct > >> platform_device *pdev) > >> priv->pdev = pdev; > >> - of_id = of_match_node(stm32_dfsdm_of_match, pnode); > >> + of_id = of_match_node(stm32_dfsdm_of_match, pdev->dev.of_node); > >> if (!of_id->data) { > >> dev_err(&pdev->dev, "Data associated to device is missing\n"); > >> return -EINVAL; > > > > FWIW, it looks like this whole lot could be cleaned up by using > > of_device_get_match_data(). > > > Right, and test of the return now seems to me an overprotection as data > structure is defined in the driver... > > Same optimization could be applied to stm32_dfsdm_adc_probe function. > > Here is the patch I tested: This looks like a good solution to me. Acked-by: Jonathan Cameron <Jonathan.Cameron@huawei.com> (given this should probably go on Mark's immutable branch). Jonathan > > --- > drivers/iio/adc/stm32-dfsdm-adc.c | 9 +-------- > drivers/iio/adc/stm32-dfsdm-core.c | 9 +-------- > 2 files changed, 2 insertions(+), 16 deletions(-) > > diff --git a/drivers/iio/adc/stm32-dfsdm-adc.c > b/drivers/iio/adc/stm32-dfsdm-adc.c > index b03ca3f..01836c9 100644 > --- a/drivers/iio/adc/stm32-dfsdm-adc.c > +++ b/drivers/iio/adc/stm32-dfsdm-adc.c > @@ -1087,18 +1087,11 @@ static int stm32_dfsdm_adc_probe(struct > platform_device *pdev) > struct device_node *np = dev->of_node; > const struct stm32_dfsdm_dev_data *dev_data; > struct iio_dev *iio; > - const struct of_device_id *of_id; > char *name; > int ret, irq, val; > > - of_id = of_match_node(stm32_dfsdm_adc_match, np); > - if (!of_id->data) { > - dev_err(&pdev->dev, "Data associated to device is missing\n"); > - return -EINVAL; > - } > - > - dev_data = (const struct stm32_dfsdm_dev_data *)of_id->data; > > + dev_data = of_device_get_match_data(dev); > iio = devm_iio_device_alloc(dev, sizeof(*adc)); > if (IS_ERR(iio)) { > dev_err(dev, "%s: Failed to allocate IIO\n", __func__); > diff --git a/drivers/iio/adc/stm32-dfsdm-core.c > b/drivers/iio/adc/stm32-dfsdm-core.c > index 7242741..6290332 100644 > --- a/drivers/iio/adc/stm32-dfsdm-core.c > +++ b/drivers/iio/adc/stm32-dfsdm-core.c > @@ -242,8 +242,6 @@ MODULE_DEVICE_TABLE(of, stm32_dfsdm_of_match); > static int stm32_dfsdm_probe(struct platform_device *pdev) > { > struct dfsdm_priv *priv; > - struct device_node *pnode = pdev->dev.of_node; > - const struct of_device_id *of_id; > const struct stm32_dfsdm_dev_data *dev_data; > struct stm32_dfsdm *dfsdm; > int ret; > @@ -254,13 +252,8 @@ static int stm32_dfsdm_probe(struct platform_device > *pdev) > > priv->pdev = pdev; > > - of_id = of_match_node(stm32_dfsdm_of_match, pnode); > - if (!of_id->data) { > - dev_err(&pdev->dev, "Data associated to device is missing\n"); > - return -EINVAL; > - } > + dev_data = of_device_get_match_data(&pdev->dev); > > - dev_data = (const struct stm32_dfsdm_dev_data *)of_id->data; > dfsdm = &priv->dfsdm; > dfsdm->fl_list = devm_kcalloc(&pdev->dev, dev_data->num_filters, > sizeof(*dfsdm->fl_list), GFP_KERNEL); ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-01-14 12:07 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2018-01-11 10:34 [PATCH] IIO: ADC: stm32-dfsdm: avoid unused-variable warning Arnd Bergmann 2018-01-11 14:27 ` Robin Murphy 2018-01-11 17:38 ` Arnaud Pouliquen 2018-01-14 12:07 ` Jonathan Cameron
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).