Hi Simon, On Thu, Aug 2, 2018 at 1:30 PM Simon Horman <horms@xxxxxxxxxxxx> wrote: > On Fri, Jul 27, 2018 at 10:22:02AM +0100, Biju Das wrote: > > Add EtherAVB groups and functions definitions for R8A77470 SoC. > > > > Signed-off-by: Biju Das <biju.das@xxxxxxxxxxxxxx> > > Reviewed-by: Fabrizio Castro <fabrizio.castro@xxxxxxxxxxxxxx> > > --- > > This patch is based on the following discussion > > https://www.mail-archive.com/linux-renesas-soc@xxxxxxxxxxxxxxx/msg27480.html > > > > and > > > > https://en.wikipedia.org/wiki/Media-independent_interface > > --- > > drivers/pinctrl/sh-pfc/pfc-r8a77470.c | 133 ++++++++++++++++++++++++++++++++++ > > 1 file changed, 133 insertions(+) > > > > diff --git a/drivers/pinctrl/sh-pfc/pfc-r8a77470.c b/drivers/pinctrl/sh-pfc/pfc-r8a77470.c > > index 9d3ed43..995c959 100644 > > --- a/drivers/pinctrl/sh-pfc/pfc-r8a77470.c > > +++ b/drivers/pinctrl/sh-pfc/pfc-r8a77470.c > > @@ -1093,6 +1093,110 @@ static const struct sh_pfc_pin pinmux_pins[] = { > > PINMUX_GPIO_GP_ALL(), > > }; > > > > +/* - AVB -------------------------------------------------------------------- */ > > +static const unsigned int avb_col_pins[] = { > > + RCAR_GP_PIN(5, 18), > > +}; > > I feel that I am missing something painfully obvious, but this does not > seem to match page 4-36 of R01UH0695EJ0100 Rev.1.00 (Oct 13 2017). Please check Function VIII (The table extends horizontally to page 4-37!). 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
