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

vimal singh vimal.newwork at gmail.com
Mon Jul 27 04:46:01 EDT 2009


On Mon, Jul 27, 2009 at 11:58 AM, vimal singh<vimal.newwork at gmail.com> wrote:
> 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.
>
To make it work for 512 bytes sector ECC in SW, below patch is required:

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

Which was posted sometime back, but unfortunately did not get merged.

David,

Can we get this patch merged? If required, I will post updated patch
after re-basing.

-vimal

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



-- 
---
Regards,
\/ | |\/| /-\ |_

____      __o
------   -\<,
-----  ( )/ ( )



More information about the linux-mtd mailing list