Patchwork [U-Boot,4/5] nitrogen6x: Change FDT loading address to avoid overlaping

login
register
mail settings
Submitter Otavio Salvador
Date Nov. 1, 2013, 11:25 a.m.
Message ID <1383305158-26019-4-git-send-email-otavio@ossystems.com.br>
Download mbox | patch
Permalink /patch/287794/
State Changes Requested
Delegated to: Stefano Babic
Headers show

Comments

Otavio Salvador - Nov. 1, 2013, 11:25 a.m.
The new FSL 3.10.9_1.0.0-alpha kernel requires more memory space and
with the previous loading address we had ovelap; change it for the
same address used in 2013.04-3.10.9_1.0.0-alpha U-Boot.

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
 include/configs/nitrogen6x.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Marek Vasut - Nov. 1, 2013, 12:19 p.m.
Dear Otavio Salvador,

> The new FSL 3.10.9_1.0.0-alpha kernel requires more memory space and
> with the previous loading address we had ovelap; change it for the
> same address used in 2013.04-3.10.9_1.0.0-alpha U-Boot.
> 
> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
> ---
>  include/configs/nitrogen6x.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/include/configs/nitrogen6x.h b/include/configs/nitrogen6x.h
> index 3454b86..a2a3bb3 100644
> --- a/include/configs/nitrogen6x.h
> +++ b/include/configs/nitrogen6x.h
> @@ -182,7 +182,7 @@
>  	"fdt_high=0xffffffff\0" \
>  	"initrd_high=0xffffffff\0" \
>  	"fdt_file=imx6q-sabrelite.dtb\0" \
> -	"fdt_addr=0x11000000\0" \
> +	"fdt_addr=0x18000000\0" \
>  	"boot_fdt=try\0" \
>  	"ip_dyn=yes\0" \
>  	"mmcdev=0\0" \

Why is the FDT not relocated close to the end of DRAM as the other boards do it?

Best regards,
Marek Vasut
Otavio Salvador - Nov. 1, 2013, 1:06 p.m.
On Fri, Nov 1, 2013 at 10:19 AM, Marek Vasut <marex@denx.de> wrote:
> Dear Otavio Salvador,
>
>> The new FSL 3.10.9_1.0.0-alpha kernel requires more memory space and
>> with the previous loading address we had ovelap; change it for the
>> same address used in 2013.04-3.10.9_1.0.0-alpha U-Boot.
>>
>> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
...
> Why is the FDT not relocated close to the end of DRAM as the other boards do it?

No idea.
Marek Vasut - Nov. 1, 2013, 2:08 p.m.
Dear Otavio Salvador,

> On Fri, Nov 1, 2013 at 10:19 AM, Marek Vasut <marex@denx.de> wrote:
> > Dear Otavio Salvador,
> > 
> >> The new FSL 3.10.9_1.0.0-alpha kernel requires more memory space and
> >> with the previous loading address we had ovelap; change it for the
> >> same address used in 2013.04-3.10.9_1.0.0-alpha U-Boot.
> >> 
> >> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
> 
> ...
> 
> > Why is the FDT not relocated close to the end of DRAM as the other boards
> > do it?
> 
> No idea.

Is this patch papering over some bug in the simplest possible way then?

btw. I mean this portion of the boot log, see the last line:

## Flattened Device Tree blob at 11000000
   Booting using the fdt blob at 0x11000000
   Loading Kernel Image ... OK
   Using Device Tree in place at 11000000, end 1100b568

Best regards,
Marek Vasut

Patch

diff --git a/include/configs/nitrogen6x.h b/include/configs/nitrogen6x.h
index 3454b86..a2a3bb3 100644
--- a/include/configs/nitrogen6x.h
+++ b/include/configs/nitrogen6x.h
@@ -182,7 +182,7 @@ 
 	"fdt_high=0xffffffff\0" \
 	"initrd_high=0xffffffff\0" \
 	"fdt_file=imx6q-sabrelite.dtb\0" \
-	"fdt_addr=0x11000000\0" \
+	"fdt_addr=0x18000000\0" \
 	"boot_fdt=try\0" \
 	"ip_dyn=yes\0" \
 	"mmcdev=0\0" \