[U-Boot] [u-boot][PATCH] omap3_beagle: Fix device tree boot
Push the device tree blob load address (fdtaddr) further apart than kernel load address (loadaddr) to accomodate a larger kernel image. We set fdtaddr to the same value as set in ti_armv7_common.h.
With this change, I'm able to boot linux-3.18-rc1 zImage built using omap2plus_defconfig.
Signed-off-by: Roger Quadros rogerq@ti.com --- include/configs/omap3_beagle.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/configs/omap3_beagle.h b/include/configs/omap3_beagle.h index f25a940..5ea048a 100644 --- a/include/configs/omap3_beagle.h +++ b/include/configs/omap3_beagle.h @@ -132,7 +132,7 @@ "loadaddr=0x80200000\0" \ "rdaddr=0x81000000\0" \ "fdt_high=0xffffffff\0" \ - "fdtaddr=0x80f80000\0" \ + "fdtaddr=0x88000000\0" \ "usbtty=cdc_acm\0" \ "bootfile=uImage\0" \ "ramdisk=ramdisk.gz\0" \
On Wed, Nov 05, 2014 at 03:57:34PM +0200, Roger Quadros wrote:
Push the device tree blob load address (fdtaddr) further apart than kernel load address (loadaddr) to accomodate a larger kernel image. We set fdtaddr to the same value as set in ti_armv7_common.h.
With this change, I'm able to boot linux-3.18-rc1 zImage built using omap2plus_defconfig.
Signed-off-by: Roger Quadros rogerq@ti.com
include/configs/omap3_beagle.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/configs/omap3_beagle.h b/include/configs/omap3_beagle.h index f25a940..5ea048a 100644 --- a/include/configs/omap3_beagle.h +++ b/include/configs/omap3_beagle.h @@ -132,7 +132,7 @@ "loadaddr=0x80200000\0" \ "rdaddr=0x81000000\0" \ "fdt_high=0xffffffff\0" \
- "fdtaddr=0x80f80000\0" \
- "fdtaddr=0x88000000\0" \
While we're in here whacking these values lets make this match up the rest of the values we use in ti_armv7_common.h (so drop fdt_high and set bootm_size and so forth). Thanks!
On 11/05/2014 04:18 PM, Tom Rini wrote:
On Wed, Nov 05, 2014 at 03:57:34PM +0200, Roger Quadros wrote:
Push the device tree blob load address (fdtaddr) further apart than kernel load address (loadaddr) to accomodate a larger kernel image. We set fdtaddr to the same value as set in ti_armv7_common.h.
With this change, I'm able to boot linux-3.18-rc1 zImage built using omap2plus_defconfig.
Signed-off-by: Roger Quadros rogerq@ti.com
include/configs/omap3_beagle.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/configs/omap3_beagle.h b/include/configs/omap3_beagle.h index f25a940..5ea048a 100644 --- a/include/configs/omap3_beagle.h +++ b/include/configs/omap3_beagle.h @@ -132,7 +132,7 @@ "loadaddr=0x80200000\0" \ "rdaddr=0x81000000\0" \ "fdt_high=0xffffffff\0" \
- "fdtaddr=0x80f80000\0" \
- "fdtaddr=0x88000000\0" \
While we're in here whacking these values lets make this match up the rest of the values we use in ti_armv7_common.h (so drop fdt_high and set bootm_size and so forth). Thanks!
It seems like omap3_beagle.h includes ti_omap3_common.h which in turn includes ti_armv7_common.h.
So I just need to remove the duplicate definitions from omap3_beagle.h
cheers, -roger
On Wed, Nov 05, 2014 at 04:25:55PM +0200, Roger Quadros wrote:
On 11/05/2014 04:18 PM, Tom Rini wrote:
On Wed, Nov 05, 2014 at 03:57:34PM +0200, Roger Quadros wrote:
Push the device tree blob load address (fdtaddr) further apart than kernel load address (loadaddr) to accomodate a larger kernel image. We set fdtaddr to the same value as set in ti_armv7_common.h.
With this change, I'm able to boot linux-3.18-rc1 zImage built using omap2plus_defconfig.
Signed-off-by: Roger Quadros rogerq@ti.com
include/configs/omap3_beagle.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/configs/omap3_beagle.h b/include/configs/omap3_beagle.h index f25a940..5ea048a 100644 --- a/include/configs/omap3_beagle.h +++ b/include/configs/omap3_beagle.h @@ -132,7 +132,7 @@ "loadaddr=0x80200000\0" \ "rdaddr=0x81000000\0" \ "fdt_high=0xffffffff\0" \
- "fdtaddr=0x80f80000\0" \
- "fdtaddr=0x88000000\0" \
While we're in here whacking these values lets make this match up the rest of the values we use in ti_armv7_common.h (so drop fdt_high and set bootm_size and so forth). Thanks!
It seems like omap3_beagle.h includes ti_omap3_common.h which in turn includes ti_armv7_common.h.
So I just need to remove the duplicate definitions from omap3_beagle.h
And references DEFAULT_LINUX_BOOT_ENV :)
participants (2)
-
Roger Quadros -
Tom Rini