Re: [PATCH v2 1/2] cmd: env: use ENV_IS_IN_DEVICE in env info

7 Jul 2020


      On Thu, Jul 02, 2020 at 05:43:44PM +0200, Patrick Delaunay wrote:
...
Use the define ENV_IS_IN_DEVICE to test if one the
CONFIG_ENV_IS_IN_...  is defined and correct the detection of
persistent storage support in the command "env info"
if CONFIG_ENV_IS_NOWHERE is activated.
Since commit 60d5ed2593c9 ("env: allow ENV_IS_NOWHERE with
other storage target") test CONFIG_ENV_IS_NOWHERE is not
enough; see also commit 953db29a1e9c6 ("env: enable saveenv
command when one CONFIG_ENV_IS_IN is activated").
This patch avoids issue for this command in stm32mp1 platform.
Signed-off-by: Patrick Delaunay patrick.delaunay@st.com
Reviewed-by: Simon Glass sjg@chromium.org
Reviewed-by: Patrice Chotard patrice.chotard@st.com
Reviewed-by: Tom Rini trini@konsulko.com
-- 
Tom