brcmfmac: remove redundant assignment to pointer 'entry'
authorColin Ian King <colin.king@canonical.com>
Fri, 4 Dec 2020 18:04:59 +0000 (18:04 +0000)
committerKalle Valo <kvalo@codeaurora.org>
Mon, 7 Dec 2020 16:36:11 +0000 (18:36 +0200)
The pointer 'entry' is being initialized with a value that is never read
and it is being updated later with a new value.  The initialization is
redundant and can be removed.

Addresses-Coverity: ("Unused value")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Link: https://lore.kernel.org/r/20201204180459.1148257-1-colin.king@canonical.com
drivers/net/wireless/broadcom/brcm80211/brcmfmac/fwsignal.c

index 437e83e..19b0f31 100644 (file)
@@ -746,7 +746,7 @@ brcmf_fws_macdesc_lookup(struct brcmf_fws_info *fws, u8 *ea)
 static struct brcmf_fws_mac_descriptor*
 brcmf_fws_macdesc_find(struct brcmf_fws_info *fws, struct brcmf_if *ifp, u8 *da)
 {
-       struct brcmf_fws_mac_descriptor *entry = &fws->desc.other;
+       struct brcmf_fws_mac_descriptor *entry;
        bool multicast;
 
        multicast = is_multicast_ether_addr(da);