Re: [PATCH 2/2] Use bit-wise majority to recover the contents of ONFI parameter

2018-05-02 Thread Boris Brezillon
On Wed, 2 May 2018 12:25:45 +0200 Boris Brezillon wrote: > Hi Jane, > > On Thu, 26 Apr 2018 17:19:56 -0700 > Jane Wan wrote: > > > Signed-off-by: Jane Wan > > --- > > drivers/mtd/nand/nand_base.c | 35

Re: [PATCH 2/2] Use bit-wise majority to recover the contents of ONFI parameter

2018-05-02 Thread Boris Brezillon
On Wed, 2 May 2018 12:25:45 +0200 Boris Brezillon wrote: > Hi Jane, > > On Thu, 26 Apr 2018 17:19:56 -0700 > Jane Wan wrote: > > > Signed-off-by: Jane Wan > > --- > > drivers/mtd/nand/nand_base.c | 35 +++ > > 1 file changed, 31 insertions(+), 4

Re: [PATCH 2/2] Use bit-wise majority to recover the contents of ONFI parameter

2018-05-02 Thread Boris Brezillon
Hi Jane, On Thu, 26 Apr 2018 17:19:56 -0700 Jane Wan wrote: > Signed-off-by: Jane Wan > --- > drivers/mtd/nand/nand_base.c | 35 +++ > 1 file changed, 31 insertions(+), 4 deletions(-) > > diff --git

Re: [PATCH 2/2] Use bit-wise majority to recover the contents of ONFI parameter

2018-05-02 Thread Boris Brezillon
Hi Jane, On Thu, 26 Apr 2018 17:19:56 -0700 Jane Wan wrote: > Signed-off-by: Jane Wan > --- > drivers/mtd/nand/nand_base.c | 35 +++ > 1 file changed, 31 insertions(+), 4 deletions(-) > > diff --git a/drivers/mtd/nand/nand_base.c

RE: [PATCH 2/2] Use bit-wise majority to recover the contents of ONFI parameter

2018-04-30 Thread Wan, Jane (Nokia - US/Sunnyvale)
Hi Miquèl, Thank you for your response and feedback. I've modified the fix based on your comments. Please see the updated patch file at the end of this message (also in attachment). My answers to your comments/questions are inline in the previous message. The new patch is rebased on top of

RE: [PATCH 2/2] Use bit-wise majority to recover the contents of ONFI parameter

2018-04-30 Thread Wan, Jane (Nokia - US/Sunnyvale)
Hi Miquèl, Thank you for your response and feedback. I've modified the fix based on your comments. Please see the updated patch file at the end of this message (also in attachment). My answers to your comments/questions are inline in the previous message. The new patch is rebased on top of

Re: [PATCH 2/2] Use bit-wise majority to recover the contents of ONFI parameter

2018-04-28 Thread Miquel Raynal
Hi Jane, Same comments as before, please: get the right maintainers, add a commit log, rebase and fix the title prefix. Have you ever needed/tried this algorithm before? On Thu, 26 Apr 2018 17:19:56 -0700, Jane Wan wrote: > Signed-off-by: Jane Wan >

Re: [PATCH 2/2] Use bit-wise majority to recover the contents of ONFI parameter

2018-04-28 Thread Miquel Raynal
Hi Jane, Same comments as before, please: get the right maintainers, add a commit log, rebase and fix the title prefix. Have you ever needed/tried this algorithm before? On Thu, 26 Apr 2018 17:19:56 -0700, Jane Wan wrote: > Signed-off-by: Jane Wan > --- > drivers/mtd/nand/nand_base.c |

[PATCH 2/2] Use bit-wise majority to recover the contents of ONFI parameter

2018-04-26 Thread Jane Wan
Signed-off-by: Jane Wan --- drivers/mtd/nand/nand_base.c | 35 +++ 1 file changed, 31 insertions(+), 4 deletions(-) diff --git a/drivers/mtd/nand/nand_base.c b/drivers/mtd/nand/nand_base.c index c2e1232..161b523 100644 ---

[PATCH 2/2] Use bit-wise majority to recover the contents of ONFI parameter

2018-04-26 Thread Jane Wan
Signed-off-by: Jane Wan --- drivers/mtd/nand/nand_base.c | 35 +++ 1 file changed, 31 insertions(+), 4 deletions(-) diff --git a/drivers/mtd/nand/nand_base.c b/drivers/mtd/nand/nand_base.c index c2e1232..161b523 100644 --- a/drivers/mtd/nand/nand_base.c +++