ar71xx: ag71xx: simplify register access functions
authorGabor Juhos <juhosg@openwrt.org>
Tue, 2 Mar 2010 13:36:12 +0000 (13:36 +0000)
committerGabor Juhos <juhosg@openwrt.org>
Tue, 2 Mar 2010 13:36:12 +0000 (13:36 +0000)
SVN-Revision: 19949

target/linux/ar71xx/files/drivers/net/ag71xx/ag71xx.h

index 712e9f1ecd0518fda08eccde0a3d964992bab06b..881ffa3693c3bb22574456c03000b51bd6570715 100644 (file)
@@ -38,7 +38,7 @@
 #define ETH_FCS_LEN    4
 
 #define AG71XX_DRV_NAME                "ag71xx"
-#define AG71XX_DRV_VERSION     "0.5.26"
+#define AG71XX_DRV_VERSION     "0.5.27"
 
 #define AG71XX_NAPI_WEIGHT     64
 #define AG71XX_OOM_REFILL      (1 + HZ/10)
@@ -343,76 +343,56 @@ static inline int ag71xx_desc_pktlen(struct ag71xx_desc *desc)
 #define MII_CTRL_SPEED_100     1
 #define MII_CTRL_SPEED_1000    2
 
-static inline void ag71xx_wr(struct ag71xx *ag, unsigned reg, u32 value)
+static inline void ag71xx_check_reg_offset(struct ag71xx *ag, unsigned reg)
 {
-       void __iomem *r;
-
        switch (reg) {
        case AG71XX_REG_MAC_CFG1 ... AG71XX_REG_MAC_MFL:
        case AG71XX_REG_MAC_IFCTL ... AG71XX_REG_INT_STATUS:
-               r = ag->mac_base + reg;
-               __raw_writel(value, r);
-
-               /* flush write */
-               (void) __raw_readl(r);
                break;
+
        default:
                BUG();
        }
 }
 
-static inline u32 ag71xx_rr(struct ag71xx *ag, unsigned reg)
+static inline void ag71xx_wr(struct ag71xx *ag, unsigned reg, u32 value)
 {
-       void __iomem *r;
-       u32 ret;
+       ag71xx_check_reg_offset(ag, reg);
 
-       switch (reg) {
-       case AG71XX_REG_MAC_CFG1 ... AG71XX_REG_MAC_MFL:
-       case AG71XX_REG_MAC_IFCTL ... AG71XX_REG_INT_STATUS:
-               r = ag->mac_base + reg;
-               ret = __raw_readl(r);
-               break;
-       default:
-               BUG();
-       }
+       __raw_writel(value, ag->mac_base + reg);
+       /* flush write */
+       (void) __raw_readl(ag->mac_base + reg);
+}
+
+static inline u32 ag71xx_rr(struct ag71xx *ag, unsigned reg)
+{
+       ag71xx_check_reg_offset(ag, reg);
 
-       return ret;
+       return __raw_readl(ag->mac_base + reg);
 }
 
 static inline void ag71xx_sb(struct ag71xx *ag, unsigned reg, u32 mask)
 {
        void __iomem *r;
 
-       switch (reg) {
-       case AG71XX_REG_MAC_CFG1 ... AG71XX_REG_MAC_MFL:
-       case AG71XX_REG_MAC_IFCTL ... AG71XX_REG_INT_STATUS:
-               r = ag->mac_base + reg;
-               __raw_writel(__raw_readl(r) | mask, r);
+       ag71xx_check_reg_offset(ag, reg);
 
-               /* flush write */
-               (void)__raw_readl(r);
-               break;
-       default:
-               BUG();
-       }
+       r = ag->mac_base + reg;
+       __raw_writel(__raw_readl(r) | mask, r);
+       /* flush write */
+       (void)__raw_readl(r);
 }
 
 static inline void ag71xx_cb(struct ag71xx *ag, unsigned reg, u32 mask)
 {
        void __iomem *r;
 
-       switch (reg) {
-       case AG71XX_REG_MAC_CFG1 ... AG71XX_REG_MAC_MFL:
-       case AG71XX_REG_MAC_IFCTL ... AG71XX_REG_INT_STATUS:
-               r = ag->mac_base + reg;
-               __raw_writel(__raw_readl(r) & ~mask, r);
+       ag71xx_check_reg_offset(ag, reg);
 
-               /* flush write */
-               (void) __raw_readl(r);
-               break;
-       default:
-               BUG();
-       }
+       r = ag->mac_base + reg;
+       __raw_writel(__raw_readl(r) & ~mask, r);
+       /* flush write */
+       (void) __raw_readl(r);
 }
 
 static inline void ag71xx_int_enable(struct ag71xx *ag, u32 ints)