On Saturday 11 August 2012 19:48:24 Mike Frysinger wrote: > On Friday 10 August 2012 13:45:57 Benoît Thébaudeau wrote: > > --- u-boot-4d3c95f.orig/README > > +++ u-boot-4d3c95f/README > > > > If this variable is defined, an environment variable > > named "ver" is created by U-Boot showing the U-Boot > > version as printed by the "version" command. > > - This variable is readonly. > > why don't we fix it to be read-only ? > -mike
I had thought about that, but there is an issue. main_loop() sets this env var, so if ver is made read-only and the env is stored somewhere (NVRAM, etc.), then after an update of U-Boot with a newer version (stored env untouched), ver will still indicate the older version. See commit 155cb01, which forgot to update the README file, which my patch does. Best regards, Benoît _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot