Messages in this thread | | | From | Jérôme Pouiller <> | Subject | Re: [PATCH 5/6] staging: wfx/hif_rx.c: Fix spelling mistake "negociation" => "negotiation" | Date | Thu, 20 Aug 2020 11:11:16 +0200 |
| |
Hello Suraj,
Thank you for your contribution.
On Thursday 20 August 2020 10:51:27 CEST Suraj Upadhyay wrote: > > Signed-off-by: Suraj Upadhyay <usuraj35@gmail.com>
Even for a small change, you have to fill the commit log body.
> --- > drivers/staging/wfx/hif_rx.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/staging/wfx/hif_rx.c b/drivers/staging/wfx/hif_rx.c > index cc7c0cf226ba..1d32973d8ec1 100644 > --- a/drivers/staging/wfx/hif_rx.c > +++ b/drivers/staging/wfx/hif_rx.c > @@ -118,7 +118,7 @@ static int hif_keys_indication(struct wfx_dev *wdev, > > // SL_PUB_KEY_EXCHANGE_STATUS_SUCCESS is used by legacy secure link > if (body->status && body->status != HIF_STATUS_SLK_NEGO_SUCCESS) > - dev_warn(wdev->dev, "secure link negociation error\n"); > + dev_warn(wdev->dev, "secure link negotiation error\n");
This change is already in the staging tree. See commit c9638363f02d ("staging: wfx: fix a handful of spelling mistakes").
Globally, if you post changes for drivers/staging you should rebase your work on the staging tree before to send it. You can use the script get_maintainer.pl to get this information:
$ ./scripts/get_maintainer.pl --scm drivers/staging/wfx [...] git git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git
-- Jérôme Pouiller
| |