Re: [U-Boot] [PATCH 5/9] apf9328: add default board configuration file

2011-10-06 Thread Wolfgang Denk
Dear Eric Jarrige, In message <20110810203325.21204.6037.st...@shuttle2.etheralp.ch> you wrote: > Signed-off-by: Eric Jarrige > --- > include/configs/apf9328.h | 1034 > + > 1 files changed, 1034 insertions(+), 0 deletions(-) > create mode 100644 inc

Re: [U-Boot] [PATCH 5/9] apf9328: add default board configuration file

2011-08-24 Thread Eric Jarrige
Hi Stefano, On 24 août 2011, at 08:22, Stefano Babic wrote: > On 08/24/2011 06:56 AM, Eric Jarrige wrote: >> Dear Wolfgang, Dear Stefano, > > Hi Eric, > We really need a set of default IP addresses for private network to simplify as much as possible the life of the armadeus project d

Re: [U-Boot] [PATCH 5/9] apf9328: add default board configuration file

2011-08-24 Thread Eric Jarrige
Dear Wolfgang, On 24 août 2011, at 08:34, Wolfgang Denk wrote: > Dear Eric, > > In message <20110824054915.0860711f9...@gemini.denx.de> I wrote: >> >>> Sure, DHCP is the best *technical* solution. >>> I meant for static IP address? >> >> Why does anybody need static IP addresses for the defaul

Re: [U-Boot] [PATCH 5/9] apf9328: add default board configuration file

2011-08-24 Thread Wolfgang Denk
Dear Eric Jarrige, In message <8f000deb-1afc-403b-9b00-fa2bff2e1...@armadeus.org> you wrote: > > > Which new interface do you mean? And why would waiting for it help > > you? > > By interface I mean the changes you mentioned to avoid to have "filesize" > and "fileaddr" with the environment varia

Re: [U-Boot] [PATCH 5/9] apf9328: add default board configuration file

2011-08-24 Thread Eric Jarrige
Dear Wolfgang, On 24 août 2011, at 07:49, Wolfgang Denk wrote: > Dear Eric Jarrige, > > In message you wrote: >> >>> Stefano is right. "filesize" and "fileaddr" are dynamic variables, >>> thet get created and updated on the fly. It makes no sense to >>> pre-define them to any specific value.

Re: [U-Boot] [PATCH 5/9] apf9328: add default board configuration file

2011-08-23 Thread Wolfgang Denk
Dear Eric, In message <20110824054915.0860711f9...@gemini.denx.de> I wrote: > > > Sure, DHCP is the best *technical* solution. > > I meant for static IP address? > > Why does anybody need static IP addresses for the default case, i. e. > for system bring up? And if you need one, what is the iss

Re: [U-Boot] [PATCH 5/9] apf9328: add default board configuration file

2011-08-23 Thread Stefano Babic
On 08/24/2011 06:56 AM, Eric Jarrige wrote: > Dear Wolfgang, Dear Stefano, Hi Eric, >>> We really need a set of default IP addresses for private network to >>> simplify as much as possible the life of the armadeus project developers. >> >> And what makes you think that 192.168.0.10 might be a f

Re: [U-Boot] [PATCH 5/9] apf9328: add default board configuration file

2011-08-23 Thread Eric Jarrige
Dear Wolfgang, Dear Stefano, On 23 août 2011, at 13:26, Wolfgang Denk wrote: > Dear Eric Jarrige, > > can you please mind your line length? It is strongly recommended that > text lines should not exceed 70 characters or so. Thanks. I will take care of that. I do apologize for the inconvenient

Re: [U-Boot] [PATCH 5/9] apf9328: add default board configuration file

2011-08-23 Thread Wolfgang Denk
Dear Eric Jarrige, In message you wrote: > > > Stefano is right. "filesize" and "fileaddr" are dynamic variables, > > thet get created and updated on the fly. It makes no sense to > > pre-define them to any specific value. [Actually I have some changes > > in mind that avoid to save such variab

Re: [U-Boot] [PATCH 5/9] apf9328: add default board configuration file

2011-08-23 Thread Eric Jarrige
Hi Stefano, On 23 août 2011, at 11:46, Stefano Babic wrote: > > > sorry for my late answer. I have missed a point. No problem on my side - my answers are sometime even longer. > >>> Please drop fix ip address. They should not be part of mainline, >> >> Here, I have a problem as our document

Re: [U-Boot] [PATCH 5/9] apf9328: add default board configuration file

2011-08-23 Thread Wolfgang Denk
Dear Eric Jarrige, can you please mind your line length? It is strongly recommended that text lines should not exceed 70 characters or so. Thanks. In message <1fad4112-2a57-415d-9cdf-8fc7a9edb...@armadeus.org> you wrote: > >> +#define CONFIG_EXTRA_ENV_SETTINGS \ > >> + "env_version="

Re: [U-Boot] [PATCH 5/9] apf9328: add default board configuration file

2011-08-23 Thread Stefano Babic
On 08/15/2011 10:25 PM, Eric Jarrige wrote: > Hi Stefano, Hi Eric, sorry for my late answer. I have missed a point. >>> +#define CONFIG_EXTRA_ENV_SETTINGS \ >>> + "env_version=" CONFIG_ENV_VERSION "\0"\ >>> + "fileaddr=" MK_STR(CONFIG_SYS_LOAD_ADDR)"

Re: [U-Boot] [PATCH 5/9] apf9328: add default board configuration file

2011-08-15 Thread Eric Jarrige
Hi Stefano, On 11 août 2011, at 11:21, Stefano Babic wrote: > On 08/10/2011 10:33 PM, Eric Jarrige wrote: >> Signed-off-by: Eric Jarrige >> --- >> include/configs/apf9328.h | 1034 >> + >> 1 files changed, 1034 insertions(+), 0 deletions(-) >> create mo

Re: [U-Boot] [PATCH 5/9] apf9328: add default board configuration file

2011-08-11 Thread Stefano Babic
On 08/10/2011 10:33 PM, Eric Jarrige wrote: > Signed-off-by: Eric Jarrige > --- > include/configs/apf9328.h | 1034 > + > 1 files changed, 1034 insertions(+), 0 deletions(-) > create mode 100644 include/configs/apf9328.h Hi Eric, > + > +#ifndef __CO

[U-Boot] [PATCH 5/9] apf9328: add default board configuration file

2011-08-10 Thread Eric Jarrige
Signed-off-by: Eric Jarrige --- include/configs/apf9328.h | 1034 + 1 files changed, 1034 insertions(+), 0 deletions(-) create mode 100644 include/configs/apf9328.h diff --git a/include/configs/apf9328.h b/include/configs/apf9328.h new file mode 10064