linux-kernel - RE: [EXT] Re: [PATCH v6 4/5] dts: marvell: Enable 10G interfaces on 9130-DB and 9131-DB boards

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: <CO6PR18MB44170F8D0403F98C664AF95ECA199@CO6PR18MB4417.namprd18.prod.outlook.com>
Date:   Thu, 8 Jul 2021 12:09:48 +0000
From:   Kostya Porotchkin <kostap@...vell.com>
To:     Stefan Chulski <stefanc@...vell.com>,
        Russell King <linux@...linux.org.uk>
CC:     "miquel.raynal@...tlin.com" <miquel.raynal@...tlin.com>,
        "kishon@...com" <kishon@...com>,
        "vkoul@...nel.org" <vkoul@...nel.org>,
        "robh+dt@...nel.org" <robh+dt@...nel.org>,
        "andrew@...n.ch" <andrew@...n.ch>,
        "gregory.clement@...tlin.com" <gregory.clement@...tlin.com>,
        "sebastian.hesselbarth@...il.com" <sebastian.hesselbarth@...il.com>,
        "vladimir.vid@...tura.hr" <vladimir.vid@...tura.hr>,
        "luka.kovacic@...tura.hr" <luka.kovacic@...tura.hr>,
        "linux-phy@...ts.infradead.org" <linux-phy@...ts.infradead.org>,
        "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "linux-arm-kernel@...ts.infradead.org" 
        <linux-arm-kernel@...ts.infradead.org>,
        "mw@...ihalf.com" <mw@...ihalf.com>,
        "jaz@...ihalf.com" <jaz@...ihalf.com>,
        Nadav Haklai <nadavh@...vell.com>,
        Ben Peled <bpeled@...vell.com>
Subject: RE: [EXT] Re: [PATCH v6 4/5] dts: marvell: Enable 10G interfaces on
 9130-DB and 9131-DB boards



> -----Original Message-----
> From: Stefan Chulski <stefanc@...vell.com>
> Sent: Thursday, July 8, 2021 14:50
> To: Russell King <linux@...linux.org.uk>; Kostya Porotchkin
> <kostap@...vell.com>
> Cc: miquel.raynal@...tlin.com; kishon@...com; vkoul@...nel.org;
> robh+dt@...nel.org; andrew@...n.ch; gregory.clement@...tlin.com;
> sebastian.hesselbarth@...il.com; vladimir.vid@...tura.hr;
> luka.kovacic@...tura.hr; linux-phy@...ts.infradead.org;
> devicetree@...r.kernel.org; linux-kernel@...r.kernel.org; linux-arm-
> kernel@...ts.infradead.org; mw@...ihalf.com; jaz@...ihalf.com; Nadav
> Haklai <nadavh@...vell.com>; Ben Peled <bpeled@...vell.com>
> Subject: RE: [EXT] Re: [PATCH v6 4/5] dts: marvell: Enable 10G interfaces on
> 9130-DB and 9131-DB boards
> 
> > External Email
> >
> > ----------------------------------------------------------------------
> > On Thu, Jul 08, 2021 at 02:25:27PM +0300, kostap@...vell.com wrote:
> > > From: Stefan Chulski <stefanc@...vell.com>
> > >
> > > This patch enables eth0 10G interface on CN9130-DB paltforms and
> > > eth0 10G and eth3 10G interfaces on CN9131-DB.
> > >
> > > Signed-off-by: Stefan Chulski <stefanc@...vell.com>
> > > Signed-off-by: Konstantin Porotchkin <kostap@...vell.com>
> > > ---
> > >  arch/arm64/boot/dts/marvell/cn9130-db.dtsi | 2 +-
> > > arch/arm64/boot/dts/marvell/cn9131-db.dtsi | 2 +-
> > >  2 files changed, 2 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/arch/arm64/boot/dts/marvell/cn9130-db.dtsi
> > > b/arch/arm64/boot/dts/marvell/cn9130-db.dtsi
> > > index 34274e061958..39fc90716454 100644
> > > --- a/arch/arm64/boot/dts/marvell/cn9130-db.dtsi
> > > +++ b/arch/arm64/boot/dts/marvell/cn9130-db.dtsi
> > > @@ -125,7 +125,7 @@
> > >
> > >  /* SLM-1521-V2, CON9 */
> > >  &cp0_eth0 {
> > > -	status = "disabled";
> > > +	status = "okay";
> > >  	phy-mode = "10gbase-kr";
> >
> > Are these really 10gbase-kr? Or should this be 10gbase-r ?
> 
> Should be 10gbase-r. By default we do not support full KR protocol.

[KP] So I will add a separate patch to this series changing "10gbase-kr" to "10gbase-r" in all CN91xx and A8K DTS.
Will it be OK?
Kosta
> 
> Regards.