diff options
| author |   Florian Fainelli <f.fainelli@gmail.com> | 2017-08-29 21:48:51 -0700 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2017-08-29 22:39:51 -0700 |
| commit | d081a16db80ef7a260fb178aa1199e01f7432625 (patch) | |
| tree | 2521136ce5239cacbfc19245ef55484549d7f739 | |
| parent | neigh: increase queue_len_bytes to match wmem_default (diff) | |
| download | linux-dev-d081a16db80ef7a260fb178aa1199e01f7432625.tar.xz linux-dev-d081a16db80ef7a260fb178aa1199e01f7432625.zip | |
net: bcmgenet: Do not return from void function
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>
| -rw-r--r-- | drivers/net/ethernet/broadcom/genet/bcmgenet.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ethernet/broadcom/genet/bcmgenet.h b/drivers/net/ethernet/broadcom/genet/bcmgenet.h index 2bfeaefcca0f..4c49d0b97748 100644 --- a/drivers/net/ethernet/broadcom/genet/bcmgenet.h +++ b/drivers/net/ethernet/broadcom/genet/bcmgenet.h @@ -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); \ } |