net/bnxt: fix compilation error with some compilers
Checks
Commit Message
The driver was defining its own version of roundup which was
conflicting with another version defined elsewhere.
Change the local definition of roundup to avoid compilation errors.
Fixes: c5d09af8aa46 ("net/bnxt: support thor controller")
Signed-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
---
drivers/net/bnxt/bnxt_ethdev.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
Comments
14/06/2019 13:20, Ajit Khaparde:
> The driver was defining its own version of roundup which was
> conflicting with another version defined elsewhere.
>
> Change the local definition of roundup to avoid compilation errors.
>
> Fixes: c5d09af8aa46 ("net/bnxt: support thor controller")
Fixed the wrong reference to
Fixes: f8168ca0e690 ("net/bnxt: support thor controller")
> Signed-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
Applied, thanks
@@ -3545,7 +3545,7 @@ static void bnxt_free_ctx_mem(struct bnxt *bp)
bp->ctx = NULL;
}
-#define roundup(x, y) ((((x) + ((y) - 1)) / (y)) * (y))
+#define bnxt_roundup(x, y) ((((x) + ((y) - 1)) / (y)) * (y))
#define min_t(type, x, y) ({ \
type __min1 = (x); \
@@ -3612,7 +3612,7 @@ int bnxt_alloc_ctx_mem(struct bnxt *bp)
return rc;
entries = ctx->qp_max_l2_entries;
- entries = roundup(entries, ctx->tqm_entries_multiple);
+ entries = bnxt_roundup(entries, ctx->tqm_entries_multiple);
entries = clamp_t(uint32_t, entries, ctx->tqm_min_entries_per_ring,
ctx->tqm_max_entries_per_ring);
for (i = 0, ena = 0; i < BNXT_MAX_Q; i++) {