[PATCH] Kconfig: clean up the efi configuration status

[Ilias Apalodimas]

Hi Simon

On mobile apologies for any weird formatting in advance.

On Fri, Aug 16, 2024, 04:34 Simon Glass <sjg at chromium.org> wrote:

> Hi Ilias,
>
> On Thu, 15 Aug 2024 at 15:11, Ilias Apalodimas
> <ilias.apalodimas at linaro.org> wrote:
> >
> >
> >
> > On Thu, Aug 15, 2024, 23:33 Simon Glass <sjg at chromium.org> wrote:
> >>
> >> Hi Ilias,
> >>
> >> On Wed, 14 Aug 2024 at 07:45, Ilias Apalodimas
> >> <ilias.apalodimas at linaro.org> wrote:
> >> >
> >> > The EFI_LOADER and EFI config options are randomly scattered under
> lib/
> >> > making it cumbersome to navigate and enable options, unless you really
> >> > know what you are doing. On top of that the existing options are in
> >> > random order instead of a logical one.
> >> >
> >> > So let's move things around a bit and present two enties for the
> >>
> >> entries
> >>
> >> > EFI_LOADER and EFI options in the main config screen.
> >> > While at it add menus for Capsules, Protocols, and Services in the
> >> > EFI_LOADER so people can find their way around easier
> >> >
> >> > Signed-off-by: Ilias Apalodimas <ilias.apalodimas at linaro.org>
> >> > ---
> >> >  Kconfig                |   4 +
> >> >  lib/Kconfig            |   2 -
> >> >  lib/efi/Kconfig        |   5 +
> >> >  lib/efi_loader/Kconfig | 202
> +++++++++++++++++++++++------------------
> >> >  4 files changed, 123 insertions(+), 90 deletions(-)
> >> >
> >> > diff --git a/Kconfig b/Kconfig
> >> > index 82df59f176ec..62c5441e3576 100644
> >> > --- a/Kconfig
> >> > +++ b/Kconfig
> >> > @@ -756,3 +756,7 @@ source "lib/Kconfig"
> >> >  source "test/Kconfig"
> >> >
> >> >  source "tools/Kconfig"
> >> > +
> >> > +source "lib/efi_loader/Kconfig"
> >> > +
> >>
> >> This should be included from boot/Kconfig as it relates to booting.
> >
> >
> > Hmm why? Booting is one of the things efi does. I prefer having it on
> the main menu since it's a pointer to the entire spec, protocols, services
> etc
>
> True. From a functionality point of view, there is booting and
> firmware update. Is there anything else? I have thought for a while
> that we should perhaps have an 'update' menu. Where, for example,
> would VBE fit in the menus?
>

Why don't we do something similar for VBE? I agree that those are the 2
main categories, but I think it's far more intuitive to have all in one
place depending on what to choose to boot and upgrade your device.

Otherwise you have to enable the 'boot method' and start navigation around
for updates etc.


> >
> >
> >>
> >> The EFI app thing should perhaps be under there too...I'm not sure.
> >
> >
> > I was toying around with the idea as well. But eventually I decided this
> needs it's own entry since it's about building U-Boot as a pe/coff. I don't
> mind changing this though.
>
> Yes, building as an app isn't really about booting. It looks very
> lonely at the end of the menu, though.
>

I'll fold this in to UEFI

Cheers
Ilias

>
> Regards,
> Simon
>
> >
> > Thanks
> > Ilias
> >>
> >>
> >> Regards,
> >> Simon
>