Re: [PATCH v2] cmd: Handle CONFIG_(SPL_|TPL_)ENV_SUPPORT for toggling nvedit object

Dear Nathan,

In message 20200202130227.7755-1-nathan@nathanrossi.com you wrote:

When CONFIG_SPL_ENV_SUPPORT is disabled, nvedit was still being included as part of the build. Use the CONFIG_ENV_SUPPORT, CONFIG_SPL_ENV_SUPPORT or CONFIG_TPL_ENV_SUPPORT values to enable/disable the inclusion of nvedit.

Why should the availability of these commands depend on any of these settings? I think this is wrong - whether someone wants to have environment commands in SPL or TPM is an independent decision. You must not dictate it in such a way for all users.

Thanks.

Best regards,

Wolfgang Denk