* [PATCH] wifi: mac80222: Fix ieee80211_i.h kernel-doc issues
@ 2024-03-28 21:08 Jeff Johnson
2024-03-28 22:16 ` Randy Dunlap
0 siblings, 1 reply; 3+ messages in thread
From: Jeff Johnson @ 2024-03-28 21:08 UTC (permalink / raw)
To: Johannes Berg, David S. Miller, Eric Dumazet, Jakub Kicinski,
Paolo Abeni
Cc: linux-wireless, netdev, linux-kernel, Jeff Johnson
kernel-doc flagged the following issues:
net/mac80211/ieee80211_i.h:146: warning: expecting prototype for enum ieee80211_corrupt_data_flags. Prototype was for enum ieee80211_bss_corrupt_data_flags instead
net/mac80211/ieee80211_i.h:163: warning: expecting prototype for enum ieee80211_valid_data_flags. Prototype was for enum ieee80211_bss_valid_data_flags instead
Correct the documentation to match the code.
Signed-off-by: Jeff Johnson <quic_jjohnson@quicinc.com>
---
This file is being modified by a quicinc.com patch and my review
process flagged these pre-existing kernel-doc issues.
---
net/mac80211/ieee80211_i.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/net/mac80211/ieee80211_i.h b/net/mac80211/ieee80211_i.h
index def611e4e55f..458f63ce9a32 100644
--- a/net/mac80211/ieee80211_i.h
+++ b/net/mac80211/ieee80211_i.h
@@ -132,7 +132,7 @@ struct ieee80211_bss {
};
/**
- * enum ieee80211_corrupt_data_flags - BSS data corruption flags
+ * enum ieee80211_bss_corrupt_data_flags - BSS data corruption flags
* @IEEE80211_BSS_CORRUPT_BEACON: last beacon frame received was corrupted
* @IEEE80211_BSS_CORRUPT_PROBE_RESP: last probe response received was corrupted
*
@@ -145,7 +145,7 @@ enum ieee80211_bss_corrupt_data_flags {
};
/**
- * enum ieee80211_valid_data_flags - BSS valid data flags
+ * enum ieee80211_bss_valid_data_flags - BSS valid data flags
* @IEEE80211_BSS_VALID_WMM: WMM/UAPSD data was gathered from non-corrupt IE
* @IEEE80211_BSS_VALID_RATES: Supported rates were gathered from non-corrupt IE
* @IEEE80211_BSS_VALID_ERP: ERP flag was gathered from non-corrupt IE
---
base-commit: b68b2beadfd30907faae944358de3a17acf6fdb6
change-id: 20240328-ieee80211_i-kerneldoc-d28d95f5c5d3
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] wifi: mac80222: Fix ieee80211_i.h kernel-doc issues
2024-03-28 21:08 [PATCH] wifi: mac80222: Fix ieee80211_i.h kernel-doc issues Jeff Johnson
@ 2024-03-28 22:16 ` Randy Dunlap
2024-03-28 23:07 ` Jeff Johnson
0 siblings, 1 reply; 3+ messages in thread
From: Randy Dunlap @ 2024-03-28 22:16 UTC (permalink / raw)
To: Jeff Johnson, Johannes Berg, David S. Miller, Eric Dumazet,
Jakub Kicinski, Paolo Abeni
Cc: linux-wireless, netdev, linux-kernel
Hi,
On 3/28/24 14:08, Jeff Johnson wrote:
> kernel-doc flagged the following issues:
>
> net/mac80211/ieee80211_i.h:146: warning: expecting prototype for enum ieee80211_corrupt_data_flags. Prototype was for enum ieee80211_bss_corrupt_data_flags instead
> net/mac80211/ieee80211_i.h:163: warning: expecting prototype for enum ieee80211_valid_data_flags. Prototype was for enum ieee80211_bss_valid_data_flags instead
>
> Correct the documentation to match the code.
>
> Signed-off-by: Jeff Johnson <quic_jjohnson@quicinc.com>
These seem to be corrected in linux-next already.
However, I do still see this one:
ieee80211_i.h:689: warning: missing initial short description on line:
* struct mesh_table
Thanks.
> ---
> This file is being modified by a quicinc.com patch and my review
> process flagged these pre-existing kernel-doc issues.
> ---
> net/mac80211/ieee80211_i.h | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/net/mac80211/ieee80211_i.h b/net/mac80211/ieee80211_i.h
> index def611e4e55f..458f63ce9a32 100644
> --- a/net/mac80211/ieee80211_i.h
> +++ b/net/mac80211/ieee80211_i.h
> @@ -132,7 +132,7 @@ struct ieee80211_bss {
> };
>
> /**
> - * enum ieee80211_corrupt_data_flags - BSS data corruption flags
> + * enum ieee80211_bss_corrupt_data_flags - BSS data corruption flags
> * @IEEE80211_BSS_CORRUPT_BEACON: last beacon frame received was corrupted
> * @IEEE80211_BSS_CORRUPT_PROBE_RESP: last probe response received was corrupted
> *
> @@ -145,7 +145,7 @@ enum ieee80211_bss_corrupt_data_flags {
> };
>
> /**
> - * enum ieee80211_valid_data_flags - BSS valid data flags
> + * enum ieee80211_bss_valid_data_flags - BSS valid data flags
> * @IEEE80211_BSS_VALID_WMM: WMM/UAPSD data was gathered from non-corrupt IE
> * @IEEE80211_BSS_VALID_RATES: Supported rates were gathered from non-corrupt IE
> * @IEEE80211_BSS_VALID_ERP: ERP flag was gathered from non-corrupt IE
>
> ---
> base-commit: b68b2beadfd30907faae944358de3a17acf6fdb6
> change-id: 20240328-ieee80211_i-kerneldoc-d28d95f5c5d3
>
>
--
#Randy
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] wifi: mac80222: Fix ieee80211_i.h kernel-doc issues
2024-03-28 22:16 ` Randy Dunlap
@ 2024-03-28 23:07 ` Jeff Johnson
0 siblings, 0 replies; 3+ messages in thread
From: Jeff Johnson @ 2024-03-28 23:07 UTC (permalink / raw)
To: Randy Dunlap, Johannes Berg, David S. Miller, Eric Dumazet,
Jakub Kicinski, Paolo Abeni
Cc: linux-wireless, netdev, linux-kernel
On 3/28/2024 3:16 PM, Randy Dunlap wrote:
> Hi,
>
> On 3/28/24 14:08, Jeff Johnson wrote:
>> kernel-doc flagged the following issues:
>>
>> net/mac80211/ieee80211_i.h:146: warning: expecting prototype for enum ieee80211_corrupt_data_flags. Prototype was for enum ieee80211_bss_corrupt_data_flags instead
>> net/mac80211/ieee80211_i.h:163: warning: expecting prototype for enum ieee80211_valid_data_flags. Prototype was for enum ieee80211_bss_valid_data_flags instead
>>
>> Correct the documentation to match the code.
>>
>> Signed-off-by: Jeff Johnson <quic_jjohnson@quicinc.com>
>
> These seem to be corrected in linux-next already.
Looks like I fixed these two earlier -- not sure why they haven't reached
wireless-next yet which was the code I was looking at today.
I've flagged as N/A in patchwork
> However, I do still see this one:
>
> ieee80211_i.h:689: warning: missing initial short description on line:
> * struct mesh_table
Ah, I haven't been adding [-Wshort-desc[ription]] (or some of the other -W
switches)
Yay, I can find even more issues.
I'll fix this one
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2024-03-28 23:08 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-03-28 21:08 [PATCH] wifi: mac80222: Fix ieee80211_i.h kernel-doc issues Jeff Johnson
2024-03-28 22:16 ` Randy Dunlap
2024-03-28 23:07 ` Jeff Johnson
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.