* [U-Boot] [PATCH v2 1/3] mx6slevk: Avoid hardcoded RAM size
@ 2016-06-09 18:28 Vanessa Maegima
2016-06-09 18:28 ` [U-Boot] [PATCH v2 2/3] mx6sxsabreauto: " Vanessa Maegima
` (3 more replies)
0 siblings, 4 replies; 7+ messages in thread
From: Vanessa Maegima @ 2016-06-09 18:28 UTC (permalink / raw)
To: u-boot
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 <vanessa.maegima@nxp.com>
---
Changes since v1:
- None
board/freescale/mx6slevk/mx6slevk.c | 2 +-
include/configs/mx6slevk.h | 1 -
2 files changed, 1 insertion(+), 2 deletions(-)
diff --git a/board/freescale/mx6slevk/mx6slevk.c b/board/freescale/mx6slevk/mx6slevk.c
index 256d602..f978e50 100644
--- a/board/freescale/mx6slevk/mx6slevk.c
+++ b/board/freescale/mx6slevk/mx6slevk.c
@@ -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;
}
diff --git a/include/configs/mx6slevk.h b/include/configs/mx6slevk.h
index d53e416..375aa0e 100644
--- a/include/configs/mx6slevk.h
+++ b/include/configs/mx6slevk.h
@@ -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
--
1.9.1
^ permalink raw reply related [flat|nested] 7+ messages in thread* [U-Boot] [PATCH v2 2/3] mx6sxsabreauto: Avoid hardcoded RAM size
2016-06-09 18:28 [U-Boot] [PATCH v2 1/3] mx6slevk: Avoid hardcoded RAM size Vanessa Maegima
@ 2016-06-09 18:28 ` Vanessa Maegima
2016-06-09 18:32 ` Fabio Estevam
2016-06-09 18:28 ` [U-Boot] [PATCH v2 3/3] mx6sxsabresd: " Vanessa Maegima
` (2 subsequent siblings)
3 siblings, 1 reply; 7+ messages in thread
From: Vanessa Maegima @ 2016-06-09 18:28 UTC (permalink / raw)
To: u-boot
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 <vanessa.maegima@nxp.com>
---
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(-)
diff --git a/board/freescale/mx6sxsabreauto/mx6sxsabreauto.c b/board/freescale/mx6sxsabreauto/mx6sxsabreauto.c
index 886373c..0ba1508 100644
--- a/board/freescale/mx6sxsabreauto/mx6sxsabreauto.c
+++ b/board/freescale/mx6sxsabreauto/mx6sxsabreauto.c
@@ -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;
}
diff --git a/include/configs/mx6sxsabreauto.h b/include/configs/mx6sxsabreauto.h
index d771f9d..439579d 100644
--- a/include/configs/mx6sxsabreauto.h
+++ b/include/configs/mx6sxsabreauto.h
@@ -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
--
1.9.1
^ permalink raw reply related [flat|nested] 7+ messages in thread* [U-Boot] [PATCH v2 3/3] mx6sxsabresd: Avoid hardcoded RAM size
2016-06-09 18:28 [U-Boot] [PATCH v2 1/3] mx6slevk: Avoid hardcoded RAM size Vanessa Maegima
2016-06-09 18:28 ` [U-Boot] [PATCH v2 2/3] mx6sxsabreauto: " Vanessa Maegima
@ 2016-06-09 18:28 ` Vanessa Maegima
2016-06-09 18:32 ` Fabio Estevam
2016-06-09 18:32 ` [U-Boot] [PATCH v2 1/3] mx6slevk: " Fabio Estevam
2016-06-18 8:55 ` Stefano Babic
3 siblings, 1 reply; 7+ messages in thread
From: Vanessa Maegima @ 2016-06-09 18:28 UTC (permalink / raw)
To: u-boot
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 <vanessa.maegima@nxp.com>
---
Changes since v1:
- None
board/freescale/mx6sxsabresd/mx6sxsabresd.c | 2 +-
include/configs/mx6sxsabresd.h | 1 -
2 files changed, 1 insertion(+), 2 deletions(-)
diff --git a/board/freescale/mx6sxsabresd/mx6sxsabresd.c b/board/freescale/mx6sxsabresd/mx6sxsabresd.c
index 25e009e..8d95c51 100644
--- a/board/freescale/mx6sxsabresd/mx6sxsabresd.c
+++ b/board/freescale/mx6sxsabresd/mx6sxsabresd.c
@@ -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;
}
diff --git a/include/configs/mx6sxsabresd.h b/include/configs/mx6sxsabresd.h
index 183a759..9281fee 100644
--- a/include/configs/mx6sxsabresd.h
+++ b/include/configs/mx6sxsabresd.h
@@ -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
--
1.9.1
^ permalink raw reply related [flat|nested] 7+ messages in thread* [U-Boot] [PATCH v2 1/3] mx6slevk: Avoid hardcoded RAM size
2016-06-09 18:28 [U-Boot] [PATCH v2 1/3] mx6slevk: Avoid hardcoded RAM size Vanessa Maegima
2016-06-09 18:28 ` [U-Boot] [PATCH v2 2/3] mx6sxsabreauto: " Vanessa Maegima
2016-06-09 18:28 ` [U-Boot] [PATCH v2 3/3] mx6sxsabresd: " Vanessa Maegima
@ 2016-06-09 18:32 ` Fabio Estevam
2016-06-18 8:55 ` Stefano Babic
3 siblings, 0 replies; 7+ messages in thread
From: Fabio Estevam @ 2016-06-09 18:32 UTC (permalink / raw)
To: u-boot
On Thu, Jun 9, 2016 at 3:28 PM, Vanessa Maegima <vanessa.maegima@nxp.com> wrote:
> 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 <vanessa.maegima@nxp.com>
Reviewed-by: Fabio Estevam <fabio.estevam@nxp.com>
^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH v2 1/3] mx6slevk: Avoid hardcoded RAM size
2016-06-09 18:28 [U-Boot] [PATCH v2 1/3] mx6slevk: Avoid hardcoded RAM size Vanessa Maegima
` (2 preceding siblings ...)
2016-06-09 18:32 ` [U-Boot] [PATCH v2 1/3] mx6slevk: " Fabio Estevam
@ 2016-06-18 8:55 ` Stefano Babic
3 siblings, 0 replies; 7+ messages in thread
From: Stefano Babic @ 2016-06-18 8:55 UTC (permalink / raw)
To: u-boot
On 09/06/2016 20:28, Vanessa Maegima wrote:
> 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 <vanessa.maegima@nxp.com>
> ---
For the series:
Applied to -next, thanks !
Best regards,
Stefano Babic
--
=====================================================================
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
=====================================================================
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2016-06-18 8:55 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-06-09 18:28 [U-Boot] [PATCH v2 1/3] mx6slevk: Avoid hardcoded RAM size Vanessa Maegima
2016-06-09 18:28 ` [U-Boot] [PATCH v2 2/3] mx6sxsabreauto: " Vanessa Maegima
2016-06-09 18:32 ` Fabio Estevam
2016-06-09 18:28 ` [U-Boot] [PATCH v2 3/3] mx6sxsabresd: " Vanessa Maegima
2016-06-09 18:32 ` Fabio Estevam
2016-06-09 18:32 ` [U-Boot] [PATCH v2 1/3] mx6slevk: " Fabio Estevam
2016-06-18 8:55 ` Stefano Babic
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox