diff options
| author | Jiri Pirko <jiri@mellanox.com> | 2015-09-24 11:02:49 +0300 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2015-09-25 08:59:22 +0300 |
| commit | f623ab7f51b1bfb523c9cd492747392abf3c4421 (patch) | |
| tree | a58ac708adc4b3c7a4151089a0815667be7d9937 /include/net | |
| parent | 79a62eb22a5fa4ee5e1f21af5e1f19f493dac5e0 (diff) | |
| download | linux-f623ab7f51b1bfb523c9cd492747392abf3c4421.tar.xz | |
switchdev: reduce transaction phase enum down to a boolean
Now, since we have only 2 values for transaction phase, just use bool.
Signed-off-by: Jiri Pirko <jiri@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include/net')
| -rw-r--r-- | include/net/switchdev.h | 11 |
1 files changed, 3 insertions, 8 deletions
diff --git a/include/net/switchdev.h b/include/net/switchdev.h index 9cf372fe1365..18207878e407 100644 --- a/include/net/switchdev.h +++ b/include/net/switchdev.h @@ -17,11 +17,6 @@ #define SWITCHDEV_F_NO_RECURSE BIT(0) -enum switchdev_trans_ph { - SWITCHDEV_TRANS_PREPARE, - SWITCHDEV_TRANS_COMMIT, -}; - struct switchdev_trans_item { struct list_head list; void *data; @@ -30,17 +25,17 @@ struct switchdev_trans_item { struct switchdev_trans { struct list_head item_list; - enum switchdev_trans_ph ph; + bool ph_prepare; }; static inline bool switchdev_trans_ph_prepare(struct switchdev_trans *trans) { - return trans && trans->ph == SWITCHDEV_TRANS_PREPARE; + return trans && trans->ph_prepare; } static inline bool switchdev_trans_ph_commit(struct switchdev_trans *trans) { - return trans && trans->ph == SWITCHDEV_TRANS_COMMIT; + return trans && !trans->ph_prepare; } enum switchdev_attr_id { |