On 1/17/2023 22:58, Shilwant, Chirag wrote:
Why are we removing am62xx-lp-evm?  Is this part of the move to remove 
am62xx-lp-evm and merge everything into am62x-evm?

- The SOC for both platforms am62xx-evm and am62xx-lp-evm are same (i.e 
am62xx). So if we have 2 separate functions and we build am62xx-lp-evm it will 
execute both do_install_prepend_am62xx() and do_install_prepend_am62xx-lp-evm() 
resulting in build failure as we have the same
                "mv am62-mcu-m4f0_0-fw 
ipc_echo_baremetal_test_mcu2_0_release_strip.xer5f;"
in both function (the first function executing used to rename the ipc fw and 
the second function won't find it).

- As the content are same for both the platforms, combining them as a single function 
with SOC will apply the same changes to both am62xx-lp-evm (as PLAT_SFX_am62xx-lp-evm = 
"am62xx") and am62xx-evm.

Thanks.


-----Original Message-----
From: Eatmon, Ryan <[email protected]>
Sent: Tuesday, January 17, 2023 9:02 PM
To: Shilwant, Chirag <[email protected]>; Bajjuri, Praneeth <[email protected]>; Denys 
Dmytriyenko <[email protected]>; [email protected]
Cc: Adivi, Sai Sree Kartheek <[email protected]>; Bhagat, Paresh <[email protected]>; 
Gupta, Gyan <[email protected]>
Subject: Re: [meta-ti][dunfell][PATCH] am62xx: Add support for secure RTOS IPC 
Firmware images



On 1/12/2023 5:27, Chirag Shilwant wrote:
- Let's add support for secure RTOS IPC Firmware images for am62xx.
This will sign RTOS IPC Firmware for all EVMs based on am62xx (i.e
AM62X and AM62X-LP-EVM)

Signed-off-by: Chirag Shilwant <[email protected]>
---
   recipes-ti/ti-rtos-bin/ti-rtos-firmware.bb | 28 +++++++---------------
   1 file changed, 8 insertions(+), 20 deletions(-)

diff --git a/recipes-ti/ti-rtos-bin/ti-rtos-firmware.bb
b/recipes-ti/ti-rtos-bin/ti-rtos-firmware.bb
index 47b29632..72d85a6b 100644
--- a/recipes-ti/ti-rtos-bin/ti-rtos-firmware.bb
+++ b/recipes-ti/ti-rtos-bin/ti-rtos-firmware.bb
@@ -174,20 +174,18 @@ do_install_prepend_am64xx() {
           )
   }
-# Update the am62xx ipc binaries to be consistent with other
platforms
-do_install_prepend_am62xx() {
-        ( cd ${RTOS_IPC_FW_DIR}; \
-                mv am62-mcu-m4f0_0-fw 
ipc_echo_baremetal_test_mcu2_0_release_strip.xer5f; \
-        )
-}
-

Why are we removing am62xx-lp-evm?  Is this part of the move to remove 
am62xx-lp-evm and merge everything into am62x-evm?


   # AM62Q HS-SE support
-do_install_prepend_am62xx-lp-evm() {
+do_install_prepend_am62xx() {
           export TI_SECURE_DEV_PKG=${TI_SECURE_DEV_PKG}
           ( cd ${RTOS_DM_FW_DIR}; \
                   mv ${DM_FIRMWARE} ${DM_FIRMWARE}.unsigned; \
                   ${TI_SECURE_DEV_PKG}/scripts/secure-binary-image.sh 
${DM_FIRMWARE}.unsigned ${DM_FIRMWARE}; \
           )
+        ( cd ${RTOS_IPC_FW_DIR}; \
+                mv am62-mcu-m4f0_0-fw 
ipc_echo_baremetal_test_mcu2_0_release_strip.xer5f; \
+                ${TI_SECURE_DEV_PKG}/scripts/secure-binary-image.sh 
ipc_echo_baremetal_test_mcu2_0_release_strip.xer5f \
+                    ipc_echo_baremetal_test_mcu2_0_release_strip.xer5f.signed; 
\
+        )
   }
#Install all R5 & DSP ipc echo test binaries in lib/firmware/pdk-ipc, with softlinks up a level
@@ -361,13 +359,8 @@ do_install_am64xx() {
   do_install_am62xx() {
       install -d ${LEGACY_IPC_FW_DIR}
       install -m 0644 
${RTOS_IPC_FW_DIR}/ipc_echo_baremetal_test_mcu2_0_release_strip.xer5f 
${LEGACY_IPC_FW_DIR}
-    # DM Firmware
-    install -m 0644 
${RTOS_DM_FW_DIR}/ipc_echo_testb_mcu1_0_release_strip.xer5f ${LEGACY_DM_FW_DIR}
-}
-
-do_install_am62xx-lp-evm() {
-    install -d ${LEGACY_IPC_FW_DIR}
-    install -m 0644 
${RTOS_IPC_FW_DIR}/ipc_echo_baremetal_test_mcu2_0_release_strip.xer5f 
${LEGACY_IPC_FW_DIR}
+    # Signed Firmware
+    install -m 0644 
${RTOS_IPC_FW_DIR}/ipc_echo_baremetal_test_mcu2_0_release_strip.xer5f.signed 
${LEGACY_IPC_FW_DIR}
       # DM Firmware
       install -m 0644 
${RTOS_DM_FW_DIR}/ipc_echo_testb_mcu1_0_release_strip.xer5f ${LEGACY_DM_FW_DIR}
   }
@@ -389,11 +382,6 @@ do_deploy_am62xx() {
       install -m 0644 
${RTOS_DM_FW_DIR}/ipc_echo_testb_mcu1_0_release_strip.xer5f ${DEPLOYDIR}
   }
-do_deploy_am62xx-lp-evm() {
-    install -d ${DEPLOYDIR}
-    install -m 0644 
${RTOS_DM_FW_DIR}/ipc_echo_testb_mcu1_0_release_strip.xer5f ${DEPLOYDIR}
-}
-
   do_deploy_am62axx() {
       install -d ${DEPLOYDIR}
       install -m 0644 
${RTOS_DM_FW_DIR}/ipc_echo_testb_mcu1_0_release_strip.xer5f ${DEPLOYDIR}


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

Reply via email to