[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <20231113023434.1186043-1-samuel.holland@sifive.com>
Date: Sun, 12 Nov 2023 18:34:32 -0800
From: Samuel Holland <samuel.holland@...ive.com>
To: Linus Walleij <linus.walleij@...aro.org>,
Bartosz Golaszewski <brgl@...ev.pl>,
Andy Shevchenko <andy@...nel.org>
Cc: Andrei Coardos <aboutphysycs@...il.com>,
Alexandru Ardelean <alex@...uggie.ro>,
Samuel Holland <samuel.holland@...ive.com>,
Palmer Dabbelt <palmer@...belt.com>,
Paul Walmsley <paul.walmsley@...ive.com>,
linux-gpio@...r.kernel.org, linux-kernel@...r.kernel.org,
linux-riscv@...ts.infradead.org
Subject: [PATCH v2] gpio: sifive: remove unneeded call to platform_set_drvdata()
From: Andrei Coardos <aboutphysycs@...il.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@...uggie.ro>
Signed-off-by: Andrei Coardos <aboutphysycs@...il.com>
Reviewed-by: Andy Shevchenko <andy@...nel.org>
Signed-off-by: Samuel Holland <samuel.holland@...ive.com>
---
Changes in v2:
- Rebase on v6.7-rc1
drivers/gpio/gpio-sifive.c | 1 -
1 file changed, 1 deletion(-)
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);
}
--
2.42.0
Powered by blists - more mailing lists