[U-Boot] [PATCH] cmd: booti: fix the image runtime location

Peng Fan van.freenix at gmail.com
Mon Sep 12 11:18:53 CEST 2016


On Mon, Sep 12, 2016 at 05:07:58PM +0800, Peng Fan wrote:
>We should not use "bi_dram[0].start + text_offset" as the image dst.
>The text_offset maybe 0 for some images, such as XEN. Then the dst
>is actually bi_dram[0].start, which maybe the location of spin table.
>
>Let's use "images->ep & ~(ih->text_offset)" as the dst address.

This patch maybe not that correct according to the doc from Linux kernel.
"
The Image must be placed text_offset bytes from a 2MB aligned base
address anywhere in usable system RAM and called there. The region
between the 2 MB aligned base address and the start of the image has no
special significance to the kernel, and may be used for other purposes.
"

Now I do not have a good idea that we may have a spin table in the start
of DRAM or even a small firmware.

Is it better to change gd->bd->bi_dram[0].start?
For example physical dram starts from 0x80000000, but 4K is reserved at the beginning.
So is it ok to change gd->bd->bi_dram[0].start to 0x80001000?

Thanks,
Peng.

>
>Signed-off-by: Peng Fan <peng.fan at nxp.com>
>Cc: Tom Rini <trini at konsulko.com>
>---
> cmd/booti.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
>diff --git a/cmd/booti.c b/cmd/booti.c
>index 6c1c998..afc87e3 100644
>--- a/cmd/booti.c
>+++ b/cmd/booti.c
>@@ -54,7 +54,9 @@ static int booti_setup(bootm_headers_t *images)
> 	 * If we are not at the correct run-time location, set the new
> 	 * correct location and then move the image there.
> 	 */
>-	dst = gd->bd->bi_dram[0].start + le64_to_cpu(ih->text_offset);
>+	dst = images->ep & ~(ih->text_offset);
>+	if (dst < gd->bd->bi_dram[0].start)
>+		dst = gd->bd->bi_dram[0].start + le64_to_cpu(ih->text_offset);
> 
> 	unmap_sysmem(ih);
> 
>-- 
>2.6.2
>


More information about the U-Boot mailing list