Re: [PATCH 7/8] binman_sym: guard with CONFIG_IS_ENABLED(BINMAN_SYMBOLS)

Alper Nebi Yasak
2022-06-04T11:52:36Z

On 03/06/2022 10:17, Peng Fan (OSS) wrote:
> From: Peng Fan <peng.fan@nxp.com>
>
> There is case that CONFIG_BINMAN is defined, but
> CONFIG_SPL_BINMAN_SYMBOLS is not defined. In that case, there will be
> build failure. So use CONFIG_SPL_BINMAN_SYMBOLS to guard the macros, and
> define CONFIG_SPL_BINMAN_SYMBOLS in binman syms test.
>
> Tested-by: Tim Harvey <tharvey@gateworks.com> #imx8m[m,n,p]-venice
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
> ---
> include/binman_sym.h | 2 +-
> tools/binman/test/Makefile | 2 +-
> tools/binman/test/generated/autoconf.h | 3 +++
> tools/binman/test/u_boot_binman_syms.c | 2 +-
> tools/binman/test/u_boot_binman_syms_size.c | 2 +-
> 5 files changed, 7 insertions(+), 4 deletions(-)
> create mode 100644 tools/binman/test/generated/autoconf.h

Reviewed-by: Alper Nebi Yasak <alpernebiyasak@gmail.com>