Dear Fabio Estevam,

> commit acc4959fc1 (Revert "i.MX28: Enable additional DRAM address bits")
> broke mx28evk boot.
> 
> Fix it by properly adjusting the HW_DRAM_CTL29 register value.
> 
> Suggested-by: Marek Vasut <ma...@denx.de>
> Signed-off-by: Fabio Estevam <fabio.este...@freescale.com>
> ---
>  board/freescale/mx28evk/iomux.c |   14 ++++++++++++++
>  1 files changed, 14 insertions(+), 0 deletions(-)
> 
> diff --git a/board/freescale/mx28evk/iomux.c

Uh, why isn't this file called splboot.c?

But
Acked-by: Marek Vasut <ma...@denx.de>

> b/board/freescale/mx28evk/iomux.c index 6587c45..1e60204 100644
> --- a/board/freescale/mx28evk/iomux.c
> +++ b/board/freescale/mx28evk/iomux.c
> @@ -161,6 +161,20 @@ const iomux_cfg_t iomux_setup[] = {
>               (MXS_PAD_3V3 | MXS_PAD_8MA | MXS_PAD_PULLUP),
>  };
> 
> +#define HW_DRAM_CTL29        (0x74 >> 2)
> +#define CS_MAP               0xf
> +#define COLUMN_SIZE  0x2
> +#define ADDR_PINS    0x1
> +#define APREBIT              0xa
> +
> +#define HW_DRAM_CTL29_CONFIG (CS_MAP << 24 | COLUMN_SIZE << 16 | \
> +                                     ADDR_PINS << 8 | APREBIT)
> +
> +void mx28_adjust_memory_params(uint32_t *dram_vals)
> +{
> +     dram_vals[HW_DRAM_CTL29] = HW_DRAM_CTL29_CONFIG;
> +}
> +
>  void board_init_ll(void)
>  {
>       mx28_common_spl_init(iomux_setup, ARRAY_SIZE(iomux_setup));

Best regards,
Marek Vasut
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to