[U-Boot] [PATCH 4/8] x86: fsp: Enlarge the size of malloc() pool before relocation

Bin Meng bmeng.cn at gmail.com
Tue Aug 18 04:11:44 CEST 2015


Hi Simon,

On Tue, Aug 18, 2015 at 10:00 AM, Simon Glass <sjg at chromium.org> wrote:
> Hi Bin,
>
> On 15 August 2015 at 01:07, Bin Meng <bmeng.cn at gmail.com> wrote:
>> After fsp_init() returns, the stack has already been switched to a
>> place within system memory as defined by CONFIG_FSP_TEMP_RAM_ADDR.
>> Enlarge the size of malloc() pool before relocation since we have
>> plenty of memory now.
>>
>> Signed-off-by: Bin Meng <bmeng.cn at gmail.com>
>> ---
>>
>>  arch/x86/Kconfig              | 7 +++++++
>>  arch/x86/cpu/start.S          | 8 ++++++++
>>  arch/x86/lib/fsp/fsp_common.c | 6 +++++-
>>  3 files changed, 20 insertions(+), 1 deletion(-)
>
> Can you please explain a bit why this patch is needed? Wouldn't it be
> better to avoid needing so much memory?
>

This is because even with pci-uclass change, the memory is still not
enough on Crown Bay to support PCI UART as the UART is behind two
bridges (at best), or five bridges (at worst).

Anyway I change this not only for Crown Bay. It's a "feature" that FSP
provides since it already switches to a new stack which is using
system memory where the size is not a problem.

Regards,
Bin


More information about the U-Boot mailing list