On 12/9/2016 11:27 AM, Bernard Iremonger wrote:
> Add the following command to testpmd:
> set vf vlan filter <port_id> <vlan_id> <vf_mask> <on|off>
> 
> Add command to the testpmd user guide.
> 
> Signed-off-by: Bernard Iremonger <bernard.iremon...@intel.com>
> ---

<...>

> +
> +cmdline_parse_inst_t cmd_set_vf_vlan_filter = {
> +     .f = cmd_set_vf_vlan_filter_parsed,
> +     .data = NULL,
> +     .help_str = "set vf vlan filter <port_id> <vlan_id> <vf_mask> <on_off>",

s/<on_off>/on|off

Only variables should be wrapped with "<>", fixed string with options
can be used as a|b|c .

similar issues in other testpmd patches.


Reply via email to