diff options
| author |   Jiri Pirko <jiri@mellanox.com> | 2015-09-24 10:02:49 +0200 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2015-09-24 22:59:22 -0700 |
| commit | f623ab7f51b1bfb523c9cd492747392abf3c4421 (patch) | |
| tree | a58ac708adc4b3c7a4151089a0815667be7d9937 /include/net/switchdev.h | |
| parent | dsa: use prepare/commit switchdev transaction helpers (diff) | |
| download | linux-dev-f623ab7f51b1bfb523c9cd492747392abf3c4421.tar.xz linux-dev-f623ab7f51b1bfb523c9cd492747392abf3c4421.zip | |
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/switchdev.h')
| -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 { |