Re: [PATCH] mtd/nand:Fix wrong usage of is_blank() in fsl_ifc_run_command

2012-01-27 Thread Artem Bityutskiy
On Wed, 2012-01-18 at 09:43 +0530, Prabhakar Kushwaha wrote: Freescale IFC NAND Machine calculates ECC on 512byte sector and same is used in fsl_ifc_run_command() during ECC status verification. Also this sector is passed to is_blank() for blank checking. It is wrong at first place because

RE: [PATCH] mtd/nand:Fix wrong usage of is_blank() in fsl_ifc_run_command

2012-01-27 Thread Kushwaha Prabhakar-B32579
wrong usage of is_blank() in fsl_ifc_run_command On Wed, 2012-01-18 at 09:43 +0530, Prabhakar Kushwaha wrote: Freescale IFC NAND Machine calculates ECC on 512byte sector and same is used in fsl_ifc_run_command() during ECC status verification. Also this sector is passed to is_blank

Re: [PATCH] mtd/nand:Fix wrong usage of is_blank() in fsl_ifc_run_command

2012-01-20 Thread Scott Wood
On 01/17/2012 10:13 PM, Prabhakar Kushwaha wrote: + for (i = sector; i = sector_end; i++) { + errors = check_read_ecc(mtd, ctrl, eccstat, i); + + if (errors == 15) { + /* + *

[PATCH] mtd/nand:Fix wrong usage of is_blank() in fsl_ifc_run_command

2012-01-17 Thread Prabhakar Kushwaha
Freescale IFC NAND Machine calculates ECC on 512byte sector and same is used in fsl_ifc_run_command() during ECC status verification. Also this sector is passed to is_blank() for blank checking. It is wrong at first place because is_blank()'s implementation checks for Page size and OOB area size.