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 <ste...@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

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

Attachment: signature.asc
Description: PGP signature

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to