All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 4/5] drivers/net/wireless/brcm80211/brcmsmac/main.c: don't use unmodified return variable name
@ 2015-03-16 14:18 Peter Teoh
  0 siblings, 0 replies; only message in thread
From: Peter Teoh @ 2015-03-16 14:18 UTC (permalink / raw)
  To: Brett Rudley, Arend van Spriel, Franky (Zhenhui) Lin,
	Hante Meuleman, LKML

Discovered by Coccinelle.   Removed the use of variable for storing
returning values, which is un-modified / un-used throughout.

Signed-off-by: Peter Teoh <htmldeveloper@gmail.com

diff -u -p a/drivers/net/wireless/brcm80211/brcmsmac/main.c
b/drivers/net/wireless/brcm80211/brcmsmac/main.c
--- a/drivers/net/wireless/brcm80211/brcmsmac/main.c
+++ b/drivers/net/wireless/brcm80211/brcmsmac/main.c
@@ -3777,7 +3777,6 @@ static void brcms_c_set_ps_ctrl(struct b
  */
 static int brcms_c_set_mac(struct brcms_bss_cfg *bsscfg)
 {
-       int err = 0;
        struct brcms_c_info *wlc = bsscfg->wlc;

        /* enter the MAC addr into the RXE match registers */
@@ -3785,7 +3784,7 @@ static int brcms_c_set_mac(struct brcms_

        brcms_c_ampdu_macaddr_upd(wlc);

-       return err;
+       return 0;
 }

 /* Write the BSS config's BSSID address to core (set_bssid in d11procs.tcl).

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2015-03-16 14:19 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-03-16 14:18 [PATCH 4/5] drivers/net/wireless/brcm80211/brcmsmac/main.c: don't use unmodified return variable name Peter Teoh

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.