[PATCH] efi: Select required CMDs from BOOTMETH_EFILOADER

[Jan Kiszka]

On 19.10.25 17:21, Simon Glass wrote:
> Hi Jan,
> 
> On Sun, 19 Oct 2025 at 16:02, Jan Kiszka <jan.kiszka at siemens.com> wrote:
>>
>> On 20.08.25 07:50, Jan Kiszka wrote:
>>> From: Jan Kiszka <jan.kiszka at siemens.com>
>>>
>>> There are needed because distro_efi_read_bootflow_net depend on symbols
>>> provided by those commands when networking is in use. Imply is too weak.
> 
> Reviewed-by: Simon Glass <sjg at chromium.org>
> 
> But I would like a bit more info on why 'imply' doesn't do the trick.
> Is it due to a depends being wrong somewhere else, perhaps?
> 

Honestly, I don't remember anymore. I think I tried, and it just didn't
work when only using "imply".

Jan

>>>
>>> Signed-off-by: Jan Kiszka <jan.kiszka at siemens.com>
>>> ---
>>>
>>> This obsoletes https://patchwork.ozlabs.org/project/uboot/patch/fffc4713-4f89-4a04-acc6-8b295e453226@siemens.com/.
>>>
>>>  boot/Kconfig | 3 ++-
>>>  1 file changed, 2 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/boot/Kconfig b/boot/Kconfig
>>> index 54ef7052c5c..2813a39b102 100644
>>> --- a/boot/Kconfig
>>> +++ b/boot/Kconfig
>>> @@ -605,7 +605,8 @@ config BOOTMETH_EXTLINUX_PXE
>>>  config BOOTMETH_EFILOADER
>>>       bool "Bootdev support for EFI boot"
>>>       depends on EFI_BINARY_EXEC
>>> -     imply CMD_TFTPBOOT if CMD_NET
>>> +     select CMD_TFTPBOOT if CMD_NET
>>> +     select CMD_DHCP if CMD_NET
>>>       default y
>>>       help
>>>         Enables support for EFI boot using bootdevs. This makes the
>>
>> Ping.
> 
> Regards,
> Simon

-- 
Siemens AG, Foundational Technologies
Linux Expert Center