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

tg3: Fix some checkpatch errors

This patch fixes the following checkpatch errors:

ERROR: do not use assignment in if condition
+	if ((mss = skb_shinfo(skb)->gso_size) != 0) {

ERROR: do not use assignment in if condition
+	if ((mss = skb_shinfo(skb)->gso_size) != 0) {

ERROR: space prohibited after that '!' (ctx:BxW)
+			if (! netif_carrier_ok(tp->dev) &&
 			    ^

ERROR: space required after that ',' (ctx:VxV)
+#define GET_REG32_LOOP(base,len)		\
                            ^

ERROR: "(foo*)" should be "(foo *)"
+		memcpy(data, ((char*)&val) + b_offset, b_count);

ERROR: do not use assignment in if condition
+		if ((err = tg3_do_mem_test(tp, mem_tbl[i].offset,
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 702e52cc
...@@ -5574,8 +5574,8 @@ static netdev_tx_t tg3_start_xmit(struct sk_buff *skb, ...@@ -5574,8 +5574,8 @@ static netdev_tx_t tg3_start_xmit(struct sk_buff *skb,
entry = tnapi->tx_prod; entry = tnapi->tx_prod;
base_flags = 0; base_flags = 0;
mss = 0; mss = skb_shinfo(skb)->gso_size;
if ((mss = skb_shinfo(skb)->gso_size) != 0) { if (mss) {
int tcp_opt_len, ip_tcp_len; int tcp_opt_len, ip_tcp_len;
u32 hdrlen; u32 hdrlen;
...@@ -5781,7 +5781,8 @@ static netdev_tx_t tg3_start_xmit_dma_bug(struct sk_buff *skb, ...@@ -5781,7 +5781,8 @@ static netdev_tx_t tg3_start_xmit_dma_bug(struct sk_buff *skb,
if (skb->ip_summed == CHECKSUM_PARTIAL) if (skb->ip_summed == CHECKSUM_PARTIAL)
base_flags |= TXD_FLAG_TCPUDP_CSUM; base_flags |= TXD_FLAG_TCPUDP_CSUM;
if ((mss = skb_shinfo(skb)->gso_size) != 0) { mss = skb_shinfo(skb)->gso_size;
if (mss) {
struct iphdr *iph; struct iphdr *iph;
u32 tcp_opt_len, hdr_len; u32 tcp_opt_len, hdr_len;
...@@ -8514,7 +8515,7 @@ static void tg3_timer(unsigned long __opaque) ...@@ -8514,7 +8515,7 @@ static void tg3_timer(unsigned long __opaque)
(mac_stat & MAC_STATUS_LNKSTATE_CHANGED)) { (mac_stat & MAC_STATUS_LNKSTATE_CHANGED)) {
need_setup = 1; need_setup = 1;
} }
if (! netif_carrier_ok(tp->dev) && if (!netif_carrier_ok(tp->dev) &&
(mac_stat & (MAC_STATUS_PCS_SYNCED | (mac_stat & (MAC_STATUS_PCS_SYNCED |
MAC_STATUS_SIGNAL_DET))) { MAC_STATUS_SIGNAL_DET))) {
need_setup = 1; need_setup = 1;
...@@ -9372,7 +9373,7 @@ static void tg3_get_regs(struct net_device *dev, ...@@ -9372,7 +9373,7 @@ static void tg3_get_regs(struct net_device *dev,
tg3_full_lock(tp, 0); tg3_full_lock(tp, 0);
#define __GET_REG32(reg) (*(p)++ = tr32(reg)) #define __GET_REG32(reg) (*(p)++ = tr32(reg))
#define GET_REG32_LOOP(base,len) \ #define GET_REG32_LOOP(base, len) \
do { p = (u32 *)(orig_p + (base)); \ do { p = (u32 *)(orig_p + (base)); \
for (i = 0; i < len; i += 4) \ for (i = 0; i < len; i += 4) \
__GET_REG32((base) + i); \ __GET_REG32((base) + i); \
...@@ -9465,7 +9466,7 @@ static int tg3_get_eeprom(struct net_device *dev, struct ethtool_eeprom *eeprom, ...@@ -9465,7 +9466,7 @@ static int tg3_get_eeprom(struct net_device *dev, struct ethtool_eeprom *eeprom,
ret = tg3_nvram_read_be32(tp, offset-b_offset, &val); ret = tg3_nvram_read_be32(tp, offset-b_offset, &val);
if (ret) if (ret)
return ret; return ret;
memcpy(data, ((char*)&val) + b_offset, b_count); memcpy(data, ((char *)&val) + b_offset, b_count);
len -= b_count; len -= b_count;
offset += b_count; offset += b_count;
eeprom->len += b_count; eeprom->len += b_count;
...@@ -10585,8 +10586,8 @@ static int tg3_test_memory(struct tg3 *tp) ...@@ -10585,8 +10586,8 @@ static int tg3_test_memory(struct tg3 *tp)
mem_tbl = mem_tbl_570x; mem_tbl = mem_tbl_570x;
for (i = 0; mem_tbl[i].offset != 0xffffffff; i++) { for (i = 0; mem_tbl[i].offset != 0xffffffff; i++) {
if ((err = tg3_do_mem_test(tp, mem_tbl[i].offset, err = tg3_do_mem_test(tp, mem_tbl[i].offset, mem_tbl[i].len);
mem_tbl[i].len)) != 0) if (err)
break; break;
} }
......
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