OSDN Git Service

net: bcmgenet: Do not return from void function
authorFlorian Fainelli <f.fainelli@gmail.com>
Wed, 30 Aug 2017 04:48:51 +0000 (21:48 -0700)
committerDavid S. Miller <davem@davemloft.net>
Wed, 30 Aug 2017 05:39:51 +0000 (22:39 -0700)
A stray return was added in the macro bcmgenet_##name##_writel where it
should not, drop it.

Reported-by: kbuild test robot <fengguang.wu@intel.com>
Fixes: 69d2ea9c7989 ("net: bcmgenet: Use correct I/O accessors")
Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/broadcom/genet/bcmgenet.h

index 2bfeaef..4c49d0b 100644 (file)
@@ -684,7 +684,7 @@ static inline void bcmgenet_##name##_writel(struct bcmgenet_priv *priv,     \
                                        u32 val, u32 off)               \
 {                                                                      \
        if (IS_ENABLED(CONFIG_MIPS) && IS_ENABLED(CONFIG_CPU_BIG_ENDIAN)) \
-               return __raw_writel(val, priv->base + offset + off);    \
+               __raw_writel(val, priv->base + offset + off);           \
        else                                                            \
                writel_relaxed(val, priv->base + offset + off);         \
 }