[U-Boot] [PATCH] nand read.jffs2 (nand_legacy) in common/cmd_nand.c

Scott Wood scottwood at freescale.com
Tue Jan 20 19:14:13 CET 2009


Scott Wood wrote:
> On Tue, Jan 20, 2009 at 04:57:55PM +0100, Schlaegl Manfred jun. wrote:
>> Error with CONFIG_NAND_LEGACY in common/cmd_nand.c:
>> With current code "nand read.jffs2s" (read and skip bad blocks) is always interpreted as 
>> "nand read.jffs2" (read and fill bad blocks with 0xff). This is because ".jffs2" is 
>> tested before ".jffs2s" and only the first two characters are compared.
>>
>> Correction:
>> Test for ".jffs2s" first and compare the first 7 characters.
> 
> I'll apply this, though I recommend switching to the non-legacy code
> (which always block-skips, BTW).  I'd like to see the legacy code go away
> soon.

And in the future, please add a Signed-off-by line.

-Scott


More information about the U-Boot mailing list