2019-09-11 09:33:11

by Colin King

Subject: [PATCH] RDMA/bnxt_re: fix spelling mistake "missin_resp" -> "missing_resp"

From: Colin Ian King <[email protected]>

There is a spelling mistake in a literal string, fix it.

Signed-off-by: Colin Ian King <[email protected]>
---
drivers/infiniband/hw/bnxt_re/hw_counters.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/infiniband/hw/bnxt_re/hw_counters.c b/drivers/infiniband/hw/bnxt_re/hw_counters.c
index 604b71875f5f..3421a0b15983 100644
--- a/drivers/infiniband/hw/bnxt_re/hw_counters.c
+++ b/drivers/infiniband/hw/bnxt_re/hw_counters.c
@@ -74,7 +74,7 @@ static const char * const bnxt_re_stat_name[] = {
[BNXT_RE_SEQ_ERR_NAKS_RCVD] = "seq_err_naks_rcvd",
[BNXT_RE_MAX_RETRY_EXCEEDED] = "max_retry_exceeded",
[BNXT_RE_RNR_NAKS_RCVD] = "rnr_naks_rcvd",
- [BNXT_RE_MISSING_RESP] = "missin_resp",
+ [BNXT_RE_MISSING_RESP] = "missing_resp",
[BNXT_RE_UNRECOVERABLE_ERR] = "unrecoverable_err",
[BNXT_RE_BAD_RESP_ERR] = "bad_resp_err",
[BNXT_RE_LOCAL_QP_OP_ERR] = "local_qp_op_err",
--
2.20.1

2019-09-12 20:46:13

by Jason Gunthorpe

Subject: Re: [PATCH] RDMA/bnxt_re: fix spelling mistake "missin_resp" -> "missing_resp"

On Wed, Sep 11, 2019 at 10:28:56AM +0100, Colin King wrote:
> From: Colin Ian King <[email protected]>
>
> There is a spelling mistake in a literal string, fix it.
>
> Signed-off-by: Colin Ian King <[email protected]>
> ---
> drivers/infiniband/hw/bnxt_re/hw_counters.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/infiniband/hw/bnxt_re/hw_counters.c b/drivers/infiniband/hw/bnxt_re/hw_counters.c
> index 604b71875f5f..3421a0b15983 100644
> --- a/drivers/infiniband/hw/bnxt_re/hw_counters.c
> +++ b/drivers/infiniband/hw/bnxt_re/hw_counters.c
> @@ -74,7 +74,7 @@ static const char * const bnxt_re_stat_name[] = {
> [BNXT_RE_SEQ_ERR_NAKS_RCVD] = "seq_err_naks_rcvd",
> [BNXT_RE_MAX_RETRY_EXCEEDED] = "max_retry_exceeded",
> [BNXT_RE_RNR_NAKS_RCVD] = "rnr_naks_rcvd",
> - [BNXT_RE_MISSING_RESP] = "missin_resp",
> + [BNXT_RE_MISSING_RESP] = "missing_resp",

Broadcom folks, can you confirm if this is OK? Is the string ABI for
this driver?

Jason

2019-09-13 14:35:47

by Selvin Xavier

Subject: Re: [PATCH] RDMA/bnxt_re: fix spelling mistake "missin_resp" -> "missing_resp"

On Thu, Sep 12, 2019 at 8:40 PM Jason Gunthorpe <[email protected]> wrote:
>
> On Wed, Sep 11, 2019 at 10:28:56AM +0100, Colin King wrote:
> > From: Colin Ian King <[email protected]>
> >
> > There is a spelling mistake in a literal string, fix it.
> >
> > Signed-off-by: Colin Ian King <[email protected]>
> > ---
> > drivers/infiniband/hw/bnxt_re/hw_counters.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/infiniband/hw/bnxt_re/hw_counters.c b/drivers/infiniband/hw/bnxt_re/hw_counters.c
> > index 604b71875f5f..3421a0b15983 100644
> > --- a/drivers/infiniband/hw/bnxt_re/hw_counters.c
> > +++ b/drivers/infiniband/hw/bnxt_re/hw_counters.c
> > @@ -74,7 +74,7 @@ static const char * const bnxt_re_stat_name[] = {
> > [BNXT_RE_SEQ_ERR_NAKS_RCVD] = "seq_err_naks_rcvd",
> > [BNXT_RE_MAX_RETRY_EXCEEDED] = "max_retry_exceeded",
> > [BNXT_RE_RNR_NAKS_RCVD] = "rnr_naks_rcvd",
> > - [BNXT_RE_MISSING_RESP] = "missin_resp",
> > + [BNXT_RE_MISSING_RESP] = "missing_resp",
>
> Broadcom folks, can you confirm if this is OK? Is the string ABI for
> this driver?
>

bnxt_re doesn't have a string ABI.
This is a spelling mistake while posting the patch and it is okay to
merge this patch.

Thanks
Acked-by: Selvin Xavier <[email protected]>

2019-09-16 19:25:52

by Jason Gunthorpe

Subject: Re: [PATCH] RDMA/bnxt_re: fix spelling mistake "missin_resp" -> "missing_resp"

On Wed, Sep 11, 2019 at 10:28:56AM +0100, Colin King wrote:
> From: Colin Ian King <[email protected]>
>
> There is a spelling mistake in a literal string, fix it.
>
> Signed-off-by: Colin Ian King <[email protected]>
> ---
> drivers/infiniband/hw/bnxt_re/hw_counters.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)

Applied to for-next, thanks

Jason