Linux GPIO subsystem development
 help / color / mirror / Atom feed
* [PATCH] pinctrl: mpfs-mssio: remove unused mutex
@ 2026-10-30 18:12 Conor Dooley
  0 siblings, 0 replies; only message in thread
From: Conor Dooley @ 2026-10-30 18:12 UTC (permalink / raw)
  To: Linus Walleij
  Cc: conor, Conor Dooley, Valentina Fernandez, Daire McNamara,
	linux-riscv, linux-gpio, linux-kernel

From: Conor Dooley <conor.dooley@microchip.com>

The pinctrl core creates a mutex that is used to protect things like the
generic dt node mapping functions. The (currently) unused mutex was added
when the driver was in RFC state and contained a custom dt node mapping
function, and used the driver's own mutex to protect it. The driver
probably could have borrowed the core's, but this is moot since the merged
version of the driver no longer has a custom dt node mapper. Remove the
unused mutex.

Signed-off-by: Conor Dooley <conor.dooley@microchip.com>
---
CC: Conor Dooley <conor.dooley@microchip.com>
CC: Valentina Fernandez <valentina.fernandezalanis@microchip.com>
CC: Daire McNamara <daire.mcnamara@microchip.com>
CC: Linus Walleij <linusw@kernel.org>
CC: linux-riscv@lists.infradead.org
CC: linux-gpio@vger.kernel.org
CC: linux-kernel@vger.kernel.org
---
 drivers/pinctrl/microchip/pinctrl-mpfs-mssio.c | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/drivers/pinctrl/microchip/pinctrl-mpfs-mssio.c b/drivers/pinctrl/microchip/pinctrl-mpfs-mssio.c
index 8da0c5312ce3..17c6d4190053 100644
--- a/drivers/pinctrl/microchip/pinctrl-mpfs-mssio.c
+++ b/drivers/pinctrl/microchip/pinctrl-mpfs-mssio.c
@@ -56,7 +56,6 @@ struct mpfs_pinctrl {
 	struct device *dev;
 	struct regmap *regmap;
 	struct regmap *sysreg_regmap;
-	struct mutex mutex;
 	struct pinctrl_desc desc;
 };
 
@@ -677,7 +676,6 @@ static int mpfs_pinctrl_probe(struct platform_device *pdev)
 {
 	struct device *dev = &pdev->dev;
 	struct mpfs_pinctrl *pctrl;
-	int ret;
 
 	pctrl = devm_kzalloc(dev, sizeof(*pctrl), GFP_KERNEL);
 	if (!pctrl)
@@ -703,10 +701,6 @@ static int mpfs_pinctrl_probe(struct platform_device *pdev)
 
 	pctrl->dev = dev;
 
-	ret = devm_mutex_init(dev, &pctrl->mutex);
-	if (ret)
-		return ret;
-
 	platform_set_drvdata(pdev, pctrl);
 
 	pctrl->pctrl = devm_pinctrl_register(&pdev->dev, &pctrl->desc, pctrl);
-- 
2.53.0


^ permalink raw reply related	[flat|nested] only message in thread

only message in thread, other threads:[~2026-10-30 18:15 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-10-30 18:12 [PATCH] pinctrl: mpfs-mssio: remove unused mutex Conor Dooley

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox