On 12/06/2011 11:53 AM, Simon Schwarz wrote: > On 12/06/2011 06:18 PM, Stefano Babic wrote: >> I found the readon of the kernel corrupt image. We are setting a very >> hard address in /nand_spl_simple.c: >> >> ecc_calc = (u_char *)(CONFIG_SYS_SDRAM_BASE + 0x10000); >> >> Because the image for a TI SOC is loaded at 0x80008000, we have a >> conflict and the image is corrupted where the ECC is computed. >> >> It is not a really good idea to fix in this way where to compute the >> ECC. Should be not better to put it in the CONFIG_SYS_INIT_RAM_ADDR >> area ?
There's a patch to fix this (with some minor changes requested): http://patchwork.ozlabs.org/patch/128018/ >> Best regards, >> Stefano Babic >> > > Hmm. This is from the former nand_spl. > > Why not use malloc for this? Thanks to changes in the FAT driver we now > have it in the SPL. > > I don't see a reason to have this in SRAM when SDRAM is available. nand_spl_simple.c is used by some very small SPL targets -- 4K or so. malloc doesn't fit. -Scott _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
