Re: [PATCH net v2 4/4] Documentation: net/mlx5: Wrap notes in admonition blocks

From: Jonathan Corbet
Date: Tue May 16 2023 - 14:27:38 EST


Bagas Sanjaya <bagasdotme@xxxxxxxxx> writes:

> Wrap note paragraphs in note:: directive as it better fit for the
> purpose of noting devlink commands.
>
> Fixes: f2d51e579359b7 ("net/mlx5: Separate mlx5 driver documentation into multiple pages")
> Fixes: cf14af140a5ad0 ("net/mlx5e: Add vnic devlink health reporter to representors")
> Reviewed-by: Leon Romanovsky <leonro@xxxxxxxxxx>
> Signed-off-by: Bagas Sanjaya <bagasdotme@xxxxxxxxx>
> ---
> .../ethernet/mellanox/mlx5/devlink.rst | 14 +++++++++-----
> 1 file changed, 9 insertions(+), 5 deletions(-)

So these changes seem harmless, but the Fixes: tags seem completely
inappropriate here. This is format tweaking, not a bug fix.

Thanks,

jon