| Message ID | 20250618170217.2577275-1-p-bhagat@ti.com |
|---|---|
| State | Under Review |
| Delegated to: | Ryan Eatmon |
| Headers | show |
| Series | [scarthgap,v3] recipes-bsp: Add TI TAD5212 DAC firmware recipe | expand |
On 6/18/25 12:02 PM, 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: > V2 -> V3: > - Removed redundant info in recipe(licesne and checksum) > - Package firmware only for am62dxx-evm > > V1 -> V2: > - Updated names of firmware in recipe > > meta-ti-bsp/conf/machine/am62dxx-evm.conf | 2 ++ > meta-ti-bsp/recipes-bsp/ti-dac-fw/ti-dac-fw.bb | 17 +++++++++++++++++ > .../recipes-bsp/ti-linux-fw/ti-linux-fw.inc | 1 + > 3 files changed, 20 insertions(+) > create mode 100644 meta-ti-bsp/recipes-bsp/ti-dac-fw/ti-dac-fw.bb > > diff --git a/meta-ti-bsp/conf/machine/am62dxx-evm.conf b/meta-ti-bsp/conf/machine/am62dxx-evm.conf > index 9fbbb90a..1e41c53c 100644 > --- a/meta-ti-bsp/conf/machine/am62dxx-evm.conf > +++ b/meta-ti-bsp/conf/machine/am62dxx-evm.conf > @@ -14,3 +14,5 @@ KERNEL_DEVICETREE = "" > FIT_CONF_DEFAULT_DTB = "ti/k3-am62d2-evm.dtb" > > UBOOT_MACHINE = "am62dx_evm_a53_defconfig" > + > +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..8f68009d > --- /dev/null > +++ b/meta-ti-bsp/recipes-bsp/ti-dac-fw/ti-dac-fw.bb > @@ -0,0 +1,17 @@ > +DESCRIPTION = "TI DAC TAD5212 config prebuild binary firmware" > + > +require recipes-bsp/ti-linux-fw/ti-linux-fw.inc > + > +PV = "${TAD5212_FW_VERSION}" > +PR = "${INC_PR}.0" > + > +COMPATIBLE_MACHINE = "am62dxx" > + > +TAD5212_1DEV = "tad5212_01.bin" > +TAD5212_4DEV = "tad5212_04.bin" You usually don't need to make variable definitions like this if you don't plan to vary the value and are only going to use the definition once, easier to just use the name directly. But that is just nitpicking, everything else LGTM, Reviewed-by: Andrew Davis <afd@ti.com> > + > +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/am62dxx-evm.conf b/meta-ti-bsp/conf/machine/am62dxx-evm.conf index 9fbbb90a..1e41c53c 100644 --- a/meta-ti-bsp/conf/machine/am62dxx-evm.conf +++ b/meta-ti-bsp/conf/machine/am62dxx-evm.conf @@ -14,3 +14,5 @@ KERNEL_DEVICETREE = "" FIT_CONF_DEFAULT_DTB = "ti/k3-am62d2-evm.dtb" UBOOT_MACHINE = "am62dx_evm_a53_defconfig" + +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..8f68009d --- /dev/null +++ b/meta-ti-bsp/recipes-bsp/ti-dac-fw/ti-dac-fw.bb @@ -0,0 +1,17 @@ +DESCRIPTION = "TI DAC TAD5212 config prebuild binary firmware" + +require recipes-bsp/ti-linux-fw/ti-linux-fw.inc + +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}"