All checks were successful
target_linux_generic / Update target_linux_generic (openwrt-25.12) (push) Successful in 8s
41 lines
1.4 KiB
Diff
41 lines
1.4 KiB
Diff
From bcbbfb4f62c4ba35783cc617997a2e92d91e3940 Mon Sep 17 00:00:00 2001
|
|
From: "SkyLake.Huang" <skylake.huang@mediatek.com>
|
|
Date: Thu, 17 Oct 2024 11:22:13 +0800
|
|
Subject: [PATCH 03/20] net: phy: mediatek-ge-soc: Propagate error code
|
|
correctly in cal_cycle()
|
|
|
|
This patch propagates error code correctly in cal_cycle()
|
|
and improve with FIELD_GET().
|
|
|
|
Signed-off-by: SkyLake.Huang <skylake.huang@mediatek.com>
|
|
Reviewed-by: Simon Horman <horms@kernel.org>
|
|
Signed-off-by: Andrew Lunn <andrew@lunn.ch>
|
|
---
|
|
drivers/net/phy/mediatek-ge-soc.c | 8 +++++---
|
|
1 file changed, 5 insertions(+), 3 deletions(-)
|
|
|
|
--- a/drivers/net/phy/mediatek-ge-soc.c
|
|
+++ b/drivers/net/phy/mediatek-ge-soc.c
|
|
@@ -110,7 +110,7 @@
|
|
#define MTK_PHY_CR_TX_AMP_OFFSET_D_MASK GENMASK(6, 0)
|
|
|
|
#define MTK_PHY_RG_AD_CAL_COMP 0x17a
|
|
-#define MTK_PHY_AD_CAL_COMP_OUT_SHIFT (8)
|
|
+#define MTK_PHY_AD_CAL_COMP_OUT_MASK GENMASK(8, 8)
|
|
|
|
#define MTK_PHY_RG_AD_CAL_CLK 0x17b
|
|
#define MTK_PHY_DA_CAL_CLK BIT(0)
|
|
@@ -351,8 +351,10 @@ static int cal_cycle(struct phy_device *
|
|
|
|
phy_clear_bits_mmd(phydev, MDIO_MMD_VEND1, MTK_PHY_RG_AD_CALIN,
|
|
MTK_PHY_DA_CALIN_FLAG);
|
|
- ret = phy_read_mmd(phydev, MDIO_MMD_VEND1, MTK_PHY_RG_AD_CAL_COMP) >>
|
|
- MTK_PHY_AD_CAL_COMP_OUT_SHIFT;
|
|
+ ret = phy_read_mmd(phydev, MDIO_MMD_VEND1, MTK_PHY_RG_AD_CAL_COMP);
|
|
+ if (ret < 0)
|
|
+ return ret;
|
|
+ ret = FIELD_GET(MTK_PHY_AD_CAL_COMP_OUT_MASK, ret);
|
|
phydev_dbg(phydev, "cal_val: 0x%x, ret: %d\n", cal_val, ret);
|
|
|
|
return ret;
|