Message ID | 20211227094526.698714-3-u.kleine-koenig@pengutronix.de (mailing list archive) |
---|---|
State | Handled Elsewhere |
Headers | show |
Series | [v2,01/23] counter: Use container_of instead of drvdata to track counter_device | expand |
On Mon, 27 Dec 2021 10:45:05 +0100 Uwe Kleine-König <u.kleine-koenig@pengutronix.de> wrote: > The driver doesn't ever use platform_get_drvdata, so drop this unused > call. > > Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> Reviewed-by: Jonathan Cameron <Jonathan.Cameron@huawei.com> > --- > drivers/counter/ftm-quaddec.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/drivers/counter/ftm-quaddec.c b/drivers/counter/ftm-quaddec.c > index 5ef0478709cd..9272f7b58beb 100644 > --- a/drivers/counter/ftm-quaddec.c > +++ b/drivers/counter/ftm-quaddec.c > @@ -269,8 +269,6 @@ static int ftm_quaddec_probe(struct platform_device *pdev) > if (!ftm) > return -ENOMEM; > > - platform_set_drvdata(pdev, ftm); > - > io = platform_get_resource(pdev, IORESOURCE_MEM, 0); > if (!io) { > dev_err(&pdev->dev, "Failed to get memory region\n");
On Mon, Dec 27, 2021 at 10:45:05AM +0100, Uwe Kleine-König wrote: > The driver doesn't ever use platform_get_drvdata, so drop this unused > call. > > Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> Acked-by: William Breathitt Gray <vilhelm.gray@gmail.com> By the way, it should be possible to remove pdev from struct ftm_quaddec because &ftm->pdev->dev is stored to ftm->counter.parent and can be accessed from there if needed. > --- > drivers/counter/ftm-quaddec.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/drivers/counter/ftm-quaddec.c b/drivers/counter/ftm-quaddec.c > index 5ef0478709cd..9272f7b58beb 100644 > --- a/drivers/counter/ftm-quaddec.c > +++ b/drivers/counter/ftm-quaddec.c > @@ -269,8 +269,6 @@ static int ftm_quaddec_probe(struct platform_device *pdev) > if (!ftm) > return -ENOMEM; > > - platform_set_drvdata(pdev, ftm); > - > io = platform_get_resource(pdev, IORESOURCE_MEM, 0); > if (!io) { > dev_err(&pdev->dev, "Failed to get memory region\n"); > -- > 2.33.0 >
diff --git a/drivers/counter/ftm-quaddec.c b/drivers/counter/ftm-quaddec.c index 5ef0478709cd..9272f7b58beb 100644 --- a/drivers/counter/ftm-quaddec.c +++ b/drivers/counter/ftm-quaddec.c @@ -269,8 +269,6 @@ static int ftm_quaddec_probe(struct platform_device *pdev) if (!ftm) return -ENOMEM; - platform_set_drvdata(pdev, ftm); - io = platform_get_resource(pdev, IORESOURCE_MEM, 0); if (!io) { dev_err(&pdev->dev, "Failed to get memory region\n");
The driver doesn't ever use platform_get_drvdata, so drop this unused call. Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> --- drivers/counter/ftm-quaddec.c | 2 -- 1 file changed, 2 deletions(-)