On Fri, Jan 20, 2012 at 06:38:14PM +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
> is_blank()'s implementation checks for Page size and OOB area size. 
> is_blank() should be called per page for main and OOB area verification.
> 
> Variables name are redefined to avoid confusion between buffer and ecc sector.
> 
> Signed-off-by: Poonam Aggrwal <poonam.aggr...@freescale.com>
> Signed-off-by: Scott Wood <scottw...@freescale.com>
> Signed-off-by: Prabhakar Kushwaha <prabha...@freescale.com>
> ---
>  Applied upon git://git.denx.de/u-boot.git (branch master)
> 
>  This patch is based upon ifc driver patch in linuxppc-dev mailing list
>  http://patchwork.ozlabs.org/patch/136547/
> 
>  It is a replacement of my earlier patch 
>   "mtd/nand:Fix wrong address read in is_blank()"
>  http://patchwork.ozlabs.org/patch/133388/ 
> 
> 
>  drivers/mtd/nand/fsl_ifc_nand.c |   52 
> +++++++++++++++++++++------------------
>  1 files changed, 28 insertions(+), 24 deletions(-)

Applied to u-boot-nand-flash...

> +                             /*
> +                              * Uncorrectable error.
> +                              * OK only if the whole page is blank.
> +                              *
> +                              * We disable ECCER reporting due to...
> +                              * erratum IFC-A002770 -- so report it now if we
> +                              * see an uncorrectable error in ECCSTAT.
> +                              */

...with the "..." removed from the comment.

-Scott

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to