Hello. On 4/25/2017 12:50 AM, Eric Anholt wrote:
Cygnus has a single amac controller connected to the B53 switch with 2 PHYs. On the BCM911360_EP platform, those two PHYs are connected to the external ethernet jacks.
My spell checker trips on "amac" and "ethernet" -- perhaps they need capitalization?
Signed-off-by: Eric Anholt <eric@xxxxxxxxxx> --- arch/arm/boot/dts/bcm-cygnus.dtsi | 60 ++++++++++++++++++++++++++++++++++ arch/arm/boot/dts/bcm911360_entphn.dts | 8 +++++ 2 files changed, 68 insertions(+) diff --git a/arch/arm/boot/dts/bcm-cygnus.dtsi b/arch/arm/boot/dts/bcm-cygnus.dtsi index 009f1346b817..318899df9972 100644 --- a/arch/arm/boot/dts/bcm-cygnus.dtsi +++ b/arch/arm/boot/dts/bcm-cygnus.dtsi @@ -142,6 +142,56 @@ interrupts = <0>; }; + mdio: mdio@18002000 { + compatible = "brcm,iproc-mdio"; + reg = <0x18002000 0x8>; + #size-cells = <1>; + #address-cells = <0>; + + gphy0: eth-gphy@0 {
The node anmes must be generic, the DT spec has standardized "ethernet-phy" name for this case.
+ reg = <0>; + max-speed = <1000>; + }; + + gphy1: eth-gphy@1 { + reg = <1>; + max-speed = <1000>; + }; + };
[...]
@@ -295,6 +345,16 @@ status = "disabled"; }; + eth0: enet@18042000 { + compatible = "brcm,amac"; + reg = <0x18042000 0x1000>, + <0x18110000 0x1000>; + reg-names = "amac_base", "idm_base";
I don't think "_base" suffixes are necessary here. [...] MBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html