netdev - RE: [EXT] Re: [PATCH net-next v3 7/7] arm64: dts: add serdes and mdio description

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <AM0PR04MB5443841EE53D93C686AB1E4AFB6E0@AM0PR04MB5443.eurprd04.prod.outlook.com>
Date:   Mon, 29 Jun 2020 12:32:45 +0000
From:   Florinel Iordache <florinel.iordache@....com>
To:     Florian Fainelli <f.fainelli@...il.com>,
        "davem@...emloft.net" <davem@...emloft.net>,
        "netdev@...r.kernel.org" <netdev@...r.kernel.org>,
        "andrew@...n.ch" <andrew@...n.ch>,
        "hkallweit1@...il.com" <hkallweit1@...il.com>,
        "linux@...linux.org.uk" <linux@...linux.org.uk>
CC:     "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        "linux-doc@...r.kernel.org" <linux-doc@...r.kernel.org>,
        "robh+dt@...nel.org" <robh+dt@...nel.org>,
        "mark.rutland@....com" <mark.rutland@....com>,
        "kuba@...nel.org" <kuba@...nel.org>,
        "corbet@....net" <corbet@....net>,
        "shawnguo@...nel.org" <shawnguo@...nel.org>,
        Leo Li <leoyang.li@....com>,
        "Madalin Bucur (OSS)" <madalin.bucur@....nxp.com>,
        Ioana Ciornei <ioana.ciornei@....com>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        Florinel Iordache <florinel.iordache@....com>
Subject: RE: [EXT] Re: [PATCH net-next v3 7/7] arm64: dts: add serdes and mdio
 description


> -----Original Message-----
> From: Florian Fainelli <f.fainelli@...il.com>
> Sent: Friday, June 26, 2020 10:08 PM
> To: Florinel Iordache <florinel.iordache@....com>; davem@...emloft.net;
> netdev@...r.kernel.org; andrew@...n.ch; hkallweit1@...il.com;
> linux@...linux.org.uk
> Cc: devicetree@...r.kernel.org; linux-doc@...r.kernel.org;
> robh+dt@...nel.org; mark.rutland@....com; kuba@...nel.org;
> corbet@....net; shawnguo@...nel.org; Leo Li <leoyang.li@....com>; Madalin
> Bucur (OSS) <madalin.bucur@....nxp.com>; Ioana Ciornei
> <ioana.ciornei@....com>; linux-kernel@...r.kernel.org
> Subject: [EXT] Re: [PATCH net-next v3 7/7] arm64: dts: add serdes and mdio
> description
> 
> Caution: EXT Email
> 
> On 6/22/20 6:35 AM, Florinel Iordache wrote:
> > Add dt nodes with serdes, lanes, mdio generic description for
> > supported
> > platform: ls1046. This is a prerequisite to enable backplane on device
> > tree for these platforms.
> >
> > Signed-off-by: Florinel Iordache <florinel.iordache@....com>
> > ---
> 
> [snip]
> 
> > @@ -21,7 +22,7 @@ fman@...0000 {
> >               fsl,fman-10g-port;
> >       };
> >
> > -     ethernet@...00 {
> > +     mac10: ethernet@...00 {
> >               cell-index = <0x9>;
> >               compatible = "fsl,fman-memac";
> >               reg = <0xf2000 0x1000>;
> > @@ -29,7 +30,7 @@ fman@...0000 {
> >               pcsphy-handle = <&pcsphy7>;
> >       };
> >
> > -     mdio@...00 {
> > +     mdio10: mdio@...00 {
> >               #address-cells = <1>;
> >               #size-cells = <0>;
> >               compatible = "fsl,fman-memac-mdio", "fsl,fman-xmdio";
> 
> That part of the patch does not seem to have any users, there is nothing in your
> patch series that references mac9, mdio9 or mac10, mdio10 AFAICT. Maybe you
> can split it up and send it aside from this series?
> --
> Florian

Hi Florian,

These labels are used in the platform DTS (e.g.: fsl-ls1046a-rdb.dts for DPAA1 / LS1046).
They are not used in the patch because backplane mode is NOT the default mode.
Backplane mode is enabled by users for custom boards by editing the DTS.
Interfaces must be used by default with a PHY like: phy-handle = <&aqr106_phy>
How to enable backplane mode in DTS is described in devicetree bindings file: ethernet-phy.yaml
(which includes an example at the end).

Here is an example of how the labels are used to enable backplane mode:
&mdio9 {
	pcsphy6: ethernet-phy@0 {
		compatible = "ethernet-phy-ieee802.3-c45";
		eq-algorithm = "bee";
		/* 10G Short cables setup: up to 30 cm cable */
		eq-init = <0x2 0x29 0x5>;
		eq-params = <0>;
		reg = <0x0>;
		lane-handle = <&lane_d>;   /* lane D */
	};
};
/* Update MAC connections to backplane PHYs */
&mac9 {
	phy-connection-type = "10gbase-kr";
	phy-handle = <&pcsphy6>;
};

However, the users could write DTS by using ethernet@...00 instead of &mac9.
Something like this:
ethernet@...00 { /* 10GEC1 */
	phy-connection-type = "10gbase-kr";
	phy-handle = <&pcsphy6>;
};
(and similar for mdio9/mdio10)
So ultimately that part of the patch could be removed indeed.
I put it there just to be more user friendly to enable backplane in DTS.

Thank you for feedback.
Florinel.