Re: [U-Boot] [U-Boot-Users] [PATCH] ads5121: support for running from memory

Could you rebase it against current HEAD?
Sure. Here it is.
From ba2fc48842d6f866e30f52f457a4c834431e1167 Mon Sep 17 00:00:00 2001 From: Nikita V. Youshchenko yoush@debian.org Date: Tue, 2 Sep 2008 07:51:05 +0400 Subject: [PATCH] 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 yoush@debian.org --- board/ads5121/ads5121.c | 5 +++-- include/configs/ads5121.h | 2 +- 2 files changed, 4 insertions(+), 3 deletions(-)
diff --git a/board/ads5121/ads5121.c b/board/ads5121/ads5121.c index 0610928..bb1bf2d 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 *) CONFIG_SYS_IMMR; u32 msize = CONFIG_SYS_DDR_SIZE * 1024 * 1024; +#if CFG_MONITOR_BASE >= CFG_FLASH_BASE + volatile immap_t *im = (immap_t *) CONFIG_SYS_IMMR; u32 msize_log2 = __ilog2 (msize); u32 i;
@@ -203,7 +204,7 @@ long int fixed_sdram (void) /* Start MDDRC */ im->mddrc.ddr_time_config0 = CONFIG_SYS_MDDRC_TIME_CFG0_RUN; im->mddrc.ddr_sys_config = CONFIG_SYS_MDDRC_SYS_CFG_RUN; - +#endif return msize; }
diff --git a/include/configs/ads5121.h b/include/configs/ads5121.h index bb3525f..7a5cdf0 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 (CONFIG_SYS_MONITOR_BASE + CONFIG_SYS_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 */

You missed some changes in the rebase.
All CFG_ macros have been renamed to CONFIG_SYS_
On Fri, Oct 24, 2008 at 3:01 PM, Nikita V. Youshchenko yoush@debian.org wrote:
Could you rebase it against current HEAD?
Sure. Here it is.
From ba2fc48842d6f866e30f52f457a4c834431e1167 Mon Sep 17 00:00:00 2001 From: Nikita V. Youshchenko yoush@debian.org Date: Tue, 2 Sep 2008 07:51:05 +0400 Subject: [PATCH] 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 yoush@debian.org
board/ads5121/ads5121.c | 5 +++-- include/configs/ads5121.h | 2 +- 2 files changed, 4 insertions(+), 3 deletions(-)
diff --git a/board/ads5121/ads5121.c b/board/ads5121/ads5121.c index 0610928..bb1bf2d 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 *) CONFIG_SYS_IMMR; u32 msize = CONFIG_SYS_DDR_SIZE * 1024 * 1024;
+#if CFG_MONITOR_BASE >= CFG_FLASH_BASE
volatile immap_t *im = (immap_t *) CONFIG_SYS_IMMR; u32 msize_log2 = __ilog2 (msize); u32 i;
@@ -203,7 +204,7 @@ long int fixed_sdram (void) /* Start MDDRC */ im->mddrc.ddr_time_config0 = CONFIG_SYS_MDDRC_TIME_CFG0_RUN; im->mddrc.ddr_sys_config = CONFIG_SYS_MDDRC_SYS_CFG_RUN;
+#endif return msize; }
diff --git a/include/configs/ads5121.h b/include/configs/ads5121.h index bb3525f..7a5cdf0 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 (CONFIG_SYS_MONITOR_BASE + CONFIG_SYS_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 */
1.5.6.5
U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

You missed some changes in the rebase.
All CFG_ macros have been renamed to CONFIG_SYS_
This version should be betterю
From ad29d5a0c120e4eee51ecbb8a294c8492cf5fdd0 Mon Sep 17 00:00:00 2001 From: Nikita V. Youshchenko yoush@debian.org Date: Tue, 2 Sep 2008 07:51:05 +0400 Subject: [PATCH] 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 yoush@debian.org --- board/ads5121/ads5121.c | 5 +++-- include/configs/ads5121.h | 2 +- 2 files changed, 4 insertions(+), 3 deletions(-)
diff --git a/board/ads5121/ads5121.c b/board/ads5121/ads5121.c index 0610928..9e4a29c 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 *) CONFIG_SYS_IMMR; u32 msize = CONFIG_SYS_DDR_SIZE * 1024 * 1024; +#if CONFIG_SYS_MONITOR_BASE >= CONFIG_SYS_FLASH_BASE + volatile immap_t *im = (immap_t *) CONFIG_SYS_IMMR; u32 msize_log2 = __ilog2 (msize); u32 i;
@@ -203,7 +204,7 @@ long int fixed_sdram (void) /* Start MDDRC */ im->mddrc.ddr_time_config0 = CONFIG_SYS_MDDRC_TIME_CFG0_RUN; im->mddrc.ddr_sys_config = CONFIG_SYS_MDDRC_SYS_CFG_RUN; - +#endif return msize; }
diff --git a/include/configs/ads5121.h b/include/configs/ads5121.h index bb3525f..7a5cdf0 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 (CONFIG_SYS_MONITOR_BASE + CONFIG_SYS_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 */
participants (2)
-
John Rigby
-
Nikita V. Youshchenko