[PATCH] efi: Select required CMDs from BOOTMETH_EFILOADER

[Simon Glass]

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?

> >
> > 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