iwlwifi: make iwl_queue_used return bool
authorEmmanuel Grumbach <emmanuel.grumbach@intel.com>
Wed, 14 Nov 2012 21:38:08 +0000 (23:38 +0200)
committerJohannes Berg <johannes.berg@intel.com>
Mon, 19 Nov 2012 14:06:17 +0000 (15:06 +0100)
Also, prefer the if(!X) notation over if(X == 0).

Signed-off-by: Emmanuel Grumbach <emmanuel.grumbach@intel.com>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
drivers/net/wireless/iwlwifi/pcie/internal.h
drivers/net/wireless/iwlwifi/pcie/tx.c

index 0d61e91c0a6f7ec66f1f49d0588e3ba12fea8831..e6b6dba982c7a4429e9d305932bbbe7c185854a9 100644 (file)
@@ -423,7 +423,7 @@ static inline void iwl_stop_queue(struct iwl_trans *trans,
                                    txq->q.id);
 }
 
-static inline int iwl_queue_used(const struct iwl_queue *q, int i)
+static inline bool iwl_queue_used(const struct iwl_queue *q, int i)
 {
        return q->write_ptr >= q->read_ptr ?
                (i >= q->read_ptr && i < q->write_ptr) :
index ba2a78c150bb5a70e189efb107f9e3f68d9ff0d6..40e5a6d6e6d3c96e7627d0a1bc3fb71e823d8437 100644 (file)
@@ -918,7 +918,7 @@ void iwl_trans_pcie_reclaim(struct iwl_trans *trans, int txq_id, int ssn,
         * the last we will free. This one must be used */
        last_to_free = iwl_queue_dec_wrap(tfd_num, q->n_bd);
 
-       if (iwl_queue_used(q, last_to_free) == 0) {
+       if (!iwl_queue_used(q, last_to_free)) {
                IWL_ERR(trans,
                        "%s: Read index for DMA queue txq id (%d), last_to_free %d is out of range [0-%d] %d %d.\n",
                        __func__, txq_id, last_to_free, q->n_bd,
@@ -969,7 +969,7 @@ static void iwl_pcie_cmdq_reclaim(struct iwl_trans *trans, int txq_id, int idx)
 
        lockdep_assert_held(&txq->lock);
 
-       if ((idx >= q->n_bd) || (iwl_queue_used(q, idx) == 0)) {
+       if ((idx >= q->n_bd) || (!iwl_queue_used(q, idx))) {
                IWL_ERR(trans,
                        "%s: Read index for DMA queue txq id (%d), index %d is out of range [0-%d] %d %d.\n",
                        __func__, txq_id, idx, q->n_bd,