[RESUBMIT] [PATCH] [MTD] NAND nand_ecc.c: rewrite for improved performance

Troy Kisky troy.kisky at boundarydevices.com
Fri Aug 15 14:56:20 EDT 2008


David Woodhouse wrote:
> On Fri, 2008-08-15 at 12:04 +0200, Frans Meulenbroeks wrote:
>> 2008/8/15, David Woodhouse <dwmw2 at infradead.org>:
>>> On Fri, 2008-08-15 at 11:23 +0200, Frans Meulenbroeks wrote:
>>>  > 2008/8/15, David Woodhouse <dwmw2 at infradead.org>:
>>>
>>> No need -- if you've thought about it and believe it should work, that's
>>>  probably enough. I just saw some 'unsigned long' data types, which are
>>>  going to have a different size between 32-bit and 64-bit systems, and
>>>  wondered if that would introduce differences.
>> I was unaware of that. For me unsigned long is more or less a synonym
>> for 32 bit. Maybe I'm just getting too old :-(
>> Anyway, if you have a suggestion for a better type, I'll happily change things.
>> Would uint32_t be better?
> 
> If it needs to be 32-bit, then yes -- uint32_t is the correct type to
> use.
> 
> If 64-bit is OK, then 'unsigned long' is likely to be more efficient on
> some platforms.
> 

You might also test it on a big endian system to be safe.

Troy



More information about the linux-mtd mailing list