| Message ID | 20250618100143.1955550-1-p-bhagat@ti.com |
|---|---|
| State | Superseded |
| Delegated to: | Ryan Eatmon |
| Headers | show |
| Series | [scarthgap,v2] recipes-bsp: Add TI TAD5212 DAC firmware recipe | expand |
On 18/06/25 15:31, 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> > --- > Boot logs > https://gist.github.com/paresh-bhagat12/38ad3188e2da2546627341cd9416ccad > > Change log: > V1 -> V2: > - Updated names of firmware in recipe > > 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..68b17aad > --- /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" Redefining LICENSE & LIC_FILES_CHKSUM seems redundant here. It would have inherited it from [0] which directly picks the LICENSE.ti from ti-linux-firmware I believe Denys had recommended the same at [1] [0]: https://git.yoctoproject.org/meta-ti/tree/meta-ti-bsp/recipes-bsp/ti-linux-fw/ti-linux-fw.inc?h=11.01.02 [1]: https://lists.yoctoproject.org/g/meta-ti/message/18648 -- Chirag > + > +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}"
On 6/18/25 5:01 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> > --- > Boot logs > https://gist.github.com/paresh-bhagat12/38ad3188e2da2546627341cd9416ccad > > Change log: > V1 -> V2: > - Updated names of firmware in recipe > > 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" This firmware is specific to a DAC found on the EVM, not all AM62D based boards will need this, above should go in the EVM config (am62pxx-evm.conf). Andrew > 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..68b17aad > --- /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" > + > +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}"
On 6/18/2025 10:37 AM, Andrew Davis wrote: > On 6/18/25 5:01 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> >> --- >> Boot logs >> https://gist.github.com/paresh-bhagat12/38ad3188e2da2546627341cd9416ccad >> >> Change log: >> V1 -> V2: >> - Updated names of firmware in recipe >> >> 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" > > This firmware is specific to a DAC found on the EVM, not all AM62D based > boards will need this, above should go in the EVM config > (am62pxx-evm.conf). Good catch. Looks like we need a v3. And I think you meant am62dxx-evm.conf > Andrew > >> 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..68b17aad >> --- /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" >> + >> +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}"
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..68b17aad --- /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" + +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}"