net: macb: add support for C45 MDIO read/write
authorMilind Parab <mparab@cadence.com>
Thu, 9 Jan 2020 08:36:46 +0000 (08:36 +0000)
committerDavid S. Miller <davem@davemloft.net>
Thu, 9 Jan 2020 18:48:29 +0000 (10:48 -0800)
This patch modify MDIO read/write functions to support
communication with C45 PHY.

Signed-off-by: Milind Parab <mparab@cadence.com>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Acked-by: Nicolas Ferre <nicolas.ferre@microchip.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/cadence/macb.h
drivers/net/ethernet/cadence/macb_main.c

index 19fe4f4..dbf7070 100644 (file)
 #define GEM_CLK_DIV96                          5
 
 /* Constants for MAN register */
-#define MACB_MAN_SOF                           1
-#define MACB_MAN_WRITE                         1
-#define MACB_MAN_READ                          2
-#define MACB_MAN_CODE                          2
+#define MACB_MAN_C22_SOF                       1
+#define MACB_MAN_C22_WRITE                     1
+#define MACB_MAN_C22_READ                      2
+#define MACB_MAN_C22_CODE                      2
+
+#define MACB_MAN_C45_SOF                       0
+#define MACB_MAN_C45_ADDR                      0
+#define MACB_MAN_C45_WRITE                     1
+#define MACB_MAN_C45_POST_READ_INCR            2
+#define MACB_MAN_C45_READ                      3
+#define MACB_MAN_C45_CODE                      2
 
 /* Capability mask bits */
 #define MACB_CAPS_ISR_CLEAR_ON_WRITE           0x00000001
index c5ee363..09a88f3 100644 (file)
@@ -337,11 +337,30 @@ static int macb_mdio_read(struct mii_bus *bus, int mii_id, int regnum)
        if (status < 0)
                goto mdio_read_exit;
 
-       macb_writel(bp, MAN, (MACB_BF(SOF, MACB_MAN_SOF)
-                             | MACB_BF(RW, MACB_MAN_READ)
-                             | MACB_BF(PHYA, mii_id)
-                             | MACB_BF(REGA, regnum)
-                             | MACB_BF(CODE, MACB_MAN_CODE)));
+       if (regnum & MII_ADDR_C45) {
+               macb_writel(bp, MAN, (MACB_BF(SOF, MACB_MAN_C45_SOF)
+                           | MACB_BF(RW, MACB_MAN_C45_ADDR)
+                           | MACB_BF(PHYA, mii_id)
+                           | MACB_BF(REGA, (regnum >> 16) & 0x1F)
+                           | MACB_BF(DATA, regnum & 0xFFFF)
+                           | MACB_BF(CODE, MACB_MAN_C45_CODE)));
+
+               status = macb_mdio_wait_for_idle(bp);
+               if (status < 0)
+                       goto mdio_read_exit;
+
+               macb_writel(bp, MAN, (MACB_BF(SOF, MACB_MAN_C45_SOF)
+                           | MACB_BF(RW, MACB_MAN_C45_READ)
+                           | MACB_BF(PHYA, mii_id)
+                           | MACB_BF(REGA, (regnum >> 16) & 0x1F)
+                           | MACB_BF(CODE, MACB_MAN_C45_CODE)));
+       } else {
+               macb_writel(bp, MAN, (MACB_BF(SOF, MACB_MAN_C22_SOF)
+                               | MACB_BF(RW, MACB_MAN_C22_READ)
+                               | MACB_BF(PHYA, mii_id)
+                               | MACB_BF(REGA, regnum)
+                               | MACB_BF(CODE, MACB_MAN_C22_CODE)));
+       }
 
        status = macb_mdio_wait_for_idle(bp);
        if (status < 0)
@@ -370,12 +389,32 @@ static int macb_mdio_write(struct mii_bus *bus, int mii_id, int regnum,
        if (status < 0)
                goto mdio_write_exit;
 
-       macb_writel(bp, MAN, (MACB_BF(SOF, MACB_MAN_SOF)
-                             | MACB_BF(RW, MACB_MAN_WRITE)
-                             | MACB_BF(PHYA, mii_id)
-                             | MACB_BF(REGA, regnum)
-                             | MACB_BF(CODE, MACB_MAN_CODE)
-                             | MACB_BF(DATA, value)));
+       if (regnum & MII_ADDR_C45) {
+               macb_writel(bp, MAN, (MACB_BF(SOF, MACB_MAN_C45_SOF)
+                           | MACB_BF(RW, MACB_MAN_C45_ADDR)
+                           | MACB_BF(PHYA, mii_id)
+                           | MACB_BF(REGA, (regnum >> 16) & 0x1F)
+                           | MACB_BF(DATA, regnum & 0xFFFF)
+                           | MACB_BF(CODE, MACB_MAN_C45_CODE)));
+
+               status = macb_mdio_wait_for_idle(bp);
+               if (status < 0)
+                       goto mdio_write_exit;
+
+               macb_writel(bp, MAN, (MACB_BF(SOF, MACB_MAN_C45_SOF)
+                           | MACB_BF(RW, MACB_MAN_C45_WRITE)
+                           | MACB_BF(PHYA, mii_id)
+                           | MACB_BF(REGA, (regnum >> 16) & 0x1F)
+                           | MACB_BF(CODE, MACB_MAN_C45_CODE)
+                           | MACB_BF(DATA, value)));
+       } else {
+               macb_writel(bp, MAN, (MACB_BF(SOF, MACB_MAN_C22_SOF)
+                               | MACB_BF(RW, MACB_MAN_C22_WRITE)
+                               | MACB_BF(PHYA, mii_id)
+                               | MACB_BF(REGA, regnum)
+                               | MACB_BF(CODE, MACB_MAN_C22_CODE)
+                               | MACB_BF(DATA, value)));
+       }
 
        status = macb_mdio_wait_for_idle(bp);
        if (status < 0)