On Fri, Feb 10, 2017 at 03:19:42AM -0800, Selvin Xavier wrote:
> Implements support for create_cq, destroy_cq and req_notify_cq
> verbs.
>
> v3: Code cleanup based on errors reported by sparse on endianness check.
>     Removes unwanted macros.
> v5: Use min_t macro to calculate the CQ entries
>
> Signed-off-by: Eddie Wai <eddie....@broadcom.com>
> Signed-off-by: Devesh Sharma <devesh.sha...@broadcom.com>
> Signed-off-by: Somnath Kotur <somnath.ko...@broadcom.com>
> Signed-off-by: Sriharsha Basavapatna <sriharsha.basavapa...@broadcom.com>
> Signed-off-by: Selvin Xavier <selvin.xav...@broadcom.com>
> ---

<...>

> diff --git a/drivers/infiniband/hw/bnxt_re/qplib_fp.h 
> b/drivers/infiniband/hw/bnxt_re/qplib_fp.h
> index 78e1717..1dfc86d 100644
> --- a/drivers/infiniband/hw/bnxt_re/qplib_fp.h
> +++ b/drivers/infiniband/hw/bnxt_re/qplib_fp.h
> @@ -38,6 +38,52 @@
>
>  #ifndef __BNXT_QPLIB_FP_H__
>  #define __BNXT_QPLIB_FP_H__
> +#define BNXT_QPLIB_MAX_CQE_ENTRY_SIZE        sizeof(struct cq_base)
> +

IMHO, this define is redundant.
Reviewed-by: Leon Romanovsky <leo...@mellanox.com>

Thanks

Attachment: signature.asc
Description: PGP signature

Reply via email to