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

Reply via email to