chelsio: Remove addressof casts to same type
authorJoe Perches <joe@perches.com>
Mon, 24 Mar 2014 17:45:12 +0000 (10:45 -0700)
committerDavid S. Miller <davem@davemloft.net>
Wed, 26 Mar 2014 19:46:15 +0000 (15:46 -0400)
Using addressof then casting to the original type is pointless,
so remove these unnecessary casts.

Done via coccinelle script:

$ cat typecast.cocci
@@
type T;
T foo;
@@

- (T *)&foo
+ &foo

Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c

index cc04d090354cc686c52d69c4125c5222e7a3a2cd..a7a41e0f3b01a48a993b624102cee3d952f28400 100644 (file)
@@ -3220,8 +3220,8 @@ static int cxgb4_clip_get(const struct net_device *dev,
        c.op_to_write = htonl(FW_CMD_OP(FW_CLIP_CMD) |
                        FW_CMD_REQUEST | FW_CMD_WRITE);
        c.alloc_to_len16 = htonl(F_FW_CLIP_CMD_ALLOC | FW_LEN16(c));
-       *(__be64 *)&c.ip_hi = *(__be64 *)(lip->s6_addr);
-       *(__be64 *)&c.ip_lo = *(__be64 *)(lip->s6_addr + 8);
+       c.ip_hi = *(__be64 *)(lip->s6_addr);
+       c.ip_lo = *(__be64 *)(lip->s6_addr + 8);
        return t4_wr_mbox_meat(adap, adap->mbox, &c, sizeof(c), &c, false);
 }
 
@@ -3236,8 +3236,8 @@ static int cxgb4_clip_release(const struct net_device *dev,
        c.op_to_write = htonl(FW_CMD_OP(FW_CLIP_CMD) |
                        FW_CMD_REQUEST | FW_CMD_READ);
        c.alloc_to_len16 = htonl(F_FW_CLIP_CMD_FREE | FW_LEN16(c));
-       *(__be64 *)&c.ip_hi = *(__be64 *)(lip->s6_addr);
-       *(__be64 *)&c.ip_lo = *(__be64 *)(lip->s6_addr + 8);
+       c.ip_hi = *(__be64 *)(lip->s6_addr);
+       c.ip_lo = *(__be64 *)(lip->s6_addr + 8);
        return t4_wr_mbox_meat(adap, adap->mbox, &c, sizeof(c), &c, false);
 }