diff options
author | Eliad Peller <eliad@wizery.com> | 2011-01-16 04:07:10 -0500 |
---|---|---|
committer | Luciano Coelho <coelho@ti.com> | 2011-01-24 15:13:55 -0500 |
commit | 1d4801f2689dc2618fdb5e83d4cb7743747491ed (patch) | |
tree | f808e5b8b644f417eead69abd606a8bc0653a61c /drivers/net/wireless/wl12xx | |
parent | 4c9cfa780643dc3f609366b85fec2444a67cad64 (diff) |
wl12xx: fix some endianess bugs
pointed out by sparse warnings:
CHECK drivers/net/wireless/wl12xx/cmd.c
drivers/net/wireless/wl12xx/cmd.c:987:20: warning: incorrect type in assignment (different base types)
drivers/net/wireless/wl12xx/cmd.c:987:20: expected restricted __le16 [usertype] aging_period
drivers/net/wireless/wl12xx/cmd.c:987:20: got int
CHECK drivers/net/wireless/wl12xx/tx.c
drivers/net/wireless/wl12xx/tx.c:197:2: warning: cast from restricted __le16
drivers/net/wireless/wl12xx/tx.c:197:2: warning: cast from restricted __le16
drivers/net/wireless/wl12xx/tx.c:197:2: warning: cast from restricted __le16
CHECK drivers/net/wireless/wl12xx/acx.c
drivers/net/wireless/wl12xx/acx.c:816:23: warning: incorrect type in assignment (different base types)
drivers/net/wireless/wl12xx/acx.c:816:23: expected restricted __le32 [usertype] rate_policy_idx
drivers/net/wireless/wl12xx/acx.c:816:23: got unsigned char [unsigned] [usertype] idx
Signed-off-by: Eliad Peller <eliad@wizery.com>
Signed-off-by: Luciano Coelho <coelho@ti.com>
Diffstat (limited to 'drivers/net/wireless/wl12xx')
-rw-r--r-- | drivers/net/wireless/wl12xx/acx.c | 2 | ||||
-rw-r--r-- | drivers/net/wireless/wl12xx/cmd.c | 2 | ||||
-rw-r--r-- | drivers/net/wireless/wl12xx/tx.c | 6 |
3 files changed, 5 insertions, 5 deletions
diff --git a/drivers/net/wireless/wl12xx/acx.c b/drivers/net/wireless/wl12xx/acx.c index 646d278bd944..679bb37f0ca4 100644 --- a/drivers/net/wireless/wl12xx/acx.c +++ b/drivers/net/wireless/wl12xx/acx.c | |||
@@ -813,7 +813,7 @@ int wl1271_acx_ap_rate_policy(struct wl1271 *wl, struct conf_tx_rate_class *c, | |||
813 | acx->rate_policy.long_retry_limit = c->long_retry_limit; | 813 | acx->rate_policy.long_retry_limit = c->long_retry_limit; |
814 | acx->rate_policy.aflags = c->aflags; | 814 | acx->rate_policy.aflags = c->aflags; |
815 | 815 | ||
816 | acx->rate_policy_idx = idx; | 816 | acx->rate_policy_idx = cpu_to_le32(idx); |
817 | 817 | ||
818 | ret = wl1271_cmd_configure(wl, ACX_RATE_POLICY, acx, sizeof(*acx)); | 818 | ret = wl1271_cmd_configure(wl, ACX_RATE_POLICY, acx, sizeof(*acx)); |
819 | if (ret < 0) { | 819 | if (ret < 0) { |
diff --git a/drivers/net/wireless/wl12xx/cmd.c b/drivers/net/wireless/wl12xx/cmd.c index e28d9cab1185..1bb8be5e805b 100644 --- a/drivers/net/wireless/wl12xx/cmd.c +++ b/drivers/net/wireless/wl12xx/cmd.c | |||
@@ -984,7 +984,7 @@ int wl1271_cmd_start_bss(struct wl1271 *wl) | |||
984 | 984 | ||
985 | memcpy(cmd->bssid, bss_conf->bssid, ETH_ALEN); | 985 | memcpy(cmd->bssid, bss_conf->bssid, ETH_ALEN); |
986 | 986 | ||
987 | cmd->aging_period = WL1271_AP_DEF_INACTIV_SEC; | 987 | cmd->aging_period = cpu_to_le16(WL1271_AP_DEF_INACTIV_SEC); |
988 | cmd->bss_index = WL1271_AP_BSS_INDEX; | 988 | cmd->bss_index = WL1271_AP_BSS_INDEX; |
989 | cmd->global_hlid = WL1271_AP_GLOBAL_HLID; | 989 | cmd->global_hlid = WL1271_AP_GLOBAL_HLID; |
990 | cmd->broadcast_hlid = WL1271_AP_BROADCAST_HLID; | 990 | cmd->broadcast_hlid = WL1271_AP_BROADCAST_HLID; |
diff --git a/drivers/net/wireless/wl12xx/tx.c b/drivers/net/wireless/wl12xx/tx.c index 2347f2552f14..3507c81c7500 100644 --- a/drivers/net/wireless/wl12xx/tx.c +++ b/drivers/net/wireless/wl12xx/tx.c | |||
@@ -195,9 +195,9 @@ static void wl1271_tx_fill_hdr(struct wl1271 *wl, struct sk_buff *skb, | |||
195 | desc->tx_attr = cpu_to_le16(tx_attr); | 195 | desc->tx_attr = cpu_to_le16(tx_attr); |
196 | 196 | ||
197 | wl1271_debug(DEBUG_TX, "tx_fill_hdr: pad: %d hlid: %d " | 197 | wl1271_debug(DEBUG_TX, "tx_fill_hdr: pad: %d hlid: %d " |
198 | "tx_attr: 0x%x len: %d life: %d mem: %d", pad, (int)desc->hlid, | 198 | "tx_attr: 0x%x len: %d life: %d mem: %d", pad, desc->hlid, |
199 | (int)desc->tx_attr, (int)desc->length, (int)desc->life_time, | 199 | le16_to_cpu(desc->tx_attr), le16_to_cpu(desc->length), |
200 | (int)desc->total_mem_blocks); | 200 | le16_to_cpu(desc->life_time), desc->total_mem_blocks); |
201 | } | 201 | } |
202 | 202 | ||
203 | /* caller must hold wl->mutex */ | 203 | /* caller must hold wl->mutex */ |