[U-Boot] [PATCH 1/2] mtd/nand:Fix wrong address read in is_blank()

Marek Vasut marek.vasut at gmail.com
Wed Dec 28 08:28:54 CET 2011


> > > IFC NAND Machine calculates ECC on 512byte sector. Same is taken care
> > > in
> > > fsl_ifc_run_command() while ECC status verification. Here buffer
> > > number is calculated assuming 512byte sector and same is passed to
> > 
> > is_blank.
> > 
> > > However in is_blank() buffer address is calculated using
> > > mdt->writesize which is wrong. It should be calculated on basis of ecc
> > 
> > sector size.
> > 
> > > Also, in fsl_ifc_run_command() bufferpage is calculated on the basis
> > > of ecc sector size instead of hard coded value.
> > > 
> > > Signed-off-by: Poonam Aggrwal <poonam.aggrwal at freescale.com>
> > > Signed-off-by: Prabhakar Kushwaha <prabhakar at freescale.com>
> > 
> > You just sent two patches based on the same commit, but those are clearly
> > dependent. This is weird. So next time:
> > 
> > 1) Submit this with git-send-email as a series
> 
> I will take care this point in future
> 
> > 2) iomem looks linuxish, so instead of specifying u-boot commit, specify
> > Linux commit/tree etc. from where this is pulled so when someone synces
> > next time, he'll know where he is in linux.
> 
> This patch is independent of Linux tree. I will float similar patch to
> Linux tree.
> 
> > 3) Cc Scott Wood
> 
> I will take care this point in future
> 
> --Prabhakar

Awesome, good going, thanks!

M


More information about the U-Boot mailing list