| Message ID | 20200821071644.109970-1-lee.jones@linaro.org |
|---|---|
| Headers | show |
| Series | Set 2: Rid W=1 warnings in Wireless | expand |
On Tue, 01 Sep 2020, Kalle Valo wrote: > Lee Jones <lee.jones@linaro.org> writes: > > > 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 Wireless. My plan > > is to work through all of them over the next few weeks. > > Hopefully it won't be too long before drivers/net/wireless > > builds clean with W=1 enabled. > > BTW, now the patches are in random order and it's quite annoying to > review when there's no logic. Grouping them by the driver would be a lot > more pleasent for reviewers. My script makes a best effort attempt to group changes by file. It takes the first warning presented by the compiler then greps the output for all issues pertaining to that file. I then split the patch by issue (i.e. different patches for; kernel-doc, unused variables, bracketing etc). One issue you might be seeing is the potential for one fixed issue to cause another i.e. when one unused variable is removed which was the only user of another, leading to a subsequent fix of the newly unused variable. Other than that, I'm not sure why they would end up out of order.
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 Wireless. My plan is to work through all of them over the next few weeks. Hopefully it won't be too long before drivers/net/wireless builds clean with W=1 enabled. This set brings the total number of (arm, arm64, x86, mips and ppc *combined* i.e. some duplicated) warnings down from 4983 (since the last set) to 2066. Lee Jones (32): wireless: marvell: mwifiex: sdio: Move 'static const struct's into their own header file wireless: rsi: rsi_91x_mac80211: Add description for function param 'sta' wireless: intel: iwlwifi: mvm: ops: Remove unused static struct 'iwl_mvm_debug_names' wireless: broadcom: brcm80211: brcmsmac: ampdu: Remove a bunch of unused variables wireless: broadcom: brcm80211: brcmfmac: p2p: Fix a bunch of function docs wireless: rsi: rsi_91x_mgmt: Add descriptions for rsi_set_vap_capabilities()'s parameters wireless: intel: iwlwifi: dvm: rx: Demote a couple of nonconformant kernel-doc headers wireless: intel: iwlwifi: mvm: utils: Fix some doc-rot wireless: broadcom: brcm80211: brcmsmac: main: Remove a bunch of unused variables wireless: rsi: rsi_91x_ps: Source file headers do not make good kernel-doc candidates wireless: intel: iwlwifi: dvm: scan: Demote a few nonconformant kernel-doc headers wireless: brcm80211: brcmfmac: firmware: Demote seemingly unintentional kernel-doc header wireless: intel: iwlwifi: dvm: rxon: Demote non-conformant kernel-doc headers wireless: rsi: rsi_91x_coex: File headers are not suitable for kernel-doc wireless: intel: iwlegacy: 4965-mac: Convert function headers to standard comment blocks wireless: brcm80211: btcoex: Update 'brcmf_btcoex_state' and demote others wireless: broadcom: b43: phy_ht: Remove 9 year old TODO wireless: intel: iwlwifi: mvm: tx: Demote misuse of kernel-doc headers wireless: intel: iwlwifi: dvm: devices: Fix function documentation formatting issues wireless: rsi: rsi_91x_debugfs: Source file headers are not suitable for kernel-doc wireless: intel: iwlwifi: iwl-drv: Provide descriptions debugfs dentries wireless: intel: iwlegacy: 4965-rs: Demote non kernel-doc headers to standard comment blocks wireless: intel: iwlegacy: 4965-calib: Demote seemingly accidental kernel-doc header wireless: brcmfmac: fwsignal: Remove unused variable 'brcmf_fws_prio2fifo' wireless: ath: wil6210: wmi: Fix formatting and demote non-conforming function headers wireless: ath: wil6210: interrupt: Demote comment header which is clearly not kernel-doc wireless: ath: wil6210: txrx: Demote obvious abuse of kernel-doc wireless: ath: wil6210: txrx_edma: Demote comments which are clearly not kernel-doc wireless: realtek: rtl8192c: phy_common: Remove unused variable 'bbvalue' wireless: ath: wil6210: pmc: Demote a few nonconformant kernel-doc function headers wireless: ath: wil6210: wil_platform: Demote kernel-doc header to standard comment block wireless: mediatek: mt76x0: Move tables used only by init.c to their own header file drivers/net/wireless/ath/wil6210/interrupt.c | 2 +- drivers/net/wireless/ath/wil6210/pmc.c | 8 +- drivers/net/wireless/ath/wil6210/txrx.c | 20 +- drivers/net/wireless/ath/wil6210/txrx_edma.c | 6 +- .../net/wireless/ath/wil6210/wil_platform.c | 2 +- drivers/net/wireless/ath/wil6210/wmi.c | 28 +- drivers/net/wireless/broadcom/b43/phy_ht.c | 3 - .../broadcom/brcm80211/brcmfmac/btcoex.c | 12 +- .../broadcom/brcm80211/brcmfmac/firmware.c | 2 +- .../broadcom/brcm80211/brcmfmac/fwsignal.c | 14 - .../broadcom/brcm80211/brcmfmac/p2p.c | 22 +- .../broadcom/brcm80211/brcmsmac/ampdu.c | 28 +- .../broadcom/brcm80211/brcmsmac/main.c | 38 +- .../net/wireless/intel/iwlegacy/4965-calib.c | 2 +- .../net/wireless/intel/iwlegacy/4965-mac.c | 55 +-- drivers/net/wireless/intel/iwlegacy/4965-rs.c | 10 +- .../net/wireless/intel/iwlwifi/dvm/devices.c | 8 +- drivers/net/wireless/intel/iwlwifi/dvm/rx.c | 4 +- drivers/net/wireless/intel/iwlwifi/dvm/rxon.c | 6 +- drivers/net/wireless/intel/iwlwifi/dvm/scan.c | 8 +- drivers/net/wireless/intel/iwlwifi/iwl-drv.c | 5 +- drivers/net/wireless/intel/iwlwifi/mvm/ops.c | 9 - drivers/net/wireless/intel/iwlwifi/mvm/tx.c | 4 +- .../net/wireless/intel/iwlwifi/mvm/utils.c | 7 +- .../wireless/marvell/mwifiex/sdio-tables.h | 451 ++++++++++++++++++ drivers/net/wireless/marvell/mwifiex/sdio.c | 1 + drivers/net/wireless/marvell/mwifiex/sdio.h | 427 ----------------- .../net/wireless/mediatek/mt76/mt76x0/init.c | 1 + .../wireless/mediatek/mt76/mt76x0/initvals.h | 145 ------ .../mediatek/mt76/mt76x0/initvals_init.h | 159 ++++++ .../realtek/rtlwifi/rtl8192c/phy_common.c | 3 +- drivers/net/wireless/rsi/rsi_91x_coex.c | 2 +- drivers/net/wireless/rsi/rsi_91x_debugfs.c | 2 +- drivers/net/wireless/rsi/rsi_91x_mac80211.c | 1 + drivers/net/wireless/rsi/rsi_91x_mgmt.c | 3 + drivers/net/wireless/rsi/rsi_91x_ps.c | 2 +- 36 files changed, 738 insertions(+), 762 deletions(-) create mode 100644 drivers/net/wireless/marvell/mwifiex/sdio-tables.h create mode 100644 drivers/net/wireless/mediatek/mt76/mt76x0/initvals_init.h