[U-Boot] at91: smartweb board fails with current HEAD

Simon Glass
2017-07-06T04:48:42Z

Hi Heiko,

On 20 June 2017 at 22:26, Heiko Schocher <hs@denx.de> wrote:
> Hello Wenyou,
>
> Am 21.06.2017 um 02:52 schrieb Wenyou.Yang at microchip.com:
>>
>>
>>
>>> -----Original Message-----
>>> From: Heiko Schocher [mailto:hs at denx.de]
>>> Sent: 2017年6月20日 22:05
>>> To: Wenyou Yang - A41535 <Wenyou.Yang@microchip.com>
>>> Cc: u-boot at lists.denx.de; Andreas Bießmann <biessmann@corscience.de>
>>> Subject: at91: smartweb board fails with current HEAD
>>>
>>> Hello Wenyou,
>>>
>>> since your patch:
>>>
>>> commit f8b7fff1d5c5297d290acab1e398bd410b0c2e68
>>> Author: Wenyou Yang <wenyou.yang@atmel.com>
>>> Date: Fri Apr 14 15:01:28 2017 +0800
>>>
>>> serial: atmel_usart: Add clk support
>>>
>>> smartweb board fails booting (and I fear at last all other at91 based
>>> siemens
>>> boards also). Reverting your patch and U-Boot boots fine again.
>>>
>>> First problem is, that I need now a bigger MALLOC_F_LEN space, but the
>>> value is
>>> for SPL and U-Boot ... but in SPL I cannot increase this value, as I have
>>> only 4k
>>> init ram.
>>>
>>> So I need to configure a lower value in SPL as in U-Boot ... this is not
>>> possible at
>>> the moment... or?
>>
>>
>> After supporting the driver model and device tree, the size of SPL is
>> beyond 4K.
>> SPL is not suitable in 4K internal SRAM
>
>
> Hmm... no?
>
> I have running a SPL (without DM/DTS support of course) from current
> mainline
> source on the board.
> Only problem is, that we need different MALLOC_F_LEN space in SPL and U-Boot
> now!
>
> added Simon Glass and Tom Rini, may I oversee some obvious config and/or
> they have an idea?

I think it would be possible to add CONFIG_SPL_MALLOC_F_LEN as a new
Kconfig option. It could default to the same value as the main
version.

Regards,
Simon …