Commit de6f31eb authored by Matt Carlson's avatar Matt Carlson Committed by David S. Miller

tg3: Remove function errors flagged by checkpatch

This patch removes the following checkpatch errors:

* return is not a function, parentheses are not required
* space prohibited between function name and open parenthesis '('
Signed-off-by: default avatarMatt Carlson <mcarlson@broadcom.com>
Reviewed-by: default avatarMichael Chan <mchan@broadcom.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 04380d40
...@@ -383,7 +383,7 @@ static void tg3_write32(struct tg3 *tp, u32 off, u32 val) ...@@ -383,7 +383,7 @@ static void tg3_write32(struct tg3 *tp, u32 off, u32 val)
static u32 tg3_read32(struct tg3 *tp, u32 off) static u32 tg3_read32(struct tg3 *tp, u32 off)
{ {
return (readl(tp->regs + off)); return readl(tp->regs + off);
} }
static void tg3_ape_write32(struct tg3 *tp, u32 off, u32 val) static void tg3_ape_write32(struct tg3 *tp, u32 off, u32 val)
...@@ -393,7 +393,7 @@ static void tg3_ape_write32(struct tg3 *tp, u32 off, u32 val) ...@@ -393,7 +393,7 @@ static void tg3_ape_write32(struct tg3 *tp, u32 off, u32 val)
static u32 tg3_ape_read32(struct tg3 *tp, u32 off) static u32 tg3_ape_read32(struct tg3 *tp, u32 off)
{ {
return (readl(tp->aperegs + off)); return readl(tp->aperegs + off);
} }
static void tg3_write_indirect_reg32(struct tg3 *tp, u32 off, u32 val) static void tg3_write_indirect_reg32(struct tg3 *tp, u32 off, u32 val)
...@@ -511,7 +511,7 @@ static void tg3_write32_tx_mbox(struct tg3 *tp, u32 off, u32 val) ...@@ -511,7 +511,7 @@ static void tg3_write32_tx_mbox(struct tg3 *tp, u32 off, u32 val)
static u32 tg3_read32_mbox_5906(struct tg3 *tp, u32 off) static u32 tg3_read32_mbox_5906(struct tg3 *tp, u32 off)
{ {
return (readl(tp->regs + off + GRCMBOX_BASE)); return readl(tp->regs + off + GRCMBOX_BASE);
} }
static void tg3_write32_mbox_5906(struct tg3 *tp, u32 off, u32 val) static void tg3_write32_mbox_5906(struct tg3 *tp, u32 off, u32 val)
...@@ -5775,7 +5775,7 @@ static netdev_tx_t tg3_start_xmit_dma_bug(struct sk_buff *skb, ...@@ -5775,7 +5775,7 @@ static netdev_tx_t tg3_start_xmit_dma_bug(struct sk_buff *skb,
hdr_len = ip_tcp_len + tcp_opt_len; hdr_len = ip_tcp_len + tcp_opt_len;
if (unlikely((ETH_HLEN + hdr_len) > 80) && if (unlikely((ETH_HLEN + hdr_len) > 80) &&
(tp->tg3_flags2 & TG3_FLG2_TSO_BUG)) (tp->tg3_flags2 & TG3_FLG2_TSO_BUG))
return (tg3_tso_bug(tp, skb)); return tg3_tso_bug(tp, skb);
base_flags |= (TXD_FLAG_CPU_PRE_DMA | base_flags |= (TXD_FLAG_CPU_PRE_DMA |
TXD_FLAG_CPU_POST_DMA); TXD_FLAG_CPU_POST_DMA);
...@@ -9285,10 +9285,10 @@ static void __tg3_set_rx_mode(struct net_device *dev) ...@@ -9285,10 +9285,10 @@ static void __tg3_set_rx_mode(struct net_device *dev)
rx_mode |= RX_MODE_PROMISC; rx_mode |= RX_MODE_PROMISC;
} else if (dev->flags & IFF_ALLMULTI) { } else if (dev->flags & IFF_ALLMULTI) {
/* Accept all multicast. */ /* Accept all multicast. */
tg3_set_multi (tp, 1); tg3_set_multi(tp, 1);
} else if (netdev_mc_empty(dev)) { } else if (netdev_mc_empty(dev)) {
/* Reject all multicast. */ /* Reject all multicast. */
tg3_set_multi (tp, 0); tg3_set_multi(tp, 0);
} else { } else {
/* Accept one or more multicast(s). */ /* Accept one or more multicast(s). */
struct netdev_hw_addr *ha; struct netdev_hw_addr *ha;
...@@ -10030,7 +10030,7 @@ static int tg3_set_tx_csum(struct net_device *dev, u32 data) ...@@ -10030,7 +10030,7 @@ static int tg3_set_tx_csum(struct net_device *dev, u32 data)
return 0; return 0;
} }
static int tg3_get_sset_count (struct net_device *dev, int sset) static int tg3_get_sset_count(struct net_device *dev, int sset)
{ {
switch (sset) { switch (sset) {
case ETH_SS_TEST: case ETH_SS_TEST:
...@@ -10042,7 +10042,7 @@ static int tg3_get_sset_count (struct net_device *dev, int sset) ...@@ -10042,7 +10042,7 @@ static int tg3_get_sset_count (struct net_device *dev, int sset)
} }
} }
static void tg3_get_strings (struct net_device *dev, u32 stringset, u8 *buf) static void tg3_get_strings(struct net_device *dev, u32 stringset, u8 *buf)
{ {
switch (stringset) { switch (stringset) {
case ETH_SS_STATS: case ETH_SS_STATS:
...@@ -10089,7 +10089,7 @@ static int tg3_phys_id(struct net_device *dev, u32 data) ...@@ -10089,7 +10089,7 @@ static int tg3_phys_id(struct net_device *dev, u32 data)
return 0; return 0;
} }
static void tg3_get_ethtool_stats (struct net_device *dev, static void tg3_get_ethtool_stats(struct net_device *dev,
struct ethtool_stats *estats, u64 *tmp_stats) struct ethtool_stats *estats, u64 *tmp_stats)
{ {
struct tg3 *tp = netdev_priv(dev); struct tg3 *tp = netdev_priv(dev);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment