On Wed, 2010-12-22 at 12:38 +0100, Eliad Peller wrote: > 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@xxxxxxxxxx> > --- Applied, thank you! -- Cheers, Luca. -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html