lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-ID: <CADaLNDnizBfxXcy8_W2WQ29X_s3FxOsU+CGV_O63ES_4zSLZcg@mail.gmail.com> Date: Tue, 2 Aug 2016 12:10:28 -0700 From: Duc Dang <dhdang@....com> To: Olof Johansson <olof@...om.net> Cc: Iyappan Subramanian <isubramanian@....com>, David Miller <davem@...emloft.net>, Network Development <netdev@...r.kernel.org>, "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>, "linux-arm-kernel@...ts.infradead.org" <linux-arm-kernel@...ts.infradead.org>, patches <patches@....com>, Matthias Brugger <matthias.bgg@...il.com>, Andrew Lunn <andrew@...n.ch>, Russell King <linux@...linux.org.uk> Subject: Re: [PATCH v5 09/11] dtb: xgene: Add MDIO node On Mon, Aug 1, 2016 at 2:13 PM, Olof Johansson <olof@...om.net> wrote: > > Hi, > > I came across this now when looking at merge conflicts when preparing > this. I wish I had seen it sooner. > > On Thu, Jul 7, 2016 at 4:02 PM, Iyappan Subramanian > <isubramanian@....com> wrote: > > Added mdio node for mdio driver. Also added phy-handle > > reference to the ethernet nodes. > > > > Removed unused mdio subnode within storm menet ethernet node. > > Removed unused clock node from storm sgenet1. > > > > Signed-off-by: Iyappan Subramanian <isubramanian@....com> > > Tested-by: Fushen Chen <fchen@....com> > > Tested-by: Toan Le <toanle@....com> > > Tested-by: Matthias Brugger <mbrugger@...e.com> > > --- > > arch/arm64/boot/dts/apm/apm-mustang.dts | 12 +++++++++++ > > arch/arm64/boot/dts/apm/apm-storm.dtsi | 36 ++++++++++++--------------------- > > 2 files changed, 25 insertions(+), 23 deletions(-) > > > > diff --git a/arch/arm64/boot/dts/apm/apm-mustang.dts b/arch/arm64/boot/dts/apm/apm-mustang.dts > > index 44db32e..b7fb5d9 100644 > > --- a/arch/arm64/boot/dts/apm/apm-mustang.dts > > +++ b/arch/arm64/boot/dts/apm/apm-mustang.dts > > @@ -79,3 +79,15 @@ > > &mmc0 { > > status = "ok"; > > }; > > + > > +&mdio { > > + menet0phy: phy@3 { > > + reg = <0x3>; > > + }; > > + sgenet0phy: phy@4 { > > + reg = <0x4>; > > + }; > > + sgenet1phy: phy@5 { > > + reg = <0x5>; > > + }; > > +}; > > diff --git a/arch/arm64/boot/dts/apm/apm-storm.dtsi b/arch/arm64/boot/dts/apm/apm-storm.dtsi > > index 5147d76..a415b3a 100644 > > --- a/arch/arm64/boot/dts/apm/apm-storm.dtsi > > +++ b/arch/arm64/boot/dts/apm/apm-storm.dtsi > > @@ -237,20 +237,11 @@ > > clocks = <&socplldiv2 0>; > > reg = <0x0 0x1f21c000 0x0 0x1000>; > > reg-names = "csr-reg"; > > - csr-mask = <0x3>; > > + csr-mask = <0xa>; > > + enable-mask = <0xf>; > > clock-output-names = "sge0clk"; > > }; > > > > - sge1clk: sge1clk@...1c000 { > > - compatible = "apm,xgene-device-clock"; > > - #clock-cells = <1>; > > - clocks = <&socplldiv2 0>; > > - reg = <0x0 0x1f21c000 0x0 0x1000>; > > - reg-names = "csr-reg"; > > - csr-mask = <0xc>; > > - clock-output-names = "sge1clk"; > > - }; > > - > > xge0clk: xge0clk@...1c000 { > > compatible = "apm,xgene-device-clock"; > > #clock-cells = <1>; > > @@ -921,6 +912,14 @@ > > clocks = <&rtcclk 0>; > > }; > > > > + mdio: mdio@...7020000 { > > + compatible = "apm,xgene-mdio-rgmii"; > > + #address-cells = <1>; > > + #size-cells = <0>; > > + reg = <0x0 0x17020000 0x0 0xd100>; > > + clocks = <&menetclk 0>; > > + }; > > + > > menet: ethernet@...20000 { > > > This new mmio node has the same unit address, and claims the same > register window as the ethernet controller. > That's doesn't seem correct? Hi Olof, Unfortunately, Iyappan is on vacation. He will respond as soon as he gets back. Regards, Duc Dang. > > > -Olof
Powered by blists - more mailing lists