> > Formatted as:
> > cmd fixed_string fixed|string|options <variable>: Description
> > 
> > If there is no description, final colon emitted.
> > 
> > Signed-off-by: Ferruh Yigit <ferruh.yi...@intel.com>
> 
> 
> Good refactoring work to give consistency. It might be worth having a look at 
> the testpmd docs to see if some of the same improvements can be made there as 
> well. That could be a separate patch.
> 
> >     .f = cmd_config_rss_hash_key_parsed,
> >     .data = NULL,
> > -   .help_str =
> > -           "port config X rss-hash-key ipv4|ipv4-frag|ipv4-tcp|ipv4-udp|"
> > -           "ipv4-sctp|ipv4-other|ipv6|ipv6-frag|ipv6-tcp|ipv6-udp|"
> > -           "ipv6-sctp|ipv6-other|l2-payload|"
> > -           "ipv6-ex|ipv6-tcp-ex|ipv6-udp-ex "
> > -           "<string of hexa digits (variable length, NIC dependent)>\n",
> > +   .help_str = "port config <port_id> rss-hash-key "
> > +           "ipv4|ipv4-frag|ipv4-tcp|ipv4-udp|ipv4-sctp|ipv4-other|"
> > +           "ipv6|ipv6-frag|ipv6-tcp|ipv6-udp|ipv6-sctp|ipv6-other|"
> > +           "l2-payload|ipv6-ex|ipv6-tcp-ex|ipv6-udp-ex "
> > +           "<string of hexa digits (variable length, NIC dependent)>",
> 
> /hex/hexa/

fixed

> Apart from that:
> 
> Acked-by: John McNamara <john.mcnam...@intel.com>

Applied, thanks

Reply via email to