On Mon, Sep 12, 2011 at 2:11 PM, Pravin Shelar <[email protected]> wrote:
> diff --git a/include/openvswitch/datapath-protocol.h 
> b/include/openvswitch/datapath-protocol.h
> index c1c9ef0..49a122c 100644
> --- a/include/openvswitch/datapath-protocol.h
> +++ b/include/openvswitch/datapath-protocol.h
> @@ -237,8 +235,8 @@ enum ovs_vport_cmd {
>  * %OVS_VPORT_ATTR_NAME attributes are required.  %OVS_VPORT_ATTR_PORT_NO is
>  * optional; if not specified a free port number is automatically selected.
>  * Whether %OVS_VPORT_ATTR_OPTIONS is required or optional depends on the type
> - * of vport.  %OVS_VPORT_ATTR_STATS, %OVS_VPORT_ATTR_ADDRESS, and
> - * %OVS_VPORT_ATTR_MTU are optional, and other attributes are ignored.
> + * of vport.  %OVS_VPORT_ATTR_STATS, %OVS_VPORT_ATTR_ADDRESS are optional,

There should be an "and" between those two OVS_VPORT_ATTR_ types.

Otherwise looks good:
Acked-by: Jesse Gross <[email protected]>
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to