[PATCH 1/6] mtd: nand_bbt: drop unnecessary header

[PATCH 1/6] mtd: nand_bbt: drop unnecessary header

[Peter Pan 潘栋 (peterpandong)]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="gb2312", Size: 737 bytes --]


Signed-off-by: Brian Norris <computersforpeace@gmail.com>
Signed-off-by: Peter Pan <peterpandong@micron.com>
---
 drivers/mtd/nand/nand_bbt.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/mtd/nand/nand_bbt.c b/drivers/mtd/nand/nand_bbt.c
index 9bb8453..307a285 100644
--- a/drivers/mtd/nand/nand_bbt.c
+++ b/drivers/mtd/nand/nand_bbt.c
@@ -64,7 +64,6 @@
 #include <linux/mtd/mtd.h>
 #include <linux/mtd/bbm.h>
 #include <linux/mtd/nand.h>
-#include <linux/mtd/nand_ecc.h>
 #include <linux/bitops.h>
 #include <linux/delay.h>
 #include <linux/vmalloc.h>
-- 
1.9.1
ÿôèº{.nÇ+‰·Ÿ®‰­†+%ŠËÿ±éݶ\x17¥Šwÿº{.nÇ+‰·¥Š{±þG«éÿŠ{ayº\x1dʇڙë,j\a­¢f£¢·hšïêÿ‘êçz_è®\x03(­éšŽŠÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?™¨è­Ú&£ø§~á¶iO•æ¬z·švØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?–I¥

Re: [PATCH 1/6] mtd: nand_bbt: drop unnecessary header

[Brian Norris]

On Thu, Apr 16, 2015 at 02:09:41AM +0000, Peter Pan 潘栋 (peterpandong) wrote:
> 
> Signed-off-by: Brian Norris <computersforpeace@gmail.com>
> Signed-off-by: Peter Pan <peterpandong@micron.com>

Why are you just resending my patches? You could Ack/Reviewed-by/etc.
instead...

http://patchwork.ozlabs.org/patch/444604/

> ---
>  drivers/mtd/nand/nand_bbt.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/drivers/mtd/nand/nand_bbt.c b/drivers/mtd/nand/nand_bbt.c
> index 9bb8453..307a285 100644
> --- a/drivers/mtd/nand/nand_bbt.c
> +++ b/drivers/mtd/nand/nand_bbt.c
> @@ -64,7 +64,6 @@
>  #include <linux/mtd/mtd.h>
>  #include <linux/mtd/bbm.h>
>  #include <linux/mtd/nand.h>
> -#include <linux/mtd/nand_ecc.h>
>  #include <linux/bitops.h>
>  #include <linux/delay.h>
>  #include <linux/vmalloc.h>
> -- 
> 1.9.1

Re: [PATCH 1/6] mtd: nand_bbt: drop unnecessary header

[Rafał Miłecki]

On 22 April 2015 at 19:50, Brian Norris <computersforpeace@gmail.com> wrote:
> On Thu, Apr 16, 2015 at 02:09:41AM +0000, Peter Pan 潘栋 (peterpandong) wrote:
>>
>> Signed-off-by: Brian Norris <computersforpeace@gmail.com>
>> Signed-off-by: Peter Pan <peterpandong@micron.com>
>
> Why are you just resending my patches? You could Ack/Reviewed-by/etc.
> instead...
>
> http://patchwork.ozlabs.org/patch/444604/

That was fishy to me too, thanks for clarifying it Brian.

Also, if you *really* need to submit somebody's patch, you *should*
use From to point the author to git.

Re: [PATCH 1/6] mtd: nand_bbt: drop unnecessary header

[Rafał Miłecki]

On 16 April 2015 at 04:09, Peter Pan 潘栋 (peterpandong)
<peterpandong@micron.com> wrote:
>
> Signed-off-by: Brian Norris <computersforpeace@gmail.com>
> Signed-off-by: Peter Pan <peterpandong@micron.com>

I'm curious, what made you picking my e-mail address directly?

-- 
Rafał

RE: [PATCH 1/6] mtd: nand_bbt: drop unnecessary header

[Peter Pan 潘栋 (peterpandong)]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="utf-8", Size: 1155 bytes --]

On 23 April 2015 at 15:47, Rafał Miłecki wrote:
> On 22 April 2015 at 19:50, Brian Norris <computersforpeace@gmail.com>
> wrote:
> > On Thu, Apr 16, 2015 at 02:09:41AM +0000, Peter Pan 潘栋
> (peterpandong) wrote:
> >>
> >> Signed-off-by: Brian Norris <computersforpeace@gmail.com>
> >> Signed-off-by: Peter Pan <peterpandong@micron.com>
> >
> > Why are you just resending my patches? You could Ack/Reviewed-by/etc.
> > instead...
> >
> > http://patchwork.ozlabs.org/patch/444604/
> 
> That was fishy to me too, thanks for clarifying it Brian.
> 
> Also, if you *really* need to submit somebody's patch, you *should*
> use From to point the author to git.


Hi Brain and Rafał,

I am really sorry for this. It is my mistake. I just start sending patch
to community recently. If I do something improper, please tell me.

I will resend Brain's and my patch with the right comment, but please look
through this patchset if you can.

Sorry for bringing troubles to you again.

Thanks
Peter Pan
ÿôèº{.nÇ+‰·Ÿ®‰­†+%ŠËÿ±éݶ\x17¥Šwÿº{.nÇ+‰·¥Š{±þG«éÿŠ{ayº\x1dʇڙë,j\a­¢f£¢·hšïêÿ‘êçz_è®\x03(­éšŽŠÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?™¨è­Ú&£ø§~á¶iO•æ¬z·švØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?–I¥

Re: [PATCH 1/6] mtd: nand_bbt: drop unnecessary header

[Rafał Miłecki]

On 24 April 2015 at 02:36, Peter Pan 潘栋 (peterpandong)
<peterpandong@micron.com> wrote:
> On 23 April 2015 at 15:47, Rafał Miłecki wrote:
>> On 22 April 2015 at 19:50, Brian Norris <computersforpeace@gmail.com>
>> wrote:
>> > On Thu, Apr 16, 2015 at 02:09:41AM +0000, Peter Pan 潘栋
>> (peterpandong) wrote:
>> >>
>> >> Signed-off-by: Brian Norris <computersforpeace@gmail.com>
>> >> Signed-off-by: Peter Pan <peterpandong@micron.com>
>> >
>> > Why are you just resending my patches? You could Ack/Reviewed-by/etc.
>> > instead...
>> >
>> > http://patchwork.ozlabs.org/patch/444604/
>>
>> That was fishy to me too, thanks for clarifying it Brian.
>>
>> Also, if you *really* need to submit somebody's patch, you *should*
>> use From to point the author to git.
>
>
> Hi Brain and Rafał,
>
> I am really sorry for this. It is my mistake. I just start sending patch
> to community recently. If I do something improper, please tell me.
>
> I will resend Brain's and my patch with the right comment, but please look
> through this patchset if you can.

I don't think there is any need to re-send Brian's patches. I'm sure
he's aware of them :)
You may try pinging him, asking what's the status. And send your own
with an appropriate comment that it depends on other stuff.

-- 
Rafał