> The "name" parameter isn't optional so don't use brackets around it.
> 

Thanks for the patch Cian, comment below.

> Fixes: 5c5c98cec21b ("docs/dpdk/bridge: Add miniflow extract section.")
> Signed-off-by: Cian Ferriter <[email protected]>
> ---
>  Documentation/topics/dpdk/bridge.rst | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/Documentation/topics/dpdk/bridge.rst
> b/Documentation/topics/dpdk/bridge.rst
> index f422eb5e9..1aa41958c 100644
> --- a/Documentation/topics/dpdk/bridge.rst
> +++ b/Documentation/topics/dpdk/bridge.rst
> @@ -318,8 +318,8 @@ command also shows whether the CPU supports each
> implementation ::
> 
>  An implementation can be selected manually by the following command ::
> 
> -    $ ovs-appctl dpif-netdev/miniflow-parser-set [-pmd core_id] [name]
> -                                                 [study_cnt]
> +    $ ovs-appctl dpif-netdev/miniflow-parser-set [-pmd core_id] name \
> +        [study_cnt]

I would keep [study_cnt] aligned under ovs-appctl above, currently seems to be 
extra white space before it.

BR
Ian
> 
>  The above command has two optional parameters: study_cnt and core_id.
>  The core_id sets a particular miniflow extract function to a specific
> --
> 2.32.0
> 
> _______________________________________________
> dev mailing list
> [email protected]
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to