diff options
author | Eliad Peller <eliad@wizery.com> | 2010-12-22 06:38:52 -0500 |
---|---|---|
committer | Luciano Coelho <coelho@ti.com> | 2011-01-24 15:11:46 -0500 |
commit | 6177eaea277527e48753d050723cd138494c98a8 (patch) | |
tree | 372618c1d42e264f00d20060441f61d3fdf3e1d1 /drivers/net/wireless/wl12xx/cmd.c | |
parent | 0dd386676497fb3097e6fbb3de6090c948c0df30 (diff) |
wl12xx: fix some sparse warnings
Note that wl1271_write32() calls cpu_to_le32() by itself, so calling
wl1271_write32(addr, cpu_to_le32(val)) is in fact a bug on BE systems.
Fix the following sparse warnings:
drivers/net/wireless/wl12xx/cmd.c:662:16: warning: incorrect type in assignment (different base types)
drivers/net/wireless/wl12xx/cmd.c:662:16: expected unsigned short [unsigned] [addressable] [usertype] llc_type
drivers/net/wireless/wl12xx/cmd.c:662:16: got restricted __be16 [usertype] <noident>
drivers/net/wireless/wl12xx/cmd.c:674:17: warning: incorrect type in assignment (different base types)
drivers/net/wireless/wl12xx/cmd.c:674:17: expected unsigned int [unsigned] [addressable] [usertype] sender_ip
drivers/net/wireless/wl12xx/cmd.c:674:17: got restricted __be32 [usertype] ip_addr
drivers/net/wireless/wl12xx/rx.c:202:4: warning: incorrect type in argument 3 (different base types)
drivers/net/wireless/wl12xx/rx.c:202:4: expected unsigned int [unsigned] [usertype] val
drivers/net/wireless/wl12xx/rx.c:202:4: got restricted __le32 [usertype] <noident>
drivers/net/wireless/wl12xx/acx.c:1247:23: warning: incorrect type in assignment (different base types)
drivers/net/wireless/wl12xx/acx.c:1247:23: expected restricted __le32 [usertype] ht_capabilites
drivers/net/wireless/wl12xx/acx.c:1247:23: got unsigned long
drivers/net/wireless/wl12xx/acx.c:1250:24: warning: invalid assignment: |=
drivers/net/wireless/wl12xx/acx.c:1250:24: left side has type restricted __le32
drivers/net/wireless/wl12xx/acx.c:1250:24: right side has type unsigned long
drivers/net/wireless/wl12xx/acx.c:1253:24: warning: invalid assignment: |=
drivers/net/wireless/wl12xx/acx.c:1253:24: left side has type restricted __le32
drivers/net/wireless/wl12xx/acx.c:1253:24: right side has type unsigned long
drivers/net/wireless/wl12xx/acx.c:1256:24: warning: invalid assignment: |=
drivers/net/wireless/wl12xx/acx.c:1256:24: left side has type restricted __le32
drivers/net/wireless/wl12xx/acx.c:1256:24: right side has type unsigned long
Signed-off-by: Eliad Peller <eliad@wizery.com>
Signed-off-by: Luciano Coelho <coelho@ti.com>
Diffstat (limited to 'drivers/net/wireless/wl12xx/cmd.c')
-rw-r--r-- | drivers/net/wireless/wl12xx/cmd.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/net/wireless/wl12xx/cmd.c b/drivers/net/wireless/wl12xx/cmd.c index 0106628aa5a2..52a6bcd5c309 100644 --- a/drivers/net/wireless/wl12xx/cmd.c +++ b/drivers/net/wireless/wl12xx/cmd.c | |||
@@ -659,15 +659,15 @@ int wl1271_cmd_build_arp_rsp(struct wl1271 *wl, __be32 ip_addr) | |||
659 | 659 | ||
660 | /* llc layer */ | 660 | /* llc layer */ |
661 | memcpy(tmpl.llc_hdr, rfc1042_header, sizeof(rfc1042_header)); | 661 | memcpy(tmpl.llc_hdr, rfc1042_header, sizeof(rfc1042_header)); |
662 | tmpl.llc_type = htons(ETH_P_ARP); | 662 | tmpl.llc_type = cpu_to_be16(ETH_P_ARP); |
663 | 663 | ||
664 | /* arp header */ | 664 | /* arp header */ |
665 | arp_hdr = &tmpl.arp_hdr; | 665 | arp_hdr = &tmpl.arp_hdr; |
666 | arp_hdr->ar_hrd = htons(ARPHRD_ETHER); | 666 | arp_hdr->ar_hrd = cpu_to_be16(ARPHRD_ETHER); |
667 | arp_hdr->ar_pro = htons(ETH_P_IP); | 667 | arp_hdr->ar_pro = cpu_to_be16(ETH_P_IP); |
668 | arp_hdr->ar_hln = ETH_ALEN; | 668 | arp_hdr->ar_hln = ETH_ALEN; |
669 | arp_hdr->ar_pln = 4; | 669 | arp_hdr->ar_pln = 4; |
670 | arp_hdr->ar_op = htons(ARPOP_REPLY); | 670 | arp_hdr->ar_op = cpu_to_be16(ARPOP_REPLY); |
671 | 671 | ||
672 | /* arp payload */ | 672 | /* arp payload */ |
673 | memcpy(tmpl.sender_hw, wl->vif->addr, ETH_ALEN); | 673 | memcpy(tmpl.sender_hw, wl->vif->addr, ETH_ALEN); |