On 12/06/2011 06:18 PM, Stefano Babic wrote:
On 31/10/2011 17:23, Simon Schwarz wrote:
From: Simon Schwarz<simonschwarz...@googlemail.com>

spl_nand overwrote BSS section because it reads a whole block everytime. Now
loads the block to spare area and just copy the needed junk to destination.
Whole block read is necessary for ecc check!

Signed-off-by: Simon Schwarz<simonschwarz...@gmail.com>
---

@@ -71,7 +71,8 @@ void spl_nand_load_image(void)
                        CONFIG_SYS_NAND_PAGE_SIZE, (void *)header);
                spl_parse_image_header(header);
                nand_spl_load_image(CONFIG_SYS_NAND_SPL_KERNEL_OFFS,
-                       spl_image.size, (void *)spl_image.load_addr);
+                       spl_image.size,
+                       (void *)spl_image.load_addr - sizeof(header));
                                                              ^---


Do you mean maybe sizeof(*header) ?

However, spl_image.load_addr was already decremented in
spl_parse_image_header() and correctly set to 64 byte before the load
address. Do we really need it ?

[SNIP]
Best regards,
Stefano Babic


You are right -> V9. Lets see if we get it to two digits ;)

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

Reply via email to