[PATCH V5 1/2] Nand driver for Nomadik 8815 SoC (on NHK8815 board)

vimal singh vimal.newwork at gmail.com
Mon Jul 27 02:28:32 EDT 2009


On Fri, Jul 24, 2009 at 6:07 PM, Alessandro Rubini<rubini-list at gnudd.com> wrote:
>> Please run ''checkpatch.pl' for this patch and try to fix the errors.
>
> Ok, will do. Although I don't agree with everything checkpatch is
> saying (I've heard checkpatch is meant as suggestions, to be
> considered but not applied blindly, so I behaved accordingly).
>
>> this can be 'NAND_ECC_SOFT' here... then
>
> As explained in the comments above, there is the hardware in there,
> only I've not been able to use it yet. So I prefer to stay HW and then
> change as little as possible.  But, unless you confirm I can do that,
> I'll write SOFT in there as requested.

As of now, I think you can make it 'NAND_ECC_SOFT'. And then once you
are able to use HW ECC, you can change it back.
Currently making it HW confuses with wrong API being used in
'nand_base.c' (uses nand_'read/write'_page_hwecc rather than
nand_read/write_page_swecc routines), although it does not make much
difference in functionality.

Regards,
vimal

>
> Will repost on monday, I'm off for the weekend.
>
> /alessandro
>



More information about the linux-mtd mailing list