Message ID | 87wpljs1rw.wl%kuninori.morimoto.gx@renesas.com (mailing list archive) |
---|---|
State | Superseded |
Delegated to: | Geert Uytterhoeven |
Headers | show |
On Tue, Jun 21, 2016 at 4:46 AM, Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> wrote: > From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> > > Now we have PINMUX_SINGLE(). Let's use it instead of PINMUX_IPSR_NOGP() > > Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> 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
Hi Geert Thank you for your review > > From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> > > > > Now we have PINMUX_SINGLE(). Let's use it instead of PINMUX_IPSR_NOGP() > > > > Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> > > Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> Oops, I didn't Cc to gpio ML I will send this patch again (with Geert's Reviewed-by)
Hi Morimoto-san, On Tue, Jun 21, 2016 at 9:19 AM, Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> wrote: >> > From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> >> > >> > Now we have PINMUX_SINGLE(). Let's use it instead of PINMUX_IPSR_NOGP() >> > >> > Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> >> >> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> > > Oops, I didn't Cc to gpio ML > I will send this patch again (with Geert's Reviewed-by) No problem, it's trivial, and it will go through my sh-pfc-for-v4.8 anyway... 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
Hi Geert > >> > From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> > >> > > >> > Now we have PINMUX_SINGLE(). Let's use it instead of PINMUX_IPSR_NOGP() > >> > > >> > Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> > >> > >> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> > > > > Oops, I didn't Cc to gpio ML > > I will send this patch again (with Geert's Reviewed-by) > > No problem, it's trivial, and it will go through my sh-pfc-for-v4.8 anyway... OK, Thank you !
diff --git a/drivers/pinctrl/sh-pfc/pfc-r8a7795.c b/drivers/pinctrl/sh-pfc/pfc-r8a7795.c index 1434a6f..cb55a9a 100644 --- a/drivers/pinctrl/sh-pfc/pfc-r8a7795.c +++ b/drivers/pinctrl/sh-pfc/pfc-r8a7795.c @@ -556,6 +556,9 @@ static const u16 pinmux_data[] = { PINMUX_SINGLE(AVS2), PINMUX_SINGLE(HDMI0_CEC), PINMUX_SINGLE(HDMI1_CEC), + PINMUX_SINGLE(I2C_SEL_0_1), + PINMUX_SINGLE(I2C_SEL_3_1), + PINMUX_SINGLE(I2C_SEL_5_1), PINMUX_SINGLE(MSIOF0_RXD), PINMUX_SINGLE(MSIOF0_SCK), PINMUX_SINGLE(MSIOF0_TXD), @@ -1405,11 +1408,6 @@ static const u16 pinmux_data[] = { PINMUX_IPSR_MSEL(IP17_7_4, STP_ISSYNC_0_E, SEL_SSP1_0_4), PINMUX_IPSR_MSEL(IP17_7_4, RIF2_D1_B, SEL_DRIF2_1), PINMUX_IPSR_GPSR(IP17_7_4, TPU0TO3), - - /* I2C */ - PINMUX_IPSR_NOGP(0, I2C_SEL_0_1), - PINMUX_IPSR_NOGP(0, I2C_SEL_3_1), - PINMUX_IPSR_NOGP(0, I2C_SEL_5_1), }; static const struct sh_pfc_pin pinmux_pins[] = {