Patchwork [U-Boot] mx23: Fix the size of internal RAM

login
register
mail settings
Submitter Fabio Estevam
Date April 26, 2013, 5 p.m.
Message ID <1366995623-26480-1-git-send-email-festevam@gmail.com>
Download mbox | patch
Permalink /patch/239892/
State Changes Requested
Delegated to: Stefano Babic
Headers show

Comments

Fabio Estevam - April 26, 2013, 5 p.m.
From: Fabio Estevam <fabio.estevam@freescale.com>

mx23 has 32kB of internal RAM.

While at it, use IRAM_BASE_ADDR and IRAM_SIZE as the other i.MX SoCs.

Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
 arch/arm/include/asm/arch-mxs/regs-base.h |    3 +++
 include/configs/mx23_olinuxino.h          |    4 ++--
 include/configs/mx23evk.h                 |    4 ++--
 3 files changed, 7 insertions(+), 4 deletions(-)
Otavio Salvador - April 26, 2013, 5:22 p.m.
On Fri, Apr 26, 2013 at 2:00 PM, Fabio Estevam <festevam@gmail.com> wrote:
> From: Fabio Estevam <fabio.estevam@freescale.com>
>
> mx23 has 32kB of internal RAM.
>
> While at it, use IRAM_BASE_ADDR and IRAM_SIZE as the other i.MX SoCs.
>
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---
>  arch/arm/include/asm/arch-mxs/regs-base.h |    3 +++
>  include/configs/mx23_olinuxino.h          |    4 ++--
>  include/configs/mx23evk.h                 |    4 ++--
>  3 files changed, 7 insertions(+), 4 deletions(-)
>
> diff --git a/arch/arm/include/asm/arch-mxs/regs-base.h b/arch/arm/include/asm/arch-mxs/regs-base.h
> index 2d9f96b..855f9d6 100644
> --- a/arch/arm/include/asm/arch-mxs/regs-base.h
> +++ b/arch/arm/include/asm/arch-mxs/regs-base.h
> @@ -28,6 +28,7 @@
>  #ifndef __MXS_REGS_BASE_H__
>  #define __MXS_REGS_BASE_H__
>
> +#include <asm/sizes.h>
>  /*
>   * Register base addresses for i.MX23
>   */
> @@ -69,6 +70,8 @@
>  #define        MXS_USBPHY0_BASE        0x8007C000
>  #define        MXS_USBCTRL0_BASE       0x80080000
>  #define        MXS_DRAM_BASE           0x800E0000
> +#define IRAM_BASE_ADDR         0x00000000
> +#define IRAM_SIZE              SZ_32K

Please fix the indenting to be the same.

--
Otavio Salvador                             O.S. Systems
E-mail: otavio@ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854              http://projetos.ossystems.com.br
Marek Vasut - April 26, 2013, 6:11 p.m.
Dear Fabio Estevam,

> From: Fabio Estevam <fabio.estevam@freescale.com>
> 
> mx23 has 32kB of internal RAM.
> 
> While at it, use IRAM_BASE_ADDR and IRAM_SIZE as the other i.MX SoCs.
> 
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---
>  arch/arm/include/asm/arch-mxs/regs-base.h |    3 +++
>  include/configs/mx23_olinuxino.h          |    4 ++--
>  include/configs/mx23evk.h                 |    4 ++--
>  3 files changed, 7 insertions(+), 4 deletions(-)
> 
> diff --git a/arch/arm/include/asm/arch-mxs/regs-base.h
> b/arch/arm/include/asm/arch-mxs/regs-base.h index 2d9f96b..855f9d6 100644
> --- a/arch/arm/include/asm/arch-mxs/regs-base.h
> +++ b/arch/arm/include/asm/arch-mxs/regs-base.h
> @@ -28,6 +28,7 @@
>  #ifndef __MXS_REGS_BASE_H__
>  #define __MXS_REGS_BASE_H__
> 
> +#include <asm/sizes.h>
>  /*
>   * Register base addresses for i.MX23
>   */
> @@ -69,6 +70,8 @@
>  #define	MXS_USBPHY0_BASE	0x8007C000
>  #define	MXS_USBCTRL0_BASE	0x80080000
>  #define	MXS_DRAM_BASE		0x800E0000
> +#define IRAM_BASE_ADDR		0x00000000
> +#define IRAM_SIZE		SZ_32K

It's called OCRAM in the datasheet. And it's 128K on MX28.

[...]

Best regards,
Marek Vasut

Patch

diff --git a/arch/arm/include/asm/arch-mxs/regs-base.h b/arch/arm/include/asm/arch-mxs/regs-base.h
index 2d9f96b..855f9d6 100644
--- a/arch/arm/include/asm/arch-mxs/regs-base.h
+++ b/arch/arm/include/asm/arch-mxs/regs-base.h
@@ -28,6 +28,7 @@ 
 #ifndef __MXS_REGS_BASE_H__
 #define __MXS_REGS_BASE_H__
 
+#include <asm/sizes.h>
 /*
  * Register base addresses for i.MX23
  */
@@ -69,6 +70,8 @@ 
 #define	MXS_USBPHY0_BASE	0x8007C000
 #define	MXS_USBCTRL0_BASE	0x80080000
 #define	MXS_DRAM_BASE		0x800E0000
+#define IRAM_BASE_ADDR		0x00000000
+#define IRAM_SIZE		SZ_32K
 
 /*
  * Register base addresses for i.MX28
diff --git a/include/configs/mx23_olinuxino.h b/include/configs/mx23_olinuxino.h
index 38ad7f4..7659f52 100644
--- a/include/configs/mx23_olinuxino.h
+++ b/include/configs/mx23_olinuxino.h
@@ -77,8 +77,8 @@ 
 #define CONFIG_SYS_SDRAM_BASE		PHYS_SDRAM_1
 /* Point initial SP in SRAM so SPL can use it too. */
 
-#define CONFIG_SYS_INIT_RAM_ADDR	0x00000000
-#define CONFIG_SYS_INIT_RAM_SIZE	(128 * 1024)
+#define CONFIG_SYS_INIT_RAM_ADDR	IRAM_BASE_ADDR
+#define CONFIG_SYS_INIT_RAM_SIZE	IRAM_SIZE
 
 #define CONFIG_SYS_INIT_SP_OFFSET \
 	(CONFIG_SYS_INIT_RAM_SIZE - GENERATED_GBL_DATA_SIZE)
diff --git a/include/configs/mx23evk.h b/include/configs/mx23evk.h
index e5a15a4..c5264a6 100644
--- a/include/configs/mx23evk.h
+++ b/include/configs/mx23evk.h
@@ -72,8 +72,8 @@ 
 #define CONFIG_SYS_SDRAM_BASE		PHYS_SDRAM_1
 /* Point initial SP in SRAM so SPL can use it too. */
 
-#define CONFIG_SYS_INIT_RAM_ADDR	0x00000000
-#define CONFIG_SYS_INIT_RAM_SIZE	(128 * 1024)
+#define CONFIG_SYS_INIT_RAM_ADDR	IRAM_BASE_ADDR
+#define CONFIG_SYS_INIT_RAM_SIZE	IRAM_SIZE
 
 #define CONFIG_SYS_INIT_SP_OFFSET \
 	(CONFIG_SYS_INIT_RAM_SIZE - GENERATED_GBL_DATA_SIZE)