public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Tom Rini <trini@konsulko.com>
To: u-boot@lists.denx.de
Cc: "Bryan O'Donoghue" <bryan.odonoghue@linaro.org>,
	Fabio Estevam <festevam@gmail.com>,
	Marek Vasut <marek.vasut@mailbox.org>,
	Patrice Chotard <patrice.chotard@foss.st.com>,
	Patrick Delaunay <patrick.delaunay@foss.st.com>,
	u-boot@dh-electronics.com
Subject: [PATCH] optee: Correct dependencies for BOOTM_OPTEE
Date: Mon, 16 Mar 2026 19:24:51 -0600	[thread overview]
Message-ID: <20260317012452.4162719-1-trini@konsulko.com> (raw)

As exposed by "make randconfig", we have an issue with the dependencies
for BOOTM_OPTEE. This symbol needs to select BOOTM_LINUX and in turn
that is only valid for CMD_BOOTM or CMD_BOOTZ. Add this dependency.

Signed-off-by: Tom Rini <trini@konsulko.com>
---
I am looking for feedback on this, so I've added the board maintainers
for the platforms which enable this option. It looks like this is only
valid on 32bit ARM platforms, to start with and not 64bit, is that
correct? That's why I excluded CMD_BOOTI from the list. I suspect that
this should really depend on CMD_BOOT{M,Z} and BOOTM_LINUX, and not
select it. If someone can confirm, I can do that instead, or someone can
just do the replacement patch for this. Thanks!

Cc: "Bryan O'Donoghue" <bryan.odonoghue@linaro.org>
Cc: Fabio Estevam <festevam@gmail.com>
Cc: Marek Vasut <marek.vasut@mailbox.org>
Cc: Patrice Chotard <patrice.chotard@foss.st.com>
Cc: Patrick Delaunay <patrick.delaunay@foss.st.com>
Cc: u-boot@dh-electronics.com
---
 lib/optee/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/lib/optee/Kconfig b/lib/optee/Kconfig
index 34b9d8afe675..9251faf73cd7 100644
--- a/lib/optee/Kconfig
+++ b/lib/optee/Kconfig
@@ -40,6 +40,7 @@ config OPTEE_TZDRAM_SIZE
 
 config BOOTM_OPTEE
 	bool "Support OPTEE bootm command"
+	depends on CMD_BOOTM || CMD_BOOTZ
 	select BOOTM_LINUX
 	select OPTEE_IMAGE
 	help
-- 
2.43.0


             reply	other threads:[~2026-03-17  1:27 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-03-17  1:24 Tom Rini [this message]
2026-03-17  2:32 ` [PATCH] optee: Correct dependencies for BOOTM_OPTEE Marek Vasut
2026-03-17 13:38   ` Tom Rini
2026-03-17 19:40     ` Marek Vasut
2026-03-20 17:24 ` [PATCH v2] " Tom Rini
2026-03-20 18:30   ` Marek Vasut
2026-04-03 19:47   ` Tom Rini

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=20260317012452.4162719-1-trini@konsulko.com \
    --to=trini@konsulko.com \
    --cc=bryan.odonoghue@linaro.org \
    --cc=festevam@gmail.com \
    --cc=marek.vasut@mailbox.org \
    --cc=patrice.chotard@foss.st.com \
    --cc=patrick.delaunay@foss.st.com \
    --cc=u-boot@dh-electronics.com \
    --cc=u-boot@lists.denx.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox