All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alexandru Gagniuc <mr.nuke.me@gmail.com>
To: u-boot@lists.denx.de, uboot-stm32@st-md-mailman.stormreply.com,
	patrick.delaunay@foss.st.com
Cc: Alexandru Gagniuc <mr.nuke.me@gmail.com>,
	patrice.chotard@foss.st.com, etienne.carriere@linaro.org
Subject: [PATCH v3 11/13] ARM: dts: stm32mp: Add OP-TEE "/firmware" node to SPL dtb
Date: Fri,  8 Oct 2021 14:56:53 -0500	[thread overview]
Message-ID: <20211008195655.46046-12-mr.nuke.me@gmail.com> (raw)
In-Reply-To: <20211008195655.46046-1-mr.nuke.me@gmail.com>

The optee "/firmware" node is normally used to load the OP-TEE driver.
SPL does not use it this way, but instead uses it to patch the kernel
devicetree when booting OP-TEE. This seems weird, as OP-TEE -- which
would run after SPL -- is capable of patching the devicetree and
adding the required "/firmware" and "/reserved-memory" nodes.

However, OP-TEE's devicetree patching (CFG_DT=y) comes with a
significant boot time penalty. Measurements put it between 700 to 1800
milliseconds. SPL can also do this patching, without incurring a
measurable increase in boot time. This is the use case for falcon
mode, which is the main reason to launch OP-TEE from SPL.

Although some OP-TEE configurations do not require any pre-patching of
the devicetree, there are good use cases for doing so. As far as
concerns with adding the nodes twice, OP-TEE will not add the node if
it already-exists.

Add the "/firmware/optee" node the SPL devicetree.

Signed-off-by: Alexandru Gagniuc <mr.nuke.me@gmail.com>
---
 arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi b/arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi
index 629451e731..fb60549f4d 100644
--- a/arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi
+++ b/arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi
@@ -35,6 +35,7 @@
 		optee {
 			compatible = "linaro,optee-tz";
 			method = "smc";
+			u-boot,dm-spl;
 		};
 	};
 
-- 
2.31.1


  parent reply	other threads:[~2021-10-08 19:59 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-10-08 19:56 [PATCH v3 00/13] stm32mp1: Support falcon mode with OP-TEE payloads Alexandru Gagniuc
2021-10-08 19:56 ` [PATCH v3 01/13] spl: Untagle spl_start_uboot() from spl_mmc.c Alexandru Gagniuc
2021-10-08 19:56 ` [PATCH v3 02/13] spl: Move SYS_MMCSD_RAW_MODE_KERNEL_SECTOR to Kconfig Alexandru Gagniuc
2021-10-08 19:56 ` [PATCH v3 03/13] stm32mp1: Add support for baudrates higher than 115200 Alexandru Gagniuc
2021-10-08 19:56 ` [PATCH v3 04/13] stm32mp1: Add support for falcon mode boot from SD card Alexandru Gagniuc
2021-10-08 19:56 ` [PATCH v3 05/13] board: stm32mp1: Implement board_fit_config_name_match() for SPL Alexandru Gagniuc
2021-10-08 19:56 ` [PATCH v3 06/13] fdt_support: Implement fdt_ethernet_set_macaddr() Alexandru Gagniuc
2021-10-08 19:56 ` [PATCH v3 07/13] arm: stm32mp: bsec: Update OTP shadow registers in SPL Alexandru Gagniuc
2021-10-08 19:56 ` [PATCH v3 08/13] arm: stm32mp: Factor out reading MAC address from OTP Alexandru Gagniuc
2021-10-08 19:56 ` [PATCH v3 09/13] stm32mp1: spl: Configure MAC address when booting OP-TEE Alexandru Gagniuc
2021-10-08 19:56 ` [PATCH v3 10/13] lib: Makefile: Make optee library available in SPL Alexandru Gagniuc
2021-10-08 19:56 ` Alexandru Gagniuc [this message]
2021-10-08 19:56 ` [PATCH v3 12/13] stm32mp1: spl: Copy optee nodes to target FDT for OP-TEE payloads Alexandru Gagniuc
2021-10-08 19:56 ` [PATCH v3 13/13] ARM: dts: stm32mp: dts: Don't remove OPTEE nodes for DK2 boards Alexandru Gagniuc

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=20211008195655.46046-12-mr.nuke.me@gmail.com \
    --to=mr.nuke.me@gmail.com \
    --cc=etienne.carriere@linaro.org \
    --cc=patrice.chotard@foss.st.com \
    --cc=patrick.delaunay@foss.st.com \
    --cc=u-boot@lists.denx.de \
    --cc=uboot-stm32@st-md-mailman.stormreply.com \
    /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.