Hi Morimoto-san,
On Tue, Jun 14, 2022 at 8:00 AM Kuninori Morimoto
<kuninori.morimoto.gx@xxxxxxxxxxx> wrote:
> From: Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx>
>
> TSN0 needs MODSEL4 settings.
> This patch adds missing MODSELx setting for these.
>
> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx>
Thanks for your patch!
> --- a/drivers/pinctrl/renesas/pfc-r8a779g0.c
> +++ b/drivers/pinctrl/renesas/pfc-r8a779g0.c
> @@ -720,27 +720,30 @@ static const u16 pinmux_data[] = {
> PINMUX_SINGLE(AVS0),
> PINMUX_SINGLE(PCIE1_CLKREQ_N),
> PINMUX_SINGLE(PCIE0_CLKREQ_N),
> +
> + /* TSN0 without MDSEL4 */
MODSEL4
> PINMUX_SINGLE(TSN0_TXCREFCLK),
> - PINMUX_SINGLE(TSN0_TD2),
> - PINMUX_SINGLE(TSN0_TD3),
> PINMUX_SINGLE(TSN0_RD2),
> PINMUX_SINGLE(TSN0_RD3),
> - PINMUX_SINGLE(TSN0_TD0),
> - PINMUX_SINGLE(TSN0_TD1),
> PINMUX_SINGLE(TSN0_RD1),
> - PINMUX_SINGLE(TSN0_TXC),
> PINMUX_SINGLE(TSN0_RXC),
> PINMUX_SINGLE(TSN0_RD0),
> - PINMUX_SINGLE(TSN0_TX_CTL),
> - PINMUX_SINGLE(TSN0_AVTP_PPS0),
> PINMUX_SINGLE(TSN0_RX_CTL),
> PINMUX_SINGLE(TSN0_AVTP_CAPTURE),
> - PINMUX_SINGLE(TSN0_AVTP_MATCH),
> PINMUX_SINGLE(TSN0_LINK),
> PINMUX_SINGLE(TSN0_PHY_INT),
> - PINMUX_SINGLE(TSN0_AVTP_PPS1),
> - PINMUX_SINGLE(TSN0_MDC),
> PINMUX_SINGLE(TSN0_MDIO),
> + /* TSN0 with MDSEL4 */
MODSEL4
> + PINMUX_IPSR_NOGM(0, TSN0_TD2, SEL_TSN0_TD2_1),
> + PINMUX_IPSR_NOGM(0, TSN0_TD3, SEL_TSN0_TD3_1),
> + PINMUX_IPSR_NOGM(0, TSN0_TD0, SEL_TSN0_TD0_1),
> + PINMUX_IPSR_NOGM(0, TSN0_TD1, SEL_TSN0_TD1_1),
> + PINMUX_IPSR_NOGM(0, TSN0_TXC, SEL_TSN0_TXC_1),
> + PINMUX_IPSR_NOGM(0, TSN0_TX_CTL, SEL_TSN0_TX_CTL_1),
> + PINMUX_IPSR_NOGM(0, TSN0_AVTP_PPS0, SEL_TSN0_AVTP_PPS0_1),
> + PINMUX_IPSR_NOGM(0, TSN0_AVTP_MATCH, SEL_TSN0_AVTP_MATCH_1),
> + PINMUX_IPSR_NOGM(0, TSN0_AVTP_PPS1, SEL_TSN0_AVTP_PPS1_1),
> + PINMUX_IPSR_NOGM(0, TSN0_MDC, SEL_TSN0_MDC_1),
>
> PINMUX_SINGLE(AVB2_RX_CTL),
> PINMUX_SINGLE(AVB2_TX_CTL),
With the above fixed:
Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx
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
