Re: [U-Boot] [PATCH] Tools: set multiple variable with fw_setenv utility

2010-05-21 Thread Wolfgang Denk
Dear Stefano, In message <4bf6b724.6090...@denx.de> you wrote: > > > Adding spaces to the variable value? This makes little sense to me. > > It's just a waste of storage space and boot time. > > Well, I can agree with you, however I have already seen this case... I believe you. I have seen stra

Re: [U-Boot] [PATCH] Tools: set multiple variable with fw_setenv utility

2010-05-21 Thread Stefano Babic
Wolfgang Denk wrote: > Adding spaces to the variable value? This makes little sense to me. > It's just a waste of storage space and boot time. Well, I can agree with you, however I have already seen this case... > If you feel your environment is so complicated to read that you want > such "forma

Re: [U-Boot] [PATCH] Tools: set multiple variable with fw_setenv utility

2010-05-21 Thread Wolfgang Denk
Dear Stefano Babic, In message <4bf6a381.7020...@denx.de> you wrote: > Wolfgang Denk wrote: > > >> a config file with a list of pairs to be set, > >> separated by a TAB character. > > > > I think we should be less restrictive here. Please split at the first > > white space, and allow for multip

Re: [U-Boot] [PATCH] Tools: set multiple variable with fw_setenv utility

2010-05-21 Thread Stefano Babic
Wolfgang Denk wrote: >> a config file with a list of pairs to be set, >> separated by a TAB character. > > I think we should be less restrictive here. Please split at the first > white space, and allow for multiple white spaces. There is a reason why I set to this format. There should be a case

Re: [U-Boot] [PATCH] Tools: set multiple variable with fw_setenv utility

2010-05-21 Thread Wolfgang Denk
Dear Stefano Babic, In message <1274439131-22807-1-git-send-email-sba...@denx.de> you wrote: > Add a sort of batch mode to fw_setenv, allowing to set > multiple variables in one shot, without updating the flash after > each set as now. It is added the possibility to pass Thanks; that's a welcome

[U-Boot] [PATCH] Tools: set multiple variable with fw_setenv utility

2010-05-21 Thread Stefano Babic
Add a sort of batch mode to fw_setenv, allowing to set multiple variables in one shot, without updating the flash after each set as now. It is added the possibility to pass a config file with a list of pairs to be set, separated by a TAB character. Signed-off-by: Stefano Babic --- tools/env/fw_