[U-Boot] [PATCH] mtd: pxa3xx_nand: Correct allocation and init bug

Kevin Smith kevin.smith at elecsyscorp.com
Fri Oct 23 22:57:47 CEST 2015


On 10/23/2015 03:34 PM, Scott Wood wrote:
> Does Linux have this problem?  Assuming no, please fix this by making the
> driver look more like Linux.  At least then it would be the same ugliness.
There are 2 problems and one improvement:
1) Invalid dereference.  This is U-Boot-only code not taken from Linux.  
Removed.
2) Bad pointer math.  This is different from Linux, and I have fixed it 
by making it more like Linux.
3) Unnecessary memory allocation.  I just noticed this while 
investigating my crashes caused by the other two issues.

> Can you explain how the change in the calculation of "chip" and the
> allocation size is relevant to the NULL dereference?  Couldn't that be fixed
> by just removing the "info->host[0]->mtd" line?
It's not, they are two separate bugs that crash when I try to load from 
NAND.  Perhaps I should submit a patch series for this?

- Kevin


More information about the U-Boot mailing list