All of lore.kernel.org
 help / color / mirror / Atom feed
* [meta-ti][master/wrynose][PATCH v2 1/3] libdce: Runtime depend on IPU firmware not the other way around
@ 2026-06-24 19:55 Andrew Davis
  2026-06-24 19:55 ` [meta-ti][master/wrynose][PATCH v2 2/3] ipumm-fw: Use prebuilt firmware from ti-linux-firmware repo Andrew Davis
                   ` (2 more replies)
  0 siblings, 3 replies; 9+ messages in thread
From: Andrew Davis @ 2026-06-24 19:55 UTC (permalink / raw)
  To: Denys Dmytriyenko, Ryan Eatmon, meta-ti; +Cc: Andrew Davis

The DCE library depends on the firmware to function, the firmware has
no dependency on any one userspace library. Switch the runtime dependency
around here.

Signed-off-by: Andrew Davis <afd@ti.com>
---

Changes for v2:
 - Use += to avoid overwriting any existing RDEPENDS

 meta-ti-extras/recipes-bsp/ipumm-fw/ipumm-fw_git.bb           | 2 --
 meta-ti-extras/recipes-multimedia/libdce/libdce_3.00.15.00.bb | 1 +
 2 files changed, 1 insertion(+), 2 deletions(-)

diff --git a/meta-ti-extras/recipes-bsp/ipumm-fw/ipumm-fw_git.bb b/meta-ti-extras/recipes-bsp/ipumm-fw/ipumm-fw_git.bb
index bca91df8..246255e3 100644
--- a/meta-ti-extras/recipes-bsp/ipumm-fw/ipumm-fw_git.bb
+++ b/meta-ti-extras/recipes-bsp/ipumm-fw/ipumm-fw_git.bb
@@ -9,8 +9,6 @@ inherit features_check
 
 REQUIRED_MACHINE_FEATURES = "mmip"
 
-RDEPENDS:${PN} = "libdce"
-
 SRC_URI = "git://git.ti.com/git/ivimm/ipumm.git;protocol=https;branch=master"
 
 SRCREV = "df4c50aecc9aad7ab3eb1ca9ebacfe473fcad7c5"
diff --git a/meta-ti-extras/recipes-multimedia/libdce/libdce_3.00.15.00.bb b/meta-ti-extras/recipes-multimedia/libdce/libdce_3.00.15.00.bb
index 2ae44086..afbfd1cd 100644
--- a/meta-ti-extras/recipes-multimedia/libdce/libdce_3.00.15.00.bb
+++ b/meta-ti-extras/recipes-multimedia/libdce/libdce_3.00.15.00.bb
@@ -5,6 +5,7 @@ LIC_FILES_CHKSUM = "file://libdce.h;beginline=1;endline=31;md5=0a398cf815b8b5f31
 inherit autotools pkgconfig
 
 DEPENDS = "libdrm ti-ipc"
+RDEPENDS:${PN} += "ipumm-fw"
 
 SRC_URI = "git://git.ti.com/git/omapzoom/libdce.git;protocol=https;branch=master"
 
-- 
2.39.2



^ permalink raw reply related	[flat|nested] 9+ messages in thread

* [meta-ti][master/wrynose][PATCH v2 2/3] ipumm-fw: Use prebuilt firmware from ti-linux-firmware repo
  2026-06-24 19:55 [meta-ti][master/wrynose][PATCH v2 1/3] libdce: Runtime depend on IPU firmware not the other way around Andrew Davis
@ 2026-06-24 19:55 ` Andrew Davis
  2026-06-24 20:08   ` Ryan Eatmon
  2026-06-24 19:55 ` [meta-ti][master/wrynose][PATCH v2 3/3] ti-codec-engine: Remove this recipe Andrew Davis
  2026-06-24 20:00 ` [meta-ti][master/wrynose][PATCH v2 1/3] libdce: Runtime depend on IPU firmware not the other way around PRC Automation
  2 siblings, 1 reply; 9+ messages in thread
From: Andrew Davis @ 2026-06-24 19:55 UTC (permalink / raw)
  To: Denys Dmytriyenko, Ryan Eatmon, meta-ti; +Cc: Andrew Davis

Use a prebuilt version of this firmware from ti-linux-firmware. This
removes the need to build this locally.

Signed-off-by: Andrew Davis <afd@ti.com>
---

Changes for v2:
 - Fix s/suppor/support in SUMMARY while here
 - Move PV definition to ti-linux-fw.inc like other prebuilt FWs

 .../recipes-bsp/ti-linux-fw/ti-linux-fw.inc   |  1 +
 .../recipes-bsp/ipumm-fw/ipumm-fw_git.bb      | 52 ++++---------------
 2 files changed, 10 insertions(+), 43 deletions(-)

diff --git a/meta-ti-bsp/recipes-bsp/ti-linux-fw/ti-linux-fw.inc b/meta-ti-bsp/recipes-bsp/ti-linux-fw/ti-linux-fw.inc
index 126c46df..88992e60 100644
--- a/meta-ti-bsp/recipes-bsp/ti-linux-fw/ti-linux-fw.inc
+++ b/meta-ti-bsp/recipes-bsp/ti-linux-fw/ti-linux-fw.inc
@@ -20,6 +20,7 @@ TI_HSM_DEMO_FW_VERSION = "11.00.09"
 TI_PKA_FW_VERSION = "2.1.0"
 TI_IPC_EXAMPLES_FW_VERSION = "3.52.00.01"
 PCM6240_FW_VERSION = "1.0.0.0"
+IPUMM_FW_VERSION = "3.00.15.00"
 
 TI_LINUX_FW_SRCREV ?= "ec029baad8b4b147af8331a5e099ef891e5ae1f4"
 SRCREV = "${TI_LINUX_FW_SRCREV}"
diff --git a/meta-ti-extras/recipes-bsp/ipumm-fw/ipumm-fw_git.bb b/meta-ti-extras/recipes-bsp/ipumm-fw/ipumm-fw_git.bb
index 246255e3..fbc6d956 100644
--- a/meta-ti-extras/recipes-bsp/ipumm-fw/ipumm-fw_git.bb
+++ b/meta-ti-extras/recipes-bsp/ipumm-fw/ipumm-fw_git.bb
@@ -1,7 +1,7 @@
-SUMMARY = "Firmware for IPU to suppor Accelerated MM decode and encode"
-LICENSE = "TI-TSPA"
+SUMMARY = "Firmware for IPU to support Accelerated MM decode and encode"
 
-LIC_FILES_CHKSUM = "file://Texas_Instruments_ipumm_Manifest.pdf;md5=5cc572579f07af266ab57fc17d762c7f"
+LICENSE = "TI-TSPA"
+LIC_FILES_CHKSUM = "file://LICENCE.ti-tspa;md5=a1b59cb7ba626b9dbbcbf00f3fbc438a"
 
 COMPATIBLE_MACHINE = "am57xx"
 
@@ -9,59 +9,25 @@ inherit features_check
 
 REQUIRED_MACHINE_FEATURES = "mmip"
 
-SRC_URI = "git://git.ti.com/git/ivimm/ipumm.git;protocol=https;branch=master"
-
-SRCREV = "df4c50aecc9aad7ab3eb1ca9ebacfe473fcad7c5"
-
-PV = "3.00.15.00"
-PR = "r7"
+require recipes-bsp/ti-linux-fw/ti-linux-fw.inc
 
-require recipes-ti/includes/ti-paths.inc
+PV = "${IPUMM_FW_VERSION}"
+PR = "r8"
 
 inherit update-alternatives
 
-DEPENDS = "ti-xdctools-native ti-sysbios ti-codec-engine ti-framework-components ti-xdais ti-cgt-arm-native ti-ipc-rtos"
-
-export HWVERSION = "ES10"
-export BIOSTOOLSROOT = "${STAGING_DIR_TARGET}/usr/share/ti"
-
-export XDCVERSION = "ti-xdctools-tree"
-export BIOSVERSION = "ti-sysbios-tree"
-export IPCVERSION = "ti-ipc-tree"
-export CEVERSION = "ti-codec-engine-tree"
-export FCVERSION = "ti-framework-components-tree"
-export XDAISVERSION = "ti-xdais-tree"
-
-export TMS470CGTOOLPATH = "${M4_TOOLCHAIN_INSTALL_DIR}"
-export IPCSRC = "${STAGING_DIR_TARGET}/usr/share/ti/ti-ipc-tree"
-
-EXTRA_OEMAKE += "XDCDIST_TREE=${STAGING_DIR_NATIVE}/usr/share/ti/${XDCVERSION}"
-
-do_configure() {
-    oe_runmake unconfig
-    oe_runmake vayu_smp_config
-}
-
-do_compile() {
-    oe_runmake
-}
-
 TARGET = "dra7-ipu2-fw.xem4"
-TARGET_MAP = "platform/ti/dce/baseimage/package/cfg/out/ipu/release/ipu.xem4.map"
 
 do_install() {
-    install -d ${D}${nonarch_base_libdir}/firmware
-    install -m 0644 ${S}/${TARGET} ${D}${nonarch_base_libdir}/firmware/${TARGET}.${BPN}
-    install -m 0644 ${S}/${TARGET_MAP} ${D}${nonarch_base_libdir}/firmware/${TARGET}.map
+    install -d ${D}${nonarch_base_libdir}/firmware/ti-ce-fw
+    install -m 0644 ${S}/ti-ce-fw/${TARGET} ${D}${nonarch_base_libdir}/firmware/ti-ce-fw/${TARGET}
 }
 
 ALTERNATIVE:${PN} = "dra7-ipu2-fw.xem4"
 ALTERNATIVE_LINK_NAME[dra7-ipu2-fw.xem4] = "${nonarch_base_libdir}/firmware/${TARGET}"
-ALTERNATIVE_TARGET[dra7-ipu2-fw.xem4] = "${nonarch_base_libdir}/firmware/${TARGET}.${BPN}"
+ALTERNATIVE_TARGET[dra7-ipu2-fw.xem4] = "${nonarch_base_libdir}/firmware/ti-ce-fw/${TARGET}"
 ALTERNATIVE_PRIORITY = "20"
 
-FILES:${PN} += "${nonarch_base_libdir}/firmware/*"
-
 # Disable the "buildpaths" check while we figure out how we are
 # going to address this issue.
 #
-- 
2.39.2



^ permalink raw reply related	[flat|nested] 9+ messages in thread

* [meta-ti][master/wrynose][PATCH v2 3/3] ti-codec-engine: Remove this recipe
  2026-06-24 19:55 [meta-ti][master/wrynose][PATCH v2 1/3] libdce: Runtime depend on IPU firmware not the other way around Andrew Davis
  2026-06-24 19:55 ` [meta-ti][master/wrynose][PATCH v2 2/3] ipumm-fw: Use prebuilt firmware from ti-linux-firmware repo Andrew Davis
@ 2026-06-24 19:55 ` Andrew Davis
  2026-06-24 20:00 ` [meta-ti][master/wrynose][PATCH v2 1/3] libdce: Runtime depend on IPU firmware not the other way around PRC Automation
  2 siblings, 0 replies; 9+ messages in thread
From: Andrew Davis @ 2026-06-24 19:55 UTC (permalink / raw)
  To: Denys Dmytriyenko, Ryan Eatmon, meta-ti; +Cc: Andrew Davis

This was a build dependency of IPUMM firmware which is now removed.
That was the only user, remove this recipe.

Signed-off-by: Andrew Davis <afd@ti.com>
---

Changes for v2:
 - None

 .../ti-codec-engine_3.24.00.08.bb             | 29 -------------------
 .../recipes-ti/includes/ti-paths.inc          |  2 --
 2 files changed, 31 deletions(-)
 delete mode 100644 meta-ti-extras/recipes-ti/codec-engine/ti-codec-engine_3.24.00.08.bb

diff --git a/meta-ti-extras/recipes-ti/codec-engine/ti-codec-engine_3.24.00.08.bb b/meta-ti-extras/recipes-ti/codec-engine/ti-codec-engine_3.24.00.08.bb
deleted file mode 100644
index b248683b..00000000
--- a/meta-ti-extras/recipes-ti/codec-engine/ti-codec-engine_3.24.00.08.bb
+++ /dev/null
@@ -1,29 +0,0 @@
-SUMMARY = "Codec Engine for TI ARM/DSP processors"
-HOMEPAGE = "http://software-dl.ti.com/dsps/dsps_public_sw/sdo_sb/targetcontent/ce"
-SECTION = "devel"
-LICENSE = "BSD-3-Clause"
-
-require recipes-ti/includes/ti-paths.inc
-require recipes-ti/includes/ti-staging.inc
-
-PV = "3_24_00_08"
-PR = "r0"
-
-LIC_FILES_CHKSUM = "file://codec_engine_${PV}_Manifest.html;md5=de7d9c2594a6f3868e42a33b6a748ce7"
-
-SRC_URI[cetarball.md5sum] = "d66a8ccbd6cdfe9f735af9f4a07e2b9b"
-SRC_URI[cetarball.sha256sum] = "ed7b7399903bbf76af06ee6457836f344ad75b903618339bae82b3967f27ffa1"
-
-S = "${UNPACKDIR}/codec_engine_${PV}"
-
-SRC_URI = "http://software-dl.ti.com/dsps/dsps_public_sw/sdo_sb/targetcontent/ce/${PV}/exports/codec_engine_${PV},lite.tar.gz;name=cetarball "
-
-do_install() {
-        CP_ARGS="-Prf --preserve=mode,timestamps --no-preserve=ownership"
-        # Install/Stage the Source Tree
-        install -d ${D}${CE_INSTALL_DIR_RECIPE}
-        cp ${CP_ARGS} ${S}/* ${D}${CE_INSTALL_DIR_RECIPE}
-}
-
-ALLOW_EMPTY:${PN} = "1"
-FILES:${PN}-dev += "${CE_INSTALL_DIR_RECIPE}"
diff --git a/meta-ti-extras/recipes-ti/includes/ti-paths.inc b/meta-ti-extras/recipes-ti/includes/ti-paths.inc
index 9f8b5897..ed199dec 100644
--- a/meta-ti-extras/recipes-ti/includes/ti-paths.inc
+++ b/meta-ti-extras/recipes-ti/includes/ti-paths.inc
@@ -3,7 +3,6 @@ installdir = "${datadir}/ti"
 export TOOLCHAIN_PATH ?= "${STAGING_DIR_NATIVE}${prefix_native}/bin/${TARGET_SYS}"
 
 # This is where do_install will put it
-export CE_INSTALL_DIR_RECIPE           = "${installdir}/ti-codec-engine-tree"
 export CODEC_INSTALL_DIR_RECIPE        = "${installdir}/ti-codecs-tree"
 export CODEGEN_ARM_INSTALL_DIR_RECIPE  = "${installdir}/ti-cgt470-tree"
 export CG_XML_INSTALL_DIR_RECIPE       = "${installdir}/ti-cg-xml-tree"
@@ -31,7 +30,6 @@ export XDAIS_INSTALL_DIR_RECIPE        = "${installdir}/ti-xdais-tree"
 export XDC_INSTALL_DIR_RECIPE          = "${installdir}/ti-xdctools-tree"
 
 # This is where the tools will end up in sysroot
-export CE_INSTALL_DIR           = "${STAGING_DIR_TARGET}${CE_INSTALL_DIR_RECIPE}"
 export CODEGEN_ARM_INSTALL_DIR  = "${STAGING_DIR_TARGET}${CODEGEN_ARM_INSTALL_DIR_RECIPE}"
 export CG_XML_INSTALL_DIR       = "${STAGING_DIR_NATIVE}${CG_XML_INSTALL_DIR_RECIPE}"
 export DSPLIB_C66_INSTALL_DIR   = "${STAGING_DIR_TARGET}${DSPLIB_C66_INSTALL_DIR_RECIPE}"
-- 
2.39.2



^ permalink raw reply related	[flat|nested] 9+ messages in thread

* Re: [meta-ti][master/wrynose][PATCH v2 1/3] libdce: Runtime depend on IPU firmware not the other way around
  2026-06-24 19:55 [meta-ti][master/wrynose][PATCH v2 1/3] libdce: Runtime depend on IPU firmware not the other way around Andrew Davis
  2026-06-24 19:55 ` [meta-ti][master/wrynose][PATCH v2 2/3] ipumm-fw: Use prebuilt firmware from ti-linux-firmware repo Andrew Davis
  2026-06-24 19:55 ` [meta-ti][master/wrynose][PATCH v2 3/3] ti-codec-engine: Remove this recipe Andrew Davis
@ 2026-06-24 20:00 ` PRC Automation
  2 siblings, 0 replies; 9+ messages in thread
From: PRC Automation @ 2026-06-24 20:00 UTC (permalink / raw)
  To: Andrew Davis; +Cc: Denys Dmytriyenko, Ryan Eatmon, meta-ti

meta-ti / na / 20260624195533.1152173-1-afd

PRC Results: FAIL

=========================================================
  check-yocto-patches: FAIL
=========================================================
Patches
----------------------------------------
FAIL - [meta-ti][master/wrynose][PATCH v2 2/3] ipumm-fw: Use prebuilt firmware from ti-linux-firmware repo
    ERROR: Missing License-Update for change in LIC_FILES_CHKSUM. (LICENSE-UPDATE-1)
        patch
    
    ERROR: Missing License-Update for change in LICENSE. (LICENSE-UPDATE-2)
        patch
    
    For details on the above errors/warnings visit: https://lists.yoctoproject.org/g/meta-ti/wiki/40887





^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [meta-ti][master/wrynose][PATCH v2 2/3] ipumm-fw: Use prebuilt firmware from ti-linux-firmware repo
  2026-06-24 19:55 ` [meta-ti][master/wrynose][PATCH v2 2/3] ipumm-fw: Use prebuilt firmware from ti-linux-firmware repo Andrew Davis
@ 2026-06-24 20:08   ` Ryan Eatmon
  2026-06-24 20:19     ` Andrew Davis
  0 siblings, 1 reply; 9+ messages in thread
From: Ryan Eatmon @ 2026-06-24 20:08 UTC (permalink / raw)
  To: Andrew Davis, Denys Dmytriyenko, meta-ti


Missing License-Update: in the commit message.

https://docs.yoctoproject.org/contributor-guide/recipe-style-guide.html#license-updates



On 6/24/2026 2:55 PM, Andrew Davis wrote:
> Use a prebuilt version of this firmware from ti-linux-firmware. This
> removes the need to build this locally.
> 
> Signed-off-by: Andrew Davis <afd@ti.com>
> ---
> 
> Changes for v2:
>   - Fix s/suppor/support in SUMMARY while here
>   - Move PV definition to ti-linux-fw.inc like other prebuilt FWs
> 
>   .../recipes-bsp/ti-linux-fw/ti-linux-fw.inc   |  1 +
>   .../recipes-bsp/ipumm-fw/ipumm-fw_git.bb      | 52 ++++---------------
>   2 files changed, 10 insertions(+), 43 deletions(-)
> 
> diff --git a/meta-ti-bsp/recipes-bsp/ti-linux-fw/ti-linux-fw.inc b/meta-ti-bsp/recipes-bsp/ti-linux-fw/ti-linux-fw.inc
> index 126c46df..88992e60 100644
> --- a/meta-ti-bsp/recipes-bsp/ti-linux-fw/ti-linux-fw.inc
> +++ b/meta-ti-bsp/recipes-bsp/ti-linux-fw/ti-linux-fw.inc
> @@ -20,6 +20,7 @@ TI_HSM_DEMO_FW_VERSION = "11.00.09"
>   TI_PKA_FW_VERSION = "2.1.0"
>   TI_IPC_EXAMPLES_FW_VERSION = "3.52.00.01"
>   PCM6240_FW_VERSION = "1.0.0.0"
> +IPUMM_FW_VERSION = "3.00.15.00"
>   
>   TI_LINUX_FW_SRCREV ?= "ec029baad8b4b147af8331a5e099ef891e5ae1f4"
>   SRCREV = "${TI_LINUX_FW_SRCREV}"
> diff --git a/meta-ti-extras/recipes-bsp/ipumm-fw/ipumm-fw_git.bb b/meta-ti-extras/recipes-bsp/ipumm-fw/ipumm-fw_git.bb
> index 246255e3..fbc6d956 100644
> --- a/meta-ti-extras/recipes-bsp/ipumm-fw/ipumm-fw_git.bb
> +++ b/meta-ti-extras/recipes-bsp/ipumm-fw/ipumm-fw_git.bb
> @@ -1,7 +1,7 @@
> -SUMMARY = "Firmware for IPU to suppor Accelerated MM decode and encode"
> -LICENSE = "TI-TSPA"
> +SUMMARY = "Firmware for IPU to support Accelerated MM decode and encode"
>   
> -LIC_FILES_CHKSUM = "file://Texas_Instruments_ipumm_Manifest.pdf;md5=5cc572579f07af266ab57fc17d762c7f"
> +LICENSE = "TI-TSPA"
> +LIC_FILES_CHKSUM = "file://LICENCE.ti-tspa;md5=a1b59cb7ba626b9dbbcbf00f3fbc438a"
>   
>   COMPATIBLE_MACHINE = "am57xx"
>   
> @@ -9,59 +9,25 @@ inherit features_check
>   
>   REQUIRED_MACHINE_FEATURES = "mmip"
>   
> -SRC_URI = "git://git.ti.com/git/ivimm/ipumm.git;protocol=https;branch=master"
> -
> -SRCREV = "df4c50aecc9aad7ab3eb1ca9ebacfe473fcad7c5"
> -
> -PV = "3.00.15.00"
> -PR = "r7"
> +require recipes-bsp/ti-linux-fw/ti-linux-fw.inc
>   
> -require recipes-ti/includes/ti-paths.inc
> +PV = "${IPUMM_FW_VERSION}"
> +PR = "r8"
>   
>   inherit update-alternatives
>   
> -DEPENDS = "ti-xdctools-native ti-sysbios ti-codec-engine ti-framework-components ti-xdais ti-cgt-arm-native ti-ipc-rtos"
> -
> -export HWVERSION = "ES10"
> -export BIOSTOOLSROOT = "${STAGING_DIR_TARGET}/usr/share/ti"
> -
> -export XDCVERSION = "ti-xdctools-tree"
> -export BIOSVERSION = "ti-sysbios-tree"
> -export IPCVERSION = "ti-ipc-tree"
> -export CEVERSION = "ti-codec-engine-tree"
> -export FCVERSION = "ti-framework-components-tree"
> -export XDAISVERSION = "ti-xdais-tree"
> -
> -export TMS470CGTOOLPATH = "${M4_TOOLCHAIN_INSTALL_DIR}"
> -export IPCSRC = "${STAGING_DIR_TARGET}/usr/share/ti/ti-ipc-tree"
> -
> -EXTRA_OEMAKE += "XDCDIST_TREE=${STAGING_DIR_NATIVE}/usr/share/ti/${XDCVERSION}"
> -
> -do_configure() {
> -    oe_runmake unconfig
> -    oe_runmake vayu_smp_config
> -}
> -
> -do_compile() {
> -    oe_runmake
> -}
> -
>   TARGET = "dra7-ipu2-fw.xem4"
> -TARGET_MAP = "platform/ti/dce/baseimage/package/cfg/out/ipu/release/ipu.xem4.map"
>   
>   do_install() {
> -    install -d ${D}${nonarch_base_libdir}/firmware
> -    install -m 0644 ${S}/${TARGET} ${D}${nonarch_base_libdir}/firmware/${TARGET}.${BPN}
> -    install -m 0644 ${S}/${TARGET_MAP} ${D}${nonarch_base_libdir}/firmware/${TARGET}.map
> +    install -d ${D}${nonarch_base_libdir}/firmware/ti-ce-fw
> +    install -m 0644 ${S}/ti-ce-fw/${TARGET} ${D}${nonarch_base_libdir}/firmware/ti-ce-fw/${TARGET}
>   }
>   
>   ALTERNATIVE:${PN} = "dra7-ipu2-fw.xem4"
>   ALTERNATIVE_LINK_NAME[dra7-ipu2-fw.xem4] = "${nonarch_base_libdir}/firmware/${TARGET}"
> -ALTERNATIVE_TARGET[dra7-ipu2-fw.xem4] = "${nonarch_base_libdir}/firmware/${TARGET}.${BPN}"
> +ALTERNATIVE_TARGET[dra7-ipu2-fw.xem4] = "${nonarch_base_libdir}/firmware/ti-ce-fw/${TARGET}"
>   ALTERNATIVE_PRIORITY = "20"
>   
> -FILES:${PN} += "${nonarch_base_libdir}/firmware/*"
> -
>   # Disable the "buildpaths" check while we figure out how we are
>   # going to address this issue.
>   #

-- 
Ryan Eatmon                reatmon@ti.com
-----------------------------------------
Texas Instruments, Inc.  -  LCPD  -  MGTS


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [meta-ti][master/wrynose][PATCH v2 2/3] ipumm-fw: Use prebuilt firmware from ti-linux-firmware repo
  2026-06-24 20:08   ` Ryan Eatmon
@ 2026-06-24 20:19     ` Andrew Davis
  2026-06-24 20:29       ` Denys Dmytriyenko
  0 siblings, 1 reply; 9+ messages in thread
From: Andrew Davis @ 2026-06-24 20:19 UTC (permalink / raw)
  To: Ryan Eatmon, Denys Dmytriyenko, meta-ti

On 6/24/26 3:08 PM, Ryan Eatmon wrote:
> 
> Missing License-Update: in the commit message.
> 
> https://docs.yoctoproject.org/contributor-guide/recipe-style-guide.html#license-updates
> 
> 

Hmm, so in this case the license isn't changing, just moving location, would something
like this work?:

License-Update: License text moved from source manifest to standalone file

Andrew

> 
> On 6/24/2026 2:55 PM, Andrew Davis wrote:
>> Use a prebuilt version of this firmware from ti-linux-firmware. This
>> removes the need to build this locally.
>>
>> Signed-off-by: Andrew Davis <afd@ti.com>
>> ---
>>
>> Changes for v2:
>>   - Fix s/suppor/support in SUMMARY while here
>>   - Move PV definition to ti-linux-fw.inc like other prebuilt FWs
>>
>>   .../recipes-bsp/ti-linux-fw/ti-linux-fw.inc   |  1 +
>>   .../recipes-bsp/ipumm-fw/ipumm-fw_git.bb      | 52 ++++---------------
>>   2 files changed, 10 insertions(+), 43 deletions(-)
>>
>> diff --git a/meta-ti-bsp/recipes-bsp/ti-linux-fw/ti-linux-fw.inc b/meta-ti-bsp/recipes-bsp/ti-linux-fw/ti-linux-fw.inc
>> index 126c46df..88992e60 100644
>> --- a/meta-ti-bsp/recipes-bsp/ti-linux-fw/ti-linux-fw.inc
>> +++ b/meta-ti-bsp/recipes-bsp/ti-linux-fw/ti-linux-fw.inc
>> @@ -20,6 +20,7 @@ TI_HSM_DEMO_FW_VERSION = "11.00.09"
>>   TI_PKA_FW_VERSION = "2.1.0"
>>   TI_IPC_EXAMPLES_FW_VERSION = "3.52.00.01"
>>   PCM6240_FW_VERSION = "1.0.0.0"
>> +IPUMM_FW_VERSION = "3.00.15.00"
>>   TI_LINUX_FW_SRCREV ?= "ec029baad8b4b147af8331a5e099ef891e5ae1f4"
>>   SRCREV = "${TI_LINUX_FW_SRCREV}"
>> diff --git a/meta-ti-extras/recipes-bsp/ipumm-fw/ipumm-fw_git.bb b/meta-ti-extras/recipes-bsp/ipumm-fw/ipumm-fw_git.bb
>> index 246255e3..fbc6d956 100644
>> --- a/meta-ti-extras/recipes-bsp/ipumm-fw/ipumm-fw_git.bb
>> +++ b/meta-ti-extras/recipes-bsp/ipumm-fw/ipumm-fw_git.bb
>> @@ -1,7 +1,7 @@
>> -SUMMARY = "Firmware for IPU to suppor Accelerated MM decode and encode"
>> -LICENSE = "TI-TSPA"
>> +SUMMARY = "Firmware for IPU to support Accelerated MM decode and encode"
>> -LIC_FILES_CHKSUM = "file://Texas_Instruments_ipumm_Manifest.pdf;md5=5cc572579f07af266ab57fc17d762c7f"
>> +LICENSE = "TI-TSPA"
>> +LIC_FILES_CHKSUM = "file://LICENCE.ti-tspa;md5=a1b59cb7ba626b9dbbcbf00f3fbc438a"
>>   COMPATIBLE_MACHINE = "am57xx"
>> @@ -9,59 +9,25 @@ inherit features_check
>>   REQUIRED_MACHINE_FEATURES = "mmip"
>> -SRC_URI = "git://git.ti.com/git/ivimm/ipumm.git;protocol=https;branch=master"
>> -
>> -SRCREV = "df4c50aecc9aad7ab3eb1ca9ebacfe473fcad7c5"
>> -
>> -PV = "3.00.15.00"
>> -PR = "r7"
>> +require recipes-bsp/ti-linux-fw/ti-linux-fw.inc
>> -require recipes-ti/includes/ti-paths.inc
>> +PV = "${IPUMM_FW_VERSION}"
>> +PR = "r8"
>>   inherit update-alternatives
>> -DEPENDS = "ti-xdctools-native ti-sysbios ti-codec-engine ti-framework-components ti-xdais ti-cgt-arm-native ti-ipc-rtos"
>> -
>> -export HWVERSION = "ES10"
>> -export BIOSTOOLSROOT = "${STAGING_DIR_TARGET}/usr/share/ti"
>> -
>> -export XDCVERSION = "ti-xdctools-tree"
>> -export BIOSVERSION = "ti-sysbios-tree"
>> -export IPCVERSION = "ti-ipc-tree"
>> -export CEVERSION = "ti-codec-engine-tree"
>> -export FCVERSION = "ti-framework-components-tree"
>> -export XDAISVERSION = "ti-xdais-tree"
>> -
>> -export TMS470CGTOOLPATH = "${M4_TOOLCHAIN_INSTALL_DIR}"
>> -export IPCSRC = "${STAGING_DIR_TARGET}/usr/share/ti/ti-ipc-tree"
>> -
>> -EXTRA_OEMAKE += "XDCDIST_TREE=${STAGING_DIR_NATIVE}/usr/share/ti/${XDCVERSION}"
>> -
>> -do_configure() {
>> -    oe_runmake unconfig
>> -    oe_runmake vayu_smp_config
>> -}
>> -
>> -do_compile() {
>> -    oe_runmake
>> -}
>> -
>>   TARGET = "dra7-ipu2-fw.xem4"
>> -TARGET_MAP = "platform/ti/dce/baseimage/package/cfg/out/ipu/release/ipu.xem4.map"
>>   do_install() {
>> -    install -d ${D}${nonarch_base_libdir}/firmware
>> -    install -m 0644 ${S}/${TARGET} ${D}${nonarch_base_libdir}/firmware/${TARGET}.${BPN}
>> -    install -m 0644 ${S}/${TARGET_MAP} ${D}${nonarch_base_libdir}/firmware/${TARGET}.map
>> +    install -d ${D}${nonarch_base_libdir}/firmware/ti-ce-fw
>> +    install -m 0644 ${S}/ti-ce-fw/${TARGET} ${D}${nonarch_base_libdir}/firmware/ti-ce-fw/${TARGET}
>>   }
>>   ALTERNATIVE:${PN} = "dra7-ipu2-fw.xem4"
>>   ALTERNATIVE_LINK_NAME[dra7-ipu2-fw.xem4] = "${nonarch_base_libdir}/firmware/${TARGET}"
>> -ALTERNATIVE_TARGET[dra7-ipu2-fw.xem4] = "${nonarch_base_libdir}/firmware/${TARGET}.${BPN}"
>> +ALTERNATIVE_TARGET[dra7-ipu2-fw.xem4] = "${nonarch_base_libdir}/firmware/ti-ce-fw/${TARGET}"
>>   ALTERNATIVE_PRIORITY = "20"
>> -FILES:${PN} += "${nonarch_base_libdir}/firmware/*"
>> -
>>   # Disable the "buildpaths" check while we figure out how we are
>>   # going to address this issue.
>>   #
> 



^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [meta-ti][master/wrynose][PATCH v2 2/3] ipumm-fw: Use prebuilt firmware from ti-linux-firmware repo
  2026-06-24 20:19     ` Andrew Davis
@ 2026-06-24 20:29       ` Denys Dmytriyenko
  2026-06-24 20:42         ` Andrew Davis
  0 siblings, 1 reply; 9+ messages in thread
From: Denys Dmytriyenko @ 2026-06-24 20:29 UTC (permalink / raw)
  To: afd; +Cc: Ryan Eatmon, Denys Dmytriyenko, meta-ti

On Wed, Jun 24, 2026 at 03:19:33PM -0500, Andrew Davis via lists.yoctoproject.org wrote:
> On 6/24/26 3:08 PM, Ryan Eatmon wrote:
> >
> >Missing License-Update: in the commit message.
> >
> >https://docs.yoctoproject.org/contributor-guide/recipe-style-guide.html#license-updates
> >
> >
> 
> Hmm, so in this case the license isn't changing, just moving location, would something
> like this work?:
> 
> License-Update: License text moved from source manifest to standalone file

Aren't they both standaline files? The old one is a pdf and the new one is 
presumably text?


> Andrew
> 
> >
> >On 6/24/2026 2:55 PM, Andrew Davis wrote:
> >>Use a prebuilt version of this firmware from ti-linux-firmware. This
> >>removes the need to build this locally.
> >>
> >>Signed-off-by: Andrew Davis <afd@ti.com>
> >>---
> >>
> >>Changes for v2:
> >>  - Fix s/suppor/support in SUMMARY while here
> >>  - Move PV definition to ti-linux-fw.inc like other prebuilt FWs
> >>
> >>  .../recipes-bsp/ti-linux-fw/ti-linux-fw.inc   |  1 +
> >>  .../recipes-bsp/ipumm-fw/ipumm-fw_git.bb      | 52 ++++---------------
> >>  2 files changed, 10 insertions(+), 43 deletions(-)
> >>
> >>diff --git a/meta-ti-bsp/recipes-bsp/ti-linux-fw/ti-linux-fw.inc b/meta-ti-bsp/recipes-bsp/ti-linux-fw/ti-linux-fw.inc
> >>index 126c46df..88992e60 100644
> >>--- a/meta-ti-bsp/recipes-bsp/ti-linux-fw/ti-linux-fw.inc
> >>+++ b/meta-ti-bsp/recipes-bsp/ti-linux-fw/ti-linux-fw.inc
> >>@@ -20,6 +20,7 @@ TI_HSM_DEMO_FW_VERSION = "11.00.09"
> >>  TI_PKA_FW_VERSION = "2.1.0"
> >>  TI_IPC_EXAMPLES_FW_VERSION = "3.52.00.01"
> >>  PCM6240_FW_VERSION = "1.0.0.0"
> >>+IPUMM_FW_VERSION = "3.00.15.00"
> >>  TI_LINUX_FW_SRCREV ?= "ec029baad8b4b147af8331a5e099ef891e5ae1f4"
> >>  SRCREV = "${TI_LINUX_FW_SRCREV}"
> >>diff --git a/meta-ti-extras/recipes-bsp/ipumm-fw/ipumm-fw_git.bb b/meta-ti-extras/recipes-bsp/ipumm-fw/ipumm-fw_git.bb
> >>index 246255e3..fbc6d956 100644
> >>--- a/meta-ti-extras/recipes-bsp/ipumm-fw/ipumm-fw_git.bb
> >>+++ b/meta-ti-extras/recipes-bsp/ipumm-fw/ipumm-fw_git.bb
> >>@@ -1,7 +1,7 @@
> >>-SUMMARY = "Firmware for IPU to suppor Accelerated MM decode and encode"
> >>-LICENSE = "TI-TSPA"
> >>+SUMMARY = "Firmware for IPU to support Accelerated MM decode and encode"
> >>-LIC_FILES_CHKSUM = "file://Texas_Instruments_ipumm_Manifest.pdf;md5=5cc572579f07af266ab57fc17d762c7f"
> >>+LICENSE = "TI-TSPA"
> >>+LIC_FILES_CHKSUM = "file://LICENCE.ti-tspa;md5=a1b59cb7ba626b9dbbcbf00f3fbc438a"
> >>  COMPATIBLE_MACHINE = "am57xx"
> >>@@ -9,59 +9,25 @@ inherit features_check
> >>  REQUIRED_MACHINE_FEATURES = "mmip"
> >>-SRC_URI = "git://git.ti.com/git/ivimm/ipumm.git;protocol=https;branch=master"
> >>-
> >>-SRCREV = "df4c50aecc9aad7ab3eb1ca9ebacfe473fcad7c5"
> >>-
> >>-PV = "3.00.15.00"
> >>-PR = "r7"
> >>+require recipes-bsp/ti-linux-fw/ti-linux-fw.inc
> >>-require recipes-ti/includes/ti-paths.inc
> >>+PV = "${IPUMM_FW_VERSION}"
> >>+PR = "r8"
> >>  inherit update-alternatives
> >>-DEPENDS = "ti-xdctools-native ti-sysbios ti-codec-engine ti-framework-components ti-xdais ti-cgt-arm-native ti-ipc-rtos"
> >>-
> >>-export HWVERSION = "ES10"
> >>-export BIOSTOOLSROOT = "${STAGING_DIR_TARGET}/usr/share/ti"
> >>-
> >>-export XDCVERSION = "ti-xdctools-tree"
> >>-export BIOSVERSION = "ti-sysbios-tree"
> >>-export IPCVERSION = "ti-ipc-tree"
> >>-export CEVERSION = "ti-codec-engine-tree"
> >>-export FCVERSION = "ti-framework-components-tree"
> >>-export XDAISVERSION = "ti-xdais-tree"
> >>-
> >>-export TMS470CGTOOLPATH = "${M4_TOOLCHAIN_INSTALL_DIR}"
> >>-export IPCSRC = "${STAGING_DIR_TARGET}/usr/share/ti/ti-ipc-tree"
> >>-
> >>-EXTRA_OEMAKE += "XDCDIST_TREE=${STAGING_DIR_NATIVE}/usr/share/ti/${XDCVERSION}"
> >>-
> >>-do_configure() {
> >>-    oe_runmake unconfig
> >>-    oe_runmake vayu_smp_config
> >>-}
> >>-
> >>-do_compile() {
> >>-    oe_runmake
> >>-}
> >>-
> >>  TARGET = "dra7-ipu2-fw.xem4"
> >>-TARGET_MAP = "platform/ti/dce/baseimage/package/cfg/out/ipu/release/ipu.xem4.map"
> >>  do_install() {
> >>-    install -d ${D}${nonarch_base_libdir}/firmware
> >>-    install -m 0644 ${S}/${TARGET} ${D}${nonarch_base_libdir}/firmware/${TARGET}.${BPN}
> >>-    install -m 0644 ${S}/${TARGET_MAP} ${D}${nonarch_base_libdir}/firmware/${TARGET}.map
> >>+    install -d ${D}${nonarch_base_libdir}/firmware/ti-ce-fw
> >>+    install -m 0644 ${S}/ti-ce-fw/${TARGET} ${D}${nonarch_base_libdir}/firmware/ti-ce-fw/${TARGET}
> >>  }
> >>  ALTERNATIVE:${PN} = "dra7-ipu2-fw.xem4"
> >>  ALTERNATIVE_LINK_NAME[dra7-ipu2-fw.xem4] = "${nonarch_base_libdir}/firmware/${TARGET}"
> >>-ALTERNATIVE_TARGET[dra7-ipu2-fw.xem4] = "${nonarch_base_libdir}/firmware/${TARGET}.${BPN}"
> >>+ALTERNATIVE_TARGET[dra7-ipu2-fw.xem4] = "${nonarch_base_libdir}/firmware/ti-ce-fw/${TARGET}"
> >>  ALTERNATIVE_PRIORITY = "20"
> >>-FILES:${PN} += "${nonarch_base_libdir}/firmware/*"
> >>-
> >>  # Disable the "buildpaths" check while we figure out how we are
> >>  # going to address this issue.
> >>  #
> >
> 


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [meta-ti][master/wrynose][PATCH v2 2/3] ipumm-fw: Use prebuilt firmware from ti-linux-firmware repo
  2026-06-24 20:29       ` Denys Dmytriyenko
@ 2026-06-24 20:42         ` Andrew Davis
  2026-06-24 20:48           ` Denys Dmytriyenko
  0 siblings, 1 reply; 9+ messages in thread
From: Andrew Davis @ 2026-06-24 20:42 UTC (permalink / raw)
  To: Denys Dmytriyenko; +Cc: Ryan Eatmon, Denys Dmytriyenko, meta-ti

On 6/24/26 3:29 PM, Denys Dmytriyenko wrote:
> On Wed, Jun 24, 2026 at 03:19:33PM -0500, Andrew Davis via lists.yoctoproject.org wrote:
>> On 6/24/26 3:08 PM, Ryan Eatmon wrote:
>>>
>>> Missing License-Update: in the commit message.
>>>
>>> https://docs.yoctoproject.org/contributor-guide/recipe-style-guide.html#license-updates
>>>
>>>
>>
>> Hmm, so in this case the license isn't changing, just moving location, would something
>> like this work?:
>>
>> License-Update: License text moved from source manifest to standalone file
> 
> Aren't they both standaline files? The old one is a pdf and the new one is
> presumably text?
> 

The old one included other information, in fact it probably wasn't even the
right file to use for the LIC_FILES_CHKSUM, it should have been this file:

https://git.ti.com/cgit/ivimm/ipumm/plain/SRAS_3443-TSPA_Click_Wrap_License.pdf

But that is an mistake someone made many years ago.. So what we are doing here
is moving the LIC_FILES_CHKSUM from the source manifest (incorrect), to the
standalone TI-TSPA license file.

Andrew

> 
>> Andrew
>>
>>>
>>> On 6/24/2026 2:55 PM, Andrew Davis wrote:
>>>> Use a prebuilt version of this firmware from ti-linux-firmware. This
>>>> removes the need to build this locally.
>>>>
>>>> Signed-off-by: Andrew Davis <afd@ti.com>
>>>> ---
>>>>
>>>> Changes for v2:
>>>>    - Fix s/suppor/support in SUMMARY while here
>>>>    - Move PV definition to ti-linux-fw.inc like other prebuilt FWs
>>>>
>>>>    .../recipes-bsp/ti-linux-fw/ti-linux-fw.inc   |  1 +
>>>>    .../recipes-bsp/ipumm-fw/ipumm-fw_git.bb      | 52 ++++---------------
>>>>    2 files changed, 10 insertions(+), 43 deletions(-)
>>>>
>>>> diff --git a/meta-ti-bsp/recipes-bsp/ti-linux-fw/ti-linux-fw.inc b/meta-ti-bsp/recipes-bsp/ti-linux-fw/ti-linux-fw.inc
>>>> index 126c46df..88992e60 100644
>>>> --- a/meta-ti-bsp/recipes-bsp/ti-linux-fw/ti-linux-fw.inc
>>>> +++ b/meta-ti-bsp/recipes-bsp/ti-linux-fw/ti-linux-fw.inc
>>>> @@ -20,6 +20,7 @@ TI_HSM_DEMO_FW_VERSION = "11.00.09"
>>>>    TI_PKA_FW_VERSION = "2.1.0"
>>>>    TI_IPC_EXAMPLES_FW_VERSION = "3.52.00.01"
>>>>    PCM6240_FW_VERSION = "1.0.0.0"
>>>> +IPUMM_FW_VERSION = "3.00.15.00"
>>>>    TI_LINUX_FW_SRCREV ?= "ec029baad8b4b147af8331a5e099ef891e5ae1f4"
>>>>    SRCREV = "${TI_LINUX_FW_SRCREV}"
>>>> diff --git a/meta-ti-extras/recipes-bsp/ipumm-fw/ipumm-fw_git.bb b/meta-ti-extras/recipes-bsp/ipumm-fw/ipumm-fw_git.bb
>>>> index 246255e3..fbc6d956 100644
>>>> --- a/meta-ti-extras/recipes-bsp/ipumm-fw/ipumm-fw_git.bb
>>>> +++ b/meta-ti-extras/recipes-bsp/ipumm-fw/ipumm-fw_git.bb
>>>> @@ -1,7 +1,7 @@
>>>> -SUMMARY = "Firmware for IPU to suppor Accelerated MM decode and encode"
>>>> -LICENSE = "TI-TSPA"
>>>> +SUMMARY = "Firmware for IPU to support Accelerated MM decode and encode"
>>>> -LIC_FILES_CHKSUM = "file://Texas_Instruments_ipumm_Manifest.pdf;md5=5cc572579f07af266ab57fc17d762c7f"
>>>> +LICENSE = "TI-TSPA"
>>>> +LIC_FILES_CHKSUM = "file://LICENCE.ti-tspa;md5=a1b59cb7ba626b9dbbcbf00f3fbc438a"
>>>>    COMPATIBLE_MACHINE = "am57xx"
>>>> @@ -9,59 +9,25 @@ inherit features_check
>>>>    REQUIRED_MACHINE_FEATURES = "mmip"
>>>> -SRC_URI = "git://git.ti.com/git/ivimm/ipumm.git;protocol=https;branch=master"
>>>> -
>>>> -SRCREV = "df4c50aecc9aad7ab3eb1ca9ebacfe473fcad7c5"
>>>> -
>>>> -PV = "3.00.15.00"
>>>> -PR = "r7"
>>>> +require recipes-bsp/ti-linux-fw/ti-linux-fw.inc
>>>> -require recipes-ti/includes/ti-paths.inc
>>>> +PV = "${IPUMM_FW_VERSION}"
>>>> +PR = "r8"
>>>>    inherit update-alternatives
>>>> -DEPENDS = "ti-xdctools-native ti-sysbios ti-codec-engine ti-framework-components ti-xdais ti-cgt-arm-native ti-ipc-rtos"
>>>> -
>>>> -export HWVERSION = "ES10"
>>>> -export BIOSTOOLSROOT = "${STAGING_DIR_TARGET}/usr/share/ti"
>>>> -
>>>> -export XDCVERSION = "ti-xdctools-tree"
>>>> -export BIOSVERSION = "ti-sysbios-tree"
>>>> -export IPCVERSION = "ti-ipc-tree"
>>>> -export CEVERSION = "ti-codec-engine-tree"
>>>> -export FCVERSION = "ti-framework-components-tree"
>>>> -export XDAISVERSION = "ti-xdais-tree"
>>>> -
>>>> -export TMS470CGTOOLPATH = "${M4_TOOLCHAIN_INSTALL_DIR}"
>>>> -export IPCSRC = "${STAGING_DIR_TARGET}/usr/share/ti/ti-ipc-tree"
>>>> -
>>>> -EXTRA_OEMAKE += "XDCDIST_TREE=${STAGING_DIR_NATIVE}/usr/share/ti/${XDCVERSION}"
>>>> -
>>>> -do_configure() {
>>>> -    oe_runmake unconfig
>>>> -    oe_runmake vayu_smp_config
>>>> -}
>>>> -
>>>> -do_compile() {
>>>> -    oe_runmake
>>>> -}
>>>> -
>>>>    TARGET = "dra7-ipu2-fw.xem4"
>>>> -TARGET_MAP = "platform/ti/dce/baseimage/package/cfg/out/ipu/release/ipu.xem4.map"
>>>>    do_install() {
>>>> -    install -d ${D}${nonarch_base_libdir}/firmware
>>>> -    install -m 0644 ${S}/${TARGET} ${D}${nonarch_base_libdir}/firmware/${TARGET}.${BPN}
>>>> -    install -m 0644 ${S}/${TARGET_MAP} ${D}${nonarch_base_libdir}/firmware/${TARGET}.map
>>>> +    install -d ${D}${nonarch_base_libdir}/firmware/ti-ce-fw
>>>> +    install -m 0644 ${S}/ti-ce-fw/${TARGET} ${D}${nonarch_base_libdir}/firmware/ti-ce-fw/${TARGET}
>>>>    }
>>>>    ALTERNATIVE:${PN} = "dra7-ipu2-fw.xem4"
>>>>    ALTERNATIVE_LINK_NAME[dra7-ipu2-fw.xem4] = "${nonarch_base_libdir}/firmware/${TARGET}"
>>>> -ALTERNATIVE_TARGET[dra7-ipu2-fw.xem4] = "${nonarch_base_libdir}/firmware/${TARGET}.${BPN}"
>>>> +ALTERNATIVE_TARGET[dra7-ipu2-fw.xem4] = "${nonarch_base_libdir}/firmware/ti-ce-fw/${TARGET}"
>>>>    ALTERNATIVE_PRIORITY = "20"
>>>> -FILES:${PN} += "${nonarch_base_libdir}/firmware/*"
>>>> -
>>>>    # Disable the "buildpaths" check while we figure out how we are
>>>>    # going to address this issue.
>>>>    #
>>>
>>



^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [meta-ti][master/wrynose][PATCH v2 2/3] ipumm-fw: Use prebuilt firmware from ti-linux-firmware repo
  2026-06-24 20:42         ` Andrew Davis
@ 2026-06-24 20:48           ` Denys Dmytriyenko
  0 siblings, 0 replies; 9+ messages in thread
From: Denys Dmytriyenko @ 2026-06-24 20:48 UTC (permalink / raw)
  To: afd; +Cc: Ryan Eatmon, Denys Dmytriyenko, meta-ti

On Wed, Jun 24, 2026 at 03:42:49PM -0500, Andrew Davis via lists.yoctoproject.org wrote:
> On 6/24/26 3:29 PM, Denys Dmytriyenko wrote:
> >On Wed, Jun 24, 2026 at 03:19:33PM -0500, Andrew Davis via lists.yoctoproject.org wrote:
> >>On 6/24/26 3:08 PM, Ryan Eatmon wrote:
> >>>
> >>>Missing License-Update: in the commit message.
> >>>
> >>>https://docs.yoctoproject.org/contributor-guide/recipe-style-guide.html#license-updates
> >>>
> >>>
> >>
> >>Hmm, so in this case the license isn't changing, just moving location, would something
> >>like this work?:
> >>
> >>License-Update: License text moved from source manifest to standalone file
> >
> >Aren't they both standaline files? The old one is a pdf and the new one is
> >presumably text?
> >
> 
> The old one included other information, in fact it probably wasn't even the
> right file to use for the LIC_FILES_CHKSUM, it should have been this file:
> 
> https://git.ti.com/cgit/ivimm/ipumm/plain/SRAS_3443-TSPA_Click_Wrap_License.pdf
> 
> But that is an mistake someone made many years ago.. So what we are doing here
> is moving the LIC_FILES_CHKSUM from the source manifest (incorrect), to the
> standalone TI-TSPA license file.

Can you just say in the License-Update: tag something along the lines:

"The license stays the same, just use the correct file for checksum"


> Andrew
> 
> >
> >>Andrew
> >>
> >>>
> >>>On 6/24/2026 2:55 PM, Andrew Davis wrote:
> >>>>Use a prebuilt version of this firmware from ti-linux-firmware. This
> >>>>removes the need to build this locally.
> >>>>
> >>>>Signed-off-by: Andrew Davis <afd@ti.com>
> >>>>---
> >>>>
> >>>>Changes for v2:
> >>>>   - Fix s/suppor/support in SUMMARY while here
> >>>>   - Move PV definition to ti-linux-fw.inc like other prebuilt FWs
> >>>>
> >>>>   .../recipes-bsp/ti-linux-fw/ti-linux-fw.inc   |  1 +
> >>>>   .../recipes-bsp/ipumm-fw/ipumm-fw_git.bb      | 52 ++++---------------
> >>>>   2 files changed, 10 insertions(+), 43 deletions(-)
> >>>>
> >>>>diff --git a/meta-ti-bsp/recipes-bsp/ti-linux-fw/ti-linux-fw.inc b/meta-ti-bsp/recipes-bsp/ti-linux-fw/ti-linux-fw.inc
> >>>>index 126c46df..88992e60 100644
> >>>>--- a/meta-ti-bsp/recipes-bsp/ti-linux-fw/ti-linux-fw.inc
> >>>>+++ b/meta-ti-bsp/recipes-bsp/ti-linux-fw/ti-linux-fw.inc
> >>>>@@ -20,6 +20,7 @@ TI_HSM_DEMO_FW_VERSION = "11.00.09"
> >>>>   TI_PKA_FW_VERSION = "2.1.0"
> >>>>   TI_IPC_EXAMPLES_FW_VERSION = "3.52.00.01"
> >>>>   PCM6240_FW_VERSION = "1.0.0.0"
> >>>>+IPUMM_FW_VERSION = "3.00.15.00"
> >>>>   TI_LINUX_FW_SRCREV ?= "ec029baad8b4b147af8331a5e099ef891e5ae1f4"
> >>>>   SRCREV = "${TI_LINUX_FW_SRCREV}"
> >>>>diff --git a/meta-ti-extras/recipes-bsp/ipumm-fw/ipumm-fw_git.bb b/meta-ti-extras/recipes-bsp/ipumm-fw/ipumm-fw_git.bb
> >>>>index 246255e3..fbc6d956 100644
> >>>>--- a/meta-ti-extras/recipes-bsp/ipumm-fw/ipumm-fw_git.bb
> >>>>+++ b/meta-ti-extras/recipes-bsp/ipumm-fw/ipumm-fw_git.bb
> >>>>@@ -1,7 +1,7 @@
> >>>>-SUMMARY = "Firmware for IPU to suppor Accelerated MM decode and encode"
> >>>>-LICENSE = "TI-TSPA"
> >>>>+SUMMARY = "Firmware for IPU to support Accelerated MM decode and encode"
> >>>>-LIC_FILES_CHKSUM = "file://Texas_Instruments_ipumm_Manifest.pdf;md5=5cc572579f07af266ab57fc17d762c7f"
> >>>>+LICENSE = "TI-TSPA"
> >>>>+LIC_FILES_CHKSUM = "file://LICENCE.ti-tspa;md5=a1b59cb7ba626b9dbbcbf00f3fbc438a"
> >>>>   COMPATIBLE_MACHINE = "am57xx"
> >>>>@@ -9,59 +9,25 @@ inherit features_check
> >>>>   REQUIRED_MACHINE_FEATURES = "mmip"
> >>>>-SRC_URI = "git://git.ti.com/git/ivimm/ipumm.git;protocol=https;branch=master"
> >>>>-
> >>>>-SRCREV = "df4c50aecc9aad7ab3eb1ca9ebacfe473fcad7c5"
> >>>>-
> >>>>-PV = "3.00.15.00"
> >>>>-PR = "r7"
> >>>>+require recipes-bsp/ti-linux-fw/ti-linux-fw.inc
> >>>>-require recipes-ti/includes/ti-paths.inc
> >>>>+PV = "${IPUMM_FW_VERSION}"
> >>>>+PR = "r8"
> >>>>   inherit update-alternatives
> >>>>-DEPENDS = "ti-xdctools-native ti-sysbios ti-codec-engine ti-framework-components ti-xdais ti-cgt-arm-native ti-ipc-rtos"
> >>>>-
> >>>>-export HWVERSION = "ES10"
> >>>>-export BIOSTOOLSROOT = "${STAGING_DIR_TARGET}/usr/share/ti"
> >>>>-
> >>>>-export XDCVERSION = "ti-xdctools-tree"
> >>>>-export BIOSVERSION = "ti-sysbios-tree"
> >>>>-export IPCVERSION = "ti-ipc-tree"
> >>>>-export CEVERSION = "ti-codec-engine-tree"
> >>>>-export FCVERSION = "ti-framework-components-tree"
> >>>>-export XDAISVERSION = "ti-xdais-tree"
> >>>>-
> >>>>-export TMS470CGTOOLPATH = "${M4_TOOLCHAIN_INSTALL_DIR}"
> >>>>-export IPCSRC = "${STAGING_DIR_TARGET}/usr/share/ti/ti-ipc-tree"
> >>>>-
> >>>>-EXTRA_OEMAKE += "XDCDIST_TREE=${STAGING_DIR_NATIVE}/usr/share/ti/${XDCVERSION}"
> >>>>-
> >>>>-do_configure() {
> >>>>-    oe_runmake unconfig
> >>>>-    oe_runmake vayu_smp_config
> >>>>-}
> >>>>-
> >>>>-do_compile() {
> >>>>-    oe_runmake
> >>>>-}
> >>>>-
> >>>>   TARGET = "dra7-ipu2-fw.xem4"
> >>>>-TARGET_MAP = "platform/ti/dce/baseimage/package/cfg/out/ipu/release/ipu.xem4.map"
> >>>>   do_install() {
> >>>>-    install -d ${D}${nonarch_base_libdir}/firmware
> >>>>-    install -m 0644 ${S}/${TARGET} ${D}${nonarch_base_libdir}/firmware/${TARGET}.${BPN}
> >>>>-    install -m 0644 ${S}/${TARGET_MAP} ${D}${nonarch_base_libdir}/firmware/${TARGET}.map
> >>>>+    install -d ${D}${nonarch_base_libdir}/firmware/ti-ce-fw
> >>>>+    install -m 0644 ${S}/ti-ce-fw/${TARGET} ${D}${nonarch_base_libdir}/firmware/ti-ce-fw/${TARGET}
> >>>>   }
> >>>>   ALTERNATIVE:${PN} = "dra7-ipu2-fw.xem4"
> >>>>   ALTERNATIVE_LINK_NAME[dra7-ipu2-fw.xem4] = "${nonarch_base_libdir}/firmware/${TARGET}"
> >>>>-ALTERNATIVE_TARGET[dra7-ipu2-fw.xem4] = "${nonarch_base_libdir}/firmware/${TARGET}.${BPN}"
> >>>>+ALTERNATIVE_TARGET[dra7-ipu2-fw.xem4] = "${nonarch_base_libdir}/firmware/ti-ce-fw/${TARGET}"
> >>>>   ALTERNATIVE_PRIORITY = "20"
> >>>>-FILES:${PN} += "${nonarch_base_libdir}/firmware/*"
> >>>>-
> >>>>   # Disable the "buildpaths" check while we figure out how we are
> >>>>   # going to address this issue.
> >>>>   #
> >>>
> >>


^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2026-06-24 20:48 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-06-24 19:55 [meta-ti][master/wrynose][PATCH v2 1/3] libdce: Runtime depend on IPU firmware not the other way around Andrew Davis
2026-06-24 19:55 ` [meta-ti][master/wrynose][PATCH v2 2/3] ipumm-fw: Use prebuilt firmware from ti-linux-firmware repo Andrew Davis
2026-06-24 20:08   ` Ryan Eatmon
2026-06-24 20:19     ` Andrew Davis
2026-06-24 20:29       ` Denys Dmytriyenko
2026-06-24 20:42         ` Andrew Davis
2026-06-24 20:48           ` Denys Dmytriyenko
2026-06-24 19:55 ` [meta-ti][master/wrynose][PATCH v2 3/3] ti-codec-engine: Remove this recipe Andrew Davis
2026-06-24 20:00 ` [meta-ti][master/wrynose][PATCH v2 1/3] libdce: Runtime depend on IPU firmware not the other way around PRC Automation

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.