On 06/05/2012 03:20 PM, Allen Martin wrote:
> This code is now included in the tegra20 SPL
> @@ -323,34 +91,10 @@ void init_pmc_scratch(void)
>
> /* ODMDATA is for kernel use to determine RAM size, LP config, etc. */
> writel(CONFIG_SYS_BOARD_ODMDATA, &pmc->pmc_scratch20);
> -
> -#ifdef CONFIG_TEGRA2_LP0
> - /* save Sdram params to PMC 2, 4, and 24 for WB0 */
> - warmboot_save_sdram_params();
> -#endif
> }
Is that change intended, or was it a rebase mistake?
> diff --git a/arch/arm/cpu/tegra20-common/board.c
> b/arch/arm/cpu/tegra20-common/board.c
> @@ -69,6 +68,11 @@ int dram_init(void)
> {
> /* We do not initialise DRAM here. We just query the size */
> gd->ram_size = query_sdram_size();
> +
> +#ifdef CONFIG_TEGRA2_LP0
> + /* save Sdram params to PMC 2, 4, and 24 for WB0 */
> + warmboot_save_sdram_params();
> +#endif
Hmmm. That's more than just removing the code that's now in the AVP
directory. Separate patch? The patch description also doesn't say why
this change is necessary.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
