bna: remove TX_E_PRIO_CHANGE event and BNA_TX_F_PRIO_CHANGED flag
authorIvan Vecera <ivecera@redhat.com>
Thu, 11 Jun 2015 13:52:24 +0000 (15:52 +0200)
committerDavid S. Miller <davem@davemloft.net>
Thu, 11 Jun 2015 22:57:16 +0000 (15:57 -0700)
TX_E_PRIO_CHANGE event is never sent for bna_tx so it doesn't need to be
handled. After this change bna_tx->flags cannot contain
BNA_TX_F_PRIO_CHANGED flag and it can be also eliminated.

Signed-off-by: Ivan Vecera <ivecera@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/brocade/bna/bna_tx_rx.c
drivers/net/ethernet/brocade/bna/bna_types.h

index 896aa8287a66ec548e78f3b191f9ccd81e5ecda1..54ad16955963275a949668f11e8a5ccc64b20ad3 100644 (file)
@@ -2901,7 +2901,6 @@ enum bna_tx_event {
        TX_E_FAIL                       = 3,
        TX_E_STARTED                    = 4,
        TX_E_STOPPED                    = 5,
-       TX_E_PRIO_CHANGE                = 6,
        TX_E_CLEANUP_DONE               = 7,
        TX_E_BW_UPDATE                  = 8,
 };
@@ -2942,9 +2941,6 @@ bna_tx_sm_stopped(struct bna_tx *tx, enum bna_tx_event event)
                /* No-op */
                break;
 
-       case TX_E_PRIO_CHANGE:
-               break;
-
        case TX_E_BW_UPDATE:
                /* No-op */
                break;
@@ -2965,28 +2961,23 @@ bna_tx_sm_start_wait(struct bna_tx *tx, enum bna_tx_event event)
 {
        switch (event) {
        case TX_E_STOP:
-               tx->flags &= ~(BNA_TX_F_PRIO_CHANGED | BNA_TX_F_BW_UPDATED);
+               tx->flags &= ~BNA_TX_F_BW_UPDATED;
                bfa_fsm_set_state(tx, bna_tx_sm_stop_wait);
                break;
 
        case TX_E_FAIL:
-               tx->flags &= ~(BNA_TX_F_PRIO_CHANGED | BNA_TX_F_BW_UPDATED);
+               tx->flags &= ~BNA_TX_F_BW_UPDATED;
                bfa_fsm_set_state(tx, bna_tx_sm_stopped);
                break;
 
        case TX_E_STARTED:
-               if (tx->flags & (BNA_TX_F_PRIO_CHANGED | BNA_TX_F_BW_UPDATED)) {
-                       tx->flags &= ~(BNA_TX_F_PRIO_CHANGED |
-                               BNA_TX_F_BW_UPDATED);
+               if (tx->flags & BNA_TX_F_BW_UPDATED) {
+                       tx->flags &= ~BNA_TX_F_BW_UPDATED;
                        bfa_fsm_set_state(tx, bna_tx_sm_prio_stop_wait);
                } else
                        bfa_fsm_set_state(tx, bna_tx_sm_started);
                break;
 
-       case TX_E_PRIO_CHANGE:
-               tx->flags |=  BNA_TX_F_PRIO_CHANGED;
-               break;
-
        case TX_E_BW_UPDATE:
                tx->flags |= BNA_TX_F_BW_UPDATED;
                break;
@@ -3028,7 +3019,6 @@ bna_tx_sm_started(struct bna_tx *tx, enum bna_tx_event event)
                tx->tx_cleanup_cbfn(tx->bna->bnad, tx);
                break;
 
-       case TX_E_PRIO_CHANGE:
        case TX_E_BW_UPDATE:
                bfa_fsm_set_state(tx, bna_tx_sm_prio_stop_wait);
                break;
@@ -3061,7 +3051,6 @@ bna_tx_sm_stop_wait(struct bna_tx *tx, enum bna_tx_event event)
                bna_tx_enet_stop(tx);
                break;
 
-       case TX_E_PRIO_CHANGE:
        case TX_E_BW_UPDATE:
                /* No-op */
                break;
@@ -3081,7 +3070,6 @@ bna_tx_sm_cleanup_wait(struct bna_tx *tx, enum bna_tx_event event)
 {
        switch (event) {
        case TX_E_FAIL:
-       case TX_E_PRIO_CHANGE:
        case TX_E_BW_UPDATE:
                /* No-op */
                break;
@@ -3119,7 +3107,6 @@ bna_tx_sm_prio_stop_wait(struct bna_tx *tx, enum bna_tx_event event)
                bfa_fsm_set_state(tx, bna_tx_sm_prio_cleanup_wait);
                break;
 
-       case TX_E_PRIO_CHANGE:
        case TX_E_BW_UPDATE:
                /* No-op */
                break;
@@ -3147,7 +3134,6 @@ bna_tx_sm_prio_cleanup_wait(struct bna_tx *tx, enum bna_tx_event event)
                bfa_fsm_set_state(tx, bna_tx_sm_failed);
                break;
 
-       case TX_E_PRIO_CHANGE:
        case TX_E_BW_UPDATE:
                /* No-op */
                break;
index 134abf7f768198a2194787891c2a3f17ca6497e0..e0e797f2ea147cd728562b9b9b0d8468d0551d04 100644 (file)
@@ -135,7 +135,6 @@ enum bna_tx_type {
 enum bna_tx_flags {
        BNA_TX_F_ENET_STARTED   = 1,
        BNA_TX_F_ENABLED        = 2,
-       BNA_TX_F_PRIO_CHANGED   = 4,
        BNA_TX_F_BW_UPDATED     = 8,
 };