2017-09-08 09:14:15

by Harsha Sharma

[permalink] [raw]
Subject: [PATCH v3] staging: pi433: Fixes minor typo errors

Fixes checkpatch warning -- "occured" and "succesfully" are misspelled

Signed-off-by: Harsha Sharma <[email protected]>
---
drivers/staging/pi433/pi433_if.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/pi433/pi433_if.c b/drivers/staging/pi433/pi433_if.c
index 6b9b7df..6a10478 100644
--- a/drivers/staging/pi433/pi433_if.c
+++ b/drivers/staging/pi433/pi433_if.c
@@ -473,7 +473,7 @@ struct pi433_instance {
}


- /* rx done, wait was interrupted or error occured */
+ /* rx done, wait was interrupted or error occurred */
abort:
dev->interrupt_rx_allowed = true;
SET_CHECKED(rf69_set_mode(dev->spi, standby));
@@ -1033,7 +1033,7 @@ static int setup_GPIOs(struct pi433_device *device)
if (retval)
return retval;

- dev_dbg(&device->spi->dev, "%s succesfully configured", name);
+ dev_dbg(&device->spi->dev, "%s successfully configured", name);
}

return 0;
--
1.9.1


2017-09-08 09:28:26

by Greg Kroah-Hartman

[permalink] [raw]
Subject: Re: [PATCH v3] staging: pi433: Fixes minor typo errors

On Fri, Sep 08, 2017 at 02:42:14PM +0530, harsha wrote:
> Fixes checkpatch warning -- "occured" and "succesfully" are misspelled
>
> Signed-off-by: Harsha Sharma <[email protected]>
> ---
> drivers/staging/pi433/pi433_if.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)

{sigh}

Ok, last time for me, I have said all of these before, yet you have not
changed anything:
- your From: name does not match the signed-off-by: line
- you did not describe what changed in the different versions of
patches, that goes below the --- line, like the documentation
tells you how to do.

greg k-h

2017-09-08 09:39:43

by Dan Carpenter

[permalink] [raw]
Subject: Re: [PATCH v3] staging: pi433: Fixes minor typo errors

Your From email header says "harsha" but it should match your signed off
by line which says "Harsha Sharma".

regards,
dan carpenter