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 <[EMAIL PROTECTED]> > > --- > > On Sun, 27 Apr 2008, Wolfgang Denk wrote: > > > In message <[EMAIL PROTECTED]> 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" \ ------------------------------------------------------------------------- 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 [email protected] https://lists.sourceforge.net/lists/listinfo/u-boot-users
