| Message ID | 20240328-ieee80211_i-kerneldoc-v1-1-e848bdec58f3@quicinc.com (mailing list archive) |
|---|---|
| State | Awaiting Upstream |
| Delegated to: | Netdev Maintainers |
| Headers | show |
| Series | wifi: mac80222: Fix ieee80211_i.h kernel-doc issues | expand |
| Context | Check | Description |
|---|---|---|
| netdev/tree_selection | success | Guessed tree name to be net-next |
| netdev/apply | fail | Patch does not apply to net-next-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 > >
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 --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
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