diff mbox series

[v2] gpio: sifive: remove unneeded call to platform_set_drvdata()

Message ID 20231113023434.1186043-1-samuel.holland@sifive.com
State New
Headers show
Series [v2] gpio: sifive: remove unneeded call to platform_set_drvdata() | expand

Commit Message

Samuel Holland Nov. 13, 2023, 2:34 a.m. UTC 
From: Andrei Coardos <aboutphysycs@gmail.com>

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>
Reviewed-by: Andy Shevchenko <andy@kernel.org>
Signed-off-by: Samuel Holland <samuel.holland@sifive.com>
---

Changes in v2:
 - Rebase on v6.7-rc1

 drivers/gpio/gpio-sifive.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Bartosz Golaszewski Nov. 15, 2023, 2:40 p.m. UTC | #1 
On Mon, Nov 13, 2023 at 3:34 AM Samuel Holland
<samuel.holland@sifive.com> wrote:
>
> From: Andrei Coardos <aboutphysycs@gmail.com>
>
> 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>
> Reviewed-by: Andy Shevchenko <andy@kernel.org>
> Signed-off-by: Samuel Holland <samuel.holland@sifive.com>
> ---

Applied, thanks!

Bart
diff mbox series

Patch

diff --git a/drivers/gpio/gpio-sifive.c b/drivers/gpio/gpio-sifive.c
index 8decd9b5d229..067c8edb62e2 100644
--- a/drivers/gpio/gpio-sifive.c
+++ b/drivers/gpio/gpio-sifive.c
@@ -250,7 +250,6 @@  static int sifive_gpio_probe(struct platform_device *pdev)
 	girq->handler = handle_bad_irq;
 	girq->default_type = IRQ_TYPE_NONE;
 
-	platform_set_drvdata(pdev, chip);
 	return gpiochip_add_data(&chip->gc, chip);
 }