[U-Boot-Users] Problems with 1.3.3-rc1 on i.MX31ADS

[U-Boot-Users] Problems with 1.3.3-rc1 on i.MX31ADS

[Felix Radensky]

Hi,

I've installed 1.3.3-rc1 on i.MX31ADS evaluation board
and encountered several problems

1. loadaddr environment variable (as reported by printenv)
   is defined as

	loadaddr=(0x80000000 + 0x800000)

   This results in image loaded at address 0x0, and boot gets stuck.
   Redefining loadaddr as 0x80800000 fixes the problem.

2. dhcp command is undefined, only bootp is available.

Thanks for adding i.mx31 support to u-boot. 
Hopefully i.mx27 support will follow soon.
 
Felix.

[U-Boot-Users] [PATCH] mx31ads: fix loadaddr environment variable define

[Guennadi Liakhovetski]

Arithmetic expressions do not get evaluated under stringification.
Thanks to Felix Radensky for reporting.

Signed-off-by: Guennadi Liakhovetski <lg@denx.de>

---

On Sun, 27 Apr 2008, Felix Radensky wrote:

> I've installed 1.3.3-rc1 on i.MX31ADS evaluation board
> and encountered several problems
> 
> 1. loadaddr environment variable (as reported by printenv)
>    is defined as
> 
> 	loadaddr=(0x80000000 + 0x800000)
> 
>    This results in image loaded at address 0x0, and boot gets stuck.
>    Redefining loadaddr as 0x80800000 fixes the problem.

Thanks for reporting, patch below. Feel free to add your Acked- or 
Tested-by.

> 2. dhcp command is undefined, only bootp is available.

It wasn't needed in the environment the configuration has been programmed 
for. Patches are welcome:-)

Thanks
Guennadi

 include/configs/mx31ads.h |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/include/configs/mx31ads.h b/include/configs/mx31ads.h
index 5286e1f..301afd5 100644
--- a/include/configs/mx31ads.h
+++ b/include/configs/mx31ads.h
@@ -87,7 +87,7 @@
 #define CONFIG_NETMASK		255.255.255.0
 #define CONFIG_IPADDR		192.168.23.168
 #define CONFIG_SERVERIP		192.168.23.2
-#define CONFIG_LOADADDR		(CSD0_BASE + 0x800000)	/* loadaddr env var */
+#define CONFIG_LOADADDR		0x80800000	/* loadaddr env var */
 
 #define	CONFIG_EXTRA_ENV_SETTINGS					\
 	"netdev=eth0\0"							\
-- 
1.5.4


> 
> -------------------------------------------------------------------------
> This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
> Don't miss this year's exciting event. There's still time to save $100. 
> Use priority code J8TL2D2. 
> http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
> _______________________________________________
> U-Boot-Users mailing list
> U-Boot-Users at lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/u-boot-users
> 

---
Guennadi Liakhovetski

[U-Boot-Users] [PATCH] mx31ads: fix loadaddr environment variable define

[Wolfgang Denk]

In message <Pine.LNX.4.64.0804272210370.6246@axis700.grange> you wrote:
>
> > 2. dhcp command is undefined, only bootp is available.
> 
> It wasn't needed in the environment the configuration has been programmed 
> for. Patches are welcome:-)

Please add it. It's default...

> Thanks
> Guennadi
> 
>  include/configs/mx31ads.h |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/include/configs/mx31ads.h b/include/configs/mx31ads.h
> index 5286e1f..301afd5 100644
> --- a/include/configs/mx31ads.h
> +++ b/include/configs/mx31ads.h
> @@ -87,7 +87,7 @@
>  #define CONFIG_NETMASK		255.255.255.0
>  #define CONFIG_IPADDR		192.168.23.168
>  #define CONFIG_SERVERIP		192.168.23.2
> -#define CONFIG_LOADADDR		(CSD0_BASE + 0x800000)	/* loadaddr env var */
> +#define CONFIG_LOADADDR		0x80800000	/* loadaddr env var */

And while we are at it: Please remove all these default network
settings. I don't like these at all.

Thanks.

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
The further the spiritual evolution of  mankind  advances,  the  more
certain  it seems to me that the path to genuine religiosity does not
lie through the fear of life, and the fear of death, and blind faith,
but through striving after rational knowledge.      - Albert Einstein

[U-Boot-Users] [PATCH v2] mx31ads: fix loadaddr environment variable define

[Guennadi Liakhovetski]

Arithmetic expressions do not get evaluated under stringification. Remove 
default network configuration, add DHCP command support. Thanks to Felix 
Radensky for reporting.

Signed-off-by: Guennadi Liakhovetski <lg@denx.de>

---

On Sun, 27 Apr 2008, Wolfgang Denk wrote:

> In message <Pine.LNX.4.64.0804272210370.6246@axis700.grange> you wrote:
> >
> > > 2. dhcp command is undefined, only bootp is available.
> > 
> > It wasn't needed in the environment the configuration has been programmed 
> > for. Patches are welcome:-)
> 
> Please add it. It's default...

Hm, but it isn't in

#include <config_cmd_default.h>

?

> > diff --git a/include/configs/mx31ads.h b/include/configs/mx31ads.h
> > index 5286e1f..301afd5 100644
> > --- a/include/configs/mx31ads.h
> > +++ b/include/configs/mx31ads.h
> > @@ -87,7 +87,7 @@
> >  #define CONFIG_NETMASK		255.255.255.0
> >  #define CONFIG_IPADDR		192.168.23.168
> >  #define CONFIG_SERVERIP		192.168.23.2
> > -#define CONFIG_LOADADDR		(CSD0_BASE + 0x800000)	/* loadaddr env var */
> > +#define CONFIG_LOADADDR		0x80800000	/* loadaddr env var */
> 
> And while we are at it: Please remove all these default network
> settings. I don't like these at all.

Is this one better?

diff --git a/include/configs/mx31ads.h b/include/configs/mx31ads.h
index 5286e1f..7614b95 100644
--- a/include/configs/mx31ads.h
+++ b/include/configs/mx31ads.h
@@ -79,15 +79,13 @@
 #include <config_cmd_default.h>
 
 #define CONFIG_CMD_PING
+#define CONFIG_CMD_DHCP
 #define CONFIG_CMD_SPI
 #define CONFIG_CMD_DATE
 
 #define CONFIG_BOOTDELAY	3
 
-#define CONFIG_NETMASK		255.255.255.0
-#define CONFIG_IPADDR		192.168.23.168
-#define CONFIG_SERVERIP		192.168.23.2
-#define CONFIG_LOADADDR		(CSD0_BASE + 0x800000)	/* loadaddr env var */
+#define CONFIG_LOADADDR		0x80800000	/* loadaddr env var */
 
 #define	CONFIG_EXTRA_ENV_SETTINGS					\
 	"netdev=eth0\0"							\

[U-Boot-Users] [PATCH v2] mx31ads: fix loadaddr environment variable define

[Felix Radensky]

Hi, 

I've verified that both problems I've reported are fixed by this
patch. Thanks a lot !

I'm sorry for not reporting this in the first place, but it
looks like ^C cannot be used to interrupt some blocking operations,
like, e.g. sending BOOTP requests.

Another problem I've noticed is that I had to manually set
ethaddr environment variable in order for DHCP/BOOTP to work.
In Redboot this worked out of the box.

Felix.
 
On Mon, 2008-04-28 at 00:25 +0200, Guennadi Liakhovetski wrote:
> Arithmetic expressions do not get evaluated under stringification. Remove 
> default network configuration, add DHCP command support. Thanks to Felix 
> Radensky for reporting.
> 
> Signed-off-by: Guennadi Liakhovetski <lg@denx.de>
> 
> ---
> 
> On Sun, 27 Apr 2008, Wolfgang Denk wrote:
> 
> > In message <Pine.LNX.4.64.0804272210370.6246@axis700.grange> you wrote:
> > >
> > > > 2. dhcp command is undefined, only bootp is available.
> > > 
> > > It wasn't needed in the environment the configuration has been programmed 
> > > for. Patches are welcome:-)
> > 
> > Please add it. It's default...
> 
> Hm, but it isn't in
> 
> #include <config_cmd_default.h>
> 
> ?
> 
> > > diff --git a/include/configs/mx31ads.h b/include/configs/mx31ads.h
> > > index 5286e1f..301afd5 100644
> > > --- a/include/configs/mx31ads.h
> > > +++ b/include/configs/mx31ads.h
> > > @@ -87,7 +87,7 @@
> > >  #define CONFIG_NETMASK		255.255.255.0
> > >  #define CONFIG_IPADDR		192.168.23.168
> > >  #define CONFIG_SERVERIP		192.168.23.2
> > > -#define CONFIG_LOADADDR		(CSD0_BASE + 0x800000)	/* loadaddr env var */
> > > +#define CONFIG_LOADADDR		0x80800000	/* loadaddr env var */
> > 
> > And while we are at it: Please remove all these default network
> > settings. I don't like these at all.
> 
> Is this one better?
> 
> diff --git a/include/configs/mx31ads.h b/include/configs/mx31ads.h
> index 5286e1f..7614b95 100644
> --- a/include/configs/mx31ads.h
> +++ b/include/configs/mx31ads.h
> @@ -79,15 +79,13 @@
>  #include <config_cmd_default.h>
>  
>  #define CONFIG_CMD_PING
> +#define CONFIG_CMD_DHCP
>  #define CONFIG_CMD_SPI
>  #define CONFIG_CMD_DATE
>  
>  #define CONFIG_BOOTDELAY	3
>  
> -#define CONFIG_NETMASK		255.255.255.0
> -#define CONFIG_IPADDR		192.168.23.168
> -#define CONFIG_SERVERIP		192.168.23.2
> -#define CONFIG_LOADADDR		(CSD0_BASE + 0x800000)	/* loadaddr env var */
> +#define CONFIG_LOADADDR		0x80800000	/* loadaddr env var */
>  
>  #define	CONFIG_EXTRA_ENV_SETTINGS					\
>  	"netdev=eth0\0"							\

[U-Boot-Users] [PATCH v2] mx31ads: fix loadaddr environment variable define

[Wolfgang Denk]

In message <Pine.LNX.4.64.0804280014100.6246@axis700.grange> you wrote:
> Arithmetic expressions do not get evaluated under stringification. Remove 
> default network configuration, add DHCP command support. Thanks to Felix 
> Radensky for reporting.
> 
> Signed-off-by: Guennadi Liakhovetski <lg@denx.de>

Applied, thanks.

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
PROGRAM - n.  A magic spell cast over a computer  allowing it to turn
one's input into error messages.
v. tr. - To engage in a pastime similar to banging one's head against
a wall, but with fewer opportunities for reward.

[U-Boot-Users] Problems with 1.3.3-rc1 on i.MX31ADS

[Robert Schwebel]

On Sun, Apr 27, 2008 at 12:06:04AM +0300, Felix Radensky wrote:
> Thanks for adding i.mx31 support to u-boot. Hopefully i.mx27 support
> will follow soon.

i.MX27 is supported in u-boot-v2.

Robert
-- 
 Dipl.-Ing. Robert Schwebel | http://www.pengutronix.de
 Pengutronix - Linux Solutions for Science and Industry
   Handelsregister:  Amtsgericht Hildesheim, HRA 2686
     Hannoversche Str. 2, 31134 Hildesheim, Germany
   Phone: +49-5121-206917-0 |  Fax: +49-5121-206917-9

[U-Boot-Users] Problems with 1.3.3-rc1 on i.MX31ADS

[Felix Radensky]

Hi, Robert,

Thanks for the info. What are the plans u-boot-v2 ?
Is going to become u-boot mainline in the near future ?
How difficult do you think would be to port i.mx27 code
to u-boot 1.3.

Thanks.

Felix.

On Tue, 2008-04-29 at 22:55 +0200, Robert Schwebel wrote:
> On Sun, Apr 27, 2008 at 12:06:04AM +0300, Felix Radensky wrote:
> > Thanks for adding i.mx31 support to u-boot. Hopefully i.mx27 support
> > will follow soon.
> 
> i.MX27 is supported in u-boot-v2.
> 
> Robert

[U-Boot-Users] Problems with 1.3.3-rc1 on i.MX31ADS

[Robert Schwebel]

On Wed, Apr 30, 2008 at 09:52:01AM +0300, Felix Radensky wrote:
> Thanks for the info. What are the plans u-boot-v2?

World domination, as usual :)

> Is going to become u-boot mainline in the near future?

Nope. It is a technology project which follows a "u-boot, done right"
concept. So over the time, things may inspire each other, but there is
no direct link. v2 tries to combine the good user experience from
u-boot-v1 with a sane internal software design.

> How difficult do you think would be to port i.mx27 code to u-boot 1.3.

Oh, that's not sooo bad. It's just that if there is already a v2 port,
why would you want to do that?

Robert
-- 
 Dipl.-Ing. Robert Schwebel | http://www.pengutronix.de
 Pengutronix - Linux Solutions for Science and Industry
   Handelsregister:  Amtsgericht Hildesheim, HRA 2686
     Hannoversche Str. 2, 31134 Hildesheim, Germany
   Phone: +49-5121-206917-0 |  Fax: +49-5121-206917-9