Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1728712 > unrolled thread

[PATCH v3] staging: pi433: Fixes minor typo errors

Back to article view | Back to linux.kernel

Contents

  [PATCH v3] staging: pi433: Fixes minor typo errors harsha <harshasharmaiitr@gmail.com> - 2017-09-08 11:20 +0200
    Re: [PATCH v3] staging: pi433: Fixes minor typo errors Greg KH <gregkh@linuxfoundation.org> - 2017-09-08 11:30 +0200
    Re: [PATCH v3] staging: pi433: Fixes minor typo errors Dan Carpenter <dan.carpenter@oracle.com> - 2017-09-08 11:40 +0200

#1728712 — [PATCH v3] staging: pi433: Fixes minor typo errors

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

Signed-off-by: Harsha Sharma <harshasharmaiitr@gmail.com>
---
 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

[toc] | [next] | [standalone]

#1728716

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 <harshasharmaiitr@gmail.com>
> ---
>  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

[toc] | [prev] | [next] | [standalone]

#1728729

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

regards,
dan carpenter

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web