mail archive of the barebox mailing list
 help / color / mirror / Atom feed
* [PATCH] ARM: dts: imx6: phycore: Fix pin settings for ADIN1300 PHY
@ 2026-10-01 10:43 Florijan Plohl
  2026-10-01 20:30 ` Sascha Hauer
  2026-10-01 20:31 ` Sascha Hauer
  0 siblings, 2 replies; 3+ messages in thread
From: Florijan Plohl @ 2026-10-01 10:43 UTC (permalink / raw)
  To: s.hauer, barebox; +Cc: upstream

From: Stefan Riedmueller <s.riedmueller@phytec.de>

Change drive strength:
 - TXC 		57 Ohm
 - TD0-3	76 Ohm
 - TX_CTL	76 Ohm

Remove pull ups from following pins so the won't interfere with strap
pin latching of the PHY:
 - ENET_TX_CLK
 - RXC
 - RD0-3
 - RX_CTL

Signed-off-by: Stefan Riedmueller <s.riedmueller@phytec.de>
Signed-off-by: Florijan Plohl <florijan.plohl@norik.com>
---
 arch/arm/dts/imx6qdl-phytec-phycore-som.dtsi | 24 ++++++++++++++++++++
 1 file changed, 24 insertions(+)

diff --git a/arch/arm/dts/imx6qdl-phytec-phycore-som.dtsi b/arch/arm/dts/imx6qdl-phytec-phycore-som.dtsi
index 60f7e2c14b..7199c416c8 100644
--- a/arch/arm/dts/imx6qdl-phytec-phycore-som.dtsi
+++ b/arch/arm/dts/imx6qdl-phytec-phycore-som.dtsi
@@ -79,6 +79,30 @@ pmic@58 {
 	};
 };
 
+&iomuxc {
+	pinctrl_enet: enetgrp {
+		fsl,pins = <
+			MX6QDL_PAD_ENET_MDIO__ENET_MDIO		0x1b0b0
+			MX6QDL_PAD_ENET_MDC__ENET_MDC		0x1b0b0
+			MX6QDL_PAD_RGMII_TXC__RGMII_TXC		0x1b020
+			MX6QDL_PAD_RGMII_TD0__RGMII_TD0		0x1b018
+			MX6QDL_PAD_RGMII_TD1__RGMII_TD1		0x1b018
+			MX6QDL_PAD_RGMII_TD2__RGMII_TD2		0x1b018
+			MX6QDL_PAD_RGMII_TD3__RGMII_TD3		0x1b018
+			MX6QDL_PAD_RGMII_TX_CTL__RGMII_TX_CTL	0x1b018
+			MX6QDL_PAD_ENET_REF_CLK__ENET_TX_CLK	0x180b0
+			MX6QDL_PAD_RGMII_RXC__RGMII_RXC		0x180b0
+			MX6QDL_PAD_RGMII_RD0__RGMII_RD0		0x180b0
+			MX6QDL_PAD_RGMII_RD1__RGMII_RD1		0x180b0
+			MX6QDL_PAD_RGMII_RD2__RGMII_RD2		0x180b0
+			MX6QDL_PAD_RGMII_RD3__RGMII_RD3		0x180b0
+			MX6QDL_PAD_RGMII_RX_CTL__RGMII_RX_CTL	0x180b0
+			MX6QDL_PAD_ENET_TX_EN__ENET_TX_EN	0x1b0b0
+			MX6QDL_PAD_SD2_DAT1__GPIO1_IO14		0x1b0b0
+		>;
+	};
+};
+
 &m25p80 {
 	status = "disabled";
 
-- 
2.43.0




^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] ARM: dts: imx6: phycore: Fix pin settings for ADIN1300 PHY
  2026-10-01 10:43 [PATCH] ARM: dts: imx6: phycore: Fix pin settings for ADIN1300 PHY Florijan Plohl
@ 2026-10-01 20:30 ` Sascha Hauer
  2026-10-01 20:31 ` Sascha Hauer
  1 sibling, 0 replies; 3+ messages in thread
From: Sascha Hauer @ 2026-10-01 20:30 UTC (permalink / raw)
  To: barebox, Florijan Plohl; +Cc: upstream


On Thu, 01 Oct 2026 12:43:09 +0200, Florijan Plohl wrote:
> Change drive strength:
>  - TXC 		57 Ohm
>  - TD0-3	76 Ohm
>  - TX_CTL	76 Ohm
> 
> Remove pull ups from following pins so the won't interfere with strap
> pin latching of the PHY:
>  - ENET_TX_CLK
>  - RXC
>  - RD0-3
>  - RX_CTL
> 
> [...]

Applied, thanks!

[1/1] ARM: dts: imx6: phycore: Fix pin settings for ADIN1300 PHY
      https://git.pengutronix.de/cgit/barebox/commit/?id=4f8473e272dc (link may not be stable)

Best regards,
-- 
Sascha Hauer <s.hauer@pengutronix.de>




^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] ARM: dts: imx6: phycore: Fix pin settings for ADIN1300 PHY
  2026-10-01 10:43 [PATCH] ARM: dts: imx6: phycore: Fix pin settings for ADIN1300 PHY Florijan Plohl
  2026-10-01 20:30 ` Sascha Hauer
@ 2026-10-01 20:31 ` Sascha Hauer
  1 sibling, 0 replies; 3+ messages in thread
From: Sascha Hauer @ 2026-10-01 20:31 UTC (permalink / raw)
  To: Florijan Plohl; +Cc: barebox, upstream

On 2026-10-01 12:43, Florijan Plohl wrote:
> From: Stefan Riedmueller <s.riedmueller@phytec.de>
> 
> Change drive strength:
>  - TXC 		57 Ohm
>  - TD0-3	76 Ohm
>  - TX_CTL	76 Ohm
> 
> Remove pull ups from following pins so the won't interfere with strap
> pin latching of the PHY:
>  - ENET_TX_CLK
>  - RXC
>  - RD0-3
>  - RX_CTL
> 
> Signed-off-by: Stefan Riedmueller <s.riedmueller@phytec.de>
> Signed-off-by: Florijan Plohl <florijan.plohl@norik.com>
> ---
>  arch/arm/dts/imx6qdl-phytec-phycore-som.dtsi | 24 ++++++++++++++++++++
>  1 file changed, 24 insertions(+)
> 
> diff --git a/arch/arm/dts/imx6qdl-phytec-phycore-som.dtsi b/arch/arm/dts/imx6qdl-phytec-phycore-som.dtsi
> index 60f7e2c14b..7199c416c8 100644
> --- a/arch/arm/dts/imx6qdl-phytec-phycore-som.dtsi
> +++ b/arch/arm/dts/imx6qdl-phytec-phycore-som.dtsi
> @@ -79,6 +79,30 @@ pmic@58 {
>  	};
>  };
>  
> +&iomuxc {
> +	pinctrl_enet: enetgrp {
> +		fsl,pins = <
> +			MX6QDL_PAD_ENET_MDIO__ENET_MDIO		0x1b0b0
> +			MX6QDL_PAD_ENET_MDC__ENET_MDC		0x1b0b0
> +			MX6QDL_PAD_RGMII_TXC__RGMII_TXC		0x1b020
> +			MX6QDL_PAD_RGMII_TD0__RGMII_TD0		0x1b018
> +			MX6QDL_PAD_RGMII_TD1__RGMII_TD1		0x1b018
> +			MX6QDL_PAD_RGMII_TD2__RGMII_TD2		0x1b018
> +			MX6QDL_PAD_RGMII_TD3__RGMII_TD3		0x1b018
> +			MX6QDL_PAD_RGMII_TX_CTL__RGMII_TX_CTL	0x1b018
> +			MX6QDL_PAD_ENET_REF_CLK__ENET_TX_CLK	0x180b0
> +			MX6QDL_PAD_RGMII_RXC__RGMII_RXC		0x180b0
> +			MX6QDL_PAD_RGMII_RD0__RGMII_RD0		0x180b0
> +			MX6QDL_PAD_RGMII_RD1__RGMII_RD1		0x180b0
> +			MX6QDL_PAD_RGMII_RD2__RGMII_RD2		0x180b0
> +			MX6QDL_PAD_RGMII_RD3__RGMII_RD3		0x180b0
> +			MX6QDL_PAD_RGMII_RX_CTL__RGMII_RX_CTL	0x180b0
> +			MX6QDL_PAD_ENET_TX_EN__ENET_TX_EN	0x1b0b0
> +			MX6QDL_PAD_SD2_DAT1__GPIO1_IO14		0x1b0b0
> +		>;
> +	};
> +};

Fine with me, so I applied it, but have you considered sending it for
kernel inclusion as well? Otherwise the values will be reverted back
under Linux when you don't start with the barebox device tree.

Sascha

--
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |




^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2026-10-01 20:33 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-10-01 10:43 [PATCH] ARM: dts: imx6: phycore: Fix pin settings for ADIN1300 PHY Florijan Plohl
2026-10-01 20:30 ` Sascha Hauer
2026-10-01 20:31 ` Sascha Hauer

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox