IB/mthca: Make functions that never fail return void

The function mthca_free_err_wqe() can never fail, so get rid of its
return value.  That means handle_error_cqe() doesn't have to check
what mthca_free_err_wqe() returns, which means it can't fail either
and doesn't have to return anything either.  All this results in
simpler source code and a slight object code improvement:

add/remove: 0/0 grow/shrink: 0/2 up/down: 0/-10 (-10)
function                                     old     new   delta
mthca_free_err_wqe                            83      81      -2
mthca_poll_cq                               1758    1750      -8

Signed-off-by: Roland Dreier <rolandd@cisco.com>

1 files changed, 10 insertions, 15 deletions

diff --git a/drivers/infiniband/hw/mthca/mthca_cq.c b/drivers/infiniband/hw/mthca/mthca_cq.c
index 96f1a86bf049..16a851b341f8 100644
--- a/drivers/infiniband/hw/mthca/mthca_cq.c
+++ b/drivers/infiniband/hw/mthca/mthca_cq.c
@@ -324,12 +324,11 @@ void mthca_cq_clean(struct mthca_dev *dev, u32 cqn, u32 qpn,
                 wake_up(&cq->wait);
 }
 
-static int handle_error_cqe(struct mthca_dev *dev, struct mthca_cq *cq,
-                            struct mthca_qp *qp, int wqe_index, int is_send,
-                            struct mthca_err_cqe *cqe,
-                            struct ib_wc *entry, int *free_cqe)
+static void handle_error_cqe(struct mthca_dev *dev, struct mthca_cq *cq,
+                             struct mthca_qp *qp, int wqe_index, int is_send,
+                             struct mthca_err_cqe *cqe,
+                             struct ib_wc *entry, int *free_cqe)
 {
-        int err;
         int dbd;
         __be32 new_wqe;
 
@@ -412,11 +411,9 @@ static int handle_error_cqe(struct mthca_dev *dev, struct mthca_cq *cq,
          * error case, so we don't have to check the doorbell count, etc.
          */
         if (mthca_is_memfree(dev))
-                return 0;
+                return;
 
-        err = mthca_free_err_wqe(dev, qp, is_send, wqe_index, &dbd, &new_wqe);
-        if (err)
-                return err;
+        mthca_free_err_wqe(dev, qp, is_send, wqe_index, &dbd, &new_wqe);
 
         /*
          * If we're at the end of the WQE chain, or we've used up our
@@ -424,15 +421,13 @@ static int handle_error_cqe(struct mthca_dev *dev, struct mthca_cq *cq,
          * the next poll operation.
          */
         if (!(new_wqe & cpu_to_be32(0x3f)) || (!cqe->db_cnt && dbd))
-                return 0;
+                return;
 
         cqe->db_cnt   = cpu_to_be16(be16_to_cpu(cqe->db_cnt) - dbd);
         cqe->wqe      = new_wqe;
         cqe->syndrome = SYNDROME_WR_FLUSH_ERR;
 
         *free_cqe = 0;
-
-        return 0;
 }
 
 static inline int mthca_poll_one(struct mthca_dev *dev,
@@ -518,9 +513,9 @@ static inline int mthca_poll_one(struct mthca_dev *dev,
         }
 
         if (is_error) {
-                err = handle_error_cqe(dev, cq, *cur_qp, wqe_index, is_send,
-                                       (struct mthca_err_cqe *) cqe,
-                                       entry, &free_cqe);
+                handle_error_cqe(dev, cq, *cur_qp, wqe_index, is_send,
+                                 (struct mthca_err_cqe *) cqe,
+                                 entry, &free_cqe);
                 goto out;
         }