On Wednesday 24 March 2021 16:39:22 Stefan Roese wrote: > On 24.03.21 16:04, Kostya Porotchkin wrote: > > Hi, Pali, > > > > > -----Original Message----- > > > From: Pali Rohár <p...@kernel.org> > > > Sent: Wednesday, March 24, 2021 16:43 > > > To: Stefan Roese <s...@denx.de> > > > Cc: u-boot@lists.denx.de; Nadav Haklai <nad...@marvell.com>; Kostya > > > Porotchkin <kos...@marvell.com>; Marek Behún <marek.be...@nic.cz>; > > > Grzegorz Jaszczyk <j...@semihalf.com>; Igal Liberman <ig...@marvell.com>; > > > Baruch Siach <bar...@tkos.co.il> > > > Subject: [EXT] Re: [PATCH v1 02/23] phy: marvell: rename comphy related > > > definitions to COMPHY_XX > > > > > > External Email > > > > > > ---------------------------------------------------------------------- > > > On Wednesday 24 March 2021 15:06:28 Stefan Roese wrote: > > > > diff --git a/arch/arm/dts/armada-3720-espressobin.dts > > > b/arch/arm/dts/armada-3720-espressobin.dts > > > > index 96a4b3d95b88..3cea5218d2b3 100644 > > > > --- a/arch/arm/dts/armada-3720-espressobin.dts > > > > +++ b/arch/arm/dts/armada-3720-espressobin.dts > > > > @@ -83,18 +83,18 @@ > > > > &comphy { > > > > max-lanes = <3>; > > > > phy0 { > > > > - phy-type = <PHY_TYPE_USB3_HOST0>; > > > > - phy-speed = <PHY_SPEED_5G>; > > > > + phy-type = <COMPHY_TYPE_PEX0>; > > > > + phy-speed = <COMPHY_SPEED_2_5G>; > > > > > > Why you are changing type USB to type PCIe? Possible typo? > > [KP] This patch is older that the one that fixed A3720 SERDES assignment. > > So you are right, this change is wrong today and I missed it. > > > > Stefan, we should either take into account the below patch, or fix this one. > > Sure. I'll either fix this in v2 and add some follow-up patch to > fix this.
Just squash fixup patch into 02/23, so code in u-boot git would be working at every commit (very useful for git bisect). > Pali, thanks for spotting this. > > Thanks, > Stefan > > > commit 9869490d13d8b47b3464e1de04bb82b7a3f39fd0 > > Author: Ken Ma <m...@marvell.com> > > Date: Fri May 12 10:55:56 2017 +0800 > > > > arm64: dts: a37xx: update comphy lane numbers > > The latest change in the comphy_a3700.c driver aligned > > the lanes assignment to the functional specification. > > Reflect that in supported device trees. > > Change-Id: Id7a239bd00e35e66b4e5023db081bb417267b8ca > > Signed-off-by: Ken Ma <m...@marvell.com> > > > > diff --git a/arch/arm/dts/armada-3720-db.dts > > b/arch/arm/dts/armada-3720-db.dts > > index 169828cbc4..8ee3bdf533 100644 > > --- a/arch/arm/dts/armada-3720-db.dts > > +++ b/arch/arm/dts/armada-3720-db.dts > > @@ -101,13 +101,13 @@ > > &comphy { > > phy0 { > > - phy-type = <COMPHY_TYPE_PEX0>; > > - phy-speed = <COMPHY_SPEED_2_5G>; > > + phy-type = <COMPHY_TYPE_USB3>; > > + phy-speed = <COMPHY_SPEED_5G>; > > }; > > phy1 { > > - phy-type = <COMPHY_TYPE_USB3>; > > - phy-speed = <COMPHY_SPEED_5G>; > > + phy-type = <COMPHY_TYPE_PEX0>; > > + phy-speed = <COMPHY_SPEED_2_5G>; > > }; > > phy2 { > > diff --git a/arch/arm/dts/armada-3720-espressobin.dts > > b/arch/arm/dts/armada-3720-espressobin.dts > > index f8fd4ed244..94cab2f4d1 100644 > > --- a/arch/arm/dts/armada-3720-espressobin.dts > > +++ b/arch/arm/dts/armada-3720-espressobin.dts > > @@ -72,13 +72,13 @@ > > &comphy { > > max-lanes = <3>; > > phy0 { > > - phy-type = <COMPHY_TYPE_PEX0>; > > - phy-speed = <COMPHY_SPEED_2_5G>; > > + phy-type = <COMPHY_TYPE_USB3>; > > + phy-speed = <COMPHY_SPEED_5G>; > > }; > > phy1 { > > - phy-type = <COMPHY_TYPE_USB3>; > > - phy-speed = <COMPHY_SPEED_5G>; > > + phy-type = <COMPHY_TYPE_PEX0>; > > + phy-speed = <COMPHY_SPEED_2_5G>; > > }; > > phy2 { > > > > > > > > > }; > > > > > > > > phy1 { > > > > - phy-type = <PHY_TYPE_PEX0>; > > > > - phy-speed = <PHY_SPEED_2_5G>; > > > > + phy-type = <COMPHY_TYPE_USB3_HOST0>; > > > > + phy-speed = <COMPHY_SPEED_5G>; > > > > > > Same here. PCIe is changed to USB. > > [KP] Yes, need to be fixed, thanks, Pali. > > > > Regards > > Kosta > > > > > > > }; > > > > > > > > phy2 { > > > > - phy-type = <PHY_TYPE_SATA0>; > > > > - phy-speed = <PHY_SPEED_5G>; > > > > + phy-type = <COMPHY_TYPE_SATA0>; > > > > + phy-speed = <COMPHY_SPEED_5G>; > > > > }; > > > > }; > > > Viele Grüße, > Stefan > > -- > DENX Software Engineering GmbH, Managing Director: Wolfgang Denk > HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany > Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: s...@denx.de