Re: [U-Boot] [PATCH u-boot] net: phy: realtek: Add functions to read PHY's extended registers

2019-01-22 Thread Joe Hershberger
On Wed, Jan 16, 2019 at 8:07 AM Carlo Caione  wrote:
>
> According to the datasheet to access the extended registers we have to:
>
> 1. Write Register 31 Data = 0x0XYZ (Page 0xXYZ)
> 2. Read/Write the target Register Data
> 3. Write Register 31 Data = 0x or 0xa42 (switch back to IEEE
>Standard Registers)
>
> Hook the missing functions so that we can use the `mdio rx/wx` command to
> easily access the extended registers.
>
> Signed-off-by: Carlo Caione 

Acked-by: Joe Hershberger 
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


[U-Boot] [PATCH u-boot] net: phy: realtek: Add functions to read PHY's extended registers

2019-01-16 Thread Carlo Caione
According to the datasheet to access the extended registers we have to:

1. Write Register 31 Data = 0x0XYZ (Page 0xXYZ)
2. Read/Write the target Register Data
3. Write Register 31 Data = 0x or 0xa42 (switch back to IEEE
   Standard Registers)

Hook the missing functions so that we can use the `mdio rx/wx` command to
easily access the extended registers.

Signed-off-by: Carlo Caione 
---
 drivers/net/phy/realtek.c | 27 +++
 1 file changed, 27 insertions(+)

diff --git a/drivers/net/phy/realtek.c b/drivers/net/phy/realtek.c
index b3e6578df9..3bea1fa9d0 100644
--- a/drivers/net/phy/realtek.c
+++ b/drivers/net/phy/realtek.c
@@ -57,6 +57,31 @@
 #define MIIM_RTL8211F_TX_DELAY 0x100
 #define MIIM_RTL8211F_LCR  0x10
 
+static int rtl8211f_phy_extread(struct phy_device *phydev, int addr,
+   int devaddr, int regnum)
+{
+   int oldpage = phy_read(phydev, MDIO_DEVAD_NONE, 
MIIM_RTL8211F_PAGE_SELECT);
+   int val;
+
+   phy_write(phydev, MDIO_DEVAD_NONE, MIIM_RTL8211F_PAGE_SELECT, devaddr);
+   val = phy_read(phydev, MDIO_DEVAD_NONE, regnum);
+   phy_write(phydev, MDIO_DEVAD_NONE, MIIM_RTL8211F_PAGE_SELECT, oldpage);
+
+   return val;
+}
+
+static int rtl8211f_phy_extwrite(struct phy_device *phydev, int addr,
+int devaddr, int regnum, u16 val)
+{
+   int oldpage = phy_read(phydev, MDIO_DEVAD_NONE, 
MIIM_RTL8211F_PAGE_SELECT);
+
+   phy_write(phydev, MDIO_DEVAD_NONE, MIIM_RTL8211F_PAGE_SELECT, devaddr);
+   phy_write(phydev, MDIO_DEVAD_NONE, regnum, val);
+   phy_write(phydev, MDIO_DEVAD_NONE, MIIM_RTL8211F_PAGE_SELECT, oldpage);
+
+   return 0;
+}
+
 static int rtl8211b_probe(struct phy_device *phydev)
 {
 #ifdef CONFIG_RTL8211X_PHY_FORCE_MASTER
@@ -336,6 +361,8 @@ static struct phy_driver RTL8211F_driver = {
.config = &rtl8211f_config,
.startup = &rtl8211f_startup,
.shutdown = &genphy_shutdown,
+   .readext = &rtl8211f_phy_extread,
+   .writeext = &rtl8211f_phy_extwrite,
 };
 
 int phy_realtek_init(void)
-- 
2.19.1

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot