net: phy: mdio-bcm-unimac: Use correct I/O accessors
authorFlorian Fainelli <f.fainelli@gmail.com>
Tue, 29 Aug 2017 20:35:18 +0000 (13:35 -0700)
committerDavid S. Miller <davem@davemloft.net>
Tue, 29 Aug 2017 21:42:17 +0000 (14:42 -0700)
The driver currently uses __raw_{read,write}l which works for all
platforms supported: Broadcom MIPS LE/BE (native endian), ARM LE (native
endian) but not ARM BE (registers are still LE). Switch to using the
proper accessors for all platforms and explain why Broadcom MIPS BE is
special here, in doing so, we introduce a couple of helper functions to
abstract these differences.

Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/phy/mdio-bcm-unimac.c

index 73c5267a11fd9241f7a23f8789a464ebbba9258a..08e0647b85e2395d5a004b9deeaa9d6a1fcc1b91 100644 (file)
@@ -47,18 +47,38 @@ struct unimac_mdio_priv {
        void                    *wait_func_data;
 };
 
+static inline u32 unimac_mdio_readl(struct unimac_mdio_priv *priv, u32 offset)
+{
+       /* MIPS chips strapped for BE will automagically configure the
+        * peripheral registers for CPU-native byte order.
+        */
+       if (IS_ENABLED(CONFIG_MIPS) && IS_ENABLED(CONFIG_CPU_BIG_ENDIAN))
+               return __raw_readl(priv->base + offset);
+       else
+               return readl_relaxed(priv->base + offset);
+}
+
+static inline void unimac_mdio_writel(struct unimac_mdio_priv *priv, u32 val,
+                                     u32 offset)
+{
+       if (IS_ENABLED(CONFIG_MIPS) && IS_ENABLED(CONFIG_CPU_BIG_ENDIAN))
+               __raw_writel(val, priv->base + offset);
+       else
+               writel_relaxed(val, priv->base + offset);
+}
+
 static inline void unimac_mdio_start(struct unimac_mdio_priv *priv)
 {
        u32 reg;
 
-       reg = __raw_readl(priv->base + MDIO_CMD);
+       reg = unimac_mdio_readl(priv, MDIO_CMD);
        reg |= MDIO_START_BUSY;
-       __raw_writel(reg, priv->base + MDIO_CMD);
+       unimac_mdio_writel(priv, reg, MDIO_CMD);
 }
 
 static inline unsigned int unimac_mdio_busy(struct unimac_mdio_priv *priv)
 {
-       return __raw_readl(priv->base + MDIO_CMD) & MDIO_START_BUSY;
+       return unimac_mdio_readl(priv, MDIO_CMD) & MDIO_START_BUSY;
 }
 
 static int unimac_mdio_poll(void *wait_func_data)
@@ -87,7 +107,7 @@ static int unimac_mdio_read(struct mii_bus *bus, int phy_id, int reg)
 
        /* Prepare the read operation */
        cmd = MDIO_RD | (phy_id << MDIO_PMD_SHIFT) | (reg << MDIO_REG_SHIFT);
-       __raw_writel(cmd, priv->base + MDIO_CMD);
+       unimac_mdio_writel(priv, cmd, MDIO_CMD);
 
        /* Start MDIO transaction */
        unimac_mdio_start(priv);
@@ -96,7 +116,7 @@ static int unimac_mdio_read(struct mii_bus *bus, int phy_id, int reg)
        if (ret)
                return ret;
 
-       cmd = __raw_readl(priv->base + MDIO_CMD);
+       cmd = unimac_mdio_readl(priv, MDIO_CMD);
 
        /* Some broken devices are known not to release the line during
         * turn-around, e.g: Broadcom BCM53125 external switches, so check for
@@ -118,7 +138,7 @@ static int unimac_mdio_write(struct mii_bus *bus, int phy_id,
        /* Prepare the write operation */
        cmd = MDIO_WR | (phy_id << MDIO_PMD_SHIFT) |
                (reg << MDIO_REG_SHIFT) | (0xffff & val);
-       __raw_writel(cmd, priv->base + MDIO_CMD);
+       unimac_mdio_writel(priv, cmd, MDIO_CMD);
 
        unimac_mdio_start(priv);