Dear AKASHI Takahiro, In message <20190717082525.891-11-takahiro.aka...@linaro.org> you wrote: > Once variable storage attribute is introduced, only NON_VOLATILE or > NON_VOLATILE_AUTOSAVE variables should be saved to backing storage.
This logis is wrong. It should read: Variables which have the "volatile" flag set will be skipped when savong the environment. > In addition, NON_VOLATILE_AUTOSAVE variables are saved immediately > at env_set[_ext](). This is completely independent, and should eventually even be a separate commit: Variables which have the 'autosave" flag set are saved with each (successful) env_set(). Yes, I made up my mind: Adding "volatile" and "autosave" is two independent features, this should be relfected in two separate commits / patch series. Probably even independent from and preceeding the context series. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de Es ist nicht genug zu wissen, man muß auch anwenden; es ist nicht ge- nug zu wollen, man muß auch tun. -- Goethe, Maximen und Reflexionen _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot