On Thu, Mar 22, 2012 at 03:10:33PM -0700, Ben Pfaff wrote:
> On Mon, Mar 19, 2012 at 09:54:45AM +0900, Simon Horman wrote:
> > This adds some more enum and struct definitions to
> > include/openflow/openflow-1.2.h. They were omitted from the patch that
> > created that file as they do not seem to be in keeping with
> > the treatment of Open Flow 1.1 in include/openflow/openflow-1.1.h.
> > That is, similar definitions are not present in 
> > include/openflow/openflow-1.1.h.
> > 
> > Signed-off-by: Simon Horman <[email protected]>
> 
> I've been adding them to openflow-1.1.h gradually as they become
> useful.  Do you have any interest in adding the similar definitions to
> openflow-1.1.h too?  I'd be happy to take them together, if you like.

I assumed that was the case. I'll see about putting together a patch
to add the missing Open Flow 1.1 definitions.

> Again, at first glance all I see is ovs_be<N> instead of uint<N>_t.

Thanks, I'll fix that throughout the series.
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to