[PATCH V2 1/7] env: Warn on force access if ENV_ACCESS_IGNORE_FORCE set

Marek Vasut
2020-10-23T09:52:45Z

On 10/23/20 10:58 AM, Simon Goldschmidt wrote:
> Am 31.07.2020 um 23:40 schrieb Tom Rini:
>> On Tue, Jul 07, 2020 at 08:51:33PM +0200, Marek Vasut wrote:
>>
>>> If the ENV_ACCESS_IGNORE_FORCE is set, inform user that the variable
>>> cannot be force-set if such attempt happens.
>>>
>>> Signed-off-by: Marek Vasut <marex@denx.de>
>>> Reviewed-by: Tom Rini <trini@konsulko.com>
>>
>> Applied to u-boot/master, thanks!
>>
>
> Sorry I haven't followed this and wasn't able to report this earlier,
> but I think this is wrong: This warning is issued when 0 is returned,
> which means the change is accepted, not rejected.

I think there was a subsequent discussion on this topic in the ML,
[PATCH] Revert "env: Warn on force access if ENV_ACCESS_IGNORE_FORCE
set", I think we reached a conclusion this patch was OK. But if you did
more digging and found a problem, please send a patch / provide details.