On Thu, Mar 23, 2023 at 04:24:54PM +0700, Bagas Sanjaya wrote:
> Commit fe4a6d2db3bad4 ("wifi: mac80211: implement support for yet
> another mesh A-MSDU format") expands amsdu_mesh_control list to
> multi-line list. However, the expansion triggers Sphinx warning:
FWIIW, checkpatch complains about the way that the commit is referenced
above. It's probably not worth respinning for this. But a way
to address this would be.
The referenced commit expands amsdu_mesh_control list to multi-line list.
However, the expansion triggers Sphinx warning:
>
> Documentation/driver-api/80211/mac80211-advanced:214: ./net/mac80211/sta_info.h:628: WARNING: Unexpected indentation.
>
> Use bullet list instead to fix the warning.
>
> Link: https://lore.kernel.org/linux-next/20230323141548.659479ef@xxxxxxxxxxxxxxxx/
> Fixes: fe4a6d2db3bad4 ("wifi: mac80211: implement support for yet another mesh A-MSDU format")
> Reported-by: Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx>
Checkpatch also says that the Link: tag should go here.
TBH, I'm not sure if that is important.
> Signed-off-by: Bagas Sanjaya <bagasdotme@xxxxxxxxx>
Patch itself looks good :)
Reviewed-by: Simon Horman <simon.horman@xxxxxxxxxxxx>
> ---
> net/mac80211/sta_info.h | 12 +++++++-----
> 1 file changed, 7 insertions(+), 5 deletions(-)
>
> diff --git a/net/mac80211/sta_info.h b/net/mac80211/sta_info.h
> index f354d470e1740c..195b563132d6c5 100644
> --- a/net/mac80211/sta_info.h
> +++ b/net/mac80211/sta_info.h
> @@ -622,11 +622,13 @@ struct link_sta_info {
> * taken from HT/VHT capabilities or VHT operating mode notification
> * @cparams: CoDel parameters for this station.
> * @reserved_tid: reserved TID (if any, otherwise IEEE80211_TID_UNRESERVED)
> - * @amsdu_mesh_control: track the mesh A-MSDU format used by the peer
> - * (-1: not yet known,
> - * 0: non-mesh A-MSDU length field
> - * 1: big-endian mesh A-MSDU length field
> - * 2: little-endian mesh A-MSDU length field)
> + * @amsdu_mesh_control: track the mesh A-MSDU format used by the peer:
> + *
> + * * -1: not yet known
> + * * 0: non-mesh A-MSDU length field
> + * * 1: big-endian mesh A-MSDU length field
> + * * 2: little-endian mesh A-MSDU length field
> + *
> * @fast_tx: TX fastpath information
> * @fast_rx: RX fastpath information
> * @tdls_chandef: a TDLS peer can have a wider chandef that is compatible to
>
> base-commit: 0dd45ebc08de2449efe1a0908147796856a5f824
> --
> An old man doll... just what I always wanted! - Clara
>
