* [PATCH] pinctrl: renesas: rzn1: enable PINMUX
@ 2023-10-04 20:00 Ralph Siemsen
2023-10-04 22:24 ` Miquel Raynal
2023-10-05 7:10 ` Geert Uytterhoeven
0 siblings, 2 replies; 3+ messages in thread
From: Ralph Siemsen @ 2023-10-04 20:00 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Miquel Raynal, Linus Walleij, linux-renesas-soc, linux-gpio,
linux-kernel, Ralph Siemsen
Enable pin muxing (eg. programmable function), so that the RZN1 GPIO
pins will be configured as specified by the pinmux in the DTS.
This used to be enabled implicitly via CONFIG_GENERIC_PINMUX_FUNCTIONS,
however that was removed in 308fb4e4eae14e6189dece3b7cf5b5f453c5d02
since the rzn1 driver does not call any of the generic pinmux functions.
Signed-off-by: Ralph Siemsen <ralph.siemsen@linaro.org>
---
drivers/pinctrl/renesas/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/pinctrl/renesas/Kconfig b/drivers/pinctrl/renesas/Kconfig
index 77730dc548ed..0ad8e14ccc52 100644
--- a/drivers/pinctrl/renesas/Kconfig
+++ b/drivers/pinctrl/renesas/Kconfig
@@ -234,6 +234,7 @@ config PINCTRL_RZN1
bool "pin control support for RZ/N1"
depends on OF
depends on ARCH_RZN1 || COMPILE_TEST
+ select PINMUX
select GENERIC_PINCONF
help
This selects pinctrl driver for Renesas RZ/N1 devices.
--
2.25.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] pinctrl: renesas: rzn1: enable PINMUX
2023-10-04 20:00 [PATCH] pinctrl: renesas: rzn1: enable PINMUX Ralph Siemsen
@ 2023-10-04 22:24 ` Miquel Raynal
2023-10-05 7:10 ` Geert Uytterhoeven
1 sibling, 0 replies; 3+ messages in thread
From: Miquel Raynal @ 2023-10-04 22:24 UTC (permalink / raw)
To: Ralph Siemsen
Cc: Geert Uytterhoeven, Linus Walleij, linux-renesas-soc, linux-gpio,
linux-kernel
Hi Ralph,
ralph.siemsen@linaro.org wrote on Wed, 4 Oct 2023 16:00:08 -0400:
> Enable pin muxing (eg. programmable function), so that the RZN1 GPIO
> pins will be configured as specified by the pinmux in the DTS.
>
> This used to be enabled implicitly via CONFIG_GENERIC_PINMUX_FUNCTIONS,
> however that was removed in 308fb4e4eae14e6189dece3b7cf5b5f453c5d02
> since the rzn1 driver does not call any of the generic pinmux functions.
>
> Signed-off-by: Ralph Siemsen <ralph.siemsen@linaro.org>
You probably need to point at this commit with a Fixes and possibly Cc
this to stable. Also you should use proper formatting for the commit
(12 digits sha then the text inside ("...")).
With this added:
Reviewed-by: Miquel Raynal <miquel.raynal@bootlin.com>
> ---
> drivers/pinctrl/renesas/Kconfig | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/pinctrl/renesas/Kconfig b/drivers/pinctrl/renesas/Kconfig
> index 77730dc548ed..0ad8e14ccc52 100644
> --- a/drivers/pinctrl/renesas/Kconfig
> +++ b/drivers/pinctrl/renesas/Kconfig
> @@ -234,6 +234,7 @@ config PINCTRL_RZN1
> bool "pin control support for RZ/N1"
> depends on OF
> depends on ARCH_RZN1 || COMPILE_TEST
> + select PINMUX
> select GENERIC_PINCONF
> help
> This selects pinctrl driver for Renesas RZ/N1 devices.
Thanks,
Miquèl
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] pinctrl: renesas: rzn1: enable PINMUX
2023-10-04 20:00 [PATCH] pinctrl: renesas: rzn1: enable PINMUX Ralph Siemsen
2023-10-04 22:24 ` Miquel Raynal
@ 2023-10-05 7:10 ` Geert Uytterhoeven
1 sibling, 0 replies; 3+ messages in thread
From: Geert Uytterhoeven @ 2023-10-05 7:10 UTC (permalink / raw)
To: Ralph Siemsen
Cc: Miquel Raynal, Linus Walleij, linux-renesas-soc, linux-gpio,
linux-kernel
Hi Ralph,
On Wed, Oct 4, 2023 at 10:00 PM Ralph Siemsen <ralph.siemsen@linaro.org> wrote:
> Enable pin muxing (eg. programmable function), so that the RZN1 GPIO
> pins will be configured as specified by the pinmux in the DTS.
>
> This used to be enabled implicitly via CONFIG_GENERIC_PINMUX_FUNCTIONS,
> however that was removed in 308fb4e4eae14e6189dece3b7cf5b5f453c5d02
You missed the first character of the commit ID.
> since the rzn1 driver does not call any of the generic pinmux functions.
>
> Signed-off-by: Ralph Siemsen <ralph.siemsen@linaro.org>
Fixes: 1308fb4e4eae14e6 ("pinctrl: rzn1: Do not select
GENERIC_PIN{CTRL_GROUPS,MUX_FUNCTIONS}")
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
i.e. will queue in renesas-pinctrl-fixes-for-v6.6...
> --- a/drivers/pinctrl/renesas/Kconfig
> +++ b/drivers/pinctrl/renesas/Kconfig
> @@ -234,6 +234,7 @@ config PINCTRL_RZN1
> bool "pin control support for RZ/N1"
> depends on OF
> depends on ARCH_RZN1 || COMPILE_TEST
> + select PINMUX
> select GENERIC_PINCONF
... with alphabetical sort order restored.
No need to resend.
> help
> This selects pinctrl driver for Renesas RZ/N1 devices.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-10-05 13:59 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-10-04 20:00 [PATCH] pinctrl: renesas: rzn1: enable PINMUX Ralph Siemsen
2023-10-04 22:24 ` Miquel Raynal
2023-10-05 7:10 ` Geert Uytterhoeven
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).