[PATCH v1] ARM: protonic: vicut1: disable on-die termination to fix PHY bootstrapping

[PATCH v1] ARM: protonic: vicut1: disable on-die termination to fix PHY bootstrapping

[Oleksij Rempel]

If on-die termination is enabled, the RXC pin of iMX6 will be pulled
high. Since we already have an 10K pull-down on board, the RXC level on
PHY reset will be ~800mV, which is mostly interpreted as 1. On some
reboots we get 0 instead and kernel can't detect the PHY properly.

Since the default 0x020e07ac value is 0, it is sufficient to remove this
entry from the affected imxcfg files.

Since we get stable 0 on pin PHYADDR[2], the PHY address is changed from
4 to 0.

Reported-by: Robin van der Gracht <robin@protonic.nl>
Fixes: 00adc1e33ef8 ("ARM: add imx6 based Protonic boads")
Signed-off-by: Oleksij Rempel <o.rempel@pengutronix.de>
---
 arch/arm/boards/protonic-imx6/flash-header-vicut1.imxcfg | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm/boards/protonic-imx6/flash-header-vicut1.imxcfg b/arch/arm/boards/protonic-imx6/flash-header-vicut1.imxcfg
index 9926fbf4a2..54f655c4c2 100644
--- a/arch/arm/boards/protonic-imx6/flash-header-vicut1.imxcfg
+++ b/arch/arm/boards/protonic-imx6/flash-header-vicut1.imxcfg
@@ -114,7 +114,6 @@ wm 32 0x020e001c 0x007f007f
 
 /* RGMII config */
 wm 32 0x020e0768 0x00080000 /* 1V2 DDR IO */
-wm 32 0x020e0788 0x00000200 /* 60 Ohm ODT */
 
 /* Debug */
 wm 32 0x020e023c 0x00000005
-- 
2.29.2


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: [PATCH v1] ARM: protonic: vicut1: disable on-die termination to fix PHY bootstrapping

[Sascha Hauer]

On Tue, May 18, 2021 at 10:37:07AM +0200, Oleksij Rempel wrote:
> If on-die termination is enabled, the RXC pin of iMX6 will be pulled
> high. Since we already have an 10K pull-down on board, the RXC level on
> PHY reset will be ~800mV, which is mostly interpreted as 1. On some
> reboots we get 0 instead and kernel can't detect the PHY properly.
> 
> Since the default 0x020e07ac value is 0, it is sufficient to remove this
> entry from the affected imxcfg files.
> 
> Since we get stable 0 on pin PHYADDR[2], the PHY address is changed from
> 4 to 0.
> 
> Reported-by: Robin van der Gracht <robin@protonic.nl>
> Fixes: 00adc1e33ef8 ("ARM: add imx6 based Protonic boads")
> Signed-off-by: Oleksij Rempel <o.rempel@pengutronix.de>
> ---
>  arch/arm/boards/protonic-imx6/flash-header-vicut1.imxcfg | 1 -
>  1 file changed, 1 deletion(-)

Applied, thanks

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 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox