diff options
| author |   Jubin John <jubin.john@intel.com> | 2016-02-14 20:22:09 -0800 |
|---|---|---|
| committer |   Doug Ledford <dledford@redhat.com> | 2016-03-10 20:45:42 -0500 |
| commit | edddfca00eecd0949a9adccf8dd490478f641cbc (patch) | |
| tree | e125f1147daef0fc48a9a102ac84ab21ffd3a2e3 /drivers/staging | |
| parent | staging/rdma/hfi1: Add braces on all arms of statement (diff) | |
| download | linux-dev-edddfca00eecd0949a9adccf8dd490478f641cbc.tar.xz linux-dev-edddfca00eecd0949a9adccf8dd490478f641cbc.zip | |
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>
Diffstat (limited to 'drivers/staging')
| -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 4f40c987303b..a17cb732277e 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; |