[U-Boot-Users] Problems with 1.3.3-rc1 on i.MX31ADS
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.
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
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.
- 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" \
In message Pine.LNX.4.64.0804272210370.6246@axis700.grange you wrote:
- 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
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:
- 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" \
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:
- 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" \
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
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
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
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
participants (4)
-
Felix Radensky -
Guennadi Liakhovetski -
Robert Schwebel -
Wolfgang Denk