linux-kernel - Re: [PATCH 5/9] arm64: dts: ls2088ardb: add PHY nodes for the CS4340 PHYs

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20201030094025.cudb2ezjaqxledt4@skbuf>
Date:   Fri, 30 Oct 2020 09:40:27 +0000
From:   Ioana Ciornei <ioana.ciornei@....com>
To:     Shawn Guo <shawnguo@...nel.org>
CC:     "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 5/9] arm64: dts: ls2088ardb: add PHY nodes for the CS4340
 PHYs

On Fri, Oct 30, 2020 at 08:10:38AM +0800, Shawn Guo wrote:
> On Fri, Sep 25, 2020 at 04:24:59PM +0300, Ioana Ciornei wrote:
> > Annotate the EMDIO1 node and describe the 4 10GBASER PHYs found on the
> > LS2088ARDB board. Also, add phy-handles for DPMACs 1-4 to their
> > associated PHY.
> > 
> > Signed-off-by: Ioana Ciornei <ioana.ciornei@....com>
> > ---
> >  .../boot/dts/freescale/fsl-ls2088a-rdb.dts    | 41 +++++++++++++++++++
> >  1 file changed, 41 insertions(+)
> > 
> > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls2088a-rdb.dts b/arch/arm64/boot/dts/freescale/fsl-ls2088a-rdb.dts
> > index f6b4d75a258b..0d1935fcd201 100644
> > --- a/arch/arm64/boot/dts/freescale/fsl-ls2088a-rdb.dts
> > +++ b/arch/arm64/boot/dts/freescale/fsl-ls2088a-rdb.dts
> > @@ -22,3 +22,44 @@ chosen {
> >  		stdout-path = "serial1:115200n8";
> >  	};
> >  };
> > +
> > +&dpmac1 {
> > +	phy-handle = <&mdio1_phy1>;
> > +};
> > +
> > +&dpmac2 {
> > +	phy-handle = <&mdio1_phy2>;
> > +};
> > +
> > +&dpmac3 {
> > +	phy-handle = <&mdio1_phy3>;
> > +};
> > +
> > +&dpmac4 {
> > +	phy-handle = <&mdio1_phy4>;
> > +};
> > +
> > +&emdio1 {
> > +	status = "okay";
> > +
> > +	mdio1_phy1: emdio1_phy@1 {
> > +		compatible = "ethernet-phy-id13e5.1002";
> > +		reg = <0x10>;
> > +		phy-connection-type = "10gbase-r";
> > +	};
> 
> Please have a newline between nodes.
> 

Sure.