Messages in this thread | | | From | Bartosz Golaszewski <> | Date | Thu, 3 Aug 2023 15:45:09 +0200 | Subject | Re: [PATCH] gpio: syscon: remove unneeded call to platform_set_drvdata() |
| |
On Tue, Aug 1, 2023 at 9:30 AM Andrei Coardos <aboutphysycs@gmail.com> wrote: > > This function call was found to be unnecessary as there is no equivalent > platform_get_drvdata() call to access the private data of the driver. Also, > the private data is defined in this driver, so there is no risk of it being > accessed outside of this driver file. > > Reviewed-by: Alexandru Ardelean <alex@shruggie.ro> > Signed-off-by: Andrei Coardos <aboutphysycs@gmail.com> > --- > drivers/gpio/gpio-syscon.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/drivers/gpio/gpio-syscon.c b/drivers/gpio/gpio-syscon.c > index 6076937b18e7..afceb7e17ad8 100644 > --- a/drivers/gpio/gpio-syscon.c > +++ b/drivers/gpio/gpio-syscon.c > @@ -249,8 +249,6 @@ static int syscon_gpio_probe(struct platform_device *pdev) > priv->chip.direction_output = syscon_gpio_dir_out; > } > > - platform_set_drvdata(pdev, priv); > - > return devm_gpiochip_add_data(&pdev->dev, &priv->chip, priv); > } > > -- > 2.34.1 >
Applied, thanks!
Bart
| |