On Thu, Feb 21, 2019 at 10:13:42AM +0000, Martyn Welch wrote: > Currently CONFIG_ENV_OFFSET_REDUND is checked regardless of the type of > build being performed, but this doesn't seem to be needed in SPL builds. > > Don't check this configuration option for SPL builds. > > Signed-off-by: Martyn Welch <martyn.we...@collabora.com> > --- > > env/nand.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/env/nand.c b/env/nand.c > index 29eda66fad..d0b95f483d 100644 > --- a/env/nand.c > +++ b/env/nand.c > @@ -26,7 +26,7 @@ > #if defined(CONFIG_CMD_SAVEENV) && defined(CONFIG_CMD_NAND) && \ > !defined(CONFIG_SPL_BUILD) > #define CMD_SAVEENV > -#elif defined(CONFIG_ENV_OFFSET_REDUND) > +#elif defined(CONFIG_ENV_OFFSET_REDUND) && !defined(CONFIG_SPL_BUILD) > #error CONFIG_ENV_OFFSET_REDUND must have CONFIG_CMD_SAVEENV & > CONFIG_CMD_NAND > #endif
I'm confused. If we have redundant env, and we have env in nand, we need to know. That said, I guess this is just a sanity check for build time, and until we have ENV_OFFSET_REDUND (and others) move to Kconfig we can't also delete those #error lines. Am I at least right about where/how you hit this problem? Thanks! -- Tom
signature.asc
Description: PGP signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot