Hi Stefano,
On Sun, 2010-05-23 at 16:29 +0200, Stefano Babic 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 a config file with a list of pairs <variable, value> to be set, separated by a TAB character.
It'd be nice to document the new functionality you're adding. Any interest in adding a runtime usage message? It seems like that'd be pretty useful. At a minimum it would be nice to update the description in fw_env_main.c to reflect the changes you're making and/or update the README.
-int fw_setenv (int argc, char *argv[]) +int fw_env_open(void) {
- int i, len;
- char *env, *nxt;
- char *oldval = NULL;
- char *name;
- return env_init();
+}
Is there a reason to keep fw_env_open around? It looks like just a call to env_init() after the changes above?
Best, Peter