[PATCH] lib/Kconfig: Fix SUPPORTS_FW_LOADER option

[Weijie Gao]

On Wed, 2026-04-15 at 16:37 -0500, David Lechner wrote:
> Change the SUPPORTS_FW_LOADER option to be enabled by default.
> 
> This is a dependency-only option intended to be used like:
>     depends on SUPPORTS_FW_LOADER
> 
> Instead of having to always remember to do both:
>     depends on CMDLINE
>     depends on ENV_SUPPORT
> 
> In order to actually work though, the option has to be enabled.
> 
> Reported-by: Weijie Gao <weijie.gao at mediatek.com>
> Closes: 
> https://lore.kernel.org/u-boot/20260410015311.4066075-1-weijie.gao@mediatek.com/
>  
> Fixes: 4ed440e6be80 ("fw_loader: Introduce SUPPORTS_FW_LOADER
> symbol")
> Signed-off-by: David Lechner <dlechner at baylibre.com>
> ---
> I tested this a bit by fiddling with a local .config. It had
> CONFIG_SUPPORTS_FW_LOADER=y already because both dependencies
> were met. Then I manually set CONFIG_CMDLINE=n which caused
> CONFIG_SUPPORTS_FW_LOADER to disappear from the .config.
> ---
>  lib/Kconfig | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/lib/Kconfig b/lib/Kconfig
> index 4e6a0c6a1b6..0950b80ede5 100644
> --- a/lib/Kconfig
> +++ b/lib/Kconfig
> @@ -72,7 +72,7 @@ config DYNAMIC_CRC_TABLE
>  	  This can be helpful when reducing the size of the build image
>  
>  config SUPPORTS_FW_LOADER
> -	bool
> +	def_bool y
>  	depends on CMDLINE
>  	depends on ENV_SUPPORT
>  

Tested-by: Weijie Gao <weijie.gao at mediatek.com>

Thanks
> 
> ---
> base-commit: 70fd0c3bb7c26d42f24d10145dd5f3168ac92eac
> change-id: 20260415-lib-fix-supports_fw_loader-af28afa23009
> 
> Best regards,
> --  
> David Lechner <dlechner at baylibre.com>
>