Please stop including (and remove) the meta-ti-bsp: from the subject. It needlessly increases the verbosity of the commit message.


On 11/28/2025 2:53 AM, Manorit Chawdhry wrote:
Adds the required TF-A and OP-TEE patches to enable LPM support on
J7200, J784s4, J742s2.

Signed-off-by: Manorit Chawdhry <[email protected]>
---
  .../trusted-firmware-a-ti.inc                 |  20 ++
  ...luster_start_id-depending-on-the-soc.patch | 116 +++++++++++
  ...essage-to-encrypt-tfa-during-suspend.patch | 195 ++++++++++++++++++
  ...uspend-in-case-of-LPM_BOARDCFG_MANAG.patch |  69 +++++++
  .../optee/optee-os-ti-overrides.inc           |  14 ++
  ...Open-TRNG-firewall-for-TIFS-on-all-k.patch |  46 +++++
  6 files changed, 460 insertions(+)
  create mode 100644 
meta-ti-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a/0001-feat-k3-choose-cluster_start_id-depending-on-the-soc.patch
  create mode 100644 
meta-ti-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a/0002-feat-ti-add-message-to-encrypt-tfa-during-suspend.patch
  create mode 100644 
meta-ti-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a/0003-feat-k3-handle-suspend-in-case-of-LPM_BOARDCFG_MANAG.patch
  create mode 100644 
meta-ti-bsp/recipes-security/optee/optee-os/0001-plat-k3-drivers-Open-TRNG-firewall-for-TIFS-on-all-k.patch

diff --git 
a/meta-ti-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a-ti.inc 
b/meta-ti-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a-ti.inc
index f188f35ee740..6058c000e865 100644
--- a/meta-ti-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a-ti.inc
+++ b/meta-ti-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a-ti.inc
@@ -1,5 +1,7 @@
  # NOTE: This .inc file with customizations only gets included for K3 platforms
+FILESEXTRAPATHS:prepend := "${THISDIR}/trusted-firmware-a:"
+
  PV = "2.13+git"
LIC_FILES_CHKSUM = "file://docs/license.rst;md5=6ed7bace7b0bc63021c6eba7b524039e"
@@ -28,3 +30,21 @@ EXTRA_OEMAKE += "${@ 'BL32_BASE=' + 
d.getVar('TFA_K3_BL32_BASE') if d.getVar('TF
  EXTRA_OEMAKE += "${@ 'PRELOADED_BL33_BASE=' + d.getVar('TFA_K3_PRELOADED_BL33') if 
d.getVar('TFA_K3_PRELOADED_BL33') else ''}"
  EXTRA_OEMAKE += "${@ 'K3_PM_SYSTEM_SUSPEND=' + d.getVar('TFA_K3_SYSTEM_SUSPEND') 
if d.getVar('TFA_K3_SYSTEM_SUSPEND') else ''}"
  EXTRA_OEMAKE:append:ti-falcon = " PRELOADED_BL33_BASE=0x82000000 
K3_HW_CONFIG_BASE=0x88000000"
+
+SRC_URI:append:j7200 = " \
+    file://0001-feat-k3-choose-cluster_start_id-depending-on-the-soc.patch \
+    file://0002-feat-ti-add-message-to-encrypt-tfa-during-suspend.patch \
+    file://0003-feat-k3-handle-suspend-in-case-of-LPM_BOARDCFG_MANAG.patch \
+"
+
+SRC_URI:append:j742s2 = " \
+    file://0001-feat-k3-choose-cluster_start_id-depending-on-the-soc.patch \
+    file://0002-feat-ti-add-message-to-encrypt-tfa-during-suspend.patch \
+    file://0003-feat-k3-handle-suspend-in-case-of-LPM_BOARDCFG_MANAG.patch \
+"
+
+SRC_URI:append:j784s4 = " \
+    file://0001-feat-k3-choose-cluster_start_id-depending-on-the-soc.patch \
+    file://0002-feat-ti-add-message-to-encrypt-tfa-during-suspend.patch \
+    file://0003-feat-k3-handle-suspend-in-case-of-LPM_BOARDCFG_MANAG.patch \
+"

I don't like seeing the same big lists repeated over and over. I'd prefer if you created a new variable and then used that to set the SRC_URI for the three platforms:

JACINTO_LPM_PATCHES = "\

file://0001-feat-k3-choose-cluster_start_id-depending-on-the-soc.patch \
    file://0002-feat-ti-add-message-to-encrypt-tfa-during-suspend.patch \

file://0003-feat-k3-handle-suspend-in-case-of-LPM_BOARDCFG_MANAG.patch \
"

SRC_URI:append:j7200 = " ${JACINTO_LPM_PATCHES}"
SRC_URI:append:j742s2 = " ${JACINTO_LPM_PATCHES}"
SRC_URI:append:j784s4 = " ${JACINTO_LPM_PATCHES}"

Feel free to pick a different variable name of course, just something unique and descriptive.



diff --git a/meta-ti-bsp/recipes-security/optee/optee-os-ti-overrides.inc 
b/meta-ti-bsp/recipes-security/optee/optee-os-ti-overrides.inc
index 61a74a069886..52aad8a72599 100644
--- a/meta-ti-bsp/recipes-security/optee/optee-os-ti-overrides.inc
+++ b/meta-ti-bsp/recipes-security/optee/optee-os-ti-overrides.inc
@@ -1,6 +1,8 @@
  # Use TI SECDEV for signing
  inherit ti-secdev
+FILESEXTRAPATHS:prepend := "${THISDIR}/optee-os:"
+
  EXTRA_OEMAKE:remove = "CFG_MAP_EXT_DT_SECURE=y"
EXTRA_OEMAKE:append:k3 = " ${@ 'CFG_CONSOLE_UART='+ d.getVar('OPTEE_K3_USART') if d.getVar('OPTEE_K3_USART') else ''}"
@@ -76,3 +78,15 @@ RDEPENDS:${PN} += "${PN}-ta"
# This is needed for bl32.elf
  INSANE_SKIP:${PN}:append:k3 = " textrel"
+
+SRC_URI:append:j7200 = " \
+    file://0001-plat-k3-drivers-Open-TRNG-firewall-for-TIFS-on-all-k.patch \
+"
+
+SRC_URI:append:j784s4 = " \
+    file://0001-plat-k3-drivers-Open-TRNG-firewall-for-TIFS-on-all-k.patch \
+"
+
+SRC_URI:append:j742s2 = " \
+    file://0001-plat-k3-drivers-Open-TRNG-firewall-for-TIFS-on-all-k.patch \
+"

Same thing here, please.


--
Ryan Eatmon                [email protected]
-----------------------------------------
Texas Instruments, Inc.  -  LCPD  -  MGTS
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#19305): 
https://lists.yoctoproject.org/g/meta-ti/message/19305
Mute This Topic: https://lists.yoctoproject.org/mt/116511092/21656
Group Owner: [email protected]
Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to