Re: [ovs-dev] [PATCH 1/4] ofp-util: Use enum ofp_table_config in struct ofputil_table_mod

2014-03-11 Thread Ben Pfaff 
On Wed, Mar 12, 2014 at 11:22:37AM +0900, Simon Horman wrote:
> Use enum ofp_table_config as the type of the 'config' field
> of struct ofputil_table_mod. This reflects the usage
> of the field.
> 
> Signed-off-by: Simon Horman 

Applied.

I hope that you will heed Yamamoto-san's comments and resubmit the rest
of the series.
___
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] [PATCH 1/4] ofp-util: Use enum ofp_table_config in struct ofputil_table_mod

2014-03-11 Thread Simon Horman 
Use enum ofp_table_config as the type of the 'config' field
of struct ofputil_table_mod. This reflects the usage
of the field.

Signed-off-by: Simon Horman 
---
 lib/ofp-util.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/ofp-util.h b/lib/ofp-util.h
index bf02b9f..7483646 100644
--- a/lib/ofp-util.h
+++ b/lib/ofp-util.h
@@ -591,7 +591,7 @@ struct ofpbuf *ofputil_encode_port_mod(const struct 
ofputil_port_mod *,
 /* Abstract ofp_table_mod. */
 struct ofputil_table_mod {
 uint8_t table_id; /* ID of the table, 0xff indicates all tables. */
-uint32_t config;
+enum ofp_table_config config;
 };
 
 enum ofperr ofputil_decode_table_mod(const struct ofp_header *,
-- 
1.8.5.2

___
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev