[U-Boot] [PATCH 1/5] mx6sabre{auto, sd}: Change FDT loading address to avoid overlaping

[Otavio Salvador]

On Fri, Nov 1, 2013 at 10:25 AM, Fabio Estevam <festevam at gmail.com> wrote:
> Hi Otavio,
>
> On Fri, Nov 1, 2013 at 9:25 AM, Otavio Salvador <otavio at ossystems.com.br> wrote:
>> 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.
>
> Care to explain in more details about the overlap?
>
> Currently we use the following addresses:
>
> - dtb: 0x11000000
> - kernel: 0x12000000
>
> Now you propose:
>
> - kernel: 0x12000000
> - dtb: 0x18000000
>
> If the kernel is larger in FSL 3.10.9 kernel, how it can overlap with
> dtb, since in the original code the kernel goes after the dtb?
>
> I have never used FSL 3.10.9 kernel, so I am curious about this fix.

I am quoting the original commit change:

    ENGR00268032 config/sabre_common: change the fdt_addr to a high address

    current the fdt_addr is just 16MB offset from the ddr base address, which
    will cause the dtb will be overritten by linux kernel(include .bss section)
    if the linux kernel is bigger than 16MB, which cause setup_machine_fdt
    failed and thus kernel failed to boot.

    This patch change the defaut fdt_addr to 128MB offset from the ddr base
    address, which should be enough for common user case. user can change it
    to other value according to their system needs.

    Signed-off-by: Jason Liu <r64343 at freescale.com>

I think I can rework the commit log and include this there.

-- 
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750