On 2023/10/8 10:47, fengchengwen wrote:
> 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>
>
OK, thanks
>> + " 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"
>>
