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 eric.jarr...@armadeus.org --- include/configs/apf9328.h | 1034 + 1 files changed, 1034 insertions(+), 0 deletions(-)

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

2011-08-24 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-24 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 free IP

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

2011-08-24 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 issue with

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 d25738a8-8a7a-475a-95f5-8408c6463...@armadeus.org you wrote: Stefano is right. filesize and fileaddr are dynamic variables, thet get created and updated on the fly. It makes no sense to

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 variables. Don't

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 default case, i. e.

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

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)\0\ +

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

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

2011-08-23 Thread Wolfgang Denk
Dear Eric Jarrige, In message d25738a8-8a7a-475a-95f5-8408c6463...@armadeus.org 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

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 eric.jarr...@armadeus.org --- include/configs/apf9328.h | 1034 + 1 files changed, 1034 insertions(+), 0

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 eric.jarr...@armadeus.org --- include/configs/apf9328.h | 1034 + 1 files changed, 1034 insertions(+), 0 deletions(-) create mode 100644 include/configs/apf9328.h Hi Eric,

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

2011-08-10 Thread Eric Jarrige
Signed-off-by: Eric Jarrige eric.jarr...@armadeus.org --- 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