* [PATCH] net: phy: dp83867: Clean force link good bit
@ 2020-02-06 15:03 Michal Simek
2020-02-10 12:07 ` Grygorii Strashko
2020-02-28 11:17 ` Michal Simek
0 siblings, 2 replies; 3+ messages in thread
From: Michal Simek @ 2020-02-06 15:03 UTC (permalink / raw)
To: u-boot
On Xilinx ZynqMP revA board initial value of PHYCR register is 0x5448 which
means FORCE_LINK_GOOD is already setup. Origin code was doing write but the
new code is doing read/modify/write and keep this bit untouched. That's why
ethernet stop to work.
The patch is cleaning this bit when PHYCR value is composed.
Tested on Xilinx zcu102-revA and zcu104-rev1.0 boards.
Fixes: 37d6265f2bfa ("net: phy: dp83867: refactor rgmii configuration")
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---
drivers/net/phy/dp83867.c | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/drivers/net/phy/dp83867.c b/drivers/net/phy/dp83867.c
index a43793cd4274..4d796e289c45 100644
--- a/drivers/net/phy/dp83867.c
+++ b/drivers/net/phy/dp83867.c
@@ -64,6 +64,7 @@
#define DP83867_PHYCR_FIFO_DEPTH_SHIFT 14
#define DP83867_PHYCR_FIFO_DEPTH_MASK GENMASK(15, 14)
#define DP83867_PHYCR_RESERVED_MASK BIT(11)
+#define DP83867_PHYCR_FORCE_LINK_GOOD BIT(10)
#define DP83867_MDI_CROSSOVER 5
#define DP83867_MDI_CROSSOVER_MDIX 2
#define DP83867_PHYCTRL_SGMIIEN 0x0800
@@ -283,6 +284,9 @@ static int dp83867_config(struct phy_device *phydev)
val &= ~DP83867_PHYCR_FIFO_DEPTH_MASK;
val |= (dp83867->fifo_depth << DP83867_PHYCR_FIFO_DEPTH_SHIFT);
+ /* Do not force link good */
+ val &= ~DP83867_PHYCR_FORCE_LINK_GOOD;
+
/* The code below checks if "port mirroring" N/A MODE4 has been
* enabled during power on bootstrap.
*
--
2.25.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH] net: phy: dp83867: Clean force link good bit
2020-02-06 15:03 [PATCH] net: phy: dp83867: Clean force link good bit Michal Simek
@ 2020-02-10 12:07 ` Grygorii Strashko
2020-02-28 11:17 ` Michal Simek
1 sibling, 0 replies; 3+ messages in thread
From: Grygorii Strashko @ 2020-02-10 12:07 UTC (permalink / raw)
To: u-boot
On 06/02/2020 17:03, Michal Simek wrote:
> On Xilinx ZynqMP revA board initial value of PHYCR register is 0x5448 which
> means FORCE_LINK_GOOD is already setup. Origin code was doing write but the
> new code is doing read/modify/write and keep this bit untouched. That's why
> ethernet stop to work.
> The patch is cleaning this bit when PHYCR value is composed.
>
> Tested on Xilinx zcu102-revA and zcu104-rev1.0 boards.
>
> Fixes: 37d6265f2bfa ("net: phy: dp83867: refactor rgmii configuration")
> Signed-off-by: Michal Simek <michal.simek@xilinx.com>
> ---
Reviewed-by: Grygorii Strashko <grygorii.strashko@ti.com>
>
> drivers/net/phy/dp83867.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/drivers/net/phy/dp83867.c b/drivers/net/phy/dp83867.c
> index a43793cd4274..4d796e289c45 100644
> --- a/drivers/net/phy/dp83867.c
> +++ b/drivers/net/phy/dp83867.c
> @@ -64,6 +64,7 @@
> #define DP83867_PHYCR_FIFO_DEPTH_SHIFT 14
> #define DP83867_PHYCR_FIFO_DEPTH_MASK GENMASK(15, 14)
> #define DP83867_PHYCR_RESERVED_MASK BIT(11)
> +#define DP83867_PHYCR_FORCE_LINK_GOOD BIT(10)
> #define DP83867_MDI_CROSSOVER 5
> #define DP83867_MDI_CROSSOVER_MDIX 2
> #define DP83867_PHYCTRL_SGMIIEN 0x0800
> @@ -283,6 +284,9 @@ static int dp83867_config(struct phy_device *phydev)
> val &= ~DP83867_PHYCR_FIFO_DEPTH_MASK;
> val |= (dp83867->fifo_depth << DP83867_PHYCR_FIFO_DEPTH_SHIFT);
>
> + /* Do not force link good */
> + val &= ~DP83867_PHYCR_FORCE_LINK_GOOD;
> +
> /* The code below checks if "port mirroring" N/A MODE4 has been
> * enabled during power on bootstrap.
> *
>
--
Best regards,
grygorii
^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH] net: phy: dp83867: Clean force link good bit
2020-02-06 15:03 [PATCH] net: phy: dp83867: Clean force link good bit Michal Simek
2020-02-10 12:07 ` Grygorii Strashko
@ 2020-02-28 11:17 ` Michal Simek
1 sibling, 0 replies; 3+ messages in thread
From: Michal Simek @ 2020-02-28 11:17 UTC (permalink / raw)
To: u-boot
?t 6. 2. 2020 v 16:03 odes?latel Michal Simek <michal.simek@xilinx.com> napsal:
>
> On Xilinx ZynqMP revA board initial value of PHYCR register is 0x5448 which
> means FORCE_LINK_GOOD is already setup. Origin code was doing write but the
> new code is doing read/modify/write and keep this bit untouched. That's why
> ethernet stop to work.
> The patch is cleaning this bit when PHYCR value is composed.
>
> Tested on Xilinx zcu102-revA and zcu104-rev1.0 boards.
>
> Fixes: 37d6265f2bfa ("net: phy: dp83867: refactor rgmii configuration")
> Signed-off-by: Michal Simek <michal.simek@xilinx.com>
> ---
>
> drivers/net/phy/dp83867.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/drivers/net/phy/dp83867.c b/drivers/net/phy/dp83867.c
> index a43793cd4274..4d796e289c45 100644
> --- a/drivers/net/phy/dp83867.c
> +++ b/drivers/net/phy/dp83867.c
> @@ -64,6 +64,7 @@
> #define DP83867_PHYCR_FIFO_DEPTH_SHIFT 14
> #define DP83867_PHYCR_FIFO_DEPTH_MASK GENMASK(15, 14)
> #define DP83867_PHYCR_RESERVED_MASK BIT(11)
> +#define DP83867_PHYCR_FORCE_LINK_GOOD BIT(10)
> #define DP83867_MDI_CROSSOVER 5
> #define DP83867_MDI_CROSSOVER_MDIX 2
> #define DP83867_PHYCTRL_SGMIIEN 0x0800
> @@ -283,6 +284,9 @@ static int dp83867_config(struct phy_device *phydev)
> val &= ~DP83867_PHYCR_FIFO_DEPTH_MASK;
> val |= (dp83867->fifo_depth << DP83867_PHYCR_FIFO_DEPTH_SHIFT);
>
> + /* Do not force link good */
> + val &= ~DP83867_PHYCR_FORCE_LINK_GOOD;
> +
> /* The code below checks if "port mirroring" N/A MODE4 has been
> * enabled during power on bootstrap.
> *
> --
> 2.25.0
>
Applied.
M
--
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Xilinx Microblaze
Maintainer of Linux kernel - Xilinx Zynq ARM and ZynqMP ARM64 SoCs
U-Boot custodian - Xilinx Microblaze/Zynq/ZynqMP/Versal SoCs
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-02-28 11:17 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-02-06 15:03 [PATCH] net: phy: dp83867: Clean force link good bit Michal Simek
2020-02-10 12:07 ` Grygorii Strashko
2020-02-28 11:17 ` Michal Simek
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.