| Message ID | 20190911092856.11146-1-colin.king@canonical.com (mailing list archive) |
|---|---|
| State | Mainlined |
| Commit | d97a3e92f33685768be04b2dfcd812f78f8ef341 |
| Delegated to: | Jason Gunthorpe |
| Headers | show |
| Series | RDMA/bnxt_re: fix spelling mistake "missin_resp" -> "missing_resp" | expand |
On Wed, Sep 11, 2019 at 10:28:56AM +0100, Colin King wrote: > From: Colin Ian King <colin.king@canonical.com> > > There is a spelling mistake in a literal string, fix it. > > Signed-off-by: Colin Ian King <colin.king@canonical.com> > --- > 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
On Thu, Sep 12, 2019 at 8:40 PM Jason Gunthorpe <jgg@mellanox.com> wrote: > > On Wed, Sep 11, 2019 at 10:28:56AM +0100, Colin King wrote: > > From: Colin Ian King <colin.king@canonical.com> > > > > There is a spelling mistake in a literal string, fix it. > > > > Signed-off-by: Colin Ian King <colin.king@canonical.com> > > --- > > 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 <selvin.xavier@broadcom.com>
On Wed, Sep 11, 2019 at 10:28:56AM +0100, Colin King wrote: > From: Colin Ian King <colin.king@canonical.com> > > There is a spelling mistake in a literal string, fix it. > > Signed-off-by: Colin Ian King <colin.king@canonical.com> > --- > drivers/infiniband/hw/bnxt_re/hw_counters.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Applied to for-next, thanks Jason
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",