On Mon, Nov 27, 2017 at 4:09 PM, Maxime Ripard maxime.ripard@free-electrons.com wrote:
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 wens@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?
That is the next step. It's also why I have these in a U-boot specific dtsi file. The driver is upgraded to support the now official bindings, then we sync up the dts files and rip out the u-boot stuff.
However I'm not sure when I'll get to that, and I'd like this feature to be available for people that need might it, such as the kernelci.org board farms.
+&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.
See above.
ChenYu