diff options
author | Naresh Gottumukkala <bgottumukkala@emulex.com> | 2013-08-07 03:22:32 -0400 |
---|---|---|
committer | Roland Dreier <roland@purestorage.com> | 2013-08-12 13:58:37 -0400 |
commit | f99b1649dbb6342d618307faef1f214fd54928b9 (patch) | |
tree | 29eeeb358a40d6d5021c90cc845e5e329043b832 /drivers/infiniband/hw/ocrdma/ocrdma.h | |
parent | c095ba7224d8edc71dcef0d655911399a8bd4a3f (diff) |
RDMA/ocrdma: Style and redundant code cleanup
Code cleanup and remove redundant code:
1) redundant initialization removed
2) braces changed as per CodingStyle.
3) redundant checks removed
4) extra braces in return statements removed.
5) removed unused pd pointer from mr.
6) reorganized get_dma_mr()
7) fixed set_av() to return error on invalid sgid index.
8) reference to ocrdma_dev removed from struct ocrdma_pd.
Signed-off-by: Naresh Gottumukkala <bgottumukkala@emulex.com>
Signed-off-by: Roland Dreier <roland@purestorage.com>
Diffstat (limited to 'drivers/infiniband/hw/ocrdma/ocrdma.h')
-rw-r--r-- | drivers/infiniband/hw/ocrdma/ocrdma.h | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/infiniband/hw/ocrdma/ocrdma.h b/drivers/infiniband/hw/ocrdma/ocrdma.h index d540180a8e42..5c00600135ed 100644 --- a/drivers/infiniband/hw/ocrdma/ocrdma.h +++ b/drivers/infiniband/hw/ocrdma/ocrdma.h | |||
@@ -317,7 +317,6 @@ struct ocrdma_mr { | |||
317 | struct ib_mr ibmr; | 317 | struct ib_mr ibmr; |
318 | struct ib_umem *umem; | 318 | struct ib_umem *umem; |
319 | struct ocrdma_hw_mr hwmr; | 319 | struct ocrdma_hw_mr hwmr; |
320 | struct ocrdma_pd *pd; | ||
321 | }; | 320 | }; |
322 | 321 | ||
323 | struct ocrdma_ucontext { | 322 | struct ocrdma_ucontext { |
@@ -393,7 +392,7 @@ static inline int is_cqe_valid(struct ocrdma_cq *cq, struct ocrdma_cqe *cqe) | |||
393 | { | 392 | { |
394 | int cqe_valid; | 393 | int cqe_valid; |
395 | cqe_valid = le32_to_cpu(cqe->flags_status_srcqpn) & OCRDMA_CQE_VALID; | 394 | cqe_valid = le32_to_cpu(cqe->flags_status_srcqpn) & OCRDMA_CQE_VALID; |
396 | return ((cqe_valid == cq->phase) ? 1 : 0); | 395 | return (cqe_valid == cq->phase); |
397 | } | 396 | } |
398 | 397 | ||
399 | static inline int is_cqe_for_sq(struct ocrdma_cqe *cqe) | 398 | static inline int is_cqe_for_sq(struct ocrdma_cqe *cqe) |