This version fixes pointed issues. Is it better?

commit 03f4cf26950e498d83c21fd040977db9633d77e2
Author: Nikita V. Youshchenko <[EMAIL PROTECTED]>
Date:   Tue Sep 2 07:51:05 2008 +0400

    ads5121: support for running from RAM
    
    This patch makes it possible to start u-boot from RAM on ads5121 board.
    
    It does two simple things:
    - skips initialization of RAM controller if already running from RAM,
    - makes CONFIG_ENV_ADDR absolute, not relative to CFG_MONITOR_BASE
    
    Signed-off-by: Nikita V. Youshchenko <[EMAIL PROTECTED]>

diff --git a/board/ads5121/ads5121.c b/board/ads5121/ads5121.c
index deaa292..644d85e 100644
--- a/board/ads5121/ads5121.c
+++ b/board/ads5121/ads5121.c
@@ -120,8 +120,9 @@ phys_size_t initdram (int board_type)
  */
 long int fixed_sdram (void)
 {
-       volatile immap_t *im = (immap_t *) CFG_IMMR;
        u32 msize = CFG_DDR_SIZE * 1024 * 1024;
+#if CFG_MONITOR_BASE >= CFG_FLASH_BASE
+       volatile immap_t *im = (immap_t *) CFG_IMMR;
        u32 msize_log2 = __ilog2 (msize);
        u32 i;
 
@@ -203,7 +204,7 @@ long int fixed_sdram (void)
        /* Start MDDRC */
        im->mddrc.ddr_time_config0 = CFG_MDDRC_TIME_CFG0_RUN;
        im->mddrc.ddr_sys_config = CFG_MDDRC_SYS_CFG_RUN;
-
+#endif
        return msize;
 }
 
diff --git a/include/configs/ads5121.h b/include/configs/ads5121.h
index d6f7e02..4086ce9 100644
--- a/include/configs/ads5121.h
+++ b/include/configs/ads5121.h
@@ -322,7 +322,7 @@
  */
 #define CONFIG_ENV_IS_IN_FLASH 1
 /* This has to be a multiple of the Flash sector size */
-#define CONFIG_ENV_ADDR                (CFG_MONITOR_BASE + CFG_MONITOR_LEN)
+#define CONFIG_ENV_ADDR                0xfff40000
 #define CONFIG_ENV_SIZE                0x2000
 #ifdef CONFIG_BKUP_FLASH
 #define CONFIG_ENV_SECT_SIZE   0x20000 /* one sector (256K) for env */

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to