Dear Linus Walleij,

In message <1310598276-9741-1-git-send-email-linus.wall...@linaro.org> you 
wrote:
> This fixes up the SDRAM memory detection code to work with the
> latest relocation code, moves it all into dram_init() and
> activates memory size detection for the Integrator AP.
> 
> Signed-off-by: Linus Walleij <linus.wall...@linaro.org>
> ---
>  board/armltd/integrator/integrator.c |   17 +++++++----------
>  include/configs/integratorap.h       |    2 +-
>  2 files changed, 8 insertions(+), 11 deletions(-)
...
> @@ -120,8 +112,13 @@ extern void dram_query(void);
>        */
>       sdram_shift              = ((cm_reg_sdram & 0x0000001C)/4)%4;
>       gd->bd->bi_dram[0].size  = 0x01000000 << sdram_shift;
> -
> +     gd->ram_size = get_ram_size((long *)CONFIG_SYS_SDRAM_BASE,
> +                                 0x01000000 << sdram_shift);
>       }
> +#else
> +        gd->bd->bi_dram[0].size = PHYS_SDRAM_1_SIZE;
> +     gd->ram_size = get_ram_size((long *)CONFIG_SYS_SDRAM_BASE,
> +                                 PHYS_SDRAM_1_SIZE);

Indentation by TAB only, please.

> diff --git a/include/configs/integratorap.h b/include/configs/integratorap.h
> index acdb37c..d724c91 100644
> --- a/include/configs/integratorap.h
> +++ b/include/configs/integratorap.h
> @@ -47,7 +47,7 @@
>  #define CONFIG_SKIP_LOWLEVEL_INIT
>  #define CONFIG_CM_INIT               1
>  #define CONFIG_CM_REMAP              1
> -#undef CONFIG_CM_SPD_DETECT
> +#define CONFIG_CM_SPD_DETECT
>  #define CONFIG_SYS_DCACHE_OFF

Are you sure this works for all affected boards?

Please also put the board maintainer on Cc: for all patches that
affedt specific boards like here.



Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
"I find this a nice feature but it is not according to  the  documen-
tation. Or is it a BUG?"   "Let's call it an accidental feature. :-)"
                       - Larry Wall in <6...@jpl-devvax.jpl.nasa.gov>
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to