* [PATCH 01/11] net: ath: ath5k: Fix wrong function name in comments
2021-05-17 5:01 [PATCH 00/11] Rid W=1 warnings in wireless Yang Shen
@ 2021-05-17 5:01 ` Yang Shen
2021-06-15 14:01 ` Kalle Valo
2021-05-17 5:01 ` [PATCH 02/11] net: ath: " Yang Shen
` (9 subsequent siblings)
10 siblings, 1 reply; 21+ messages in thread
From: Yang Shen @ 2021-05-17 5:01 UTC (permalink / raw)
To: kvalo
Cc: linux-wireless, linux-kernel, Yang Shen, Jiri Slaby,
Nick Kossifidis, Luis Chamberlain
Fixes the following W=1 kernel build warning(s):
drivers/net/wireless/ath/ath5k/pcu.c:865: warning: expecting prototype for at5k_hw_stop_rx_pcu(). Prototype was for ath5k_hw_stop_rx_pcu() instead
Cc: Jiri Slaby <jirislaby@kernel.org>
Cc: Nick Kossifidis <mickflemm@gmail.com>
Cc: Luis Chamberlain <mcgrof@kernel.org>
Signed-off-by: Yang Shen <shenyang39@huawei.com>
---
drivers/net/wireless/ath/ath5k/pcu.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/wireless/ath/ath5k/pcu.c b/drivers/net/wireless/ath/ath5k/pcu.c
index f2db7cf16566..3f4ce4e9c532 100644
--- a/drivers/net/wireless/ath/ath5k/pcu.c
+++ b/drivers/net/wireless/ath/ath5k/pcu.c
@@ -855,7 +855,7 @@ ath5k_hw_start_rx_pcu(struct ath5k_hw *ah)
}
/**
- * at5k_hw_stop_rx_pcu() - Stop RX engine
+ * ath5k_hw_stop_rx_pcu() - Stop RX engine
* @ah: The &struct ath5k_hw
*
* Stops RX engine on PCU
--
2.17.1
^ permalink raw reply related [flat|nested] 21+ messages in thread* [PATCH 02/11] net: ath: Fix wrong function name in comments
2021-05-17 5:01 [PATCH 00/11] Rid W=1 warnings in wireless Yang Shen
2021-05-17 5:01 ` [PATCH 01/11] net: ath: ath5k: Fix wrong function name in comments Yang Shen
@ 2021-05-17 5:01 ` Yang Shen
2021-05-17 5:01 ` [PATCH 03/11] net: ath: wil6210: " Yang Shen
` (8 subsequent siblings)
10 siblings, 0 replies; 21+ messages in thread
From: Yang Shen @ 2021-05-17 5:01 UTC (permalink / raw)
To: kvalo; +Cc: linux-wireless, linux-kernel, Yang Shen
Fixes the following W=1 kernel build warning(s):
drivers/net/wireless/ath/hw.c:119: warning: expecting prototype for ath_hw_set_bssid_mask(). Prototype was for ath_hw_setbssidmask() instead
Signed-off-by: Yang Shen <shenyang39@huawei.com>
---
drivers/net/wireless/ath/hw.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/wireless/ath/hw.c b/drivers/net/wireless/ath/hw.c
index eae9abf540a7..b53ebb3ac9a2 100644
--- a/drivers/net/wireless/ath/hw.c
+++ b/drivers/net/wireless/ath/hw.c
@@ -24,7 +24,7 @@
#define REG_WRITE(_ah, _reg, _val) (common->ops->write)(_ah, _val, _reg)
/**
- * ath_hw_set_bssid_mask - filter out bssids we listen
+ * ath_hw_setbssidmask - filter out bssids we listen
*
* @common: the ath_common struct for the device.
*
--
2.17.1
^ permalink raw reply related [flat|nested] 21+ messages in thread* [PATCH 03/11] net: ath: wil6210: Fix wrong function name in comments
2021-05-17 5:01 [PATCH 00/11] Rid W=1 warnings in wireless Yang Shen
2021-05-17 5:01 ` [PATCH 01/11] net: ath: ath5k: Fix wrong function name in comments Yang Shen
2021-05-17 5:01 ` [PATCH 02/11] net: ath: " Yang Shen
@ 2021-05-17 5:01 ` Yang Shen
2021-05-17 5:01 ` [PATCH 04/11] net: broadcom: brcmfmac: Demote non-compliant kernel-doc headers Yang Shen
` (7 subsequent siblings)
10 siblings, 0 replies; 21+ messages in thread
From: Yang Shen @ 2021-05-17 5:01 UTC (permalink / raw)
To: kvalo; +Cc: linux-wireless, linux-kernel, Yang Shen, Maya Erez
Fixes the following W=1 kernel build warning(s):
drivers/net/wireless/ath/wil6210/interrupt.c:28: warning: expecting prototype for Theory of operation(). Prototype was for WIL6210_IRQ_DISABLE() instead
drivers/net/wireless/ath/wil6210/wmi.c:227: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
drivers/net/wireless/ath/wil6210/wmi.c:245: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
drivers/net/wireless/ath/wil6210/wmi.c:263: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
Cc: Maya Erez <merez@codeaurora.org>
Signed-off-by: Yang Shen <shenyang39@huawei.com>
---
drivers/net/wireless/ath/wil6210/interrupt.c | 2 +-
drivers/net/wireless/ath/wil6210/wmi.c | 6 +++---
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/net/wireless/ath/wil6210/interrupt.c b/drivers/net/wireless/ath/wil6210/interrupt.c
index d13d081fdcc6..67172385a5d6 100644
--- a/drivers/net/wireless/ath/wil6210/interrupt.c
+++ b/drivers/net/wireless/ath/wil6210/interrupt.c
@@ -9,7 +9,7 @@
#include "wil6210.h"
#include "trace.h"
-/**
+/*
* Theory of operation:
*
* There is ISR pseudo-cause register,
diff --git a/drivers/net/wireless/ath/wil6210/wmi.c b/drivers/net/wireless/ath/wil6210/wmi.c
index 02ad44997e87..2dc8406736f4 100644
--- a/drivers/net/wireless/ath/wil6210/wmi.c
+++ b/drivers/net/wireless/ath/wil6210/wmi.c
@@ -224,7 +224,7 @@ struct auth_no_hdr {
u8 led_polarity = LED_POLARITY_LOW_ACTIVE;
/**
- * return AHB address for given firmware internal (linker) address
+ * wmi_addr_remap - return AHB address for given firmware internal (linker) address
* @x: internal address
* If address have no valid AHB mapping, return 0
*/
@@ -242,7 +242,7 @@ static u32 wmi_addr_remap(u32 x)
}
/**
- * find fw_mapping entry by section name
+ * wil_find_fw_mapping - find fw_mapping entry by section name
* @section: section name
*
* Return pointer to section or NULL if not found
@@ -260,7 +260,7 @@ struct fw_map *wil_find_fw_mapping(const char *section)
}
/**
- * Check address validity for WMI buffer; remap if needed
+ * wmi_buffer_block - Check address validity for WMI buffer; remap if needed
* @wil: driver data
* @ptr_: internal (linker) fw/ucode address
* @size: if non zero, validate the block does not
--
2.17.1
^ permalink raw reply related [flat|nested] 21+ messages in thread* [PATCH 04/11] net: broadcom: brcmfmac: Demote non-compliant kernel-doc headers
2021-05-17 5:01 [PATCH 00/11] Rid W=1 warnings in wireless Yang Shen
` (2 preceding siblings ...)
2021-05-17 5:01 ` [PATCH 03/11] net: ath: wil6210: " Yang Shen
@ 2021-05-17 5:01 ` Yang Shen
2021-05-19 12:41 ` Arend van Spriel
2021-06-15 10:37 ` [04/11] " Kalle Valo
2021-05-17 5:01 ` [PATCH 05/11] net: intel: ipw2x00: Fix wrong function name in comments Yang Shen
` (6 subsequent siblings)
10 siblings, 2 replies; 21+ messages in thread
From: Yang Shen @ 2021-05-17 5:01 UTC (permalink / raw)
To: kvalo; +Cc: linux-wireless, linux-kernel, Yang Shen, Franky Lin, Hante Meuleman
Fixes the following W=1 kernel build warning(s):
drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c:2040: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
drivers/net/wireless/broadcom/brcm80211/brcmfmac/sdio.c:1295: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
Cc: Franky Lin <franky.lin@broadcom.com>
Cc: Hante Meuleman <hante.meuleman@broadcom.com>
Signed-off-by: Yang Shen <shenyang39@huawei.com>
---
drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c | 2 +-
drivers/net/wireless/broadcom/brcm80211/brcmfmac/sdio.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c
index 34cd8a7401fe..9ac0d8c73d5a 100644
--- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c
+++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c
@@ -2037,7 +2037,7 @@ static void brcmf_p2p_get_current_chanspec(struct brcmf_p2p_info *p2p,
}
/**
- * Change a P2P Role.
+ * brcmf_p2p_ifchange - Change a P2P Role.
* @cfg: driver private data for cfg80211 interface.
* @if_type: interface type.
* Returns 0 if success.
diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/sdio.c b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/sdio.c
index 16ed325795a8..d1dda851e176 100644
--- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/sdio.c
+++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/sdio.c
@@ -1291,7 +1291,7 @@ static void brcmf_sdio_free_glom(struct brcmf_sdio *bus)
}
}
-/**
+/*
* brcmfmac sdio bus specific header
* This is the lowest layer header wrapped on the packets transmitted between
* host and WiFi dongle which contains information needed for SDIO core and
--
2.17.1
^ permalink raw reply related [flat|nested] 21+ messages in thread* Re: [PATCH 04/11] net: broadcom: brcmfmac: Demote non-compliant kernel-doc headers
2021-05-17 5:01 ` [PATCH 04/11] net: broadcom: brcmfmac: Demote non-compliant kernel-doc headers Yang Shen
@ 2021-05-19 12:41 ` Arend van Spriel
2021-06-15 10:37 ` [04/11] " Kalle Valo
1 sibling, 0 replies; 21+ messages in thread
From: Arend van Spriel @ 2021-05-19 12:41 UTC (permalink / raw)
To: Yang Shen, kvalo; +Cc: linux-wireless, linux-kernel, Franky Lin, Hante Meuleman
On 17-05-2021 07:01, Yang Shen wrote:
> Fixes the following W=1 kernel build warning(s):
>
> drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c:2040: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
> drivers/net/wireless/broadcom/brcm80211/brcmfmac/sdio.c:1295: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
Acked-by: Arend van Spriel <arend.vanspriel@broadcom.com>
> Cc: Franky Lin <franky.lin@broadcom.com>
> Cc: Hante Meuleman <hante.meuleman@broadcom.com>
> Signed-off-by: Yang Shen <shenyang39@huawei.com>
> ---
> drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c | 2 +-
> drivers/net/wireless/broadcom/brcm80211/brcmfmac/sdio.c | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [04/11] brcmfmac: Demote non-compliant kernel-doc headers
2021-05-17 5:01 ` [PATCH 04/11] net: broadcom: brcmfmac: Demote non-compliant kernel-doc headers Yang Shen
2021-05-19 12:41 ` Arend van Spriel
@ 2021-06-15 10:37 ` Kalle Valo
1 sibling, 0 replies; 21+ messages in thread
From: Kalle Valo @ 2021-06-15 10:37 UTC (permalink / raw)
To: Yang Shen
Cc: linux-wireless, linux-kernel, Yang Shen, Franky Lin, Hante Meuleman
Yang Shen <shenyang39@huawei.com> wrote:
> Fixes the following W=1 kernel build warning(s):
>
> drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c:2040: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
> drivers/net/wireless/broadcom/brcm80211/brcmfmac/sdio.c:1295: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
>
> Cc: Franky Lin <franky.lin@broadcom.com>
> Cc: Hante Meuleman <hante.meuleman@broadcom.com>
> Signed-off-by: Yang Shen <shenyang39@huawei.com>
> Acked-by: Arend van Spriel <arend.vanspriel@broadcom.com>
Patch applied to wireless-drivers-next.git, thanks.
ea3f903caea0 brcmfmac: Demote non-compliant kernel-doc headers
--
https://patchwork.kernel.org/project/linux-wireless/patch/20210517050141.61488-5-shenyang39@huawei.com/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
^ permalink raw reply [flat|nested] 21+ messages in thread
* [PATCH 05/11] net: intel: ipw2x00: Fix wrong function name in comments
2021-05-17 5:01 [PATCH 00/11] Rid W=1 warnings in wireless Yang Shen
` (3 preceding siblings ...)
2021-05-17 5:01 ` [PATCH 04/11] net: broadcom: brcmfmac: Demote non-compliant kernel-doc headers Yang Shen
@ 2021-05-17 5:01 ` Yang Shen
2021-05-17 16:30 ` Stanislav Yakovlev
2021-06-15 13:12 ` [05/11] " Kalle Valo
2021-05-17 5:01 ` [PATCH 06/11] net: intel: iwlwifi: Demote non-compliant kernel-doc headers Yang Shen
` (5 subsequent siblings)
10 siblings, 2 replies; 21+ messages in thread
From: Yang Shen @ 2021-05-17 5:01 UTC (permalink / raw)
To: kvalo; +Cc: linux-wireless, linux-kernel, Yang Shen, Stanislav Yakovlev
Fixes the following W=1 kernel build warning(s):
drivers/net/wireless/intel/ipw2x00/ipw2100.c:5359: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
drivers/net/wireless/intel/ipw2x00/ipw2100.c:6533: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
drivers/net/wireless/intel/ipw2x00/ipw2100.c:6565: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
Cc: Stanislav Yakovlev <stas.yakovlev@gmail.com>
Signed-off-by: Yang Shen <shenyang39@huawei.com>
---
drivers/net/wireless/intel/ipw2x00/ipw2100.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/net/wireless/intel/ipw2x00/ipw2100.c b/drivers/net/wireless/intel/ipw2x00/ipw2100.c
index 23fbddd0c1f8..a1c7b04930f5 100644
--- a/drivers/net/wireless/intel/ipw2x00/ipw2100.c
+++ b/drivers/net/wireless/intel/ipw2x00/ipw2100.c
@@ -5356,7 +5356,7 @@ struct ipw2100_wep_key {
#define WEP_STR_128(x) x[0],x[1],x[2],x[3],x[4],x[5],x[6],x[7],x[8],x[9],x[10]
/**
- * Set a the wep key
+ * ipw2100_set_key - Set a the wep key
*
* @priv: struct to work on
* @idx: index of the key we want to set
@@ -6530,7 +6530,7 @@ static struct pci_driver ipw2100_pci_driver = {
};
/**
- * Initialize the ipw2100 driver/module
+ * ipw2100_init - Initialize the ipw2100 driver/module
*
* @returns 0 if ok, < 0 errno node con error.
*
@@ -6562,7 +6562,7 @@ static int __init ipw2100_init(void)
}
/**
- * Cleanup ipw2100 driver registration
+ * ipw2100_exit - Cleanup ipw2100 driver registration
*/
static void __exit ipw2100_exit(void)
{
@@ -7197,7 +7197,7 @@ static int ipw2100_wx_set_txpow(struct net_device *dev,
{
struct ipw2100_priv *priv = libipw_priv(dev);
int err = 0, value;
-
+
if (ipw_radio_kill_sw(priv, wrqu->txpower.disabled))
return -EINPROGRESS;
--
2.17.1
^ permalink raw reply related [flat|nested] 21+ messages in thread* Re: [PATCH 05/11] net: intel: ipw2x00: Fix wrong function name in comments
2021-05-17 5:01 ` [PATCH 05/11] net: intel: ipw2x00: Fix wrong function name in comments Yang Shen
@ 2021-05-17 16:30 ` Stanislav Yakovlev
2021-06-15 13:12 ` [05/11] " Kalle Valo
1 sibling, 0 replies; 21+ messages in thread
From: Stanislav Yakovlev @ 2021-05-17 16:30 UTC (permalink / raw)
To: Yang Shen; +Cc: Kalle Valo, wireless, linux-kernel
On Mon, 17 May 2021 at 12:14, Yang Shen <shenyang39@huawei.com> wrote:
>
> Fixes the following W=1 kernel build warning(s):
>
> drivers/net/wireless/intel/ipw2x00/ipw2100.c:5359: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
> drivers/net/wireless/intel/ipw2x00/ipw2100.c:6533: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
> drivers/net/wireless/intel/ipw2x00/ipw2100.c:6565: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
>
> Cc: Stanislav Yakovlev <stas.yakovlev@gmail.com>
> Signed-off-by: Yang Shen <shenyang39@huawei.com>
> ---
> drivers/net/wireless/intel/ipw2x00/ipw2100.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
Looks fine, thanks!
Stanislav.
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [05/11] ipw2x00: Fix wrong function name in comments
2021-05-17 5:01 ` [PATCH 05/11] net: intel: ipw2x00: Fix wrong function name in comments Yang Shen
2021-05-17 16:30 ` Stanislav Yakovlev
@ 2021-06-15 13:12 ` Kalle Valo
2021-06-18 7:00 ` Yang Shen
1 sibling, 1 reply; 21+ messages in thread
From: Kalle Valo @ 2021-06-15 13:12 UTC (permalink / raw)
To: Yang Shen; +Cc: linux-wireless, linux-kernel, Yang Shen, Stanislav Yakovlev
Yang Shen <shenyang39@huawei.com> wrote:
> Fixes the following W=1 kernel build warning(s):
>
> drivers/net/wireless/intel/ipw2x00/ipw2100.c:5359: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
> drivers/net/wireless/intel/ipw2x00/ipw2100.c:6533: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
> drivers/net/wireless/intel/ipw2x00/ipw2100.c:6565: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
>
> Cc: Stanislav Yakovlev <stas.yakovlev@gmail.com>
> Signed-off-by: Yang Shen <shenyang39@huawei.com>
Failed to apply, please rebase:
Recorded preimage for 'drivers/net/wireless/intel/ipw2x00/ipw2100.c'
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch' to see the failed patch
Applying: ipw2x00: Fix wrong function name in comments
Using index info to reconstruct a base tree...
M drivers/net/wireless/intel/ipw2x00/ipw2100.c
Falling back to patching base and 3-way merge...
Auto-merging drivers/net/wireless/intel/ipw2x00/ipw2100.c
CONFLICT (content): Merge conflict in drivers/net/wireless/intel/ipw2x00/ipw2100.c
Patch failed at 0001 ipw2x00: Fix wrong function name in comments
Patch set to Changes Requested.
--
https://patchwork.kernel.org/project/linux-wireless/patch/20210517050141.61488-6-shenyang39@huawei.com/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [05/11] ipw2x00: Fix wrong function name in comments
2021-06-15 13:12 ` [05/11] " Kalle Valo
@ 2021-06-18 7:00 ` Yang Shen
0 siblings, 0 replies; 21+ messages in thread
From: Yang Shen @ 2021-06-18 7:00 UTC (permalink / raw)
To: Kalle Valo; +Cc: linux-wireless, linux-kernel, Stanislav Yakovlev
Got it, I'll rebase on the latest branch.
On 2021/6/15 21:12, Kalle Valo wrote:
> Yang Shen <shenyang39@huawei.com> wrote:
>
>> Fixes the following W=1 kernel build warning(s):
>>
>> drivers/net/wireless/intel/ipw2x00/ipw2100.c:5359: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
>> drivers/net/wireless/intel/ipw2x00/ipw2100.c:6533: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
>> drivers/net/wireless/intel/ipw2x00/ipw2100.c:6565: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
>>
>> Cc: Stanislav Yakovlev <stas.yakovlev@gmail.com>
>> Signed-off-by: Yang Shen <shenyang39@huawei.com>
>
> Failed to apply, please rebase:
>
> Recorded preimage for 'drivers/net/wireless/intel/ipw2x00/ipw2100.c'
> error: Failed to merge in the changes.
> hint: Use 'git am --show-current-patch' to see the failed patch
> Applying: ipw2x00: Fix wrong function name in comments
> Using index info to reconstruct a base tree...
> M drivers/net/wireless/intel/ipw2x00/ipw2100.c
> Falling back to patching base and 3-way merge...
> Auto-merging drivers/net/wireless/intel/ipw2x00/ipw2100.c
> CONFLICT (content): Merge conflict in drivers/net/wireless/intel/ipw2x00/ipw2100.c
> Patch failed at 0001 ipw2x00: Fix wrong function name in comments
>
> Patch set to Changes Requested.
>
^ permalink raw reply [flat|nested] 21+ messages in thread
* [PATCH 06/11] net: intel: iwlwifi: Demote non-compliant kernel-doc headers
2021-05-17 5:01 [PATCH 00/11] Rid W=1 warnings in wireless Yang Shen
` (4 preceding siblings ...)
2021-05-17 5:01 ` [PATCH 05/11] net: intel: ipw2x00: Fix wrong function name in comments Yang Shen
@ 2021-05-17 5:01 ` Yang Shen
2021-05-18 3:08 ` Yang Shen
2021-05-17 5:01 ` [PATCH 07/11] net: marvell: libertas_tf: Fix wrong function name in comments Yang Shen
` (4 subsequent siblings)
10 siblings, 1 reply; 21+ messages in thread
From: Yang Shen @ 2021-05-17 5:01 UTC (permalink / raw)
To: kvalo; +Cc: linux-wireless, linux-kernel, Yang Shen, Stanislav Yakovlev
Fixes the following W=1 kernel build warning(s):
drivers/net/wireless/intel/iwlwifi/mvm/rfi.c:11: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
drivers/net/wireless/intel/iwlwifi/mvm/rfi.c:14:1: warning: ‘static’ is not at beginning of declaration [-Wold-style-declaration]
Cc: Stanislav Yakovlev <stas.yakovlev@gmail.com>
Signed-off-by: Yang Shen <shenyang39@huawei.com>
---
drivers/net/wireless/intel/iwlwifi/mvm/rfi.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/net/wireless/intel/iwlwifi/mvm/rfi.c b/drivers/net/wireless/intel/iwlwifi/mvm/rfi.c
index 0b818067067c..f1ed73c9f3dd 100644
--- a/drivers/net/wireless/intel/iwlwifi/mvm/rfi.c
+++ b/drivers/net/wireless/intel/iwlwifi/mvm/rfi.c
@@ -7,11 +7,11 @@
#include "fw/api/commands.h"
#include "fw/api/phy-ctxt.h"
-/**
+/*
* DDR needs frequency in units of 16.666MHz, so provide FW with the
* frequency values in the adjusted format.
*/
-const static struct iwl_rfi_lut_entry iwl_rfi_table[IWL_RFI_LUT_SIZE] = {
+static const struct iwl_rfi_lut_entry iwl_rfi_table[IWL_RFI_LUT_SIZE] = {
/* LPDDR4 */
/* frequency 3733MHz */
--
2.17.1
^ permalink raw reply related [flat|nested] 21+ messages in thread* Re: [PATCH 06/11] net: intel: iwlwifi: Demote non-compliant kernel-doc headers
2021-05-17 5:01 ` [PATCH 06/11] net: intel: iwlwifi: Demote non-compliant kernel-doc headers Yang Shen
@ 2021-05-18 3:08 ` Yang Shen
0 siblings, 0 replies; 21+ messages in thread
From: Yang Shen @ 2021-05-18 3:08 UTC (permalink / raw)
To: Luca Coelho; +Cc: kvalo, linux-wireless, linux-kernel, Stanislav Yakovlev
Hello:
I'm sorry for getting wrong maintainer information.
Best Regards,
Yang
On 2021/5/17 13:01, Yang Shen wrote:
> Fixes the following W=1 kernel build warning(s):
>
> drivers/net/wireless/intel/iwlwifi/mvm/rfi.c:11: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
> drivers/net/wireless/intel/iwlwifi/mvm/rfi.c:14:1: warning: ‘static’ is not at beginning of declaration [-Wold-style-declaration]
>
> Cc: Stanislav Yakovlev <stas.yakovlev@gmail.com>
> Signed-off-by: Yang Shen <shenyang39@huawei.com>
> ---
> drivers/net/wireless/intel/iwlwifi/mvm/rfi.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/net/wireless/intel/iwlwifi/mvm/rfi.c b/drivers/net/wireless/intel/iwlwifi/mvm/rfi.c
> index 0b818067067c..f1ed73c9f3dd 100644
> --- a/drivers/net/wireless/intel/iwlwifi/mvm/rfi.c
> +++ b/drivers/net/wireless/intel/iwlwifi/mvm/rfi.c
> @@ -7,11 +7,11 @@
> #include "fw/api/commands.h"
> #include "fw/api/phy-ctxt.h"
>
> -/**
> +/*
> * DDR needs frequency in units of 16.666MHz, so provide FW with the
> * frequency values in the adjusted format.
> */
> -const static struct iwl_rfi_lut_entry iwl_rfi_table[IWL_RFI_LUT_SIZE] = {
> +static const struct iwl_rfi_lut_entry iwl_rfi_table[IWL_RFI_LUT_SIZE] = {
> /* LPDDR4 */
>
> /* frequency 3733MHz */
> --
> 2.17.1
>
> .
>
^ permalink raw reply [flat|nested] 21+ messages in thread
* [PATCH 07/11] net: marvell: libertas_tf: Fix wrong function name in comments
2021-05-17 5:01 [PATCH 00/11] Rid W=1 warnings in wireless Yang Shen
` (5 preceding siblings ...)
2021-05-17 5:01 ` [PATCH 06/11] net: intel: iwlwifi: Demote non-compliant kernel-doc headers Yang Shen
@ 2021-05-17 5:01 ` Yang Shen
2021-06-15 13:11 ` [07/11] " Kalle Valo
2021-05-17 5:01 ` [PATCH 08/11] net: realtek: rtlwifi: " Yang Shen
` (3 subsequent siblings)
10 siblings, 1 reply; 21+ messages in thread
From: Yang Shen @ 2021-05-17 5:01 UTC (permalink / raw)
To: kvalo; +Cc: linux-wireless, linux-kernel, Yang Shen
Fixes the following W=1 kernel build warning(s):
drivers/net/wireless/marvell/libertas_tf/if_usb.c:56: warning: expecting prototype for if_usb_wrike_bulk_callback(). Prototype was for if_usb_write_bulk_callback() instead
Signed-off-by: Yang Shen <shenyang39@huawei.com>
---
drivers/net/wireless/marvell/libertas_tf/if_usb.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/wireless/marvell/libertas_tf/if_usb.c b/drivers/net/wireless/marvell/libertas_tf/if_usb.c
index a92916dc81a9..fe0a69e804d8 100644
--- a/drivers/net/wireless/marvell/libertas_tf/if_usb.c
+++ b/drivers/net/wireless/marvell/libertas_tf/if_usb.c
@@ -48,7 +48,7 @@ static int if_usb_submit_rx_urb(struct if_usb_card *cardp);
static int if_usb_reset_device(struct lbtf_private *priv);
/**
- * if_usb_wrike_bulk_callback - call back to handle URB status
+ * if_usb_write_bulk_callback - call back to handle URB status
*
* @urb: pointer to urb structure
*/
--
2.17.1
^ permalink raw reply related [flat|nested] 21+ messages in thread* [PATCH 08/11] net: realtek: rtlwifi: Fix wrong function name in comments
2021-05-17 5:01 [PATCH 00/11] Rid W=1 warnings in wireless Yang Shen
` (6 preceding siblings ...)
2021-05-17 5:01 ` [PATCH 07/11] net: marvell: libertas_tf: Fix wrong function name in comments Yang Shen
@ 2021-05-17 5:01 ` Yang Shen
2021-05-17 5:01 ` [PATCH 09/11] net: rsi: Fix missing " Yang Shen
` (2 subsequent siblings)
10 siblings, 0 replies; 21+ messages in thread
From: Yang Shen @ 2021-05-17 5:01 UTC (permalink / raw)
To: kvalo; +Cc: linux-wireless, linux-kernel, Yang Shen, Ping-Ke Shih
Fixes the following W=1 kernel build warning(s):
drivers/net/wireless/realtek/rtlwifi/rtl8192cu/mac.c:124: warning: expecting prototype for writeLLT(). Prototype was for rtl92c_llt_write() instead
drivers/net/wireless/realtek/rtlwifi/rtl8192cu/mac.c:154: warning: expecting prototype for rtl92c_init_LLT_table(). Prototype was for rtl92c_init_llt_table() instead
Cc: Ping-Ke Shih <pkshih@realtek.com>
Signed-off-by: Yang Shen <shenyang39@huawei.com>
---
drivers/net/wireless/realtek/rtlwifi/rtl8192cu/mac.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8192cu/mac.c b/drivers/net/wireless/realtek/rtlwifi/rtl8192cu/mac.c
index 8d2c6d8d32d9..4ff0d4118193 100644
--- a/drivers/net/wireless/realtek/rtlwifi/rtl8192cu/mac.c
+++ b/drivers/net/wireless/realtek/rtlwifi/rtl8192cu/mac.c
@@ -112,7 +112,7 @@ void rtl92c_read_chip_version(struct ieee80211_hw *hw)
}
/**
- * writeLLT - LLT table write access
+ * rtl92c_llt_write - LLT table write access
* @hw: Pointer to the ieee80211_hw structure.
* @address: LLT logical address.
* @data: LLT data content
@@ -144,7 +144,7 @@ bool rtl92c_llt_write(struct ieee80211_hw *hw, u32 address, u32 data)
}
/**
- * rtl92c_init_LLT_table - Init LLT table
+ * rtl92c_init_llt_table - Init LLT table
* @hw: Pointer to the ieee80211_hw structure.
* @boundary: Page boundary.
*
--
2.17.1
^ permalink raw reply related [flat|nested] 21+ messages in thread* [PATCH 09/11] net: rsi: Fix missing function name in comments
2021-05-17 5:01 [PATCH 00/11] Rid W=1 warnings in wireless Yang Shen
` (7 preceding siblings ...)
2021-05-17 5:01 ` [PATCH 08/11] net: realtek: rtlwifi: " Yang Shen
@ 2021-05-17 5:01 ` Yang Shen
2021-05-17 5:01 ` [PATCH 10/11] net: ti: wl1251: " Yang Shen
2021-05-17 5:01 ` [PATCH 11/11] net: ti: wlcore: " Yang Shen
10 siblings, 0 replies; 21+ messages in thread
From: Yang Shen @ 2021-05-17 5:01 UTC (permalink / raw)
To: kvalo
Cc: linux-wireless, linux-kernel, Yang Shen, Amitkumar Karwar,
Siva Rebbagondla
Fixes the following W=1 kernel build warning(s):
drivers/net/wireless/rsi/rsi_91x_mgmt.c:1550: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
Cc: Amitkumar Karwar <amitkarwar@gmail.com>
Cc: Siva Rebbagondla <siva8118@gmail.com>
Signed-off-by: Yang Shen <shenyang39@huawei.com>
---
drivers/net/wireless/rsi/rsi_91x_mgmt.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/net/wireless/rsi/rsi_91x_mgmt.c b/drivers/net/wireless/rsi/rsi_91x_mgmt.c
index 33c76d39a8e9..dffe1d6cc592 100644
--- a/drivers/net/wireless/rsi/rsi_91x_mgmt.c
+++ b/drivers/net/wireless/rsi/rsi_91x_mgmt.c
@@ -1547,8 +1547,8 @@ static int rsi_eeprom_read(struct rsi_common *common)
}
/**
- * This function sends a frame to block/unblock
- * data queues in the firmware
+ * rsi_send_block_unblock_frame() - This function sends a frame to block/unblock
+ * data queues in the firmware
*
* @common: Pointer to the driver private structure.
* @block_event: Event block if true, unblock if false
--
2.17.1
^ permalink raw reply related [flat|nested] 21+ messages in thread* [PATCH 10/11] net: ti: wl1251: Fix missing function name in comments
2021-05-17 5:01 [PATCH 00/11] Rid W=1 warnings in wireless Yang Shen
` (8 preceding siblings ...)
2021-05-17 5:01 ` [PATCH 09/11] net: rsi: Fix missing " Yang Shen
@ 2021-05-17 5:01 ` Yang Shen
2021-06-15 13:39 ` [10/11] " Kalle Valo
2021-05-17 5:01 ` [PATCH 11/11] net: ti: wlcore: " Yang Shen
10 siblings, 1 reply; 21+ messages in thread
From: Yang Shen @ 2021-05-17 5:01 UTC (permalink / raw)
To: kvalo; +Cc: linux-wireless, linux-kernel, Yang Shen
Fixes the following W=1 kernel build warning(s):
drivers/net/wireless/ti/wl1251/cmd.c:15: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
drivers/net/wireless/ti/wl1251/cmd.c:62: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
drivers/net/wireless/ti/wl1251/cmd.c:103: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
drivers/net/wireless/ti/wl1251/cmd.c:141: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
Signed-off-by: Yang Shen <shenyang39@huawei.com>
---
drivers/net/wireless/ti/wl1251/cmd.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/net/wireless/ti/wl1251/cmd.c b/drivers/net/wireless/ti/wl1251/cmd.c
index 498c8db2eb48..eb92dde786e7 100644
--- a/drivers/net/wireless/ti/wl1251/cmd.c
+++ b/drivers/net/wireless/ti/wl1251/cmd.c
@@ -12,7 +12,7 @@
#include "acx.h"
/**
- * send command to firmware
+ * wl1251_cmd_send - Send command to firmware
*
* @wl: wl struct
* @id: command id
@@ -59,7 +59,7 @@ int wl1251_cmd_send(struct wl1251 *wl, u16 id, void *buf, size_t len)
}
/**
- * send test command to firmware
+ * wl1251_cmd_test - Send test command to firmware
*
* @wl: wl struct
* @buf: buffer containing the command, with all headers, must work with dma
@@ -100,7 +100,7 @@ int wl1251_cmd_test(struct wl1251 *wl, void *buf, size_t buf_len, u8 answer)
}
/**
- * read acx from firmware
+ * wl1251_cmd_interrogate - Read acx from firmware
*
* @wl: wl struct
* @id: acx id
@@ -138,7 +138,7 @@ int wl1251_cmd_interrogate(struct wl1251 *wl, u16 id, void *buf, size_t len)
}
/**
- * write acx value to firmware
+ * wl1251_cmd_configure - Write acx value to firmware
*
* @wl: wl struct
* @id: acx id
--
2.17.1
^ permalink raw reply related [flat|nested] 21+ messages in thread* [PATCH 11/11] net: ti: wlcore: Fix missing function name in comments
2021-05-17 5:01 [PATCH 00/11] Rid W=1 warnings in wireless Yang Shen
` (9 preceding siblings ...)
2021-05-17 5:01 ` [PATCH 10/11] net: ti: wl1251: " Yang Shen
@ 2021-05-17 5:01 ` Yang Shen
10 siblings, 0 replies; 21+ messages in thread
From: Yang Shen @ 2021-05-17 5:01 UTC (permalink / raw)
To: kvalo; +Cc: linux-wireless, linux-kernel, Yang Shen
Fixes the following W=1 kernel build warning(s):
drivers/net/wireless/ti/wlcore/cmd.c:824: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
drivers/net/wireless/ti/wlcore/cmd.c:853: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
drivers/net/wireless/ti/wlcore/cmd.c:882: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
Signed-off-by: Yang Shen <shenyang39@huawei.com>
---
drivers/net/wireless/ti/wlcore/cmd.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/net/wireless/ti/wlcore/cmd.c b/drivers/net/wireless/ti/wlcore/cmd.c
index 32a2e27cc561..8b798b5fcaf5 100644
--- a/drivers/net/wireless/ti/wlcore/cmd.c
+++ b/drivers/net/wireless/ti/wlcore/cmd.c
@@ -821,7 +821,7 @@ int wl12xx_cmd_role_start_ibss(struct wl1271 *wl, struct wl12xx_vif *wlvif)
/**
- * send test command to firmware
+ * wl1271_cmd_test - send test command to firmware
*
* @wl: wl struct
* @buf: buffer containing the command, with all headers, must work with dma
@@ -850,7 +850,7 @@ int wl1271_cmd_test(struct wl1271 *wl, void *buf, size_t buf_len, u8 answer)
EXPORT_SYMBOL_GPL(wl1271_cmd_test);
/**
- * read acx from firmware
+ * wl1271_cmd_interrogate - read acx from firmware
*
* @wl: wl struct
* @id: acx id
@@ -879,7 +879,7 @@ int wl1271_cmd_interrogate(struct wl1271 *wl, u16 id, void *buf,
}
/**
- * write acx value to firmware
+ * wlcore_cmd_configure_failsafe - write acx value to firmware
*
* @wl: wl struct
* @id: acx id
--
2.17.1
^ permalink raw reply related [flat|nested] 21+ messages in thread