Problems with 1.3.3-rc1 on i.MX31ADS

U-Boot-Users Superseded Felix Radensky 4 participants 9 messages Apr 26, 2008 – Apr 30, 2008

lore

Share & tools

lore

Felix Radensky

author

2008-04-26 21:06 UTC #0 lore raw

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.

Guennadi Liakhovetski

2008-04-27 20:14 UTC #1 lore raw

PATCH 1/1 mx31ads: fix loadaddr environment variable define

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

Signed-off-by Guennadi Liakhovetski <[email protected]>

---

Quoted 10 lines · Felix Radensky

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.

Quoted 1 line

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(-)

include/configs/mx31ads.h +1 -1

index 5286e1f..301afd5 100644
@@ -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"							\

Signature · 1 line

1.5.4

Quoted 11 lines

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

Wolfgang Denk

2008-04-27 21:44 UTC #2 lore raw

PATCH 1/1 mx31ads: fix loadaddr environment variable define

Quoted 5 lines · In message <[email protected]> you wrote:

Quoted 2 lines
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...

Quoted 11 lines

Thanks
Guennadi

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

include/configs/mx31ads.h +0 -0

index 5286e1f..301afd5 100644

include/configs/mx31ads.h line 90

Quoted 5 lines

include/configs/mx31ads.h +1 -1

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

Signature · 7 lines

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

Guennadi Liakhovetski

2008-04-27 22:25 UTC #3 lore raw

PATCH v2 1/1 mx31ads: fix loadaddr environment variable define

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 <[email protected]>

---

Quoted 8 lines · Wolfgang Denk

Quoted 5 lines · In message <[email protected]> you wrote:
Quoted 2 lines
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>

Quoted 5 lines

Quoted 5 lines
include/configs/mx31ads.h +0 -0
index 5286e1f..301afd5 100644
@@ -87,7 +87,7 @@

Quoted 8 lines

include/configs/mx31ads.h line 90
Quoted 5 lines
include/configs/mx31ads.h +1 -1
 #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?

include/configs/mx31ads.h +2 -4

index 5286e1f..7614b95 100644
@@ -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"							\

Felix Radensky

author

2008-04-28 08:10 UTC #4 lore raw

PATCH v2 1/1 mx31ads: fix loadaddr environment variable define

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.

Quoted 63 lines · 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 <[email protected]>
---
Quoted 8 lines · Wolfgang Denk
Quoted 5 lines · In message <[email protected]> you wrote:
Quoted 2 lines
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>
?
Quoted 5 lines
Quoted 5 lines
include/configs/mx31ads.h +0 -0
index 5286e1f..301afd5 100644
@@ -87,7 +87,7 @@
Quoted 8 lines
include/configs/mx31ads.h line 90
Quoted 5 lines
include/configs/mx31ads.h +1 -1
 #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?
include/configs/mx31ads.h +2 -4
index 5286e1f..7614b95 100644
@@ -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"							\

Wolfgang Denk

Applied

2008-04-28 09:10 UTC #5 lore raw

PATCH v2 1/1 mx31ads: fix loadaddr environment variable define

In reply to Guennadi Liakhovetski

Quoted 5 lines · In message <[email protected]> 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 <[email protected]>

Applied, thanks.

Best regards,

Wolfgang Denk

Signature · 7 lines

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.

Robert Schwebel

2008-04-29 20:55 UTC #6 lore raw

In reply to Felix Radensky

Quoted 2 lines · Felix Radensky

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

Signature · 5 lines

 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

Felix Radensky

author

2008-04-30 06:52 UTC #7 lore raw

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.

Quoted 7 lines · Robert Schwebel

Quoted 2 lines · Felix Radensky
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

Robert Schwebel

2008-04-30 20:55 UTC #8 lore raw

Quoted 1 line · Felix Radensky

Thanks for the info. What are the plans u-boot-v2?

World domination, as usual :)

Quoted 1 line

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.

Quoted 1 line

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

Signature · 5 lines

 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