[PATCH v3] mtd: gpmi: add NAND write verify support
Marek Vasut
marex at denx.de
Sat Aug 11 08:47:17 EDT 2012
Dear Huang Shijie,
> On Sat, Aug 11, 2012 at 3:13 PM, Marek Vasut <marex at denx.de> wrote:
> > Dear Huang Shijie,
> >
> > Please re-add credit to Fabio. Also, Fabio fixed the subject to be more
>
> sorry. Should I add "Reported by : Fabio ...." to the patch?
Yes, at least.
> I really appriaciate Fabio's work, but I do not think Fabio's patch is
> more accurate.
> The nand chips can be 2K page, 4K page and 8K page. But most of the
> time, the nand chip is 4K page.
> If we fix the verify_buf to 8K size as Fabio's patch does. It will
> waste much memory.
> This is my concern now. :)
That's all right, but the rest of your patch is Fabio's work, correct me if I'm
wrong. Just give the guy some credit, he found the flaw and diag'd it.
> My patch maybe not good. So If Fabio also sends out a new patch which
> allocates the verify_buf with the mtd->writesize size.
> I will ack it.
Either way is OK.
[...]
> > Best regards,
> > Marek Vasut
More information about the linux-arm-kernel
mailing list