Re: [U-Boot] [PATCH 5/7] JFFS2: Change DEFAULT_EMPTY_SCAN_SIZE to 256 Bytes

2011-04-30 Thread Wolfgang Denk
Dear Baidu Liu, In message BANLkTin7REQN=pv1ythlfowrqtoj2pa...@mail.gmail.com you wrote: A patch description should include _all_ changes. Either you also put that change into the patch description or you make a separate change. Doing changes not described in the commit log are a good

Re: [U-Boot] [PATCH 5/7] JFFS2: Change DEFAULT_EMPTY_SCAN_SIZE to 256 Bytes

2011-04-29 Thread Detlev Zundel
Hi Baidu, 1/ Syncs up with jffs2 in the linux kernel: If the first 256 Bytes is 0xff,we get the conclusion that the sector is empty. Signed-off-by: Baidu Liu liucai@gmail.com --- fs/jffs2/jffs2_1pass.c | 11 ++- fs/jffs2/jffs2_nand_1pass.c | 13 ++---

Re: [U-Boot] [PATCH 5/7] JFFS2: Change DEFAULT_EMPTY_SCAN_SIZE to 256 Bytes

2011-04-29 Thread Baidu Liu
Hi, Detlev :                               default:                                       /* unknown */ -                                     putLabeledWord(UNKOWN COMPRESSION METHOD = , jNode-compr); +                                     putLabeledWord(UNKNOWN COMPRESSION METHOD = ,

Re: [U-Boot] [PATCH 5/7] JFFS2: Change DEFAULT_EMPTY_SCAN_SIZE to 256 Bytes

2011-04-29 Thread Baidu Liu
Hi,Detlev 2011/4/30 Detlev Zundel d...@denx.de: Hi Baidu, Hi,Detlev                               default:                                       /* unknown */ -                                     putLabeledWord(UNKOWN COMPRESSION METHOD = , jNode-compr); +                                

[U-Boot] [PATCH 5/7] JFFS2: Change DEFAULT_EMPTY_SCAN_SIZE to 256 Bytes

2011-04-23 Thread Baidu Liu
1/ Syncs up with jffs2 in the linux kernel: If the first 256 Bytes is 0xff,we get the conclusion that the sector is empty. Signed-off-by: Baidu Liu liucai@gmail.com --- fs/jffs2/jffs2_1pass.c | 11 ++- fs/jffs2/jffs2_nand_1pass.c | 13 ++--- include/jffs2/jffs2.h