On 16/02/16 16:10, Vladimir Murzin wrote: > On 16/02/16 11:01, Arnd Bergmann wrote: >> On Tuesday 16 February 2016 10:08:14 Vladimir Murzin wrote: >>> + >>> + ethernet@40200000 { >>> + compatible = "smsc,lan9220", "smsc,lan9115"; >>> + reg = <0x40200000 0x10000>; >>> + interrupts = <13>; >>> + interrupt-parent = <&nvic>; >>> + smsc,irq-active-high; >>> + }; >>> +}; >>> + >>> >> >> This node seems slightly misplaced. Is there some external bus interface >> that this is connected to? The address suggests that it should be somewhere >> below the /soc node, and you probably want to list the external bus >> interface with a "ranges" property that identifies the addresses visibile >> there, and put the external chip under there. >> > > I might messed it up since the MAC/PHY connects to the same 16-bit > interface as the 16MB PSRAM external memory and both connected via AHB. > > Not sure how it should be expressed, so some help form DT camp would be > appreciated. Arnd, Does following fixup address your point on where/how ethernet node should be placed? diff --git a/arch/arm/boot/dts/mps2-an385.dts b/arch/arm/boot/dts/an385.dts index 976f86d..50c8d24 100644 --- a/arch/arm/boot/dts/mps2-an385.dts +++ b/arch/arm/boot/dts/mps2-an385.dts @@ -63,14 +63,10 @@ device_type = "memory"; reg = <0x21000000 0x1000000>; }; +}; - ethernet@40200000 { - compatible = "smsc,lan9220", "smsc,lan9115"; - reg = <0x40200000 0x10000>; - interrupts = <13>; - interrupt-parent = <&nvic>; - smsc,irq-active-high; - }; +&mb { + ranges = <0 0x40200000 0x10000>; }; &uart0 { diff --git a/arch/arm/boot/dts/mps2.dtsi b/arch/arm/boot/dts/mps2.dtsi index 5d2c539..1673472 100644 --- a/arch/arm/boot/dts/mps2.dtsi +++ b/arch/arm/boot/dts/mps2.dtsi @@ -223,5 +223,21 @@ default-state = "off"; }; }; + + }; + + mb: smb { + compatible = "simple-bus"; + #address-cells = <1>; + #size-cells = <1>; + ranges; + + ethernet@0 { + compatible = "smsc,lan9220", "smsc,lan9115"; + reg = <0x0 0x10000>; + interrupts = <13>; + interrupt-parent = <&nvic>; + smsc,irq-active-high; + }; }; }; Cheers Vladimir > > Cheers > Vladimir > >> Arnd >> >> >> > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@xxxxxxxxxxxxxxxxxxx > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel > > > -- 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