brcmsmac: delete useless variable
authorJulia Lawall <Julia.Lawall@lip6.fr>
Mon, 13 Jan 2014 16:17:26 +0000 (17:17 +0100)
committerJohn W. Linville <linville@tuxdriver.com>
Mon, 13 Jan 2014 19:50:06 +0000 (14:50 -0500)
Delete a variable that is at most only assigned to a constant, but never
used otherwise.

A simplified version of the semantic patch that fixes this problem is as
follows: (http://coccinelle.lip6.fr/)

// <smpl>
@@
type T;
identifier i;
constant c;
@@

-T i;
<... when != i
-i = c;
...>
// </smpl>

Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/brcm80211/brcmsmac/main.c

index 8138f1cff4e56a3bd801dc48f1917963012653ca..9417cb5a2553f70fc2d2733f80c65d61477dc185 100644 (file)
@@ -7108,7 +7108,6 @@ prep_mac80211_status(struct brcms_c_info *wlc, struct d11rxhdr *rxh,
                     struct sk_buff *p,
                     struct ieee80211_rx_status *rx_status)
 {
-       int preamble;
        int channel;
        u32 rspec;
        unsigned char *plcp;
@@ -7191,7 +7190,6 @@ prep_mac80211_status(struct brcms_c_info *wlc, struct d11rxhdr *rxh,
                        rx_status->rate_idx -= BRCMS_LEGACY_5G_RATE_OFFSET;
 
                /* Determine short preamble and rate_idx */
-               preamble = 0;
                if (is_cck_rate(rspec)) {
                        if (rxh->PhyRxStatus_0 & PRXS0_SHORTH)
                                rx_status->flag |= RX_FLAG_SHORTPRE;