Skip to contentSkip to conversation

mx6slevk: Avoid hardcoded RAM size

PATCH New Vanessa Maegima 3 participants 7 messages Jun 9, 2016 – Jun 18, 2016

lore
Share & tools
lore

Vanessa Maegima

author
#0 lore raw

PATCH v2 1/3 mx6slevk: Avoid hardcoded RAM size

Instead of passing the total RAM size via PHYS_SDRAM_SIZE option, we should better use imx_ddr_size() function, which automatically determines the RAM size.

Signed-off-by Vanessa Maegima <[email protected]>

---
Changes since v1:
- None

 board/freescale/mx6slevk/mx6slevk.c | 2 +-
 include/configs/mx6slevk.h          | 1 -
 2 files changed, 1 insertion(+), 2 deletions(-)
board/freescale/mx6slevk/mx6slevk.c +1 -1
index 256d602..f978e50 100644
@@ -61,7 +61,7 @@ DECLARE_GLOBAL_DATA_PTR;
 
 int dram_init(void)
 {
-	gd->ram_size = get_ram_size((void *)PHYS_SDRAM, PHYS_SDRAM_SIZE);
+	gd->ram_size = imx_ddr_size();
 
 	return 0;
 }
include/configs/mx6slevk.h +0 -1
index d53e416..375aa0e 100644
@@ -137,7 +137,6 @@
 /* Physical Memory Map */
 #define CONFIG_NR_DRAM_BANKS		1
 #define PHYS_SDRAM			MMDC0_ARB_BASE_ADDR
-#define PHYS_SDRAM_SIZE			SZ_1G
 
 #define CONFIG_SYS_SDRAM_BASE		PHYS_SDRAM
 #define CONFIG_SYS_INIT_RAM_ADDR	IRAM_BASE_ADDR
Signature · 1 line
1.9.1
author
#1 lore raw

PATCH v2 2/3 mx6sxsabreauto: Avoid hardcoded RAM size

Instead of passing the total RAM size via PHYS_SDRAM_SIZE option, we should better use imx_ddr_size() function, which automatically determines the RAM size.

Signed-off-by Vanessa Maegima <[email protected]>

---
Changes since v1:
- remove typo in function name

 board/freescale/mx6sxsabreauto/mx6sxsabreauto.c | 2 +-
 include/configs/mx6sxsabreauto.h                | 1 -
 2 files changed, 1 insertion(+), 2 deletions(-)
board/freescale/mx6sxsabreauto/mx6sxsabreauto.c +1 -1
index 886373c..0ba1508 100644
@@ -106,7 +106,7 @@ static int port_exp_direction_output(unsigned gpio, int value)
 
 int dram_init(void)
 {
-	gd->ram_size = PHYS_SDRAM_SIZE;
+	gd->ram_size = imx_ddr_size();
 
 	return 0;
 }
include/configs/mx6sxsabreauto.h +0 -1
index d771f9d..439579d 100644
@@ -103,7 +103,6 @@
 /* Physical Memory Map */
 #define CONFIG_NR_DRAM_BANKS		1
 #define PHYS_SDRAM			MMDC0_ARB_BASE_ADDR
-#define PHYS_SDRAM_SIZE			SZ_2G
 
 #define CONFIG_SYS_SDRAM_BASE		PHYS_SDRAM
 #define CONFIG_SYS_INIT_RAM_ADDR	IRAM_BASE_ADDR
Signature · 1 line
1.9.1
author
#3 lore raw

PATCH v2 3/3 mx6sxsabresd: Avoid hardcoded RAM size

In reply to Vanessa Maegima

Instead of passing the total RAM size via PHYS_SDRAM_SIZE option, we should better use imx_ddr_size() function, which automatically determines the RAM size.

Signed-off-by Vanessa Maegima <[email protected]>

---
Changes since v1:
- None

 board/freescale/mx6sxsabresd/mx6sxsabresd.c | 2 +-
 include/configs/mx6sxsabresd.h              | 1 -
 2 files changed, 1 insertion(+), 2 deletions(-)
board/freescale/mx6sxsabresd/mx6sxsabresd.c +1 -1
index 25e009e..8d95c51 100644
@@ -64,7 +64,7 @@ DECLARE_GLOBAL_DATA_PTR;
 
 int dram_init(void)
 {
-	gd->ram_size = PHYS_SDRAM_SIZE;
+	gd->ram_size = imx_ddr_size();
 
 	return 0;
 }
include/configs/mx6sxsabresd.h +0 -1
index 183a759..9281fee 100644
@@ -132,7 +132,6 @@
 /* Physical Memory Map */
 #define CONFIG_NR_DRAM_BANKS		1
 #define PHYS_SDRAM			MMDC0_ARB_BASE_ADDR
-#define PHYS_SDRAM_SIZE			SZ_1G
 
 #define CONFIG_SYS_SDRAM_BASE		PHYS_SDRAM
 #define CONFIG_SYS_INIT_RAM_ADDR	IRAM_BASE_ADDR
Signature · 1 line
1.9.1
Applied
#6 lore raw

PATCH v2 1/3 mx6slevk: Avoid hardcoded RAM size

In reply to Vanessa Maegima
Quoted 6 lines · Vanessa Maegima

Instead of passing the total RAM size via PHYS_SDRAM_SIZE option, we should better use imx_ddr_size() function, which automatically determines the RAM size.

Signed-off-by Vanessa Maegima <[email protected]>

---

For the series:

Applied to -next, thanks !

Best regards,
Stefano Babic

Signature · 5 lines
=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================