Hi Simon,
On Thu, Aug 2, 2018 at 1:30 PM Simon Horman <[email protected]> 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 <[email protected]>
> > Reviewed-by: Fabrizio Castro <[email protected]>
> > ---
> > This patch is based on the following discussion
> > https://www.mail-archive.com/[email protected]/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 -- [email protected]
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