Hi Stefan,
You should also put Jagan Teki as recipients.
On Wed, Sep 27, 2017 at 07:19:10AM +0000, Stefan Mavrodiev wrote:
From revision J the board uses new phy chip LAN8710. Compared with RTL8201, RA17 pin is TXERR. It has pullup which causes phy not to work. To fix this PA17 is muxed with GMAC function. This makes the pin output-low.
Signed-off-by: Stefan Mavrodiev stefan@olimex.com
arch/arm/dts/sun7i-a20-olinuxino-micro.dts | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/arch/arm/dts/sun7i-a20-olinuxino-micro.dts b/arch/arm/dts/sun7i-a20-olinuxino-micro.dts index 7e3006f..0bc5ea9 100644 --- a/arch/arm/dts/sun7i-a20-olinuxino-micro.dts +++ b/arch/arm/dts/sun7i-a20-olinuxino-micro.dts @@ -95,7 +95,7 @@
&gmac { pinctrl-names = "default";
- pinctrl-0 = <&gmac_pins_mii_a>;
- pinctrl-0 = <&gmac_pins_mii_a>,<&gmac_txerr>; phy = <&phy1>; phy-mode = "mii"; status = "okay";
@@ -226,6 +226,13 @@ };
&pio {
- gmac_txerr: gmac_txerr@0 {
allwinner,pins = "PA17";allwinner,function = "gpio_out";allwinner,drive = <SUN4I_PINCTRL_10_MA>;allwinner,pull = <SUN4I_PINCTRL_NO_PULL>;- };
There's two issues with that patch: - In general we try to sync the DT with Linux when we can, and the patch that ended up in Linux wasn't this one. - And we don't have any pinctrl support in U-Boot, that patch will basically do nothing
The code that changes the muxing is here: http://git.denx.de/?p=u-boot.git;a=blob;f=board/sunxi/gmac.c#l34
Maxime