diff mbox series

[net-next,1/2] net: phy: add genphy_c45_loopback

Message ID 20210323164641.26059-2-vee.khee.wong@linux.intel.com
State New
Headers show
Series Add support for Clause-45 PHY Loopback | expand

Commit Message

Wong Vee Khee March 23, 2021, 4:46 p.m. UTC
Add generic code to enable C45 PHY loopback into the common phy-c45.c
file. This will allow C45 PHY drivers aceess this by setting
.set_loopback.

Suggested-by: Heiner Kallweit <hkallweit1@gmail.com>
Signed-off-by: Wong Vee Khee <vee.khee.wong@linux.intel.com>
---
 drivers/net/phy/phy-c45.c | 8 ++++++++
 include/linux/phy.h       | 1 +
 2 files changed, 9 insertions(+)

Comments

Heiner Kallweit March 24, 2021, 11:02 a.m. UTC | #1
On 23.03.2021 17:46, Wong Vee Khee wrote:
> Add generic code to enable C45 PHY loopback into the common phy-c45.c

> file. This will allow C45 PHY drivers aceess this by setting

> .set_loopback.

> 

> Suggested-by: Heiner Kallweit <hkallweit1@gmail.com>

> Signed-off-by: Wong Vee Khee <vee.khee.wong@linux.intel.com>

> ---

>  drivers/net/phy/phy-c45.c | 8 ++++++++

>  include/linux/phy.h       | 1 +

>  2 files changed, 9 insertions(+)

> 


LGTM

Reviewed-by: Heiner Kallweit <hkallweit1@gmail.com>
Andrew Lunn March 24, 2021, 10:04 p.m. UTC | #2
On Wed, Mar 24, 2021 at 12:46:40AM +0800, Wong Vee Khee wrote:
> Add generic code to enable C45 PHY loopback into the common phy-c45.c

> file. This will allow C45 PHY drivers aceess this by setting

> .set_loopback.

> 

> Suggested-by: Heiner Kallweit <hkallweit1@gmail.com>

> Signed-off-by: Wong Vee Khee <vee.khee.wong@linux.intel.com>


Reviewed-by: Andrew Lunn <andrew@lunn.ch>


    Andrew
diff mbox series

Patch

diff --git a/drivers/net/phy/phy-c45.c b/drivers/net/phy/phy-c45.c
index 077f2929c45e..91e3acb9e397 100644
--- a/drivers/net/phy/phy-c45.c
+++ b/drivers/net/phy/phy-c45.c
@@ -560,6 +560,14 @@  int gen10g_config_aneg(struct phy_device *phydev)
 }
 EXPORT_SYMBOL_GPL(gen10g_config_aneg);
 
+int genphy_c45_loopback(struct phy_device *phydev, bool enable)
+{
+	return phy_modify_mmd(phydev, MDIO_MMD_PCS, MDIO_CTRL1,
+			      MDIO_PCS_CTRL1_LOOPBACK,
+			      enable ? MDIO_PCS_CTRL1_LOOPBACK : 0);
+}
+EXPORT_SYMBOL_GPL(genphy_c45_loopback);
+
 struct phy_driver genphy_c45_driver = {
 	.phy_id         = 0xffffffff,
 	.phy_id_mask    = 0xffffffff,
diff --git a/include/linux/phy.h b/include/linux/phy.h
index 1a12e4436b5b..8e2cf84b2318 100644
--- a/include/linux/phy.h
+++ b/include/linux/phy.h
@@ -1532,6 +1532,7 @@  int genphy_c45_read_mdix(struct phy_device *phydev);
 int genphy_c45_pma_read_abilities(struct phy_device *phydev);
 int genphy_c45_read_status(struct phy_device *phydev);
 int genphy_c45_config_aneg(struct phy_device *phydev);
+int genphy_c45_loopback(struct phy_device *phydev, bool enable);
 
 /* Generic C45 PHY driver */
 extern struct phy_driver genphy_c45_driver;