[U-Boot] [Patch v2] common: Add get_effective_memsize() to memsize.c

[York Sun <yorksun@freescale.com>]

On 02/11/2014 11:34 PM, Albert ARIBAUD wrote:
> Hi York,
> 
> On Tue, 11 Feb 2014 11:57:26 -0800, York Sun <yorksun@freescale.com>
> wrote:
> 
>> This function has been around for powerpc. It is used for systems with
>> memory more than CONFIG_MAX_MEM_MAPPED. In case of non-contiguous memory,
>> this feature can limit U-boot to one block without going over the limit.
>>
>> Signed-off-by: York Sun <yorksun@freescale.com>
>> ---
>> Change log:
>>  v1: The function is added to arch/arm/lib/board.c without changing others
>>  v2: The function is moved to common/memsize.c, using __weak. The prototype
>>      is added to include/common.h. Also consolidate existing functions for
>>      powerpc.
>>
>>  arch/arm/lib/board.c               |    2 +-
>>  arch/powerpc/cpu/mpc512x/traps.c   |    1 -
>>  arch/powerpc/cpu/mpc85xx/traps.c   |    1 -
>>  arch/powerpc/cpu/mpc86xx/traps.c   |    1 -
>>  arch/powerpc/lib/board.c           |   18 ------------------
>>  arch/powerpc/lib/bootm.c           |    1 -
>>  board/freescale/c29xpcie/spl.c     |    2 +-
>>  board/freescale/p1022ds/spl.c      |    2 +-
>>  board/freescale/p1_p2_rdb_pc/spl.c |    2 +-
>>  common/board_f.c                   |   11 -----------
>>  common/cmd_log.c                   |    2 +-
>>  common/memsize.c                   |   16 +++++++++++++++-
>>  include/common.h                   |    1 +
>>  13 files changed, 21 insertions(+), 39 deletions(-)
> 
> For the trivial ARM change:
> 
> Acked-by: Albert ARIBAUD <albert.u.boot@aribaud.net>
> 
> Since the rest is essentially PPC, though, it makes sense not to go
> through the ARM repo for this patch.
> 

Agree. Originally I put the change mainly to arch/arm/lib/board.c. That's why it
went to you.

I will merge it.

York