diff mbox series

wifi: mac80222: Fix ieee80211_i.h kernel-doc issues

Message ID 20240328-ieee80211_i-kerneldoc-v1-1-e848bdec58f3@quicinc.com (mailing list archive)
State Not Applicable
Delegated to: Johannes Berg
Headers show
Series wifi: mac80222: Fix ieee80211_i.h kernel-doc issues | expand

Commit Message

Jeff Johnson March 28, 2024, 9:08 p.m. UTC
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(-)


---
base-commit: b68b2beadfd30907faae944358de3a17acf6fdb6
change-id: 20240328-ieee80211_i-kerneldoc-d28d95f5c5d3

Comments

Randy Dunlap March 28, 2024, 10:16 p.m. UTC | #1
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
> 
>
Jeff Johnson March 28, 2024, 11:07 p.m. UTC | #2
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
diff mbox series

Patch

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