Dear AKASHI Takahiro,
In message 20190717082525.891-11-takahiro.akashi@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