Skip to content

Commit de6f31e

Browse files
Matt Carlsondavem330
authored andcommitted
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: Matt Carlson <mcarlson@broadcom.com> Reviewed-by: Michael Chan <mchan@broadcom.com> Signed-off-by: David S. Miller <davem@davemloft.net>
1 parent 04380d4 commit de6f31e

File tree

1 file changed

+9
-9
lines changed

1 file changed

+9
-9
lines changed

drivers/net/tg3.c

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -383,7 +383,7 @@ static void tg3_write32(struct tg3 *tp, u32 off, u32 val)
383383

384384
static u32 tg3_read32(struct tg3 *tp, u32 off)
385385
{
386-
return (readl(tp->regs + off));
386+
return readl(tp->regs + off);
387387
}
388388

389389
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)
393393

394394
static u32 tg3_ape_read32(struct tg3 *tp, u32 off)
395395
{
396-
return (readl(tp->aperegs + off));
396+
return readl(tp->aperegs + off);
397397
}
398398

399399
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)
511511

512512
static u32 tg3_read32_mbox_5906(struct tg3 *tp, u32 off)
513513
{
514-
return (readl(tp->regs + off + GRCMBOX_BASE));
514+
return readl(tp->regs + off + GRCMBOX_BASE);
515515
}
516516

517517
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,
57755775
hdr_len = ip_tcp_len + tcp_opt_len;
57765776
if (unlikely((ETH_HLEN + hdr_len) > 80) &&
57775777
(tp->tg3_flags2 & TG3_FLG2_TSO_BUG))
5778-
return (tg3_tso_bug(tp, skb));
5778+
return tg3_tso_bug(tp, skb);
57795779

57805780
base_flags |= (TXD_FLAG_CPU_PRE_DMA |
57815781
TXD_FLAG_CPU_POST_DMA);
@@ -9285,10 +9285,10 @@ static void __tg3_set_rx_mode(struct net_device *dev)
92859285
rx_mode |= RX_MODE_PROMISC;
92869286
} else if (dev->flags & IFF_ALLMULTI) {
92879287
/* Accept all multicast. */
9288-
tg3_set_multi (tp, 1);
9288+
tg3_set_multi(tp, 1);
92899289
} else if (netdev_mc_empty(dev)) {
92909290
/* Reject all multicast. */
9291-
tg3_set_multi (tp, 0);
9291+
tg3_set_multi(tp, 0);
92929292
} else {
92939293
/* Accept one or more multicast(s). */
92949294
struct netdev_hw_addr *ha;
@@ -10030,7 +10030,7 @@ static int tg3_set_tx_csum(struct net_device *dev, u32 data)
1003010030
return 0;
1003110031
}
1003210032

10033-
static int tg3_get_sset_count (struct net_device *dev, int sset)
10033+
static int tg3_get_sset_count(struct net_device *dev, int sset)
1003410034
{
1003510035
switch (sset) {
1003610036
case ETH_SS_TEST:
@@ -10042,7 +10042,7 @@ static int tg3_get_sset_count (struct net_device *dev, int sset)
1004210042
}
1004310043
}
1004410044

10045-
static void tg3_get_strings (struct net_device *dev, u32 stringset, u8 *buf)
10045+
static void tg3_get_strings(struct net_device *dev, u32 stringset, u8 *buf)
1004610046
{
1004710047
switch (stringset) {
1004810048
case ETH_SS_STATS:
@@ -10089,7 +10089,7 @@ static int tg3_phys_id(struct net_device *dev, u32 data)
1008910089
return 0;
1009010090
}
1009110091

10092-
static void tg3_get_ethtool_stats (struct net_device *dev,
10092+
static void tg3_get_ethtool_stats(struct net_device *dev,
1009310093
struct ethtool_stats *estats, u64 *tmp_stats)
1009410094
{
1009510095
struct tg3 *tp = netdev_priv(dev);

0 commit comments

Comments
 (0)