[PATCH 00/30] Rid W=1 warnings in Networking

From: Lee Jones
Date: Fri Aug 14 2020 - 07:39:43 EST


This set is part of a larger effort attempting to clean-up W=1
kernel builds, which are currently overwhelmingly riddled with
niggly little warnings.

There are quite a few W=1 warnings in the Networking code. My
plan is to work through all of them over the next few weeks.
Hopefully it won't be too long before drivers/net builds clean
with W=1 enabled.

Lee Jones (30):
net: bonding: bond_3ad: Fix a bunch of kerneldoc parameter issues
net: bonding: bond_main: Document 'proto' and rename 'new_active'
parameters
net: ethernet: 3com: 3c574_cs: Remove set but unused variables 'tx'
and 'rx'
net: bonding: bond_alb: Describe
alb_handle_addr_collision_on_attach()'s 'bond' and 'addr' params
net: wireless: atmel: atmel: Demote non-kerneldoc header to standard
comment block
net: ethernet: 8390: axnet_cs: Document unused parameter 'txqueue'
net: wireless: broadcom: b43: main: Add braces around empty statements
net: wireless: ath: carl9170: Mark 'ar9170_qmap' as __maybe_unused
net: wireless: ath: ath5k: pcu: Add a description for 'band' remove
one for 'mode'
net: wan: dlci: Remove set but not used variable 'err'
net: wireless: cisco: airo: Place brackets around empty statement
net: wireless: cisco: airo: Fix a myriad of coding style issues
net: wireless: ath: wil6210: cfg80211: Demote non-kerneldoc headers to
standard comment blocks
net: wireless: intel: iwlegacy: common: Remove set but not used
variable 'len'
net: wireless: intel: iwlegacy: common: Demote kerneldoc headers to
standard comment blocks
net: wireless: intel: ipw2200: Remove set but unused variables 'rc'
and 'w'
net: wireless: broadcom: b43legacy: main: Provide braces around empty
'if' body
net: fddi: skfp: hwmtm: Remove seemingly unused variable 'ID_sccs'
net: fddi: skfp: fplustm: Remove seemingly unused variable 'ID_sccs'
net: wireless: ath: ath5k: base: Fix kerneldoc formatting issue
net: fddi: skfp: smt: Place definition of 'smt_pdef' under same
stipulations as its use
net: wireless: broadcom: brcm80211: brcmfmac: fweh: Remove set but
unused variable 'err'
net: wireless: broadcom: brcm80211: brcmfmac: fweh: Fix docrot related
function documentation issues
net: wireless: broadcom: brcm80211: brcmsmac: mac80211_if: Demote a
few non-conformant kerneldoc headers
net: wireless: intel: ipw2x00: ipw2200: Demote lots of nonconformant
kerneldoc comments
net: wireless: broadcom: b43: phy_common: Demote non-conformant
kerneldoc header
net: fddi: skfp: smt: Remove seemingly unused variable 'ID_sccs'
net: fddi: skfp: cfm: Remove set but unused variable 'oldstate'
net: fddi: skfp: cfm: Remove seemingly unused variable 'ID_sccs'
net: wireless: broadcom: b43: phy_n: Add empty braces around empty
statements

drivers/net/bonding/bond_3ad.c | 15 +-
drivers/net/bonding/bond_alb.c | 4 +-
drivers/net/bonding/bond_main.c | 4 +-
drivers/net/ethernet/3com/3c574_cs.c | 6 +-
drivers/net/ethernet/8390/axnet_cs.c | 1 +
drivers/net/fddi/skfp/cfm.c | 17 +-
drivers/net/fddi/skfp/fplustm.c | 4 -
drivers/net/fddi/skfp/hwmtm.c | 4 -
drivers/net/fddi/skfp/smt.c | 7 +-
drivers/net/wan/dlci.c | 3 +-
drivers/net/wireless/ath/ath5k/base.c | 2 +-
drivers/net/wireless/ath/ath5k/pcu.c | 2 +-
drivers/net/wireless/ath/carl9170/carl9170.h | 2 +-
drivers/net/wireless/ath/wil6210/cfg80211.c | 4 +-
drivers/net/wireless/atmel/atmel.c | 2 +-
drivers/net/wireless/broadcom/b43/main.c | 6 +-
.../net/wireless/broadcom/b43/phy_common.c | 2 +-
drivers/net/wireless/broadcom/b43/phy_n.c | 19 +-
.../net/wireless/broadcom/b43legacy/main.c | 3 +-
.../broadcom/brcm80211/brcmfmac/fweh.c | 13 +-
.../broadcom/brcm80211/brcmsmac/mac80211_if.c | 11 +-
drivers/net/wireless/cisco/airo.c | 898 +++++++++---------
drivers/net/wireless/intel/ipw2x00/ipw2200.c | 45 +-
drivers/net/wireless/intel/iwlegacy/common.c | 72 +-
24 files changed, 582 insertions(+), 564 deletions(-)

--
2.25.1