On Thu, Aug 15, 2024, 23:33 Simon Glass <[email protected]> wrote: > Hi Ilias, > > On Wed, 14 Aug 2024 at 07:45, Ilias Apalodimas > <[email protected]> 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 <[email protected]> > > --- > > 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 > 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. Thanks Ilias > > Regards, > Simon >
