[U-Boot] [PATCH 1/2] MX31: mx31pdk: fix nand_spl

[U-Boot] [PATCH 1/2] MX31: mx31pdk: fix nand_spl

[Stefano Babic]

Commit f326cbba98bae21d41df8daac0bd78121d557af1 breaks mx31pdk,
as the _end section in u-boot.lds is missing for the nand_spl
production.

Signed-off-by: Stefano Babic <sbabic@denx.de>
CC: Fabio Estevam <fabio.estevam@freescale.com>
---
 nand_spl/board/freescale/mx31pdk/u-boot.lds |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/nand_spl/board/freescale/mx31pdk/u-boot.lds b/nand_spl/board/freescale/mx31pdk/u-boot.lds
index 324a932..d2b08f6 100644
--- a/nand_spl/board/freescale/mx31pdk/u-boot.lds
+++ b/nand_spl/board/freescale/mx31pdk/u-boot.lds
@@ -63,6 +63,8 @@ SECTIONS
 		*(.dynsym)
 	}
 
+	_end = .;
+
 	.bss __rel_dyn_start (OVERLAY) : {
 		__bss_start = .;
 		*(.bss)
-- 
1.7.1

[U-Boot] [PATCH 1/2] MX31: mx31pdk: fix nand_spl

[Albert ARIBAUD]

Hi Stefano,

Le 03/04/2011 12:01, Stefano Babic a ?crit :
> Commit f326cbba98bae21d41df8daac0bd78121d557af1 breaks mx31pdk,
> as the _end section in u-boot.lds is missing for the nand_spl
> production.

What command line should one use to reproduce the issue? I just did a 
./MAKEALL mx31pdk from the current u-boot-arm master branch and it 
builds fine, which I suspect sould not be possible if it missed the _end 
symbol.

Amicalement,
-- 
Albert.

[U-Boot] [PATCH 1/2] MX31: mx31pdk: fix nand_spl

[Stefano Babic]

On 04/03/2011 02:08 PM, Albert ARIBAUD wrote:
> Hi Stefano,
>

Hi Albert,

> Le 03/04/2011 12:01, Stefano Babic a ?crit :
>> Commit f326cbba98bae21d41df8daac0bd78121d557af1 breaks mx31pdk,
>> as the _end section in u-boot.lds is missing for the nand_spl
>> production.
> 
> What command line should one use to reproduce the issue?

you should issue mx31pdk_nand. I do not know why there is a special
target in the Makefile, I do not think this is necessary and only
boards.cfg could be used. But this has nothing to do with the reported
error, answering your question I have seen something else strange ;-)

> I just did a 
> ./MAKEALL mx31pdk from the current u-boot-arm master branch and it 
> builds fine, which I suspect sould not be possible if it missed the _end 
> symbol.

The mx31pdk supports booting from NOR or from NAND, using two different
configuration. You compiled for NOR, and everything works fine - only
the nand_spl part has this issue.

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
=====================================================================

[U-Boot] [PATCH 1/2] MX31: mx31pdk: fix nand_spl

[Fabio Estevam]

Hi Albert,

--- On Sun, 4/3/11, Albert ARIBAUD <albert.aribaud@free.fr> wrote:

> From: Albert ARIBAUD <albert.aribaud@free.fr>
> Subject: Re: [U-Boot] [PATCH 1/2] MX31: mx31pdk: fix nand_spl
> To: u-boot at lists.denx.de
> Date: Sunday, April 3, 2011, 9:08 AM
> Hi Stefano,
> 
> Le 03/04/2011 12:01, Stefano Babic a ?crit :
> > Commit f326cbba98bae21d41df8daac0bd78121d557af1 breaks
> mx31pdk,
> > as the _end section in u-boot.lds is missing for the
> nand_spl
> > production.
> 
> What command line should one use to reproduce the issue? I

Please do a ?make mx31pdk_nand_config?

Regards,

Fabio Estevam

[U-Boot] [PATCH 1/2] MX31: mx31pdk: fix nand_spl

[Fabio Estevam]

Hi Stefano,

On 4/3/2011 7:01 AM, Stefano Babic wrote:
> Commit f326cbba98bae21d41df8daac0bd78121d557af1 breaks mx31pdk,
> as the _end section in u-boot.lds is missing for the nand_spl
> production.

After applying your patch I was able to build and boot MX31PDK from NAND,
so here goes my:
 
> Signed-off-by: Stefano Babic <sbabic@denx.de>
Tested-by: Fabio Estevam <fabio.estevam@freescale.com>

Regards,

Fabio Estevam

[U-Boot] [PATCH 1/2] MX31: mx31pdk: fix nand_spl

[Albert ARIBAUD]

Le 04/04/2011 00:04, Fabio Estevam a ?crit :
> Hi Stefano,
>
> On 4/3/2011 7:01 AM, Stefano Babic wrote:
>> Commit f326cbba98bae21d41df8daac0bd78121d557af1 breaks mx31pdk,
>> as the _end section in u-boot.lds is missing for the nand_spl
>> production.
>
> After applying your patch I was able to build and boot MX31PDK from NAND,
> so here goes my:
>
>> Signed-off-by: Stefano Babic<sbabic@denx.de>
> Tested-by: Fabio Estevam<fabio.estevam@freescale.com>
>
> Regards,
>
> Fabio Estevam

Applied to u-boot-arm/master, thanks!

Amicalement,
-- 
Albert.