Re: [PATCH] wifi: mac80211: use bullet list for amsdu_mesh_control formats list

[Simon Horman]

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
>