On 24.03.21 16:47, Pali Rohár wrote:
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 <[email protected]>
Sent: Wednesday, March 24, 2021 16:43
To: Stefan Roese <[email protected]>
Cc: [email protected]; Nadav Haklai <[email protected]>; Kostya
Porotchkin <[email protected]>; Marek Behún <[email protected]>;
Grzegorz Jaszczyk <[email protected]>; Igal Liberman <[email protected]>;
Baruch Siach <[email protected]>
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).
Okay, done. I've send v2 of only this patch.
Thanks,
Stefan
Pali, thanks for spotting this.
Thanks,
Stefan
commit 9869490d13d8b47b3464e1de04bb82b7a3f39fd0
Author: Ken Ma <[email protected]>
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 <[email protected]>
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: [email protected]
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: [email protected]
