[U-Boot] [PATCH 3/3] Consolidate arch-specific mem_malloc_init() implementations

Graeme Russ
2009-08-22T22:48:49Z

On Sun, Aug 23, 2009 at 2:45 AM, Peter Tyser<ptyser@xes-inc.com> wrote:
> Jean-Christophe PLAGNIOL-VILLARD wrote:
>> On 23:05 Fri 21 Aug , Peter Tyser wrote:
>>> Signed-off-by: Peter Tyser <ptyser@xes-inc.com>
>>> ---

[snip]

>>>
>>> +#ifndef CONFIG_X86
>> could we have a generic name (not arch)
>> CONFIG_GENERIC_MALLOC_INIT
>>
>> and then define it in the configs/arch.h
>
> I thought the cleanest fix would be to update the x86 call to
> mem_malloc_init() to match all the other architectures', but didn't want
> to delve into the x86 arch init code.
>
> I think adding a new CONFIG_* option to handle 1 non-conforming
> architecture is dirtier than syncing up the x86 init code in any case.
>

I am currently undertaking a fairly significant tidy up of some x86
loose ends. One that I noticed (and have not touched just yet) is how
the x86 allocates the malloc pool. After these (Peter's) patches get
applied, I will revisit the x86 malloc and bring it into line with the
other arches

Regards,

G
> Best,
> Peter
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
>