diff options
| author | Jubin John <jubin.john@intel.com> | 2016-02-15 04:22:09 (GMT) |
|---|---|---|
| committer | Doug Ledford <dledford@redhat.com> | 2016-03-11 01:45:42 (GMT) |
| commit | edddfca00eecd0949a9adccf8dd490478f641cbc (patch) | |
| tree | e125f1147daef0fc48a9a102ac84ab21ffd3a2e3 | |
| parent | e490974e675e8ddec795137c1db7f38e0308cbcd (diff) | |
| download | linux-edddfca00eecd0949a9adccf8dd490478f641cbc.tar.xz | |
staging/rdma/hfi1: Remove else after break
Remove else after break to fix checkpatch warning:
WARNING: else is not generally useful after a break or return
Reviewed-by: Dennis Dalessandro <dennis.dalessandro@intel.com>
Reviewed-by: Ira Weiny <ira.weiny@intel.com>
Reviewed-by: Mike Marciniszyn <mike.marciniszyn@intel.com>
Signed-off-by: Jubin John <jubin.john@intel.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>
| -rw-r--r-- | drivers/staging/rdma/hfi1/qp.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/staging/rdma/hfi1/qp.c b/drivers/staging/rdma/hfi1/qp.c index 4f40c98..a17cb73 100644 --- a/drivers/staging/rdma/hfi1/qp.c +++ b/drivers/staging/rdma/hfi1/qp.c @@ -329,11 +329,9 @@ __be32 hfi1_compute_aeth(struct rvt_qp *qp) if (credit_table[x] > credits) { max = x; } else { - if (min == x) { + if (min == x) break; - } else { - min = x; - } + min = x; } } aeth |= x << HFI1_AETH_CREDIT_SHIFT; |