[U-Boot] [PATCH 2/3] env: Don't check CONFIG_ENV_OFFSET_REDUND for SPL build
Tom Rini
trini at konsulko.com
Fri Feb 22 03:33:38 UTC 2019
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.welch at 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20190221/c32d604d/attachment.sig>
More information about the U-Boot
mailing list