[PATCH 1/9] configs: meson64: permit redefining SYS_MALLOC_LEN
Mattijs Korpershoek
mkorpershoek at baylibre.com
Fri Aug 6 11:24:54 CEST 2021
Hi Tom, Neil,
Neil Armstrong <narmstrong at baylibre.com> writes:
> On 05/08/2021 19:16, Tom Rini wrote:
>> On Thu, Aug 05, 2021 at 06:20:53PM +0200, Neil Armstrong wrote:
>>> On 05/08/2021 17:17, Mattijs Korpershoek wrote:
>>>> Permit redefining SYS_MALLOC_LEN for board specific configs.
>>>> This is especially useful for Android with AVB, which requires a malloc
>>>> length of 128M.
>>>>
>>>> Signed-off-by: Guillaume La Roque <glaroque at baylibre.com>
>>>> Signed-off-by: Mattijs Korpershoek <mkorpershoek at baylibre.com>
>>>> ---
>>>> include/configs/meson64.h | 2 ++
>>>> 1 file changed, 2 insertions(+)
>>>>
>>>> diff --git a/include/configs/meson64.h b/include/configs/meson64.h
>>>> index fb7f16d148..f9bb0240d2 100644
>>>> --- a/include/configs/meson64.h
>>>> +++ b/include/configs/meson64.h
>>>> @@ -32,7 +32,9 @@
>>>> #define CONFIG_CPU_ARMV8
>>>> #define CONFIG_REMAKE_ELF
>>>> #define CONFIG_SYS_MAXARGS 32
>>>> +#ifndef CONFIG_SYS_MALLOC_LEN
>>>> #define CONFIG_SYS_MALLOC_LEN (32 << 20)
>>>> +#endif
>>>> #define CONFIG_SYS_CBSIZE 1024
>>>>
>>>> #define CONFIG_SYS_SDRAM_BASE 0
>>>>
>>>
>>> Acked-by: Neil Armstrong <narmstrong at baylibre.com>
>>
>> I prefer to see these as SZ_xxx, but also what's the minimum DRAM you're
>> going to see on a meson64 platform? And if it's 1GB or more, why not
>> just use 128MB for everyone?
>>
>
> We can get down to 256MiB DRAM (very rare), but 512MiB is used on La Frite.
In that case, I think it's more reasonable to keep the change as-is.
I'll send a v2 replacing (32 << 20) by SZ_32M
Do you agree ?
Mattijs
>
> Neil
More information about the U-Boot
mailing list