All of lore.kernel.org
 help / color / mirror / Atom feed
From: Simon Glass <sjg@chromium.org>
To: U-Boot Mailing List <u-boot@lists.denx.de>
Cc: "Ilias Apalodimas" <ilias.apalodimas@linaro.org>,
	"Pali Rohár" <pali@kernel.org>,
	"Mark Kettenis" <mark.kettenis@xs4all.nl>,
	"Heinrich Schuchardt" <xypron.glpk@gmx.de>,
	"Simon Glass" <sjg@chromium.org>
Subject: [PATCH v2 7/9] Make EFI_LOADER depend on DM and OF_CONTROL
Date: Fri,  2 Jul 2021 12:36:18 -0600	[thread overview]
Message-ID: <20210702183620.1542680-4-sjg@chromium.org> (raw)
In-Reply-To: <20210702183620.1542680-1-sjg@chromium.org>

This feature should never have been made available when driver model
or devicetree are disabled. Add these as conditions, so that we don't
create even more barriers to migration.

Add a note about the substantial size increment associated with this
option.

Signed-off-by: Simon Glass <sjg@chromium.org>
---

Changes in v2:
- Split out new patch to make EFI_LOADER depend on DM and OF_CONTROL
- Note the approximate size of this feature in the help

 lib/efi_loader/Kconfig | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/lib/efi_loader/Kconfig b/lib/efi_loader/Kconfig
index 6242caceb7f..466abfed300 100644
--- a/lib/efi_loader/Kconfig
+++ b/lib/efi_loader/Kconfig
@@ -1,6 +1,6 @@
 config EFI_LOADER
 	bool "Support running UEFI applications"
-	depends on OF_LIBFDT && ( \
+	depends on OF_LIBFDT && DM && OF_CONTROL && ( \
 		ARM && (SYS_CPU = arm1136 || \
 			SYS_CPU = arm1176 || \
 			SYS_CPU = armv7   || \
@@ -25,6 +25,8 @@ config EFI_LOADER
 	  will expose the UEFI API to a loaded application, enabling it to
 	  reuse U-Boot's device drivers.
 
+	  For ARM 32-bit, this adds about 90KB to the size of U-Boot.
+
 if EFI_LOADER
 
 config CMD_BOOTEFI_BOOTMGR
-- 
2.32.0.93.g670b81a890-goog


  parent reply	other threads:[~2021-07-02 19:07 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-07-02 18:36 [PATCH v2 0/9] efi: Various tidy-ups and drop the default Simon Glass
2021-07-02 18:36 ` [PATCH v2 1/9] configs: Resync with savedefconfig Simon Glass
2021-07-02 18:36 ` [PATCH v2 2/9] Makefile: Drop include/asm directory as well as symlink Simon Glass
2021-07-28 22:56   ` Tom Rini
2021-07-02 18:36 ` [PATCH v2 3/9] disk: Tidy up #ifdefs in part_efi Simon Glass
2021-07-28 22:56   ` Tom Rini
2021-07-28 23:21   ` Heinrich Schuchardt
2021-07-02 18:36 ` [PATCH v2 4/9] Use LIB_UUID with ACPIGEN and FS_BTRFS Simon Glass
2021-07-28 22:56   ` Tom Rini
2021-07-02 18:36 ` [PATCH v2 5/9] Allow efi_loader header to be included always Simon Glass
2021-07-28 22:56   ` Tom Rini
2021-07-02 18:36 ` [PATCH v2 6/9] lib: Create a new Kconfig option for charset conversion Simon Glass
2021-07-28 22:57   ` Tom Rini
2021-07-02 18:36 ` Simon Glass [this message]
2021-07-26 22:07   ` [PATCH v2 7/9] Make EFI_LOADER depend on DM and OF_CONTROL Tom Rini
2021-07-28 23:45     ` Heinrich Schuchardt
2021-07-28 23:55       ` Tom Rini
2021-07-29  1:44         ` Simon Glass
2021-07-29 13:52           ` Tom Rini
2021-07-30 19:02             ` Simon Glass
2021-07-30 21:33               ` Tom Rini
2021-07-30 21:48                 ` Simon Glass
2021-07-30 22:08                   ` Tom Rini
2022-03-28  6:35                     ` Simon Glass
2022-03-28 14:15                       ` Tom Rini
2021-07-02 18:36 ` [PATCH v2 8/9] Add an option for EBBR Simon Glass
2021-07-29  0:12   ` Heinrich Schuchardt
2021-07-29  0:40     ` Tom Rini
2021-07-02 18:36 ` [PATCH v2 9/9] efi: Make EBBR optional Simon Glass
2021-07-02 20:11   ` Tom Rini
2021-07-02 20:38     ` Simon Glass
2021-07-29  0:35       ` Heinrich Schuchardt
2021-07-02 19:50 ` [PATCH v2 0/9] efi: Various tidy-ups and drop the default Mark Kettenis
2021-07-02 20:04   ` Simon Glass
2021-07-02 20:19     ` Tom Rini
2021-07-02 20:50       ` Simon Glass

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20210702183620.1542680-4-sjg@chromium.org \
    --to=sjg@chromium.org \
    --cc=ilias.apalodimas@linaro.org \
    --cc=mark.kettenis@xs4all.nl \
    --cc=pali@kernel.org \
    --cc=u-boot@lists.denx.de \
    --cc=xypron.glpk@gmx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.