* [U-Boot] [PATCH 1/5] mx31ads: Provide proper range for memtest
@ 2012-02-12 15:38 Fabio Estevam
2012-02-12 15:38 ` [U-Boot] [PATCH 2/5] mx35pdk: " Fabio Estevam
` (5 more replies)
0 siblings, 6 replies; 15+ messages in thread
From: Fabio Estevam @ 2012-02-12 15:38 UTC (permalink / raw)
To: u-boot
memtest adresses should contain proper DRAM address range.
Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
include/configs/mx31ads.h | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/include/configs/mx31ads.h b/include/configs/mx31ads.h
index 87638a4..fe073ba 100644
--- a/include/configs/mx31ads.h
+++ b/include/configs/mx31ads.h
@@ -146,8 +146,8 @@
#define CONFIG_SYS_MAXARGS 16 /* max number of command args */
#define CONFIG_SYS_BARGSIZE CONFIG_SYS_CBSIZE /* Boot Argument Buffer Size */
-#define CONFIG_SYS_MEMTEST_START 0 /* memtest works on */
-#define CONFIG_SYS_MEMTEST_END 0x10000
+#define CONFIG_SYS_MEMTEST_START (PHYS_SDRAM_1 + PHYS_SDRAM_1_SIZE/2)
+#define CONFIG_SYS_MEMTEST_END (PHYS_SDRAM_1 + PHYS_SDRAM_1_SIZE)
#define CONFIG_SYS_LOAD_ADDR CONFIG_LOADADDR
--
1.7.1
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 2/5] mx35pdk: Provide proper range for memtest
2012-02-12 15:38 [U-Boot] [PATCH 1/5] mx31ads: Provide proper range for memtest Fabio Estevam
@ 2012-02-12 15:38 ` Fabio Estevam
2012-02-13 7:26 ` Stefano Babic
2012-03-07 10:10 ` Stefano Babic
2012-02-12 15:38 ` [U-Boot] [PATCH 3/5] flea3: " Fabio Estevam
` (4 subsequent siblings)
5 siblings, 2 replies; 15+ messages in thread
From: Fabio Estevam @ 2012-02-12 15:38 UTC (permalink / raw)
To: u-boot
From: Fabio Estevam <fabio.estevam@freescale.com>
memtest adresses should contain proper DRAM address range.
Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
include/configs/mx35pdk.h | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/include/configs/mx35pdk.h b/include/configs/mx35pdk.h
index 0c62b9f..f2d4112 100644
--- a/include/configs/mx35pdk.h
+++ b/include/configs/mx35pdk.h
@@ -156,8 +156,8 @@
#define CONFIG_SYS_MAXARGS 16 /* max number of command args */
#define CONFIG_SYS_BARGSIZE CONFIG_SYS_CBSIZE /* Boot Argument Buffer Size */
-#define CONFIG_SYS_MEMTEST_START 0 /* memtest works on */
-#define CONFIG_SYS_MEMTEST_END 0x10000
+#define CONFIG_SYS_MEMTEST_START (PHYS_SDRAM_1 + PHYS_SDRAM_1_SIZE/2)
+#define CONFIG_SYS_MEMTEST_END (PHYS_SDRAM_1 + PHYS_SDRAM_1_SIZE)
#undef CONFIG_SYS_CLKS_IN_HZ /* everything, incl board info, in Hz */
--
1.7.1
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 3/5] flea3: Provide proper range for memtest
2012-02-12 15:38 [U-Boot] [PATCH 1/5] mx31ads: Provide proper range for memtest Fabio Estevam
2012-02-12 15:38 ` [U-Boot] [PATCH 2/5] mx35pdk: " Fabio Estevam
@ 2012-02-12 15:38 ` Fabio Estevam
2012-03-07 10:10 ` Stefano Babic
2012-02-12 15:38 ` [U-Boot] [PATCH 4/5] imx31_litekit: " Fabio Estevam
` (3 subsequent siblings)
5 siblings, 1 reply; 15+ messages in thread
From: Fabio Estevam @ 2012-02-12 15:38 UTC (permalink / raw)
To: u-boot
memtest adresses should contain proper DRAM address range.
Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
include/configs/flea3.h | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/include/configs/flea3.h b/include/configs/flea3.h
index 649e272..748d2b4 100644
--- a/include/configs/flea3.h
+++ b/include/configs/flea3.h
@@ -140,8 +140,8 @@
#define CONFIG_SYS_MAXARGS 16 /* max number of command args */
#define CONFIG_SYS_BARGSIZE CONFIG_SYS_CBSIZE /* Boot Argument Buffer Size */
-#define CONFIG_SYS_MEMTEST_START 0 /* memtest works on */
-#define CONFIG_SYS_MEMTEST_END 0x10000
+#define CONFIG_SYS_MEMTEST_START (PHYS_SDRAM_1 + PHYS_SDRAM_1_SIZE/2)
+#define CONFIG_SYS_MEMTEST_END (PHYS_SDRAM_1 + PHYS_SDRAM_1_SIZE)
#undef CONFIG_SYS_CLKS_IN_HZ /* everything, incl board info, in Hz */
--
1.7.1
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 4/5] imx31_litekit: Provide proper range for memtest
2012-02-12 15:38 [U-Boot] [PATCH 1/5] mx31ads: Provide proper range for memtest Fabio Estevam
2012-02-12 15:38 ` [U-Boot] [PATCH 2/5] mx35pdk: " Fabio Estevam
2012-02-12 15:38 ` [U-Boot] [PATCH 3/5] flea3: " Fabio Estevam
@ 2012-02-12 15:38 ` Fabio Estevam
2012-03-07 10:10 ` Stefano Babic
2012-02-12 15:38 ` [U-Boot] [PATCH 5/5] imx31_phycore: " Fabio Estevam
` (2 subsequent siblings)
5 siblings, 1 reply; 15+ messages in thread
From: Fabio Estevam @ 2012-02-12 15:38 UTC (permalink / raw)
To: u-boot
memtest adresses should contain proper DRAM address range.
Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
include/configs/imx31_litekit.h | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/include/configs/imx31_litekit.h b/include/configs/imx31_litekit.h
index bbcbce1..fcbd4a5 100644
--- a/include/configs/imx31_litekit.h
+++ b/include/configs/imx31_litekit.h
@@ -130,8 +130,8 @@
#define CONFIG_SYS_MAXARGS 16 /* max number of command args */
#define CONFIG_SYS_BARGSIZE CONFIG_SYS_CBSIZE /* Boot Argument Buffer Size */
-#define CONFIG_SYS_MEMTEST_START 0 /* memtest works on */
-#define CONFIG_SYS_MEMTEST_END 0x10000
+#define CONFIG_SYS_MEMTEST_START (PHYS_SDRAM_1 + PHYS_SDRAM_1_SIZE/2)
+#define CONFIG_SYS_MEMTEST_END (PHYS_SDRAM_1 + PHYS_SDRAM_1_SIZE)
#define CONFIG_SYS_LOAD_ADDR 0 /* default load address */
--
1.7.1
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 5/5] imx31_phycore: Provide proper range for memtest
2012-02-12 15:38 [U-Boot] [PATCH 1/5] mx31ads: Provide proper range for memtest Fabio Estevam
` (2 preceding siblings ...)
2012-02-12 15:38 ` [U-Boot] [PATCH 4/5] imx31_litekit: " Fabio Estevam
@ 2012-02-12 15:38 ` Fabio Estevam
2012-03-07 10:11 ` Stefano Babic
2012-02-13 7:12 ` [U-Boot] [PATCH 1/5] mx31ads: " Marek Vasut
2012-03-07 10:11 ` Stefano Babic
5 siblings, 1 reply; 15+ messages in thread
From: Fabio Estevam @ 2012-02-12 15:38 UTC (permalink / raw)
To: u-boot
memtest adresses should contain proper DRAM address range.
Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
include/configs/imx31_phycore.h | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/include/configs/imx31_phycore.h b/include/configs/imx31_phycore.h
index 3153eb5..f0766d4 100644
--- a/include/configs/imx31_phycore.h
+++ b/include/configs/imx31_phycore.h
@@ -134,8 +134,8 @@
/* Boot Argument Buffer Size */
#define CONFIG_SYS_BARGSIZE CONFIG_SYS_CBSIZE
-#define CONFIG_SYS_MEMTEST_START 0 /* memtest works on */
-#define CONFIG_SYS_MEMTEST_END 0x10000
+#define CONFIG_SYS_MEMTEST_START (PHYS_SDRAM_1 + PHYS_SDRAM_1_SIZE/2)
+#define CONFIG_SYS_MEMTEST_END (PHYS_SDRAM_1 + PHYS_SDRAM_1_SIZE)
#define CONFIG_SYS_LOAD_ADDR 0 /* default load address */
--
1.7.1
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 1/5] mx31ads: Provide proper range for memtest
2012-02-12 15:38 [U-Boot] [PATCH 1/5] mx31ads: Provide proper range for memtest Fabio Estevam
` (3 preceding siblings ...)
2012-02-12 15:38 ` [U-Boot] [PATCH 5/5] imx31_phycore: " Fabio Estevam
@ 2012-02-13 7:12 ` Marek Vasut
2012-03-07 10:11 ` Stefano Babic
5 siblings, 0 replies; 15+ messages in thread
From: Marek Vasut @ 2012-02-13 7:12 UTC (permalink / raw)
To: u-boot
> memtest adresses should contain proper DRAM address range.
>
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---
> include/configs/mx31ads.h | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/include/configs/mx31ads.h b/include/configs/mx31ads.h
> index 87638a4..fe073ba 100644
> --- a/include/configs/mx31ads.h
> +++ b/include/configs/mx31ads.h
> @@ -146,8 +146,8 @@
> #define CONFIG_SYS_MAXARGS 16 /* max number of command
args */
> #define CONFIG_SYS_BARGSIZE CONFIG_SYS_CBSIZE /* Boot Argument
Buffer
> Size */
>
> -#define CONFIG_SYS_MEMTEST_START 0 /* memtest works on */
> -#define CONFIG_SYS_MEMTEST_END 0x10000
> +#define CONFIG_SYS_MEMTEST_START (PHYS_SDRAM_1 + PHYS_SDRAM_1_SIZE/2)
> +#define CONFIG_SYS_MEMTEST_END (PHYS_SDRAM_1 +
PHYS_SDRAM_1_SIZE)
Won't this overwrite teh u-boot at the end of the RAM?
>
> #define CONFIG_SYS_LOAD_ADDR CONFIG_LOADADDR
^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 2/5] mx35pdk: Provide proper range for memtest
2012-02-12 15:38 ` [U-Boot] [PATCH 2/5] mx35pdk: " Fabio Estevam
@ 2012-02-13 7:26 ` Stefano Babic
2012-03-07 10:10 ` Stefano Babic
1 sibling, 0 replies; 15+ messages in thread
From: Stefano Babic @ 2012-02-13 7:26 UTC (permalink / raw)
To: u-boot
On 12/02/2012 16:38, Fabio Estevam wrote:
> From: Fabio Estevam <fabio.estevam@freescale.com>
>
> memtest adresses should contain proper DRAM address range.
>
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---
> include/configs/mx35pdk.h | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/include/configs/mx35pdk.h b/include/configs/mx35pdk.h
> index 0c62b9f..f2d4112 100644
> --- a/include/configs/mx35pdk.h
> +++ b/include/configs/mx35pdk.h
> @@ -156,8 +156,8 @@
> #define CONFIG_SYS_MAXARGS 16 /* max number of command args */
> #define CONFIG_SYS_BARGSIZE CONFIG_SYS_CBSIZE /* Boot Argument Buffer Size */
>
> -#define CONFIG_SYS_MEMTEST_START 0 /* memtest works on */
> -#define CONFIG_SYS_MEMTEST_END 0x10000
> +#define CONFIG_SYS_MEMTEST_START (PHYS_SDRAM_1 + PHYS_SDRAM_1_SIZE/2)
> +#define CONFIG_SYS_MEMTEST_END (PHYS_SDRAM_1 + PHYS_SDRAM_1_SIZE)
>
> #undef CONFIG_SYS_CLKS_IN_HZ /* everything, incl board info, in Hz */
>
Personally I do not like that the memory test runs taking automatically
the input parameters - even if the test must run automatically, I think
it is better to put the parameters inside a script else to compile them
with U-Boot.
But if we want to change them, why to put the start address at the
middle of RAM ? The mx35pdk can have two banks of RAM, and the second
one will still not tested. What I mean is that it is quite arbitrary
where to set CONFIG_SYS_MEMTEST_START and CONFIG_SYS_MEMTEST_END, and
with the possibility on some boards to have different RAM configurations
this setup should be not decided at compile time.
And setting the end of test at the end of RAM (but this can open a
debate: really the RAM size is computed at runtime with get_ram_size())
will overwrite U-Boot.
Stefano
--
=====================================================================
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office at denx.de
=====================================================================
^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 2/5] mx35pdk: Provide proper range for memtest
2012-02-12 15:38 ` [U-Boot] [PATCH 2/5] mx35pdk: " Fabio Estevam
2012-02-13 7:26 ` Stefano Babic
@ 2012-03-07 10:10 ` Stefano Babic
2012-03-07 12:38 ` Wolfgang Denk
1 sibling, 1 reply; 15+ messages in thread
From: Stefano Babic @ 2012-03-07 10:10 UTC (permalink / raw)
To: u-boot
On 12/02/2012 16:38, Fabio Estevam wrote:
> From: Fabio Estevam <fabio.estevam@freescale.com>
>
> memtest adresses should contain proper DRAM address range.
>
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---
> include/configs/mx35pdk.h | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/include/configs/mx35pdk.h b/include/configs/mx35pdk.h
> index 0c62b9f..f2d4112 100644
> --- a/include/configs/mx35pdk.h
> +++ b/include/configs/mx35pdk.h
> @@ -156,8 +156,8 @@
> #define CONFIG_SYS_MAXARGS 16 /* max number of command args */
> #define CONFIG_SYS_BARGSIZE CONFIG_SYS_CBSIZE /* Boot Argument Buffer Size */
>
> -#define CONFIG_SYS_MEMTEST_START 0 /* memtest works on */
> -#define CONFIG_SYS_MEMTEST_END 0x10000
> +#define CONFIG_SYS_MEMTEST_START (PHYS_SDRAM_1 + PHYS_SDRAM_1_SIZE/2)
> +#define CONFIG_SYS_MEMTEST_END (PHYS_SDRAM_1 + PHYS_SDRAM_1_SIZE)
>
> #undef CONFIG_SYS_CLKS_IN_HZ /* everything, incl board info, in Hz */
>
Applied to u-boot-imx, thans.
Best regards,
Stefano Babic
--
=====================================================================
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office at denx.de
=====================================================================
^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 3/5] flea3: Provide proper range for memtest
2012-02-12 15:38 ` [U-Boot] [PATCH 3/5] flea3: " Fabio Estevam
@ 2012-03-07 10:10 ` Stefano Babic
2012-03-07 12:39 ` Wolfgang Denk
0 siblings, 1 reply; 15+ messages in thread
From: Stefano Babic @ 2012-03-07 10:10 UTC (permalink / raw)
To: u-boot
On 12/02/2012 16:38, Fabio Estevam wrote:
> memtest adresses should contain proper DRAM address range.
>
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---
> include/configs/flea3.h | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/include/configs/flea3.h b/include/configs/flea3.h
> index 649e272..748d2b4 100644
> --- a/include/configs/flea3.h
> +++ b/include/configs/flea3.h
> @@ -140,8 +140,8 @@
> #define CONFIG_SYS_MAXARGS 16 /* max number of command args */
> #define CONFIG_SYS_BARGSIZE CONFIG_SYS_CBSIZE /* Boot Argument Buffer Size */
>
> -#define CONFIG_SYS_MEMTEST_START 0 /* memtest works on */
> -#define CONFIG_SYS_MEMTEST_END 0x10000
> +#define CONFIG_SYS_MEMTEST_START (PHYS_SDRAM_1 + PHYS_SDRAM_1_SIZE/2)
> +#define CONFIG_SYS_MEMTEST_END (PHYS_SDRAM_1 + PHYS_SDRAM_1_SIZE)
>
> #undef CONFIG_SYS_CLKS_IN_HZ /* everything, incl board info, in Hz */
>
Applied to u-boot-imx, thans.
Best regards,
Stefano Babic
--
=====================================================================
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office at denx.de
=====================================================================
^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 4/5] imx31_litekit: Provide proper range for memtest
2012-02-12 15:38 ` [U-Boot] [PATCH 4/5] imx31_litekit: " Fabio Estevam
@ 2012-03-07 10:10 ` Stefano Babic
0 siblings, 0 replies; 15+ messages in thread
From: Stefano Babic @ 2012-03-07 10:10 UTC (permalink / raw)
To: u-boot
On 12/02/2012 16:38, Fabio Estevam wrote:
> memtest adresses should contain proper DRAM address range.
>
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---
> include/configs/imx31_litekit.h | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/include/configs/imx31_litekit.h b/include/configs/imx31_litekit.h
> index bbcbce1..fcbd4a5 100644
> --- a/include/configs/imx31_litekit.h
> +++ b/include/configs/imx31_litekit.h
> @@ -130,8 +130,8 @@
> #define CONFIG_SYS_MAXARGS 16 /* max number of command args */
> #define CONFIG_SYS_BARGSIZE CONFIG_SYS_CBSIZE /* Boot Argument Buffer Size */
>
> -#define CONFIG_SYS_MEMTEST_START 0 /* memtest works on */
> -#define CONFIG_SYS_MEMTEST_END 0x10000
> +#define CONFIG_SYS_MEMTEST_START (PHYS_SDRAM_1 + PHYS_SDRAM_1_SIZE/2)
> +#define CONFIG_SYS_MEMTEST_END (PHYS_SDRAM_1 + PHYS_SDRAM_1_SIZE)
>
> #define CONFIG_SYS_LOAD_ADDR 0 /* default load address */
>
Applied to u-boot-imx, thans.
Best regards,
Stefano Babic
--
=====================================================================
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office at denx.de
=====================================================================
^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 5/5] imx31_phycore: Provide proper range for memtest
2012-02-12 15:38 ` [U-Boot] [PATCH 5/5] imx31_phycore: " Fabio Estevam
@ 2012-03-07 10:11 ` Stefano Babic
0 siblings, 0 replies; 15+ messages in thread
From: Stefano Babic @ 2012-03-07 10:11 UTC (permalink / raw)
To: u-boot
On 12/02/2012 16:38, Fabio Estevam wrote:
> memtest adresses should contain proper DRAM address range.
>
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---
> include/configs/imx31_phycore.h | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/include/configs/imx31_phycore.h b/include/configs/imx31_phycore.h
> index 3153eb5..f0766d4 100644
> --- a/include/configs/imx31_phycore.h
> +++ b/include/configs/imx31_phycore.h
> @@ -134,8 +134,8 @@
> /* Boot Argument Buffer Size */
> #define CONFIG_SYS_BARGSIZE CONFIG_SYS_CBSIZE
>
> -#define CONFIG_SYS_MEMTEST_START 0 /* memtest works on */
> -#define CONFIG_SYS_MEMTEST_END 0x10000
> +#define CONFIG_SYS_MEMTEST_START (PHYS_SDRAM_1 + PHYS_SDRAM_1_SIZE/2)
> +#define CONFIG_SYS_MEMTEST_END (PHYS_SDRAM_1 + PHYS_SDRAM_1_SIZE)
>
> #define CONFIG_SYS_LOAD_ADDR 0 /* default load address */
>
Applied to u-boot-imx, thans.
Best regards,
Stefano Babic
--
=====================================================================
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office at denx.de
=====================================================================
^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 1/5] mx31ads: Provide proper range for memtest
2012-02-12 15:38 [U-Boot] [PATCH 1/5] mx31ads: Provide proper range for memtest Fabio Estevam
` (4 preceding siblings ...)
2012-02-13 7:12 ` [U-Boot] [PATCH 1/5] mx31ads: " Marek Vasut
@ 2012-03-07 10:11 ` Stefano Babic
5 siblings, 0 replies; 15+ messages in thread
From: Stefano Babic @ 2012-03-07 10:11 UTC (permalink / raw)
To: u-boot
On 12/02/2012 16:38, Fabio Estevam wrote:
> memtest adresses should contain proper DRAM address range.
>
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---
Applied to u-boot-imx, thans.
Best regards,
Stefano Babic
--
=====================================================================
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office at denx.de
=====================================================================
^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 2/5] mx35pdk: Provide proper range for memtest
2012-03-07 10:10 ` Stefano Babic
@ 2012-03-07 12:38 ` Wolfgang Denk
2012-03-07 13:08 ` Stefano Babic
0 siblings, 1 reply; 15+ messages in thread
From: Wolfgang Denk @ 2012-03-07 12:38 UTC (permalink / raw)
To: u-boot
Dear Stefano Babic,
In message <4F5733F8.6030806@denx.de> you wrote:
>
> > +#define CONFIG_SYS_MEMTEST_START (PHYS_SDRAM_1 + PHYS_SDRAM_1_SIZE/2)
> > +#define CONFIG_SYS_MEMTEST_END (PHYS_SDRAM_1 + PHYS_SDRAM_1_SIZE)
> >
> > #undef CONFIG_SYS_CLKS_IN_HZ /* everything, incl board info, in Hz */
> >
>
> Applied to u-boot-imx, thans.
But this looks broken to me.
U-Boot will relocate itself to the end of RAM, and pout malloc arena
and stack below. All this will be overwritten with this setup.
Has anybody actually tested this configuration?
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Always borrow money from a pessimist; they don't expect to be paid
back.
^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 3/5] flea3: Provide proper range for memtest
2012-03-07 10:10 ` Stefano Babic
@ 2012-03-07 12:39 ` Wolfgang Denk
0 siblings, 0 replies; 15+ messages in thread
From: Wolfgang Denk @ 2012-03-07 12:39 UTC (permalink / raw)
To: u-boot
Dear Stefano Babic,
In message <4F57340E.4090901@denx.de> you wrote:
>
> > +#define CONFIG_SYS_MEMTEST_START (PHYS_SDRAM_1 + PHYS_SDRAM_1_SIZE/2)
> > +#define CONFIG_SYS_MEMTEST_END (PHYS_SDRAM_1 + PHYS_SDRAM_1_SIZE)
> >
> > #undef CONFIG_SYS_CLKS_IN_HZ /* everything, incl board info, in Hz */
> >
>
> Applied to u-boot-imx, thans.
Ditto - same for the other "Provide proper range for memtest" patches.
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Business is like a wheelbarrow. Nothing ever happens until you start
pushing.
^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 2/5] mx35pdk: Provide proper range for memtest
2012-03-07 12:38 ` Wolfgang Denk
@ 2012-03-07 13:08 ` Stefano Babic
0 siblings, 0 replies; 15+ messages in thread
From: Stefano Babic @ 2012-03-07 13:08 UTC (permalink / raw)
To: u-boot
On 07/03/2012 13:38, Wolfgang Denk wrote:
> Dear Stefano Babic,
>
> In message <4F5733F8.6030806@denx.de> you wrote:
>>
>>> +#define CONFIG_SYS_MEMTEST_START (PHYS_SDRAM_1 + PHYS_SDRAM_1_SIZE/2)
>>> +#define CONFIG_SYS_MEMTEST_END (PHYS_SDRAM_1 + PHYS_SDRAM_1_SIZE)
>>>
>>> #undef CONFIG_SYS_CLKS_IN_HZ /* everything, incl board info, in Hz */
>>>
>>
>> Applied to u-boot-imx, thans.
>
> But this looks broken to me.
Me too. I have not applied this patch. Rather I sent the "applied"
message to the wrong patchset.
>
> U-Boot will relocate itself to the end of RAM, and pout malloc arena
> and stack below. All this will be overwritten with this setup.
This was stated also in the thread related this patchset. Sorry for the
confusion, I have really applied another patchset by Fabio, as you can
see on u-boot-imx:
mx31pdk: Fix CONFIG_SYS_MEMTEST_END
efikamx: Fix CONFIG_SYS_MEMTEST_END
mx53smd: Fix CONFIG_SYS_MEMTEST_END
mx53evk: Fix CONFIG_SYS_MEMTEST_END
mx51evk: Fix CONFIG_SYS_MEMTEST_END
In patchworks things are ok, only my e-mail was wrong (reply to wrong
e-mail).
Best regards,
Stefano Babic
--
=====================================================================
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office at denx.de
=====================================================================
^ permalink raw reply [flat|nested] 15+ messages in thread
end of thread, other threads:[~2012-03-07 13:08 UTC | newest]
Thread overview: 15+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-02-12 15:38 [U-Boot] [PATCH 1/5] mx31ads: Provide proper range for memtest Fabio Estevam
2012-02-12 15:38 ` [U-Boot] [PATCH 2/5] mx35pdk: " Fabio Estevam
2012-02-13 7:26 ` Stefano Babic
2012-03-07 10:10 ` Stefano Babic
2012-03-07 12:38 ` Wolfgang Denk
2012-03-07 13:08 ` Stefano Babic
2012-02-12 15:38 ` [U-Boot] [PATCH 3/5] flea3: " Fabio Estevam
2012-03-07 10:10 ` Stefano Babic
2012-03-07 12:39 ` Wolfgang Denk
2012-02-12 15:38 ` [U-Boot] [PATCH 4/5] imx31_litekit: " Fabio Estevam
2012-03-07 10:10 ` Stefano Babic
2012-02-12 15:38 ` [U-Boot] [PATCH 5/5] imx31_phycore: " Fabio Estevam
2012-03-07 10:11 ` Stefano Babic
2012-02-13 7:12 ` [U-Boot] [PATCH 1/5] mx31ads: " Marek Vasut
2012-03-07 10:11 ` Stefano Babic
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox