* [PATCH] ARM: dts: imx7d-pinfunc: add input mux for ENET2 mdio
@ 2020-05-15 11:41 Steffen Trumtrar
2020-05-18 3:54 ` Aisheng Dong
2020-05-20 2:50 ` Shawn Guo
0 siblings, 2 replies; 5+ messages in thread
From: Steffen Trumtrar @ 2020-05-15 11:41 UTC (permalink / raw)
To: Shawn Guo
Cc: linux-arm-kernel, Rob Herring, Fabio Estevam, NXP Linux Team,
Pengutronix Kernel Team
Add the missing input mux for ENET2 mdio. Without this setting, it is not
possible to read the MDIO answers back from the PHY.
Signed-off-by: Steffen Trumtrar <s.trumtrar@pengutronix.de>
---
arch/arm/boot/dts/imx7d-pinfunc.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/imx7d-pinfunc.h b/arch/arm/boot/dts/imx7d-pinfunc.h
index 08ca1608fdb1..69f2c1ec8254 100644
--- a/arch/arm/boot/dts/imx7d-pinfunc.h
+++ b/arch/arm/boot/dts/imx7d-pinfunc.h
@@ -592,7 +592,7 @@
#define MX7D_PAD_UART2_RX_DATA__ECSPI1_SS3 0x0130 0x03A0 0x0000 0x3 0x0
#define MX7D_PAD_UART2_RX_DATA__ENET2_1588_EVENT1_IN 0x0130 0x03A0 0x0000 0x4 0x0
#define MX7D_PAD_UART2_RX_DATA__GPIO4_IO2 0x0130 0x03A0 0x0000 0x5 0x0
-#define MX7D_PAD_UART2_RX_DATA__ENET2_MDIO 0x0130 0x03A0 0x0000 0x6 0x0
+#define MX7D_PAD_UART2_RX_DATA__ENET2_MDIO 0x0130 0x03A0 0x0574 0x6 0x1
#define MX7D_PAD_UART2_TX_DATA__UART2_DCE_TX 0x0134 0x03A4 0x0000 0x0 0x0
#define MX7D_PAD_UART2_TX_DATA__UART2_DTE_RX 0x0134 0x03A4 0x06FC 0x0 0x3
#define MX7D_PAD_UART2_TX_DATA__I2C2_SDA 0x0134 0x03A4 0x05E0 0x1 0x0
--
2.26.2
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 5+ messages in thread
* RE: [PATCH] ARM: dts: imx7d-pinfunc: add input mux for ENET2 mdio
2020-05-15 11:41 [PATCH] ARM: dts: imx7d-pinfunc: add input mux for ENET2 mdio Steffen Trumtrar
@ 2020-05-18 3:54 ` Aisheng Dong
2020-05-18 4:35 ` Andy Duan
2020-05-20 2:50 ` Shawn Guo
1 sibling, 1 reply; 5+ messages in thread
From: Aisheng Dong @ 2020-05-18 3:54 UTC (permalink / raw)
To: Steffen Trumtrar, Shawn Guo, Andy Duan
Cc: linux-arm-kernel@lists.infradead.org, Rob Herring, Fabio Estevam,
dl-linux-imx, Pengutronix Kernel Team
> From: Steffen Trumtrar <s.trumtrar@pengutronix.de>
> Sent: Friday, May 15, 2020 7:42 PM
>
> Add the missing input mux for ENET2 mdio. Without this setting, it is not
> possible to read the MDIO answers back from the PHY.
>
> Signed-off-by: Steffen Trumtrar <s.trumtrar@pengutronix.de>
> ---
> arch/arm/boot/dts/imx7d-pinfunc.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/imx7d-pinfunc.h
> b/arch/arm/boot/dts/imx7d-pinfunc.h
> index 08ca1608fdb1..69f2c1ec8254 100644
> --- a/arch/arm/boot/dts/imx7d-pinfunc.h
> +++ b/arch/arm/boot/dts/imx7d-pinfunc.h
> @@ -592,7 +592,7 @@
> #define MX7D_PAD_UART2_RX_DATA__ECSPI1_SS3
> 0x0130 0x03A0 0x0000 0x3 0x0
> #define MX7D_PAD_UART2_RX_DATA__ENET2_1588_EVENT1_IN
> 0x0130 0x03A0 0x0000 0x4 0x0
> #define MX7D_PAD_UART2_RX_DATA__GPIO4_IO2
> 0x0130 0x03A0 0x0000 0x5 0x0
> -#define MX7D_PAD_UART2_RX_DATA__ENET2_MDIO
> 0x0130 0x03A0 0x0000 0x6 0x0
> +#define MX7D_PAD_UART2_RX_DATA__ENET2_MDIO
> 0x0130 0x03A0 0x0574 0x6 0x1
It's strange that I didn't find this select input setting from latest RM.
Anything I missed?
Also copy Andy to comment.
Regards
Aisheng
> #define MX7D_PAD_UART2_TX_DATA__UART2_DCE_TX
> 0x0134 0x03A4 0x0000 0x0 0x0
> #define MX7D_PAD_UART2_TX_DATA__UART2_DTE_RX
> 0x0134 0x03A4 0x06FC 0x0 0x3
> #define MX7D_PAD_UART2_TX_DATA__I2C2_SDA
> 0x0134 0x03A4 0x05E0 0x1 0x0
> --
> 2.26.2
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: [PATCH] ARM: dts: imx7d-pinfunc: add input mux for ENET2 mdio
2020-05-18 3:54 ` Aisheng Dong
@ 2020-05-18 4:35 ` Andy Duan
2020-05-18 6:08 ` Aisheng Dong
0 siblings, 1 reply; 5+ messages in thread
From: Andy Duan @ 2020-05-18 4:35 UTC (permalink / raw)
To: Aisheng Dong, Steffen Trumtrar, Shawn Guo
Cc: linux-arm-kernel@lists.infradead.org, Rob Herring, Fabio Estevam,
dl-linux-imx, Pengutronix Kernel Team
From: Aisheng Dong <aisheng.dong@nxp.com> Sent: Monday, May 18, 2020 11:55 AM
> > From: Steffen Trumtrar <s.trumtrar@pengutronix.de>
> > Sent: Friday, May 15, 2020 7:42 PM
> >
> > Add the missing input mux for ENET2 mdio. Without this setting, it is
> > not possible to read the MDIO answers back from the PHY.
> >
> > Signed-off-by: Steffen Trumtrar <s.trumtrar@pengutronix.de>
> > ---
> > arch/arm/boot/dts/imx7d-pinfunc.h | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/arch/arm/boot/dts/imx7d-pinfunc.h
> > b/arch/arm/boot/dts/imx7d-pinfunc.h
> > index 08ca1608fdb1..69f2c1ec8254 100644
> > --- a/arch/arm/boot/dts/imx7d-pinfunc.h
> > +++ b/arch/arm/boot/dts/imx7d-pinfunc.h
> > @@ -592,7 +592,7 @@
> > #define MX7D_PAD_UART2_RX_DATA__ECSPI1_SS3
> > 0x0130 0x03A0 0x0000 0x3 0x0
> > #define MX7D_PAD_UART2_RX_DATA__ENET2_1588_EVENT1_IN
> > 0x0130 0x03A0 0x0000 0x4 0x0
> > #define MX7D_PAD_UART2_RX_DATA__GPIO4_IO2
> > 0x0130 0x03A0 0x0000 0x5 0x0
> > -#define MX7D_PAD_UART2_RX_DATA__ENET2_MDIO
> > 0x0130 0x03A0 0x0000 0x6 0x0
> > +#define MX7D_PAD_UART2_RX_DATA__ENET2_MDIO
> > 0x0130 0x03A0 0x0574 0x6 0x1
>
> It's strange that I didn't find this select input setting from latest RM.
> Anything I missed?
>
> Also copy Andy to comment.
>
Check the RM in my local, it define the signal select input from UART2_RX_DATA,
it is correct with the change.
Aisheng, we can check together for our RM version.
> Regards
> Aisheng
>
> > #define MX7D_PAD_UART2_TX_DATA__UART2_DCE_TX
> > 0x0134 0x03A4 0x0000 0x0 0x0
> > #define MX7D_PAD_UART2_TX_DATA__UART2_DTE_RX
> > 0x0134 0x03A4 0x06FC 0x0 0x3
> > #define MX7D_PAD_UART2_TX_DATA__I2C2_SDA
> > 0x0134 0x03A4 0x05E0 0x1 0x0
> > --
> > 2.26.2
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: [PATCH] ARM: dts: imx7d-pinfunc: add input mux for ENET2 mdio
2020-05-18 4:35 ` Andy Duan
@ 2020-05-18 6:08 ` Aisheng Dong
0 siblings, 0 replies; 5+ messages in thread
From: Aisheng Dong @ 2020-05-18 6:08 UTC (permalink / raw)
To: Andy Duan, Steffen Trumtrar, Shawn Guo
Cc: linux-arm-kernel@lists.infradead.org, Rob Herring, Fabio Estevam,
dl-linux-imx, Pengutronix Kernel Team
> From: Andy Duan <fugang.duan@nxp.com>
> Sent: Monday, May 18, 2020 12:36 PM
>
> From: Aisheng Dong <aisheng.dong@nxp.com> Sent: Monday, May 18, 2020
> 11:55 AM
> > > From: Steffen Trumtrar <s.trumtrar@pengutronix.de>
> > > Sent: Friday, May 15, 2020 7:42 PM
> > >
> > > Add the missing input mux for ENET2 mdio. Without this setting, it
> > > is not possible to read the MDIO answers back from the PHY.
> > >
> > > Signed-off-by: Steffen Trumtrar <s.trumtrar@pengutronix.de>
> > > ---
> > > arch/arm/boot/dts/imx7d-pinfunc.h | 2 +-
> > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/arch/arm/boot/dts/imx7d-pinfunc.h
> > > b/arch/arm/boot/dts/imx7d-pinfunc.h
> > > index 08ca1608fdb1..69f2c1ec8254 100644
> > > --- a/arch/arm/boot/dts/imx7d-pinfunc.h
> > > +++ b/arch/arm/boot/dts/imx7d-pinfunc.h
> > > @@ -592,7 +592,7 @@
> > > #define MX7D_PAD_UART2_RX_DATA__ECSPI1_SS3
> > > 0x0130 0x03A0 0x0000 0x3 0x0
> > > #define MX7D_PAD_UART2_RX_DATA__ENET2_1588_EVENT1_IN
> > > 0x0130 0x03A0 0x0000 0x4 0x0
> > > #define MX7D_PAD_UART2_RX_DATA__GPIO4_IO2
> > > 0x0130 0x03A0 0x0000 0x5 0x0
> > > -#define MX7D_PAD_UART2_RX_DATA__ENET2_MDIO
> > > 0x0130 0x03A0 0x0000 0x6 0x0
> > > +#define MX7D_PAD_UART2_RX_DATA__ENET2_MDIO
> > > 0x0130 0x03A0 0x0574 0x6 0x1
> >
> > It's strange that I didn't find this select input setting from latest RM.
> > Anything I missed?
> >
> > Also copy Andy to comment.
> >
> Check the RM in my local, it define the signal select input from UART2_RX_DATA,
> it is correct with the change.
>
> Aisheng, we can check together for our RM version.
You're right. I checked the wrong place.
So for this patch:
Reviewed-by: Dong Aisheng <aisheng.dong@nxp.com>
Regards
Aisheng
> > Regards
> > Aisheng
> >
> > > #define MX7D_PAD_UART2_TX_DATA__UART2_DCE_TX
> > > 0x0134 0x03A4 0x0000 0x0 0x0
> > > #define MX7D_PAD_UART2_TX_DATA__UART2_DTE_RX
> > > 0x0134 0x03A4 0x06FC 0x0 0x3
> > > #define MX7D_PAD_UART2_TX_DATA__I2C2_SDA
> > > 0x0134 0x03A4 0x05E0 0x1 0x0
> > > --
> > > 2.26.2
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] ARM: dts: imx7d-pinfunc: add input mux for ENET2 mdio
2020-05-15 11:41 [PATCH] ARM: dts: imx7d-pinfunc: add input mux for ENET2 mdio Steffen Trumtrar
2020-05-18 3:54 ` Aisheng Dong
@ 2020-05-20 2:50 ` Shawn Guo
1 sibling, 0 replies; 5+ messages in thread
From: Shawn Guo @ 2020-05-20 2:50 UTC (permalink / raw)
To: Steffen Trumtrar
Cc: linux-arm-kernel, Rob Herring, Fabio Estevam, NXP Linux Team,
Pengutronix Kernel Team
On Fri, May 15, 2020 at 01:41:51PM +0200, Steffen Trumtrar wrote:
> Add the missing input mux for ENET2 mdio. Without this setting, it is not
> possible to read the MDIO answers back from the PHY.
>
> Signed-off-by: Steffen Trumtrar <s.trumtrar@pengutronix.de>
Applied, thanks.
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2020-05-20 2:50 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-05-15 11:41 [PATCH] ARM: dts: imx7d-pinfunc: add input mux for ENET2 mdio Steffen Trumtrar
2020-05-18 3:54 ` Aisheng Dong
2020-05-18 4:35 ` Andy Duan
2020-05-18 6:08 ` Aisheng Dong
2020-05-20 2:50 ` Shawn Guo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).