2023-06-25 01:15:02

by Yang Li

[permalink] [raw]
Subject: [PATCH -next] RDMA/bnxt_re: Remove unneeded semicolon

./drivers/infiniband/hw/bnxt_re/main.c: ib_verbs.h is included more than once.

Reported-by: Abaci Robot <[email protected]>
Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=5588
Signed-off-by: Yang Li <[email protected]>
---
drivers/infiniband/hw/bnxt_re/main.c | 1 -
1 file changed, 1 deletion(-)

diff --git a/drivers/infiniband/hw/bnxt_re/main.c b/drivers/infiniband/hw/bnxt_re/main.c
index 1e5b60f9269e..b42166fe7454 100644
--- a/drivers/infiniband/hw/bnxt_re/main.c
+++ b/drivers/infiniband/hw/bnxt_re/main.c
@@ -66,7 +66,6 @@
#include <rdma/bnxt_re-abi.h>
#include "bnxt.h"
#include "hw_counters.h"
-#include "ib_verbs.h"

static char version[] =
BNXT_RE_DESC "\n";
--
2.20.1.7.g153144c



2023-06-25 11:44:54

by Leon Romanovsky

[permalink] [raw]
Subject: Re: [PATCH -next] RDMA/bnxt_re: Remove unneeded semicolon

On Sun, Jun 25, 2023 at 08:57:43AM +0800, Yang Li wrote:
> ./drivers/infiniband/hw/bnxt_re/main.c: ib_verbs.h is included more than once.
>
> Reported-by: Abaci Robot <[email protected]>
> Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=5588
> Signed-off-by: Yang Li <[email protected]>
> ---
> drivers/infiniband/hw/bnxt_re/main.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/infiniband/hw/bnxt_re/main.c b/drivers/infiniband/hw/bnxt_re/main.c
> index 1e5b60f9269e..b42166fe7454 100644
> --- a/drivers/infiniband/hw/bnxt_re/main.c
> +++ b/drivers/infiniband/hw/bnxt_re/main.c
> @@ -66,7 +66,6 @@
> #include <rdma/bnxt_re-abi.h>
> #include "bnxt.h"
> #include "hw_counters.h"
> -#include "ib_verbs.h"

The subject line doesn't correlate with commit message and the proposed
change.

Thanks