Patchwork [U-Boot,4/4,V2] SMDK5250: Modify IRAM stack pointer address

login
register
mail settings
Submitter Rajeshwari Birje
Date Jan. 7, 2013, 1:08 p.m.
Message ID <1357564126-13275-5-git-send-email-rajeshwari.s@samsung.com>
Download mbox | patch
Permalink /patch/209915/
State Changes Requested
Delegated to: Minkyu Kang
Headers show

Comments

Rajeshwari Birje - Jan. 7, 2013, 1:08 p.m.
RAM stack pointer address to enable u-boot booting
after conversion of lowlevel_init.S to lowlevel_init_c.c.
Added CONFIG_CMD_BOOTZ to boot zImage.dtb kernel image.
Configs required for early serial initialisation added.

Signed-off-by: Rajeshwari Shinde <rajeshwari.s@samsung.com>
---
Changes in V2:
	- Addded Configs required for early serial initialisation.
 include/configs/exynos5250-dt.h |   11 ++++++++---
 1 files changed, 8 insertions(+), 3 deletions(-)
Simon Glass - Jan. 11, 2013, 5:59 a.m.
Hi,

On Mon, Jan 7, 2013 at 5:08 AM, Rajeshwari Shinde
<rajeshwari.s@samsung.com> wrote:
> RAM stack pointer address to enable u-boot booting
> after conversion of lowlevel_init.S to lowlevel_init_c.c.
> Added CONFIG_CMD_BOOTZ to boot zImage.dtb kernel image.
> Configs required for early serial initialisation added.

Perhaps should separate out the BOOTZ and early serial things into a
separate patch?

>
> Signed-off-by: Rajeshwari Shinde <rajeshwari.s@samsung.com>
> ---
> Changes in V2:
>         - Addded Configs required for early serial initialisation.
>  include/configs/exynos5250-dt.h |   11 ++++++++---
>  1 files changed, 8 insertions(+), 3 deletions(-)
>
> diff --git a/include/configs/exynos5250-dt.h b/include/configs/exynos5250-dt.h
> index 59182f4..66e1d8d 100644
> --- a/include/configs/exynos5250-dt.h
> +++ b/include/configs/exynos5250-dt.h
> @@ -114,6 +114,10 @@
>
>  /* MMC SPL */
>  #define CONFIG_SPL
> +#define CONFIG_SPL_LIBCOMMON_SUPPORT
> +#define CONFIG_SPL_SERIAL_SUPPORT
> +#define CONFIG_SPL_GPIO_SUPPORT
> +
>  #define COPY_BL2_FNPTR_ADDR    0x02020030

BTW, do you still need this one - or do you have function pointers
elsewhere in a table now?

>
>  /* specific .lds file */
> @@ -199,9 +203,9 @@
>
>  #define CONFIG_DOS_PARTITION
>
> -#define CONFIG_IRAM_STACK      0x02050000
> -
> -#define CONFIG_SYS_INIT_SP_ADDR        (CONFIG_SYS_LOAD_ADDR - 0x1000000)
> +#define CONFIG_IRAM_TOP                0x02050000
> +#define CONFIG_SYS_INIT_SP_ADDR        (CONFIG_IRAM_TOP - 0x800)
> +#define CONFIG_IRAM_STACK      CONFIG_SYS_INIT_SP_ADDR
>
>  /* I2C */
>  #define CONFIG_SYS_I2C_INIT_BOARD
> @@ -297,4 +301,5 @@
>  #define CONFIG_SHA1
>  #define CONFIG_SHA256
>
> +#define CONFIG_CMD_BOOTZ
>  #endif /* __CONFIG_H */
> --
> 1.7.4.4
>

Regards,
Simon

Patch

diff --git a/include/configs/exynos5250-dt.h b/include/configs/exynos5250-dt.h
index 59182f4..66e1d8d 100644
--- a/include/configs/exynos5250-dt.h
+++ b/include/configs/exynos5250-dt.h
@@ -114,6 +114,10 @@ 
 
 /* MMC SPL */
 #define CONFIG_SPL
+#define CONFIG_SPL_LIBCOMMON_SUPPORT
+#define CONFIG_SPL_SERIAL_SUPPORT
+#define CONFIG_SPL_GPIO_SUPPORT
+
 #define COPY_BL2_FNPTR_ADDR	0x02020030
 
 /* specific .lds file */
@@ -199,9 +203,9 @@ 
 
 #define CONFIG_DOS_PARTITION
 
-#define CONFIG_IRAM_STACK	0x02050000
-
-#define CONFIG_SYS_INIT_SP_ADDR	(CONFIG_SYS_LOAD_ADDR - 0x1000000)
+#define CONFIG_IRAM_TOP		0x02050000
+#define CONFIG_SYS_INIT_SP_ADDR	(CONFIG_IRAM_TOP - 0x800)
+#define CONFIG_IRAM_STACK	CONFIG_SYS_INIT_SP_ADDR
 
 /* I2C */
 #define CONFIG_SYS_I2C_INIT_BOARD
@@ -297,4 +301,5 @@ 
 #define CONFIG_SHA1
 #define CONFIG_SHA256
 
+#define CONFIG_CMD_BOOTZ
 #endif	/* __CONFIG_H */