Hi, On Sat, Nov 25, 2017 at 01:08:30PM +0800, Chen-Yu Tsai wrote: > The Cubietruck Plus has an RTL8211E PHY connected to the EMAC using > RGMII. The PHY is powered by DLDO4 @ 3.3V, while the I/O pins are > powered by DLDO3 @ 2.5V. > > This patch adds a U-boot specific dtsi file for the board adding > an enabled EMAC node, and enables the EMAC driver in the defconfig. > The binding used here is the old revision currently supported in > U-boot. The U-boot driver has not been updated to support the new > binding. > > Signed-off-by: Chen-Yu Tsai <w...@csie.org> > --- > .../arm/dts/sun8i-a83t-cubietruck-plus-u-boot.dtsi | 38 > ++++++++++++++++++++++ > configs/Cubietruck_plus_defconfig | 2 ++ > 2 files changed, 40 insertions(+) > create mode 100644 arch/arm/dts/sun8i-a83t-cubietruck-plus-u-boot.dtsi > > diff --git a/arch/arm/dts/sun8i-a83t-cubietruck-plus-u-boot.dtsi > b/arch/arm/dts/sun8i-a83t-cubietruck-plus-u-boot.dtsi > new file mode 100644 > index 000000000000..4637e128f76e > --- /dev/null > +++ b/arch/arm/dts/sun8i-a83t-cubietruck-plus-u-boot.dtsi > @@ -0,0 +1,38 @@ > +#include "sunxi-u-boot.dtsi" > + > +/ { > + aliases { > + ethernet0 = &emac; > + }; > + > + soc { > + emac: ethernet@01c30000 { > + compatible = "allwinner,sun8i-a83t-emac"; > + reg = <0x01c30000 0x2000>, <0x01c00030 0x4>; > + reg-names = "emac", "syscon"; > + interrupts = <GIC_SPI 82 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + pinctrl-names = "default"; > + pinctrl-0 = <&rgmii_pins>; > + phy-mode = "rgmii"; > + phy = <&phy1>; > + status = "okay"; > + > + phy1: ethernet-phy@1 { > + reg = <1>; > + }; > + }; > + }; > +};
Now that we have a binding merged, can't we use that instead? > +&pio { > + rgmii_pins: rgmii_pins { > + allwinner,pins = "PD2", "PD3", "PD4", "PD5", "PD6", "PD7", > + "PD11", "PD12", "PD13", "PD14", "PD18", > + "PD19", "PD21", "PD22", "PD23"; > + allwinner,function = "emac"; > + allwinner,drive = <3>; > + allwinner,pull = <0>; > + }; > +}; You should also use the new bindings here. Thanks! Maxime -- Maxime Ripard, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com
signature.asc
Description: PGP signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot