diff options
author | Larry Finger <Larry.Finger@lwfinger.net> | 2011-05-22 21:54:25 -0400 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2011-06-01 15:12:29 -0400 |
commit | 9c1cacd2c08ede12765e838c35b6d1e9eb900f60 (patch) | |
tree | 952cedf9bdfdfc7a65a3d6c607cbe6a77fe6b795 | |
parent | 24f7580e852b5472b51eea322bb78454df0054b8 (diff) |
b43: Fix warnings from gcc 4.6.0
gcc 4.6.0 warnings for b43:
CC [M] drivers/net/wireless/b43/lo.o
drivers/net/wireless/b43/lo.c: In function ‘lo_measure_gain_values’:
drivers/net/wireless/b43/lo.c:304:7: warning: variable ‘trsw_rx’ set but not used [-Wunused-but-set-variable]
CC [M] drivers/net/wireless/b43/dma.o
drivers/net/wireless/b43/dma.c: In function ‘free_all_descbuffers’:
drivers/net/wireless/b43/dma.c:760:30: warning: variable ‘desc’ set but not used [-Wunused-but-set-variable]
drivers/net/wireless/b43/dma.c: In function ‘b43_dma_handle_txstatus’:
drivers/net/wireless/b43/dma.c:1391:30: warning: variable ‘desc’ set but not used [-Wunused-but-set-variable]
Signed-off-by: Larry Finger <Larry.Finger@lwfinger.net>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
-rw-r--r-- | drivers/net/wireless/b43/dma.c | 8 | ||||
-rw-r--r-- | drivers/net/wireless/b43/lo.c | 2 |
2 files changed, 4 insertions, 6 deletions
diff --git a/drivers/net/wireless/b43/dma.c b/drivers/net/wireless/b43/dma.c index 11e66cd2e26c..d02cf8300e3e 100644 --- a/drivers/net/wireless/b43/dma.c +++ b/drivers/net/wireless/b43/dma.c | |||
@@ -757,14 +757,14 @@ static void dmacontroller_cleanup(struct b43_dmaring *ring) | |||
757 | 757 | ||
758 | static void free_all_descbuffers(struct b43_dmaring *ring) | 758 | static void free_all_descbuffers(struct b43_dmaring *ring) |
759 | { | 759 | { |
760 | struct b43_dmadesc_generic *desc; | ||
761 | struct b43_dmadesc_meta *meta; | 760 | struct b43_dmadesc_meta *meta; |
762 | int i; | 761 | int i; |
763 | 762 | ||
764 | if (!ring->used_slots) | 763 | if (!ring->used_slots) |
765 | return; | 764 | return; |
766 | for (i = 0; i < ring->nr_slots; i++) { | 765 | for (i = 0; i < ring->nr_slots; i++) { |
767 | desc = ring->ops->idx2desc(ring, i, &meta); | 766 | /* get meta - ignore returned value */ |
767 | ring->ops->idx2desc(ring, i, &meta); | ||
768 | 768 | ||
769 | if (!meta->skb || b43_dma_ptr_is_poisoned(meta->skb)) { | 769 | if (!meta->skb || b43_dma_ptr_is_poisoned(meta->skb)) { |
770 | B43_WARN_ON(!ring->tx); | 770 | B43_WARN_ON(!ring->tx); |
@@ -1388,7 +1388,6 @@ void b43_dma_handle_txstatus(struct b43_wldev *dev, | |||
1388 | { | 1388 | { |
1389 | const struct b43_dma_ops *ops; | 1389 | const struct b43_dma_ops *ops; |
1390 | struct b43_dmaring *ring; | 1390 | struct b43_dmaring *ring; |
1391 | struct b43_dmadesc_generic *desc; | ||
1392 | struct b43_dmadesc_meta *meta; | 1391 | struct b43_dmadesc_meta *meta; |
1393 | int slot, firstused; | 1392 | int slot, firstused; |
1394 | bool frame_succeed; | 1393 | bool frame_succeed; |
@@ -1416,7 +1415,8 @@ void b43_dma_handle_txstatus(struct b43_wldev *dev, | |||
1416 | ops = ring->ops; | 1415 | ops = ring->ops; |
1417 | while (1) { | 1416 | while (1) { |
1418 | B43_WARN_ON(slot < 0 || slot >= ring->nr_slots); | 1417 | B43_WARN_ON(slot < 0 || slot >= ring->nr_slots); |
1419 | desc = ops->idx2desc(ring, slot, &meta); | 1418 | /* get meta - ignore returned value */ |
1419 | ops->idx2desc(ring, slot, &meta); | ||
1420 | 1420 | ||
1421 | if (b43_dma_ptr_is_poisoned(meta->skb)) { | 1421 | if (b43_dma_ptr_is_poisoned(meta->skb)) { |
1422 | b43dbg(dev->wl, "Poisoned TX slot %d (first=%d) " | 1422 | b43dbg(dev->wl, "Poisoned TX slot %d (first=%d) " |
diff --git a/drivers/net/wireless/b43/lo.c b/drivers/net/wireless/b43/lo.c index b32efd45c080..a3dc8bb8ca95 100644 --- a/drivers/net/wireless/b43/lo.c +++ b/drivers/net/wireless/b43/lo.c | |||
@@ -301,14 +301,12 @@ static void lo_measure_gain_values(struct b43_wldev *dev, | |||
301 | max_rx_gain = 0; | 301 | max_rx_gain = 0; |
302 | 302 | ||
303 | if (has_loopback_gain(phy)) { | 303 | if (has_loopback_gain(phy)) { |
304 | int trsw_rx = 0; | ||
305 | int trsw_rx_gain; | 304 | int trsw_rx_gain; |
306 | 305 | ||
307 | if (use_trsw_rx) { | 306 | if (use_trsw_rx) { |
308 | trsw_rx_gain = gphy->trsw_rx_gain / 2; | 307 | trsw_rx_gain = gphy->trsw_rx_gain / 2; |
309 | if (max_rx_gain >= trsw_rx_gain) { | 308 | if (max_rx_gain >= trsw_rx_gain) { |
310 | trsw_rx_gain = max_rx_gain - trsw_rx_gain; | 309 | trsw_rx_gain = max_rx_gain - trsw_rx_gain; |
311 | trsw_rx = 0x20; | ||
312 | } | 310 | } |
313 | } else | 311 | } else |
314 | trsw_rx_gain = max_rx_gain; | 312 | trsw_rx_gain = max_rx_gain; |