diff options
author | Kalle Valo <kvalo@qca.qualcomm.com> | 2012-01-17 08:05:46 -0500 |
---|---|---|
committer | Kalle Valo <kvalo@qca.qualcomm.com> | 2012-01-18 06:41:13 -0500 |
commit | e572602884c4f979cbba4fed413af24797fd01d9 (patch) | |
tree | ca24cfce177a47c7a0240c5c451b91b16e447d6e /drivers/net/wireless | |
parent | 3462735dadb90efd5bed9d2b81222ed7da1390a0 (diff) |
ath6kl: fix uninitialized warning in ath6kl_process_uapsdq()
Before I commited patch c1762a3fe ("ath6kl: Add support for uAPSD") I
did a minor change how up variable is initialised in
ath6kl_process_uapsdq(). But I was sloppy and caused this compiler
warning:
txrx.c:88:5: warning: 'up' may be used uninitialized in this function
Revert my change to fix the warning.
Signed-off-by: Kalle Valo <kvalo@qca.qualcomm.com>
Diffstat (limited to 'drivers/net/wireless')
-rw-r--r-- | drivers/net/wireless/ath/ath6kl/txrx.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/net/wireless/ath/ath6kl/txrx.c b/drivers/net/wireless/ath/ath6kl/txrx.c index 91bbc1ffa493..dd6337142603 100644 --- a/drivers/net/wireless/ath/ath6kl/txrx.c +++ b/drivers/net/wireless/ath/ath6kl/txrx.c | |||
@@ -85,7 +85,7 @@ static bool ath6kl_process_uapsdq(struct ath6kl_sta *conn, | |||
85 | struct ath6kl *ar = vif->ar; | 85 | struct ath6kl *ar = vif->ar; |
86 | bool is_apsdq_empty = false; | 86 | bool is_apsdq_empty = false; |
87 | struct ethhdr *datap = (struct ethhdr *) skb->data; | 87 | struct ethhdr *datap = (struct ethhdr *) skb->data; |
88 | u8 up, traffic_class, *ip_hdr; | 88 | u8 up = 0, traffic_class, *ip_hdr; |
89 | u16 ether_type; | 89 | u16 ether_type; |
90 | struct ath6kl_llc_snap_hdr *llc_hdr; | 90 | struct ath6kl_llc_snap_hdr *llc_hdr; |
91 | 91 | ||
@@ -122,8 +122,6 @@ static bool ath6kl_process_uapsdq(struct ath6kl_sta *conn, | |||
122 | if (ether_type == IP_ETHERTYPE) | 122 | if (ether_type == IP_ETHERTYPE) |
123 | up = ath6kl_wmi_determine_user_priority( | 123 | up = ath6kl_wmi_determine_user_priority( |
124 | ip_hdr, 0); | 124 | ip_hdr, 0); |
125 | } else { | ||
126 | up = 0; | ||
127 | } | 125 | } |
128 | 126 | ||
129 | traffic_class = ath6kl_wmi_get_traffic_class(up); | 127 | traffic_class = ath6kl_wmi_get_traffic_class(up); |