On Mon, Sep 12, 2011 at 4:53 PM, Jesse Gross <je...@nicira.com> wrote:
> On Mon, Sep 12, 2011 at 2:11 PM, Pravin Shelar <pshe...@nicira.com> 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 <je...@nicira.com>
>
pushed.
Thanks,
Pravin.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
