Patchwork [V2,3/6] mtd: bcm47xxnflash: fix message

login
register
mail settings
Submitter Rafał Miłecki
Date Jan. 28, 2013, 10:25 a.m.
Message ID <1359368750-19763-3-git-send-email-zajec5@gmail.com>
Download mbox | patch
Permalink /patch/216153/
State New
Headers show

Comments

Rafał Miłecki - Jan. 28, 2013, 10:25 a.m.
From: Hauke Mehrtens <hauke@hauke-m.de>

This is not a serial flash driver, but a nand flash driver

Signed-off-by: Hauke Mehrtens <hauke@hauke-m.de>
---
V2: break line to match 80 chars
---
 drivers/mtd/nand/bcm47xxnflash/main.c |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
Artem Bityutskiy - Feb. 4, 2013, 8:35 a.m.
On Mon, 2013-01-28 at 11:25 +0100, Rafał Miłecki wrote:
> From: Hauke Mehrtens <hauke@hauke-m.de>
> 
> This is not a serial flash driver, but a nand flash driver
> 
> Signed-off-by: Hauke Mehrtens <hauke@hauke-m.de>
> ---
> V2: break line to match 80 chars

When sending v2 for individual patches, please, remember 2 things:

1. Keep them part of the original thread (ensure correct In-Reply-To:).
2. Add your own signed-off-by.

Thanks!

P.S. I took your versions of patches, thanks!
Rafał Miłecki - Feb. 4, 2013, 8:42 a.m.
2013/2/4 Artem Bityutskiy <dedekind1@gmail.com>:
> On Mon, 2013-01-28 at 11:25 +0100, Rafał Miłecki wrote:
>> From: Hauke Mehrtens <hauke@hauke-m.de>
>>
>> This is not a serial flash driver, but a nand flash driver
>>
>> Signed-off-by: Hauke Mehrtens <hauke@hauke-m.de>
>> ---
>> V2: break line to match 80 chars
>
> When sending v2 for individual patches, please, remember 2 things:
>
> 1. Keep them part of the original thread (ensure correct In-Reply-To:).

Whoops, I've to learn how to make that with git send-email.


> 2. Add your own signed-off-by.

It was so trivial... I wasn't sure if it's worth signing-it-by... ;)
Artem Bityutskiy - Feb. 4, 2013, 9:02 a.m.
On Mon, 2013-02-04 at 09:42 +0100, Rafał Miłecki wrote:
> 2013/2/4 Artem Bityutskiy <dedekind1@gmail.com>:
> > On Mon, 2013-01-28 at 11:25 +0100, Rafał Miłecki wrote:
> >> From: Hauke Mehrtens <hauke@hauke-m.de>
> >>
> >> This is not a serial flash driver, but a nand flash driver
> >>
> >> Signed-off-by: Hauke Mehrtens <hauke@hauke-m.de>
> >> ---
> >> V2: break line to match 80 chars
> >
> > When sending v2 for individual patches, please, remember 2 things:
> >
> > 1. Keep them part of the original thread (ensure correct In-Reply-To:).
> 
> Whoops, I've to learn how to make that with git send-email.

 --in-reply-to=<identifier>

Pick the identifier from the Message-ID tag of the e-mail you are
replying to. E.g., save the e-mail as hauke.mbox file and do:

$ grep -i '^Message-ID:' hauke.mbox
Message-Id: <1359045599-1305-2-git-send-email-hauke@hauke-m.de>

which means that for this one the 'git send-email' option will be:

--in-reply-to="<1359045599-1305-2-git-send-email-hauke@hauke-m.de>"

> > 2. Add your own signed-off-by.
> 
> It was so trivial... I wasn't sure if it's worth signing-it-by... ;)

Signed-off-by is not about how big were your changes, it roughly
speaking means that you confirm that the source of this patch for you is
Hauke Mehrtens.

See Documentation/SubmittingPatches

The Signed-off-by: tag indicates that the signer was involved in the
development of the patch, or that he/she was in the patch's delivery
path.

I googled a bit and found also this:

http://kerneltrap.org/files/Jeremy/DCO.txt

which gives additional clue about what can be behind SOB.

Patch

diff --git a/drivers/mtd/nand/bcm47xxnflash/main.c b/drivers/mtd/nand/bcm47xxnflash/main.c
index 8363a9a..3e20762 100644
--- a/drivers/mtd/nand/bcm47xxnflash/main.c
+++ b/drivers/mtd/nand/bcm47xxnflash/main.c
@@ -94,7 +94,8 @@  static int __init bcm47xxnflash_init(void)
 	 */
 	err = platform_driver_probe(&bcm47xxnflash_driver, bcm47xxnflash_probe);
 	if (err)
-		pr_err("Failed to register serial flash driver: %d\n", err);
+		pr_err("Failed to register bcm47xx nand flash driver: %d\n",
+		       err);
 
 	return err;
 }