Re: [PATCH] cmd: fpga: Fix wrong preprocessor guard for loadb help text

[Alexander Dahl <ada@thorsis.com>]

Hello Michal,

Am Tue, Apr 07, 2026 at 10:07:11AM +0200 schrieb Michal Simek:
> The help text for the "fpga loadb" command was incorrectly guarded by
> CONFIG_CMD_FPGA_LOADP instead of CONFIG_CMD_FPGA_LOADB. This meant the
> loadb help text would only be shown when LOADP was enabled, rather than
> when LOADB was enabled.
> 
> Fix the preprocessor condition to use the correct config option.
> 
> Signed-off-by: Michal Simek <michal.simek@amd.com>
> ---
> 
>  cmd/fpga.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/cmd/fpga.c b/cmd/fpga.c
> index d51c380d7b3e..d347a7002906 100644
> --- a/cmd/fpga.c
> +++ b/cmd/fpga.c
> @@ -417,7 +417,7 @@ U_BOOT_CMD(fpga, 6, 1, do_fpga_wrapper,
>  	 "info   [dev]                  List known device information\n"
>  	 "fpga dump   <dev> <address> <size> Load device to memory buffer\n"
>  	 "fpga load   <dev> <address> <size> Load device from memory buffer\n"
> -#if defined(CONFIG_CMD_FPGA_LOADP)
> +#if defined(CONFIG_CMD_FPGA_LOADB)
>  	 "fpga loadb  <dev> <address> <size> Load device from bitstream buffer\n"
>  #endif
>  #if defined(CONFIG_CMD_FPGA_LOADP)

Fixes: f8f378877b8f ("fpga: add new symbol for fpga_loadb")
Reviewed-by: Alexander Dahl <ada@thorsis.com>

Added original patch author to Cc.

Greets
Alex