Hi Simon:

On 2017年07月07日 11:58, Simon Glass wrote:

Hi Andy,

On 30 June 2017 at 01:47, Andy Yan andy.yan@rock-chips.com wrote:

...

Some platform like rk3036 has no enough sapce for malloc and also has no necessary to use malloc function in spl stage. But the malloc pool is need in u-boot stage before relocation.Make the reserve functions as weak, so we can override it with a no malloc reserve in spl stage and use the weak function to reserve malloc pool in u-boot stage.

Signed-off-by: Andy Yan andy.yan@rock-chips.com

---

common/init/board_init.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)

I am not keen on weak functions for this sort of thing.

Can we instead be explicit and introduce a new control CONFIG_SPL_SYS_MALLOC_F?

I also have this ideas, but with a grep I found the CONFIG_SYS_MALLOC_F_LEN is spread all over the project. So maybe it's a little hard to seperate it. Anyway, I will have a try.

Regards, Simon