Skip to contentSkip to conversation

omap3_beagle: Fix device tree boot

PATCH New Roger Quadros 2 participants 4 messages Nov 5, 2014

lore
Share & tools
lore

Roger Quadros

author
#0 lore raw

PATCH 1/1 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 <[email protected]>
---
 include/configs/omap3_beagle.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
include/configs/omap3_beagle.h +1 -1
index f25a940..5ea048a 100644
@@ -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" \
Signature · 1 line
1.8.3.2
#1 lore raw

PATCH 1/1 omap3_beagle: Fix device tree boot

Quoted 17 lines · Roger Quadros

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 <[email protected]>
---
 include/configs/omap3_beagle.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
include/configs/omap3_beagle.h +0 -0
index f25a940..5ea048a 100644

include/configs/omap3_beagle.h line 135

Quoted 5 lines
include/configs/omap3_beagle.h +1 -1
@@ -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!

Signature · 8 lines
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141105/444c8be8/attachment.pgp>
author
#2 lore raw

PATCH 1/1 omap3_beagle: Fix device tree boot

Quoted 14 lines · Tom Rini
Quoted 12 lines · Roger Quadros

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 <[email protected]>
---
 include/configs/omap3_beagle.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Quoted 15 lines

include/configs/omap3_beagle.h line 135

Quoted 10 lines
include/configs/omap3_beagle.h +1 -1
index f25a940..5ea048a 100644
@@ -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

#3 lore raw

PATCH 1/1 omap3_beagle: Fix device tree boot

Quoted 30 lines · Roger Quadros
Quoted 14 lines · Tom Rini
Quoted 12 lines · Roger Quadros

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 <[email protected]>
---
 include/configs/omap3_beagle.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Quoted 15 lines

include/configs/omap3_beagle.h line 135

Quoted 10 lines
include/configs/omap3_beagle.h +1 -1
index f25a940..5ea048a 100644
@@ -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!

Quoted 4 lines

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 :)

Signature · 8 lines
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141105/33a5b27f/attachment.pgp>