? 2018?7?26? GMT+08:00 ??10:06:44, Andre Przywara [off-list ref] ??:

Hi,

On 26/07/18 14:41, Icenowy Zheng wrote:

quoted

? 2018?7?26? GMT+08:00 ??9:39:56, Sergey Suloev

[off-list ref] ??:

quoted

quoted

Hi,

On 07/26/2018 03:35 AM, Andre Przywara wrote:

quoted

The NanoPi-A64 has the usual Realtek Gbit PHY connected to the

EMAC,

quoted

quoted

quoted

so add the respective nodes to the DT. The PHY is powered by the
VDD_SYS_3.3V line, which is always on.

Signed-off-by: Andre Przywara <andre.przywara@arm.com>
---
  arch/arm64/boot/dts/allwinner/sun50i-a64-nanopi-a64.dts | 17

+++++++++++++++++

quoted

  1 file changed, 17 insertions(+)

diff --git

a/arch/arm64/boot/dts/allwinner/sun50i-a64-nanopi-a64.dts

quoted

quoted

b/arch/arm64/boot/dts/allwinner/sun50i-a64-nanopi-a64.dts

quoted

index 5caba225b4f7..bd35a093e6cd 100644

--- a/arch/arm64/boot/dts/allwinner/sun50i-a64-nanopi-a64.dts
+++ b/arch/arm64/boot/dts/allwinner/sun50i-a64-nanopi-a64.dts

@@ -51,6 +51,7 @@
  	compatible = "friendlyarm,nanopi-a64", "allwinner,sun50i-a64";
  
  	aliases {
+		ethernet0 = &emac;
  		serial0 = &uart0;
  	};
  

@@ -67,6 +68,15 @@
  	status = "okay";
  };
  
+&emac {
+	pinctrl-names = "default";
+	pinctrl-0 = <&rgmii_pins>;
+	phy-mode = "rgmii";
+	phy-handle = <&ext_rgmii_phy>;
+	phy-supply = <&reg_dcdc1>;
+	status = "okay";
+};
+
  /* i2c1 connected with gpio headers like pine64, bananapi */
  &i2c1 {
  	pinctrl-names = "default";

@@ -78,6 +88,13 @@
  	bias-pull-up;
  };
  
+&mdio {
+	ext_rgmii_phy: ethernet-phy at 1 {
+		compatible = "ethernet-phy-ieee802.3-c22";
+		reg = <1>;
+	};
+};
+
  &mmc0 {
  	pinctrl-names = "default";
  	pinctrl-0 = <&mmc0_pins>;

did you test? "reg = <1>" on a real hardware ?

No, thanks for pointing this out.

According to the PHY's datasheet the PHY address is configured by the
three pins also used for the LEDs. By looking at the schematics back
then I somehow managed to convince myself that it should be 1, as most
other boards use, but from looking again it indeed looks more like 7
(the other LED pin connected to 3.3V).

Can you please try with reg = <7> and confirm that it works that way?

In my memory many nanopi's use 7.

Many thanks,
Andre.

quoted

quoted

I have? "reg = <0>" in my dts? and? it has been working on my board

for

quoted

0 is a wildcard address.

For nano pi, if 1 doesn't work, then it should be 7. 0 can work,
but it won't be accepted by DT maintainers.

quoted

ages.

Thanks

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel at lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel