[PATCH 1/1] pinctrl: pinctrl-generic-mux: use mux_state_try_select()

[PATCH 1/1] pinctrl: pinctrl-generic-mux: use mux_state_try_select()

[Frank.Li]

From: Frank Li <Frank.Li@nxp.com>

Use mux_state_try_select() instead of mux_state_select() so that the
consumer driver does not block during probe when the mux state has
already been selected.

mux_state_try_select() returns -EBUSY if the requested state is already
selected, allowing the driver to handle the condition without waiting.

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
Previous earily version pinctrl-generic-mux manage this resouce to avoid
call mux_state_select(). But review or respin patch, remove these check.
anyways mux_state_try_select() better method.
---
 drivers/pinctrl/pinctrl-generic-mux.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/pinctrl/pinctrl-generic-mux.c b/drivers/pinctrl/pinctrl-generic-mux.c
index da5a5ec015833..202b72351efb7 100644
--- a/drivers/pinctrl/pinctrl-generic-mux.c
+++ b/drivers/pinctrl/pinctrl-generic-mux.c
@@ -93,7 +93,7 @@ static int mux_pinmux_set_mux(struct pinctrl_dev *pctldev,
 	function = pinmux_generic_get_function(pctldev, func_selector);
 	func = function->data;
 
-	ret = mux_state_select(func->mux_state);
+	ret = mux_state_try_select(func->mux_state);
 	if (ret)
 		return ret;
 
-- 
2.43.0

Re: [PATCH 1/1] pinctrl: pinctrl-generic-mux: use mux_state_try_select()

[Linus Walleij]

On Wed, Jul 15, 2026 at 9:36 PM <Frank.Li@oss.nxp.com> wrote:

> Use mux_state_try_select() instead of mux_state_select() so that the
> consumer driver does not block during probe when the mux state has
> already been selected.
>
> mux_state_try_select() returns -EBUSY if the requested state is already
> selected, allowing the driver to handle the condition without waiting.
>
> Signed-off-by: Frank Li <Frank.Li@nxp.com>
> ---
> Previous earily version pinctrl-generic-mux manage this resouce to avoid
> call mux_state_select(). But review or respin patch, remove these check.
> anyways mux_state_try_select() better method.

OK makes sense, patch applied!

Yours,
Linus Walleij