Hi Dengdui,

On 2023/10/8 9:56, Dengdui Huang wrote:
> Command help string is missing 'mcast_addr add|remove'.
> This patch add it.
> 
> Fixes: 8fff667578a7 ("app/testpmd: new command to add/remove multicast MAC 
> addresses")
> Cc: sta...@dpdk.org
> 
> Signed-off-by: Dengdui Huang <huangdeng...@huawei.com>
> ---
>  app/test-pmd/cmdline.c | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c
> index a0e97719b3..3165347a05 100644
> --- a/app/test-pmd/cmdline.c
> +++ b/app/test-pmd/cmdline.c
> @@ -516,6 +516,12 @@ static void cmd_help_long_parsed(void *parsed_result,
>                       "set allmulti (port_id|all) (on|off)\n"
>                       "    Set the allmulti mode on port_id, or all.\n\n"
>  
> +                     "mcast_addr add (port_id) (mcast_addr)\n"
> +                     "    Add a multicast MAC addresses on port_id.\n\n"
> +
> +                     "mcast_addr remove (port_id) (mcast_addr)"

Please add '\n' at last.

with above fixed,
Acked-by: Chengwen Feng <fengcheng...@huawei.com>

> +                     "    Remove a multicast MAC address from port_id.\n\n"
> +
>                       "set flow_ctrl rx (on|off) tx (on|off) (high_water)"
>                       " (low_water) (pause_time) (send_xon) 
> mac_ctrl_frame_fwd"
>                       " (on|off) autoneg (on|off) (port_id)\n"
> 

Reply via email to