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

[Kostya Porotchkin]

> -----Original Message-----
> From: Stefan Chulski <stefanc@xxxxxxxxxxx>
> Sent: Thursday, July 8, 2021 14:50
> To: Russell King <linux@xxxxxxxxxxxxxxx>; Kostya Porotchkin
> <kostap@xxxxxxxxxxx>
> Cc: miquel.raynal@xxxxxxxxxxx; kishon@xxxxxx; vkoul@xxxxxxxxxx;
> robh+dt@xxxxxxxxxx; andrew@xxxxxxx; gregory.clement@xxxxxxxxxxx;
> sebastian.hesselbarth@xxxxxxxxx; vladimir.vid@xxxxxxxxxx;
> luka.kovacic@xxxxxxxxxx; linux-phy@xxxxxxxxxxxxxxxxxxx;
> devicetree@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; linux-arm-
> kernel@xxxxxxxxxxxxxxxxxxx; mw@xxxxxxxxxxxx; jaz@xxxxxxxxxxxx; Nadav
> Haklai <nadavh@xxxxxxxxxxx>; Ben Peled <bpeled@xxxxxxxxxxx>
> 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@xxxxxxxxxxx wrote:
> > > From: Stefan Chulski <stefanc@xxxxxxxxxxx>
> > >
> > > 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@xxxxxxxxxxx>
> > > Signed-off-by: Konstantin Porotchkin <kostap@xxxxxxxxxxx>
> > > ---
> > >  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.