diff mbox series

[meta-ti,master/wrynose,v2,2/3] ipumm-fw: Use prebuilt firmware from ti-linux-firmware repo

Message ID 20260624195533.1152173-2-afd@ti.com
State New
Headers show
Series [meta-ti,master/wrynose,v2,1/3] libdce: Runtime depend on IPU firmware not the other way around | expand

Commit Message

Andrew Davis June 24, 2026, 7:55 p.m. UTC
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(-)

Comments

Ryan Eatmon June 24, 2026, 8:08 p.m. UTC | #1
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.
>   #
Andrew Davis June 24, 2026, 8:19 p.m. UTC | #2
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.
>>   #
>
Denys Dmytriyenko June 24, 2026, 8:29 p.m. UTC | #3
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.
> >>  #
> >
>
Andrew Davis June 24, 2026, 8:42 p.m. UTC | #4
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.
>>>>    #
>>>
>>
Denys Dmytriyenko June 24, 2026, 8:48 p.m. UTC | #5
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.
> >>>>   #
> >>>
> >>
diff mbox series

Patch

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.
 #