* [PATCH] pinctrl: sh-pfc: r8a77470: Add EtherAVB pin groups
@ 2018-07-27 9:22 Biju Das
2018-07-27 9:29 ` Biju Das
2018-08-02 11:30 ` Simon Horman
0 siblings, 2 replies; 6+ messages in thread
From: Biju Das @ 2018-07-27 9:22 UTC (permalink / raw)
To: Laurent Pinchart, Geert Uytterhoeven, Linus Walleij
Cc: Biju Das, linux-renesas-soc, linux-gpio, Simon Horman,
Chris Paterson, Fabrizio Castro
Add EtherAVB groups and functions definitions for R8A77470 SoC.
Signed-off-by: Biju Das <biju.das@bp.renesas.com>
Reviewed-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com>
---
This patch is based on the following discussion
https://www.mail-archive.com/linux-renesas-soc@vger.kernel.org/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),
+};
+static const unsigned int avb_col_mux[] = {
+ AVB_COL_MARK,
+};
+static const unsigned int avb_crs_pins[] = {
+ RCAR_GP_PIN(5, 17),
+};
+static const unsigned int avb_crs_mux[] = {
+ AVB_CRS_MARK,
+};
+static const unsigned int avb_link_pins[] = {
+ RCAR_GP_PIN(5, 14),
+};
+static const unsigned int avb_link_mux[] = {
+ AVB_LINK_MARK,
+};
+static const unsigned int avb_magic_pins[] = {
+ RCAR_GP_PIN(5, 15),
+};
+static const unsigned int avb_magic_mux[] = {
+ AVB_MAGIC_MARK,
+};
+static const unsigned int avb_phy_int_pins[] = {
+ RCAR_GP_PIN(5, 16),
+};
+static const unsigned int avb_phy_int_mux[] = {
+ AVB_PHY_INT_MARK,
+};
+static const unsigned int avb_mdio_pins[] = {
+ RCAR_GP_PIN(5, 12), RCAR_GP_PIN(5, 13),
+};
+static const unsigned int avb_mdio_mux[] = {
+ AVB_MDC_MARK, AVB_MDIO_MARK,
+};
+static const unsigned int avb_mii_tx_rx_pins[] = {
+ RCAR_GP_PIN(3, 12), RCAR_GP_PIN(3, 14), RCAR_GP_PIN(3, 15),
+ RCAR_GP_PIN(3, 16), RCAR_GP_PIN(3, 27), RCAR_GP_PIN(3, 13),
+
+ RCAR_GP_PIN(3, 0), RCAR_GP_PIN(3, 2), RCAR_GP_PIN(3, 3),
+ RCAR_GP_PIN(3, 4), RCAR_GP_PIN(3, 5), RCAR_GP_PIN(3, 1),
+ RCAR_GP_PIN(3, 10),
+};
+static const unsigned int avb_mii_tx_rx_mux[] = {
+ AVB_TX_CLK_MARK, AVB_TXD0_MARK, AVB_TXD1_MARK, AVB_TXD2_MARK,
+ AVB_TXD3_MARK, AVB_TX_EN_MARK,
+
+ AVB_RX_CLK_MARK, AVB_RXD0_MARK, AVB_RXD1_MARK, AVB_RXD2_MARK,
+ AVB_RXD3_MARK, AVB_RX_DV_MARK, AVB_RX_ER_MARK,
+};
+static const unsigned int avb_mii_tx_er_pins[] = {
+ RCAR_GP_PIN(5, 23),
+};
+static const unsigned int avb_mii_tx_er_mux[] = {
+ AVB_TX_ER_MARK,
+};
+static const unsigned int avb_gmii_tx_rx_pins[] = {
+ RCAR_GP_PIN(4, 1), RCAR_GP_PIN(3, 11), RCAR_GP_PIN(3, 12),
+ RCAR_GP_PIN(3, 14), RCAR_GP_PIN(3, 15), RCAR_GP_PIN(3, 16),
+ RCAR_GP_PIN(3, 27), RCAR_GP_PIN(3, 28), RCAR_GP_PIN(3, 29),
+ RCAR_GP_PIN(4, 0), RCAR_GP_PIN(5, 22), RCAR_GP_PIN(3, 13),
+ RCAR_GP_PIN(5, 23),
+
+ RCAR_GP_PIN(3, 0), RCAR_GP_PIN(3, 2), RCAR_GP_PIN(3, 3),
+ RCAR_GP_PIN(3, 4), RCAR_GP_PIN(3, 5), RCAR_GP_PIN(3, 6),
+ RCAR_GP_PIN(3, 7), RCAR_GP_PIN(3, 8), RCAR_GP_PIN(3, 9),
+ RCAR_GP_PIN(3, 1), RCAR_GP_PIN(3, 10),
+};
+static const unsigned int avb_gmii_tx_rx_mux[] = {
+ AVB_GTX_CLK_MARK, AVB_GTXREFCLK_MARK, AVB_TX_CLK_MARK, AVB_TXD0_MARK,
+ AVB_TXD1_MARK, AVB_TXD2_MARK, AVB_TXD3_MARK, AVB_TXD4_MARK,
+ AVB_TXD5_MARK, AVB_TXD6_MARK, AVB_TXD7_MARK, AVB_TX_EN_MARK,
+ AVB_TX_ER_MARK,
+
+ AVB_RX_CLK_MARK, AVB_RXD0_MARK, AVB_RXD1_MARK, AVB_RXD2_MARK,
+ AVB_RXD3_MARK, AVB_RXD4_MARK, AVB_RXD5_MARK, AVB_RXD6_MARK,
+ AVB_RXD7_MARK, AVB_RX_DV_MARK, AVB_RX_ER_MARK,
+};
+static const unsigned int avb_avtp_match_a_pins[] = {
+ RCAR_GP_PIN(1, 15),
+};
+static const unsigned int avb_avtp_match_a_mux[] = {
+ AVB_AVTP_MATCH_A_MARK,
+};
+static const unsigned int avb_avtp_capture_a_pins[] = {
+ RCAR_GP_PIN(1, 14),
+};
+static const unsigned int avb_avtp_capture_a_mux[] = {
+ AVB_AVTP_CAPTURE_A_MARK,
+};
+static const unsigned int avb_avtp_match_b_pins[] = {
+ RCAR_GP_PIN(5, 20),
+};
+static const unsigned int avb_avtp_match_b_mux[] = {
+ AVB_AVTP_MATCH_B_MARK,
+};
+static const unsigned int avb_avtp_capture_b_pins[] = {
+ RCAR_GP_PIN(5, 19),
+};
+static const unsigned int avb_avtp_capture_b_mux[] = {
+ AVB_AVTP_CAPTURE_B_MARK,
+};
/* - MMC -------------------------------------------------------------------- */
static const unsigned int mmc_data1_pins[] = {
/* D0 */
@@ -1370,6 +1474,19 @@ static const unsigned int scif_clk_b_mux[] = {
};
static const struct sh_pfc_pin_group pinmux_groups[] = {
+ SH_PFC_PIN_GROUP(avb_col),
+ SH_PFC_PIN_GROUP(avb_crs),
+ SH_PFC_PIN_GROUP(avb_link),
+ SH_PFC_PIN_GROUP(avb_magic),
+ SH_PFC_PIN_GROUP(avb_phy_int),
+ SH_PFC_PIN_GROUP(avb_mdio),
+ SH_PFC_PIN_GROUP(avb_mii_tx_rx),
+ SH_PFC_PIN_GROUP(avb_mii_tx_er),
+ SH_PFC_PIN_GROUP(avb_gmii_tx_rx),
+ SH_PFC_PIN_GROUP(avb_avtp_match_a),
+ SH_PFC_PIN_GROUP(avb_avtp_capture_a),
+ SH_PFC_PIN_GROUP(avb_avtp_match_b),
+ SH_PFC_PIN_GROUP(avb_avtp_capture_b),
SH_PFC_PIN_GROUP(mmc_data1),
SH_PFC_PIN_GROUP(mmc_data4),
SH_PFC_PIN_GROUP(mmc_data8),
@@ -1409,6 +1526,21 @@ static const struct sh_pfc_pin_group pinmux_groups[] = {
SH_PFC_PIN_GROUP(scif_clk_b),
};
+static const char * const avb_groups[] = {
+ "avb_col",
+ "avb_crs",
+ "avb_link",
+ "avb_magic",
+ "avb_phy_int",
+ "avb_mdio",
+ "avb_mii_tx_rx",
+ "avb_mii_tx_er",
+ "avb_gmii_tx_rx",
+ "avb_avtp_match_a",
+ "avb_avtp_capture_a",
+ "avb_avtp_match_b",
+ "avb_avtp_capture_b",
+};
static const char * const mmc_groups[] = {
"mmc_data1",
"mmc_data4",
@@ -1471,6 +1603,7 @@ static const char * const scif_clk_groups[] = {
};
static const struct sh_pfc_function pinmux_functions[] = {
+ SH_PFC_FUNCTION(avb),
SH_PFC_FUNCTION(mmc),
SH_PFC_FUNCTION(scif0),
SH_PFC_FUNCTION(scif1),
--
2.7.4
^ permalink raw reply related [flat|nested] 6+ messages in thread* RE: [PATCH] pinctrl: sh-pfc: r8a77470: Add EtherAVB pin groups
2018-07-27 9:22 [PATCH] pinctrl: sh-pfc: r8a77470: Add EtherAVB pin groups Biju Das
@ 2018-07-27 9:29 ` Biju Das
2018-07-30 15:01 ` Geert Uytterhoeven
2018-08-02 11:30 ` Simon Horman
1 sibling, 1 reply; 6+ messages in thread
From: Biju Das @ 2018-07-27 9:29 UTC (permalink / raw)
To: Biju Das, Laurent Pinchart, Geert Uytterhoeven, Linus Walleij
Cc: linux-renesas-soc, linux-gpio, Simon Horman, Chris Paterson,
Fabrizio Castro, Sergei Shtylyov
+ sergie
> -----Original Message-----
> From: Biju Das [mailto:biju.das@bp.renesas.com]
> Sent: 27 July 2018 10:22
> To: Laurent Pinchart <laurent.pinchart@ideasonboard.com>; Geert
> Uytterhoeven <geert+renesas@glider.be>; Linus Walleij
> <linus.walleij@linaro.org>
> Cc: Biju Das <biju.das@bp.renesas.com>; linux-renesas-
> soc@vger.kernel.org; linux-gpio@vger.kernel.org; Simon Horman
> <horms@verge.net.au>; Chris Paterson <Chris.Paterson2@renesas.com>;
> Fabrizio Castro <fabrizio.castro@bp.renesas.com>
> Subject: [PATCH] pinctrl: sh-pfc: r8a77470: Add EtherAVB pin groups
>
> Add EtherAVB groups and functions definitions for R8A77470 SoC.
>
> Signed-off-by: Biju Das <biju.das@bp.renesas.com>
> Reviewed-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com>
> ---
> This patch is based on the following discussion https://www.mail-
> archive.com/linux-renesas-soc@vger.kernel.org/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),
> +};
> +static const unsigned int avb_col_mux[] = {
> +AVB_COL_MARK,
> +};
> +static const unsigned int avb_crs_pins[] = {
> +RCAR_GP_PIN(5, 17),
> +};
> +static const unsigned int avb_crs_mux[] = {
> +AVB_CRS_MARK,
> +};
> +static const unsigned int avb_link_pins[] = {
> +RCAR_GP_PIN(5, 14),
> +};
> +static const unsigned int avb_link_mux[] = {
> +AVB_LINK_MARK,
> +};
> +static const unsigned int avb_magic_pins[] = {
> +RCAR_GP_PIN(5, 15),
> +};
> +static const unsigned int avb_magic_mux[] = {
> +AVB_MAGIC_MARK,
> +};
> +static const unsigned int avb_phy_int_pins[] = {
> +RCAR_GP_PIN(5, 16),
> +};
> +static const unsigned int avb_phy_int_mux[] = {
> +AVB_PHY_INT_MARK,
> +};
> +static const unsigned int avb_mdio_pins[] = {
> +RCAR_GP_PIN(5, 12), RCAR_GP_PIN(5, 13), }; static const unsigned
> int
> +avb_mdio_mux[] = {
> +AVB_MDC_MARK, AVB_MDIO_MARK,
> +};
> +static const unsigned int avb_mii_tx_rx_pins[] = {
> +RCAR_GP_PIN(3, 12), RCAR_GP_PIN(3, 14), RCAR_GP_PIN(3, 15),
> +RCAR_GP_PIN(3, 16), RCAR_GP_PIN(3, 27), RCAR_GP_PIN(3, 13),
> +
> +RCAR_GP_PIN(3, 0), RCAR_GP_PIN(3, 2), RCAR_GP_PIN(3, 3),
> +RCAR_GP_PIN(3, 4), RCAR_GP_PIN(3, 5), RCAR_GP_PIN(3, 1),
> +RCAR_GP_PIN(3, 10),
> +};
> +static const unsigned int avb_mii_tx_rx_mux[] = {
> +AVB_TX_CLK_MARK, AVB_TXD0_MARK, AVB_TXD1_MARK,
> AVB_TXD2_MARK,
> +AVB_TXD3_MARK, AVB_TX_EN_MARK,
> +
> +AVB_RX_CLK_MARK, AVB_RXD0_MARK, AVB_RXD1_MARK,
> AVB_RXD2_MARK,
> +AVB_RXD3_MARK, AVB_RX_DV_MARK, AVB_RX_ER_MARK, }; static
> const
> +unsigned int avb_mii_tx_er_pins[] = {
> +RCAR_GP_PIN(5, 23),
> +};
> +static const unsigned int avb_mii_tx_er_mux[] = {
> +AVB_TX_ER_MARK,
> +};
> +static const unsigned int avb_gmii_tx_rx_pins[] = {
> +RCAR_GP_PIN(4, 1), RCAR_GP_PIN(3, 11), RCAR_GP_PIN(3, 12),
> +RCAR_GP_PIN(3, 14), RCAR_GP_PIN(3, 15), RCAR_GP_PIN(3, 16),
> +RCAR_GP_PIN(3, 27), RCAR_GP_PIN(3, 28), RCAR_GP_PIN(3, 29),
> +RCAR_GP_PIN(4, 0), RCAR_GP_PIN(5, 22), RCAR_GP_PIN(3, 13),
> +RCAR_GP_PIN(5, 23),
> +
> +RCAR_GP_PIN(3, 0), RCAR_GP_PIN(3, 2), RCAR_GP_PIN(3, 3),
> +RCAR_GP_PIN(3, 4), RCAR_GP_PIN(3, 5), RCAR_GP_PIN(3, 6),
> +RCAR_GP_PIN(3, 7), RCAR_GP_PIN(3, 8), RCAR_GP_PIN(3, 9),
> +RCAR_GP_PIN(3, 1), RCAR_GP_PIN(3, 10), }; static const unsigned int
> +avb_gmii_tx_rx_mux[] = {
> +AVB_GTX_CLK_MARK, AVB_GTXREFCLK_MARK,
> AVB_TX_CLK_MARK, AVB_TXD0_MARK,
> +AVB_TXD1_MARK, AVB_TXD2_MARK, AVB_TXD3_MARK,
> AVB_TXD4_MARK,
> +AVB_TXD5_MARK, AVB_TXD6_MARK, AVB_TXD7_MARK,
> AVB_TX_EN_MARK,
> +AVB_TX_ER_MARK,
> +
> +AVB_RX_CLK_MARK, AVB_RXD0_MARK, AVB_RXD1_MARK,
> AVB_RXD2_MARK,
> +AVB_RXD3_MARK, AVB_RXD4_MARK, AVB_RXD5_MARK,
> AVB_RXD6_MARK,
> +AVB_RXD7_MARK, AVB_RX_DV_MARK, AVB_RX_ER_MARK, }; static
> const
> +unsigned int avb_avtp_match_a_pins[] = {
> +RCAR_GP_PIN(1, 15),
> +};
> +static const unsigned int avb_avtp_match_a_mux[] = {
> +AVB_AVTP_MATCH_A_MARK,
> +};
> +static const unsigned int avb_avtp_capture_a_pins[] = {
> +RCAR_GP_PIN(1, 14),
> +};
> +static const unsigned int avb_avtp_capture_a_mux[] = {
> +AVB_AVTP_CAPTURE_A_MARK,
> +};
> +static const unsigned int avb_avtp_match_b_pins[] = {
> +RCAR_GP_PIN(5, 20),
> +};
> +static const unsigned int avb_avtp_match_b_mux[] = {
> +AVB_AVTP_MATCH_B_MARK,
> +};
> +static const unsigned int avb_avtp_capture_b_pins[] = {
> +RCAR_GP_PIN(5, 19),
> +};
> +static const unsigned int avb_avtp_capture_b_mux[] = {
> +AVB_AVTP_CAPTURE_B_MARK,
> +};
> /* - MMC -------------------------------------------------------------------- */ static
> const unsigned int mmc_data1_pins[] = {
> /* D0 */
> @@ -1370,6 +1474,19 @@ static const unsigned int scif_clk_b_mux[] = { };
>
> static const struct sh_pfc_pin_group pinmux_groups[] = {
> +SH_PFC_PIN_GROUP(avb_col),
> +SH_PFC_PIN_GROUP(avb_crs),
> +SH_PFC_PIN_GROUP(avb_link),
> +SH_PFC_PIN_GROUP(avb_magic),
> +SH_PFC_PIN_GROUP(avb_phy_int),
> +SH_PFC_PIN_GROUP(avb_mdio),
> +SH_PFC_PIN_GROUP(avb_mii_tx_rx),
> +SH_PFC_PIN_GROUP(avb_mii_tx_er),
> +SH_PFC_PIN_GROUP(avb_gmii_tx_rx),
> +SH_PFC_PIN_GROUP(avb_avtp_match_a),
> +SH_PFC_PIN_GROUP(avb_avtp_capture_a),
> +SH_PFC_PIN_GROUP(avb_avtp_match_b),
> +SH_PFC_PIN_GROUP(avb_avtp_capture_b),
> SH_PFC_PIN_GROUP(mmc_data1),
> SH_PFC_PIN_GROUP(mmc_data4),
> SH_PFC_PIN_GROUP(mmc_data8),
> @@ -1409,6 +1526,21 @@ static const struct sh_pfc_pin_group
> pinmux_groups[] = {
> SH_PFC_PIN_GROUP(scif_clk_b),
> };
>
> +static const char * const avb_groups[] = {
> +"avb_col",
> +"avb_crs",
> +"avb_link",
> +"avb_magic",
> +"avb_phy_int",
> +"avb_mdio",
> +"avb_mii_tx_rx",
> +"avb_mii_tx_er",
> +"avb_gmii_tx_rx",
> +"avb_avtp_match_a",
> +"avb_avtp_capture_a",
> +"avb_avtp_match_b",
> +"avb_avtp_capture_b",
> +};
> static const char * const mmc_groups[] = {
> "mmc_data1",
> "mmc_data4",
> @@ -1471,6 +1603,7 @@ static const char * const scif_clk_groups[] = { };
>
> static const struct sh_pfc_function pinmux_functions[] = {
> +SH_PFC_FUNCTION(avb),
> SH_PFC_FUNCTION(mmc),
> SH_PFC_FUNCTION(scif0),
> SH_PFC_FUNCTION(scif1),
> --
> 2.7.4
Renesas Electronics Europe Ltd, Dukes Meadow, Millboard Road, Bourne End, Buckinghamshire, SL8 5FH, UK. Registered in England & Wales under Registered No. 04586709.
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] pinctrl: sh-pfc: r8a77470: Add EtherAVB pin groups
2018-07-27 9:29 ` Biju Das
@ 2018-07-30 15:01 ` Geert Uytterhoeven
0 siblings, 0 replies; 6+ messages in thread
From: Geert Uytterhoeven @ 2018-07-30 15:01 UTC (permalink / raw)
To: Biju Das
Cc: Laurent Pinchart, Geert Uytterhoeven, Linus Walleij,
Linux-Renesas, open list:GPIO SUBSYSTEM, Simon Horman,
Chris Paterson, Fabrizio Castro, Sergei Shtylyov
Hi Biju,
On Fri, Jul 27, 2018 at 11:29 AM Biju Das <biju.das@bp.renesas.com> wrote:
>
> + sergie
>
> > -----Original Message-----
> > From: Biju Das [mailto:biju.das@bp.renesas.com]
> > Sent: 27 July 2018 10:22
> > To: Laurent Pinchart <laurent.pinchart@ideasonboard.com>; Geert
> > Uytterhoeven <geert+renesas@glider.be>; Linus Walleij
> > <linus.walleij@linaro.org>
> > Cc: Biju Das <biju.das@bp.renesas.com>; linux-renesas-
> > soc@vger.kernel.org; linux-gpio@vger.kernel.org; Simon Horman
> > <horms@verge.net.au>; Chris Paterson <Chris.Paterson2@renesas.com>;
> > Fabrizio Castro <fabrizio.castro@bp.renesas.com>
> > Subject: [PATCH] pinctrl: sh-pfc: r8a77470: Add EtherAVB pin groups
> >
> > Add EtherAVB groups and functions definitions for R8A77470 SoC.
> >
> > Signed-off-by: Biju Das <biju.das@bp.renesas.com>
> > Reviewed-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com>
> > ---
> > This patch is based on the following discussion https://www.mail-
> > archive.com/linux-renesas-soc@vger.kernel.org/msg27480.html
> >
> > and
> >
> > https://en.wikipedia.org/wiki/Media-independent_interface
> > ---
> > drivers/pinctrl/sh-pfc/pfc-r8a77470.c | 133
Thanks for the update!
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
I'll wait to see if Sergei has some comments before applying...
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] 6+ messages in thread
* Re: [PATCH] pinctrl: sh-pfc: r8a77470: Add EtherAVB pin groups
2018-07-27 9:22 [PATCH] pinctrl: sh-pfc: r8a77470: Add EtherAVB pin groups Biju Das
2018-07-27 9:29 ` Biju Das
@ 2018-08-02 11:30 ` Simon Horman
2018-08-02 11:34 ` Geert Uytterhoeven
1 sibling, 1 reply; 6+ messages in thread
From: Simon Horman @ 2018-08-02 11:30 UTC (permalink / raw)
To: Biju Das
Cc: Laurent Pinchart, Geert Uytterhoeven, Linus Walleij,
linux-renesas-soc, linux-gpio, Chris Paterson, Fabrizio Castro
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@bp.renesas.com>
> Reviewed-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com>
> ---
> This patch is based on the following discussion
> https://www.mail-archive.com/linux-renesas-soc@vger.kernel.org/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).
> +static const unsigned int avb_col_mux[] = {
> + AVB_COL_MARK,
> +};
> +static const unsigned int avb_crs_pins[] = {
> + RCAR_GP_PIN(5, 17),
> +};
> +static const unsigned int avb_crs_mux[] = {
> + AVB_CRS_MARK,
> +};
> +static const unsigned int avb_link_pins[] = {
> + RCAR_GP_PIN(5, 14),
> +};
> +static const unsigned int avb_link_mux[] = {
> + AVB_LINK_MARK,
> +};
> +static const unsigned int avb_magic_pins[] = {
> + RCAR_GP_PIN(5, 15),
> +};
> +static const unsigned int avb_magic_mux[] = {
> + AVB_MAGIC_MARK,
> +};
> +static const unsigned int avb_phy_int_pins[] = {
> + RCAR_GP_PIN(5, 16),
> +};
> +static const unsigned int avb_phy_int_mux[] = {
> + AVB_PHY_INT_MARK,
> +};
> +static const unsigned int avb_mdio_pins[] = {
> + RCAR_GP_PIN(5, 12), RCAR_GP_PIN(5, 13),
> +};
> +static const unsigned int avb_mdio_mux[] = {
> + AVB_MDC_MARK, AVB_MDIO_MARK,
> +};
> +static const unsigned int avb_mii_tx_rx_pins[] = {
> + RCAR_GP_PIN(3, 12), RCAR_GP_PIN(3, 14), RCAR_GP_PIN(3, 15),
> + RCAR_GP_PIN(3, 16), RCAR_GP_PIN(3, 27), RCAR_GP_PIN(3, 13),
> +
> + RCAR_GP_PIN(3, 0), RCAR_GP_PIN(3, 2), RCAR_GP_PIN(3, 3),
> + RCAR_GP_PIN(3, 4), RCAR_GP_PIN(3, 5), RCAR_GP_PIN(3, 1),
> + RCAR_GP_PIN(3, 10),
> +};
> +static const unsigned int avb_mii_tx_rx_mux[] = {
> + AVB_TX_CLK_MARK, AVB_TXD0_MARK, AVB_TXD1_MARK, AVB_TXD2_MARK,
> + AVB_TXD3_MARK, AVB_TX_EN_MARK,
> +
> + AVB_RX_CLK_MARK, AVB_RXD0_MARK, AVB_RXD1_MARK, AVB_RXD2_MARK,
> + AVB_RXD3_MARK, AVB_RX_DV_MARK, AVB_RX_ER_MARK,
> +};
> +static const unsigned int avb_mii_tx_er_pins[] = {
> + RCAR_GP_PIN(5, 23),
> +};
> +static const unsigned int avb_mii_tx_er_mux[] = {
> + AVB_TX_ER_MARK,
> +};
> +static const unsigned int avb_gmii_tx_rx_pins[] = {
> + RCAR_GP_PIN(4, 1), RCAR_GP_PIN(3, 11), RCAR_GP_PIN(3, 12),
> + RCAR_GP_PIN(3, 14), RCAR_GP_PIN(3, 15), RCAR_GP_PIN(3, 16),
> + RCAR_GP_PIN(3, 27), RCAR_GP_PIN(3, 28), RCAR_GP_PIN(3, 29),
> + RCAR_GP_PIN(4, 0), RCAR_GP_PIN(5, 22), RCAR_GP_PIN(3, 13),
> + RCAR_GP_PIN(5, 23),
> +
> + RCAR_GP_PIN(3, 0), RCAR_GP_PIN(3, 2), RCAR_GP_PIN(3, 3),
> + RCAR_GP_PIN(3, 4), RCAR_GP_PIN(3, 5), RCAR_GP_PIN(3, 6),
> + RCAR_GP_PIN(3, 7), RCAR_GP_PIN(3, 8), RCAR_GP_PIN(3, 9),
> + RCAR_GP_PIN(3, 1), RCAR_GP_PIN(3, 10),
> +};
> +static const unsigned int avb_gmii_tx_rx_mux[] = {
> + AVB_GTX_CLK_MARK, AVB_GTXREFCLK_MARK, AVB_TX_CLK_MARK, AVB_TXD0_MARK,
> + AVB_TXD1_MARK, AVB_TXD2_MARK, AVB_TXD3_MARK, AVB_TXD4_MARK,
> + AVB_TXD5_MARK, AVB_TXD6_MARK, AVB_TXD7_MARK, AVB_TX_EN_MARK,
> + AVB_TX_ER_MARK,
> +
> + AVB_RX_CLK_MARK, AVB_RXD0_MARK, AVB_RXD1_MARK, AVB_RXD2_MARK,
> + AVB_RXD3_MARK, AVB_RXD4_MARK, AVB_RXD5_MARK, AVB_RXD6_MARK,
> + AVB_RXD7_MARK, AVB_RX_DV_MARK, AVB_RX_ER_MARK,
> +};
> +static const unsigned int avb_avtp_match_a_pins[] = {
> + RCAR_GP_PIN(1, 15),
> +};
> +static const unsigned int avb_avtp_match_a_mux[] = {
> + AVB_AVTP_MATCH_A_MARK,
> +};
> +static const unsigned int avb_avtp_capture_a_pins[] = {
> + RCAR_GP_PIN(1, 14),
> +};
> +static const unsigned int avb_avtp_capture_a_mux[] = {
> + AVB_AVTP_CAPTURE_A_MARK,
> +};
> +static const unsigned int avb_avtp_match_b_pins[] = {
> + RCAR_GP_PIN(5, 20),
> +};
> +static const unsigned int avb_avtp_match_b_mux[] = {
> + AVB_AVTP_MATCH_B_MARK,
> +};
> +static const unsigned int avb_avtp_capture_b_pins[] = {
> + RCAR_GP_PIN(5, 19),
> +};
> +static const unsigned int avb_avtp_capture_b_mux[] = {
> + AVB_AVTP_CAPTURE_B_MARK,
> +};
> /* - MMC -------------------------------------------------------------------- */
> static const unsigned int mmc_data1_pins[] = {
> /* D0 */
> @@ -1370,6 +1474,19 @@ static const unsigned int scif_clk_b_mux[] = {
> };
>
> static const struct sh_pfc_pin_group pinmux_groups[] = {
> + SH_PFC_PIN_GROUP(avb_col),
> + SH_PFC_PIN_GROUP(avb_crs),
> + SH_PFC_PIN_GROUP(avb_link),
> + SH_PFC_PIN_GROUP(avb_magic),
> + SH_PFC_PIN_GROUP(avb_phy_int),
> + SH_PFC_PIN_GROUP(avb_mdio),
> + SH_PFC_PIN_GROUP(avb_mii_tx_rx),
> + SH_PFC_PIN_GROUP(avb_mii_tx_er),
> + SH_PFC_PIN_GROUP(avb_gmii_tx_rx),
> + SH_PFC_PIN_GROUP(avb_avtp_match_a),
> + SH_PFC_PIN_GROUP(avb_avtp_capture_a),
> + SH_PFC_PIN_GROUP(avb_avtp_match_b),
> + SH_PFC_PIN_GROUP(avb_avtp_capture_b),
> SH_PFC_PIN_GROUP(mmc_data1),
> SH_PFC_PIN_GROUP(mmc_data4),
> SH_PFC_PIN_GROUP(mmc_data8),
> @@ -1409,6 +1526,21 @@ static const struct sh_pfc_pin_group pinmux_groups[] = {
> SH_PFC_PIN_GROUP(scif_clk_b),
> };
>
> +static const char * const avb_groups[] = {
> + "avb_col",
> + "avb_crs",
> + "avb_link",
> + "avb_magic",
> + "avb_phy_int",
> + "avb_mdio",
> + "avb_mii_tx_rx",
> + "avb_mii_tx_er",
> + "avb_gmii_tx_rx",
> + "avb_avtp_match_a",
> + "avb_avtp_capture_a",
> + "avb_avtp_match_b",
> + "avb_avtp_capture_b",
> +};
> static const char * const mmc_groups[] = {
> "mmc_data1",
> "mmc_data4",
> @@ -1471,6 +1603,7 @@ static const char * const scif_clk_groups[] = {
> };
>
> static const struct sh_pfc_function pinmux_functions[] = {
> + SH_PFC_FUNCTION(avb),
> SH_PFC_FUNCTION(mmc),
> SH_PFC_FUNCTION(scif0),
> SH_PFC_FUNCTION(scif1),
> --
> 2.7.4
>
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] pinctrl: sh-pfc: r8a77470: Add EtherAVB pin groups
2018-08-02 11:30 ` Simon Horman
@ 2018-08-02 11:34 ` Geert Uytterhoeven
2018-08-09 11:01 ` Simon Horman
0 siblings, 1 reply; 6+ messages in thread
From: Geert Uytterhoeven @ 2018-08-02 11:34 UTC (permalink / raw)
To: Simon Horman
Cc: Biju Das, Laurent Pinchart, Geert Uytterhoeven, Linus Walleij,
Linux-Renesas, open list:GPIO SUBSYSTEM, Chris Paterson,
Fabrizio Castro
Hi Simon,
On Thu, Aug 2, 2018 at 1:30 PM Simon Horman <horms@verge.net.au> 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@bp.renesas.com>
> > Reviewed-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com>
> > ---
> > This patch is based on the following discussion
> > https://www.mail-archive.com/linux-renesas-soc@vger.kernel.org/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@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] 6+ messages in thread* Re: [PATCH] pinctrl: sh-pfc: r8a77470: Add EtherAVB pin groups
2018-08-02 11:34 ` Geert Uytterhoeven
@ 2018-08-09 11:01 ` Simon Horman
0 siblings, 0 replies; 6+ messages in thread
From: Simon Horman @ 2018-08-09 11:01 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Biju Das, Laurent Pinchart, Geert Uytterhoeven, Linus Walleij,
Linux-Renesas, open list:GPIO SUBSYSTEM, Chris Paterson,
Fabrizio Castro
On Thu, Aug 02, 2018 at 01:34:48PM +0200, Geert Uytterhoeven wrote:
> Hi Simon,
>
> On Thu, Aug 2, 2018 at 1:30 PM Simon Horman <horms@verge.net.au> 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@bp.renesas.com>
> > > Reviewed-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com>
> > > ---
> > > This patch is based on the following discussion
> > > https://www.mail-archive.com/linux-renesas-soc@vger.kernel.org/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!).
Thanks, that is the obvious thing that I was missing.
Reviewed-by: Simon Horman <horms+renesas@verge.net.au>
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2018-08-09 13:26 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-07-27 9:22 [PATCH] pinctrl: sh-pfc: r8a77470: Add EtherAVB pin groups Biju Das
2018-07-27 9:29 ` Biju Das
2018-07-30 15:01 ` Geert Uytterhoeven
2018-08-02 11:30 ` Simon Horman
2018-08-02 11:34 ` Geert Uytterhoeven
2018-08-09 11:01 ` Simon Horman
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.