Re: [scarthgap][PATCH] recipes-bsp: Add TI TAD5212 DAC firmware recipe

[Denys Dmytriyenko <denis@denix.org>]

On Tue, Jun 17, 2025 at 12:10:37PM -0500, Bajjuri, Praneeth wrote:
> 
> 
> On 6/17/2025 5:19 AM, Paresh Bhagat wrote:
> >From: Vishnu Singh <v-singh1@ti.com>
> >
> >Add TAD5212 DAC firmware images for am62dxx-evm to support two DAC
> >configurations:
> >- TAD5212_01.bin: Configures 1 DAC with stereo channel in i2s mode.
> >- TAD5212_04.bin: Configures 4 DAC with each having stereo channel
> >		  support in i2s mode.
> >
> >Signed-off-by: Vishnu Singh <v-singh1@ti.com>
> >Signed-off-by: Paresh Bhagat <p-bhagat@ti.com>
> >---
> >
> >This patch depends on new DAC firmware patch on ti-linux-firmware.
> >
> >Boot logs
> >https://gist.github.com/paresh-bhagat12/e4e2a7a1291d0f2d7acd0971bf2ed539
> >
> >
> >  meta-ti-bsp/conf/machine/include/am62dxx.inc  |  2 ++
> >  .../recipes-bsp/ti-dac-fw/ti-dac-fw.bb        | 20 +++++++++++++++++++
> >  .../recipes-bsp/ti-linux-fw/ti-linux-fw.inc   |  1 +
> >  3 files changed, 23 insertions(+)
> >  create mode 100644 meta-ti-bsp/recipes-bsp/ti-dac-fw/ti-dac-fw.bb
> >
> >diff --git a/meta-ti-bsp/conf/machine/include/am62dxx.inc b/meta-ti-bsp/conf/machine/include/am62dxx.inc
> >index ac4e8e17..3c60acb1 100644
> >--- a/meta-ti-bsp/conf/machine/include/am62dxx.inc
> >+++ b/meta-ti-bsp/conf/machine/include/am62dxx.inc
> >@@ -13,3 +13,5 @@ IMAGE_BOOT_FILES += "tiboot3-am62dx-hs-evm.bin"
> >  TFA_BOARD = "lite"
> >  OPTEEMACHINE = "k3-am62x"
> >+
> >+MACHINE_ESSENTIAL_EXTRA_RRECOMMENDS += "ti-dac-fw"
> >diff --git a/meta-ti-bsp/recipes-bsp/ti-dac-fw/ti-dac-fw.bb b/meta-ti-bsp/recipes-bsp/ti-dac-fw/ti-dac-fw.bb
> >new file mode 100644
> >index 00000000..7d333b9e
> >--- /dev/null
> >+++ b/meta-ti-bsp/recipes-bsp/ti-dac-fw/ti-dac-fw.bb
> >@@ -0,0 +1,20 @@
> >+DESCRIPTION = "TI DAC TAD5212 config prebuild binary firmware"
> >+
> >+require recipes-bsp/ti-linux-fw/ti-linux-fw.inc
> >+
> >+LICENSE = "TI-TFL"
> >+LIC_FILES_CHKSUM = "file://${THISDIR}/../../licenses/TI-TFL;md5=a1b59cb7ba626b9dbbcbf00f3fbc438a"
> 
> not the correct license file.
> update to match https://git.ti.com/cgit/processor-firmware/ti-linux-firmware/commit/?h=ti-linux-firmware-next&id=ef1e9d0dd2c3ba0af102cea0edb7044a12c8fb5b
> 
> TI-TSPA

Praneeth,

TI-TSPA was replaced by TI-TFL back in 2014.

And LICENSE.ti in ti-linux-firmware repo seems to be the TI-TFL license:
https://git.ti.com/cgit/processor-firmware/ti-linux-firmware/tree/LICENSE.ti?h=ti-linux-firmware-next&id=ef1e9d0dd2c3ba0af102cea0edb7044a12c8fb5b

Which matches the TI-TFL license in meta-ti:
https://git.yoctoproject.org/meta-ti/tree/meta-ti-bsp/licenses/TI-TFL

For the reference, here's the old TI-TSPA license:
https://git.yoctoproject.org/meta-ti/tree/meta-ti-bsp/licenses/TI-TSPA

And here's the commit that refers to deprecating TI-TSPA in favor of TI-TFL:
https://git.yoctoproject.org/meta-ti/commit/?id=0442fa03bc07bd70376522c290c1af254f7e77d4


> >+
> >+PV = "${TAD5212_FW_VERSION}"
> >+PR = "${INC_PR}.0"
> >+
> >+COMPATIBLE_MACHINE = "am62dxx"
> >+
> >+TAD5212_1DEV = "TAD5212_01.bin"
> >+TAD5212_4DEV = "TAD5212_04.bin"
> >+
> >+do_install() {
> >+	install -d ${D}${nonarch_base_libdir}/firmware
> >+	install -m 0644 ${S}/ti/tad5212/${TAD5212_1DEV} ${D}${nonarch_base_libdir}/firmware/
> >+	install -m 0644 ${S}/ti/tad5212/${TAD5212_4DEV} ${D}${nonarch_base_libdir}/firmware/
> >+}
> >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 d1dd9df3..08cd0b30 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
> >@@ -17,6 +17,7 @@ CNM_WAVE521_FW_VERSION = "1.0.8"
> >  TI_DM_FW_VERSION = "11.00.09"
> >  TI_SYSFW_VERSION = "11.01.00"
> >  TI_HSM_DEMO_FW_VERSION = "11.00.09"
> >+TAD5212_FW_VERSION = "1.0.0.0"
> >  TI_LINUX_FW_SRCREV ?= "a37c82f0e25e0e3c99fdf03f1d57f7b08314a2bc"
> >  SRCREV = "${TI_LINUX_FW_SRCREV}"