| Message ID | 20220504213844.20764-1-reatmon@ti.com |
|---|---|
| State | Accepted |
| Delegated to: | Ryan Eatmon |
| Headers | show |
| Series | [meta-ti,dunfell] recipes-bsp: tifs-lpm-stub: Introduce new recipe | expand |
On Wed, May 04, 2022 at 04:38:44PM -0500, Ryan Eatmon via lists.yoctoproject.org wrote: > This firmware stub is required by the TI foundational security > software while transitioning to low power mode. > > Add a recipe for this compatible with am62xx-evm Is it GP only FW? > Signed-off-by: Nikhil Devshatwar <nikhil.nd@ti.com> > Signed-off-by: Ryan Eatmon <reatmon@ti.com> > --- > recipes-bsp/ti-linux-fw/ti-linux-fw.inc | 1 + > .../tifs-lpm-stub/tifs-lpm-stub_git.bb | 24 +++++++++++++++++++ > recipes-kernel/linux/kernel-rdepends.inc | 4 ++++ > 3 files changed, 29 insertions(+) > create mode 100644 recipes-bsp/tifs-lpm-stub/tifs-lpm-stub_git.bb > > diff --git a/recipes-bsp/ti-linux-fw/ti-linux-fw.inc b/recipes-bsp/ti-linux-fw/ti-linux-fw.inc > index 3ebdae30..47e76347 100644 > --- a/recipes-bsp/ti-linux-fw/ti-linux-fw.inc > +++ b/recipes-bsp/ti-linux-fw/ti-linux-fw.inc > @@ -18,6 +18,7 @@ GOODIX_FW_VERSION = "1.0.0.0" > CADENCE_MHDP_FW_VERSION = "1.2.17" > IMG_DEC_FW_VERSION = "1.0" > CNM_WAVE521_FW_VERSION = "1.0.00" > +TIFS_LPM_STUB_FW_VERSION = "08.03.02" > > TI_LINUX_FW_SRCREV ?= "917625c2721eef8677df9e47d919005b0d4f93aa" > SRCREV = "${TI_LINUX_FW_SRCREV}" > diff --git a/recipes-bsp/tifs-lpm-stub/tifs-lpm-stub_git.bb b/recipes-bsp/tifs-lpm-stub/tifs-lpm-stub_git.bb > new file mode 100644 > index 00000000..642a8d3a > --- /dev/null > +++ b/recipes-bsp/tifs-lpm-stub/tifs-lpm-stub_git.bb > @@ -0,0 +1,24 @@ > +SUMMARY = "TI Foundational security Low Power Management Stub" > + > +require recipes-bsp/ti-linux-fw/ti-linux-fw.inc > + > +PV = "${TIFS_LPM_STUB_FW_VERSION}" > +PR = "${INC_PR}.0" > + > +CLEANBROKEN = "1" > + > +COMPATIBLE_MACHINE = "am62xx-evm" > + > +PACKAGE_ARCH = "${MACHINE_ARCH}" > + > +S = "${WORKDIR}/git" > + > +TARGET = "ti-fs-stub-firmware-am62x-gp-signed.bin" > + > +do_install() { > + install -d ${D}${base_libdir}/firmware/ti-sysfw/ > + install -m 0644 ${S}/ti-sysfw/${TARGET} ${D}${base_libdir}/firmware/ti-sysfw/${TARGET} > +} > + > +FILES_${PN} = "${base_libdir}/firmware" > + > diff --git a/recipes-kernel/linux/kernel-rdepends.inc b/recipes-kernel/linux/kernel-rdepends.inc > index 522d9674..17aee4c2 100644 > --- a/recipes-kernel/linux/kernel-rdepends.inc > +++ b/recipes-kernel/linux/kernel-rdepends.inc > @@ -45,3 +45,7 @@ RDEPENDS_${KERNEL_PACKAGE_NAME}-base_append_j7 = " vxd-dec-fw" > > # Add run-time dependency for Chips&Media Wave521 firmware to the rootfs > RDEPENDS_${KERNEL_PACKAGE_NAME}-base_append_j721s2-evm = " cnm-wave-fw" > + > +# Add run-time dependency for TIFS Low Power Module stub > +RDEPENDS_${KERNEL_PACKAGE_NAME}-base_append_am62xx-evm = " tifs-lpm-stub" > + > -- > 2.17.1
On 5/4/2022 16:43, Denys Dmytriyenko wrote: > On Wed, May 04, 2022 at 04:38:44PM -0500, Ryan Eatmon via lists.yoctoproject.org wrote: >> This firmware stub is required by the TI foundational security >> software while transitioning to low power mode. >> >> Add a recipe for this compatible with am62xx-evm > > Is it GP only FW? Not sure how to answer that. I was just told to pick up this change for the release and I cleaned it up to match a more consistent style. > >> Signed-off-by: Nikhil Devshatwar <nikhil.nd@ti.com> >> Signed-off-by: Ryan Eatmon <reatmon@ti.com> >> --- >> recipes-bsp/ti-linux-fw/ti-linux-fw.inc | 1 + >> .../tifs-lpm-stub/tifs-lpm-stub_git.bb | 24 +++++++++++++++++++ >> recipes-kernel/linux/kernel-rdepends.inc | 4 ++++ >> 3 files changed, 29 insertions(+) >> create mode 100644 recipes-bsp/tifs-lpm-stub/tifs-lpm-stub_git.bb >> >> diff --git a/recipes-bsp/ti-linux-fw/ti-linux-fw.inc b/recipes-bsp/ti-linux-fw/ti-linux-fw.inc >> index 3ebdae30..47e76347 100644 >> --- a/recipes-bsp/ti-linux-fw/ti-linux-fw.inc >> +++ b/recipes-bsp/ti-linux-fw/ti-linux-fw.inc >> @@ -18,6 +18,7 @@ GOODIX_FW_VERSION = "1.0.0.0" >> CADENCE_MHDP_FW_VERSION = "1.2.17" >> IMG_DEC_FW_VERSION = "1.0" >> CNM_WAVE521_FW_VERSION = "1.0.00" >> +TIFS_LPM_STUB_FW_VERSION = "08.03.02" >> >> TI_LINUX_FW_SRCREV ?= "917625c2721eef8677df9e47d919005b0d4f93aa" >> SRCREV = "${TI_LINUX_FW_SRCREV}" >> diff --git a/recipes-bsp/tifs-lpm-stub/tifs-lpm-stub_git.bb b/recipes-bsp/tifs-lpm-stub/tifs-lpm-stub_git.bb >> new file mode 100644 >> index 00000000..642a8d3a >> --- /dev/null >> +++ b/recipes-bsp/tifs-lpm-stub/tifs-lpm-stub_git.bb >> @@ -0,0 +1,24 @@ >> +SUMMARY = "TI Foundational security Low Power Management Stub" >> + >> +require recipes-bsp/ti-linux-fw/ti-linux-fw.inc >> + >> +PV = "${TIFS_LPM_STUB_FW_VERSION}" >> +PR = "${INC_PR}.0" >> + >> +CLEANBROKEN = "1" >> + >> +COMPATIBLE_MACHINE = "am62xx-evm" >> + >> +PACKAGE_ARCH = "${MACHINE_ARCH}" >> + >> +S = "${WORKDIR}/git" >> + >> +TARGET = "ti-fs-stub-firmware-am62x-gp-signed.bin" >> + >> +do_install() { >> + install -d ${D}${base_libdir}/firmware/ti-sysfw/ >> + install -m 0644 ${S}/ti-sysfw/${TARGET} ${D}${base_libdir}/firmware/ti-sysfw/${TARGET} >> +} >> + >> +FILES_${PN} = "${base_libdir}/firmware" >> + >> diff --git a/recipes-kernel/linux/kernel-rdepends.inc b/recipes-kernel/linux/kernel-rdepends.inc >> index 522d9674..17aee4c2 100644 >> --- a/recipes-kernel/linux/kernel-rdepends.inc >> +++ b/recipes-kernel/linux/kernel-rdepends.inc >> @@ -45,3 +45,7 @@ RDEPENDS_${KERNEL_PACKAGE_NAME}-base_append_j7 = " vxd-dec-fw" >> >> # Add run-time dependency for Chips&Media Wave521 firmware to the rootfs >> RDEPENDS_${KERNEL_PACKAGE_NAME}-base_append_j721s2-evm = " cnm-wave-fw" >> + >> +# Add run-time dependency for TIFS Low Power Module stub >> +RDEPENDS_${KERNEL_PACKAGE_NAME}-base_append_am62xx-evm = " tifs-lpm-stub" >> + >> -- >> 2.17.1
On Wed, May 04, 2022 at 04:49:23PM -0500, Ryan Eatmon wrote: > > > On 5/4/2022 16:43, Denys Dmytriyenko wrote: > >On Wed, May 04, 2022 at 04:38:44PM -0500, Ryan Eatmon via lists.yoctoproject.org wrote: > >>This firmware stub is required by the TI foundational security > >>software while transitioning to low power mode. > >> > >>Add a recipe for this compatible with am62xx-evm > > > >Is it GP only FW? > > Not sure how to answer that. I was just told to pick up this change > for the release and I cleaned it up to match a more consistent > style. Adding Nikhil to direct recipients. The reason I'm asking - the change is specific to am62xx-evm and not a more generic am62xx family (GP, HS, custom(er) boards). Plus the FW name is "ti-fs-stub-firmware-am62x-gp-signed.bin" with GP in the name, hence the question. > >>Signed-off-by: Nikhil Devshatwar <nikhil.nd@ti.com> > >>Signed-off-by: Ryan Eatmon <reatmon@ti.com> > >>--- > >> recipes-bsp/ti-linux-fw/ti-linux-fw.inc | 1 + > >> .../tifs-lpm-stub/tifs-lpm-stub_git.bb | 24 +++++++++++++++++++ > >> recipes-kernel/linux/kernel-rdepends.inc | 4 ++++ > >> 3 files changed, 29 insertions(+) > >> create mode 100644 recipes-bsp/tifs-lpm-stub/tifs-lpm-stub_git.bb > >> > >>diff --git a/recipes-bsp/ti-linux-fw/ti-linux-fw.inc b/recipes-bsp/ti-linux-fw/ti-linux-fw.inc > >>index 3ebdae30..47e76347 100644 > >>--- a/recipes-bsp/ti-linux-fw/ti-linux-fw.inc > >>+++ b/recipes-bsp/ti-linux-fw/ti-linux-fw.inc > >>@@ -18,6 +18,7 @@ GOODIX_FW_VERSION = "1.0.0.0" > >> CADENCE_MHDP_FW_VERSION = "1.2.17" > >> IMG_DEC_FW_VERSION = "1.0" > >> CNM_WAVE521_FW_VERSION = "1.0.00" > >>+TIFS_LPM_STUB_FW_VERSION = "08.03.02" > >> TI_LINUX_FW_SRCREV ?= "917625c2721eef8677df9e47d919005b0d4f93aa" > >> SRCREV = "${TI_LINUX_FW_SRCREV}" > >>diff --git a/recipes-bsp/tifs-lpm-stub/tifs-lpm-stub_git.bb b/recipes-bsp/tifs-lpm-stub/tifs-lpm-stub_git.bb > >>new file mode 100644 > >>index 00000000..642a8d3a > >>--- /dev/null > >>+++ b/recipes-bsp/tifs-lpm-stub/tifs-lpm-stub_git.bb > >>@@ -0,0 +1,24 @@ > >>+SUMMARY = "TI Foundational security Low Power Management Stub" > >>+ > >>+require recipes-bsp/ti-linux-fw/ti-linux-fw.inc > >>+ > >>+PV = "${TIFS_LPM_STUB_FW_VERSION}" > >>+PR = "${INC_PR}.0" > >>+ > >>+CLEANBROKEN = "1" > >>+ > >>+COMPATIBLE_MACHINE = "am62xx-evm" > >>+ > >>+PACKAGE_ARCH = "${MACHINE_ARCH}" > >>+ > >>+S = "${WORKDIR}/git" > >>+ > >>+TARGET = "ti-fs-stub-firmware-am62x-gp-signed.bin" > >>+ > >>+do_install() { > >>+ install -d ${D}${base_libdir}/firmware/ti-sysfw/ > >>+ install -m 0644 ${S}/ti-sysfw/${TARGET} ${D}${base_libdir}/firmware/ti-sysfw/${TARGET} > >>+} > >>+ > >>+FILES_${PN} = "${base_libdir}/firmware" > >>+ > >>diff --git a/recipes-kernel/linux/kernel-rdepends.inc b/recipes-kernel/linux/kernel-rdepends.inc > >>index 522d9674..17aee4c2 100644 > >>--- a/recipes-kernel/linux/kernel-rdepends.inc > >>+++ b/recipes-kernel/linux/kernel-rdepends.inc > >>@@ -45,3 +45,7 @@ RDEPENDS_${KERNEL_PACKAGE_NAME}-base_append_j7 = " vxd-dec-fw" > >> # Add run-time dependency for Chips&Media Wave521 firmware to the rootfs > >> RDEPENDS_${KERNEL_PACKAGE_NAME}-base_append_j721s2-evm = " cnm-wave-fw" > >>+ > >>+# Add run-time dependency for TIFS Low Power Module stub > >>+RDEPENDS_${KERNEL_PACKAGE_NAME}-base_append_am62xx-evm = " tifs-lpm-stub" > >>+ > >>-- > >>2.17.1
On 5/4/2022 4:49 PM, Ryan Eatmon wrote: > > > On 5/4/2022 16:43, Denys Dmytriyenko wrote: >> On Wed, May 04, 2022 at 04:38:44PM -0500, Ryan Eatmon via >> lists.yoctoproject.org wrote: >>> This firmware stub is required by the TI foundational security >>> software while transitioning to low power mode. >>> >>> Add a recipe for this compatible with am62xx-evm >> >> Is it GP only FW? > > Not sure how to answer that. I was just told to pick up this change for > the release and I cleaned it up to match a more consistent style. Yes, this signed firmware is applicable for GP only. We dont have a HS variant yet. > >> >>> Signed-off-by: Nikhil Devshatwar <nikhil.nd@ti.com> >>> Signed-off-by: Ryan Eatmon <reatmon@ti.com> >>> --- >>> recipes-bsp/ti-linux-fw/ti-linux-fw.inc | 1 + >>> .../tifs-lpm-stub/tifs-lpm-stub_git.bb | 24 +++++++++++++++++++ >>> recipes-kernel/linux/kernel-rdepends.inc | 4 ++++ >>> 3 files changed, 29 insertions(+) >>> create mode 100644 recipes-bsp/tifs-lpm-stub/tifs-lpm-stub_git.bb >>> >>> diff --git a/recipes-bsp/ti-linux-fw/ti-linux-fw.inc >>> b/recipes-bsp/ti-linux-fw/ti-linux-fw.inc >>> index 3ebdae30..47e76347 100644 >>> --- a/recipes-bsp/ti-linux-fw/ti-linux-fw.inc >>> +++ b/recipes-bsp/ti-linux-fw/ti-linux-fw.inc >>> @@ -18,6 +18,7 @@ GOODIX_FW_VERSION = "1.0.0.0" >>> CADENCE_MHDP_FW_VERSION = "1.2.17" >>> IMG_DEC_FW_VERSION = "1.0" >>> CNM_WAVE521_FW_VERSION = "1.0.00" >>> +TIFS_LPM_STUB_FW_VERSION = "08.03.02" >>> TI_LINUX_FW_SRCREV ?= "917625c2721eef8677df9e47d919005b0d4f93aa" >>> SRCREV = "${TI_LINUX_FW_SRCREV}" >>> diff --git a/recipes-bsp/tifs-lpm-stub/tifs-lpm-stub_git.bb >>> b/recipes-bsp/tifs-lpm-stub/tifs-lpm-stub_git.bb >>> new file mode 100644 >>> index 00000000..642a8d3a >>> --- /dev/null >>> +++ b/recipes-bsp/tifs-lpm-stub/tifs-lpm-stub_git.bb >>> @@ -0,0 +1,24 @@ >>> +SUMMARY = "TI Foundational security Low Power Management Stub" >>> + >>> +require recipes-bsp/ti-linux-fw/ti-linux-fw.inc >>> + >>> +PV = "${TIFS_LPM_STUB_FW_VERSION}" >>> +PR = "${INC_PR}.0" >>> + >>> +CLEANBROKEN = "1" >>> + >>> +COMPATIBLE_MACHINE = "am62xx-evm" >>> + >>> +PACKAGE_ARCH = "${MACHINE_ARCH}" >>> + >>> +S = "${WORKDIR}/git" >>> + >>> +TARGET = "ti-fs-stub-firmware-am62x-gp-signed.bin" >>> + >>> +do_install() { >>> + install -d ${D}${base_libdir}/firmware/ti-sysfw/ >>> + install -m 0644 ${S}/ti-sysfw/${TARGET} >>> ${D}${base_libdir}/firmware/ti-sysfw/${TARGET} >>> +} >>> + >>> +FILES_${PN} = "${base_libdir}/firmware" >>> + >>> diff --git a/recipes-kernel/linux/kernel-rdepends.inc >>> b/recipes-kernel/linux/kernel-rdepends.inc >>> index 522d9674..17aee4c2 100644 >>> --- a/recipes-kernel/linux/kernel-rdepends.inc >>> +++ b/recipes-kernel/linux/kernel-rdepends.inc >>> @@ -45,3 +45,7 @@ RDEPENDS_${KERNEL_PACKAGE_NAME}-base_append_j7 = " >>> vxd-dec-fw" >>> # Add run-time dependency for Chips&Media Wave521 firmware to the >>> rootfs >>> RDEPENDS_${KERNEL_PACKAGE_NAME}-base_append_j721s2-evm = " >>> cnm-wave-fw" >>> + >>> +# Add run-time dependency for TIFS Low Power Module stub >>> +RDEPENDS_${KERNEL_PACKAGE_NAME}-base_append_am62xx-evm = " >>> tifs-lpm-stub" >>> + >>> -- >>> 2.17.1 >
diff --git a/recipes-bsp/ti-linux-fw/ti-linux-fw.inc b/recipes-bsp/ti-linux-fw/ti-linux-fw.inc index 3ebdae30..47e76347 100644 --- a/recipes-bsp/ti-linux-fw/ti-linux-fw.inc +++ b/recipes-bsp/ti-linux-fw/ti-linux-fw.inc @@ -18,6 +18,7 @@ GOODIX_FW_VERSION = "1.0.0.0" CADENCE_MHDP_FW_VERSION = "1.2.17" IMG_DEC_FW_VERSION = "1.0" CNM_WAVE521_FW_VERSION = "1.0.00" +TIFS_LPM_STUB_FW_VERSION = "08.03.02" TI_LINUX_FW_SRCREV ?= "917625c2721eef8677df9e47d919005b0d4f93aa" SRCREV = "${TI_LINUX_FW_SRCREV}" diff --git a/recipes-bsp/tifs-lpm-stub/tifs-lpm-stub_git.bb b/recipes-bsp/tifs-lpm-stub/tifs-lpm-stub_git.bb new file mode 100644 index 00000000..642a8d3a --- /dev/null +++ b/recipes-bsp/tifs-lpm-stub/tifs-lpm-stub_git.bb @@ -0,0 +1,24 @@ +SUMMARY = "TI Foundational security Low Power Management Stub" + +require recipes-bsp/ti-linux-fw/ti-linux-fw.inc + +PV = "${TIFS_LPM_STUB_FW_VERSION}" +PR = "${INC_PR}.0" + +CLEANBROKEN = "1" + +COMPATIBLE_MACHINE = "am62xx-evm" + +PACKAGE_ARCH = "${MACHINE_ARCH}" + +S = "${WORKDIR}/git" + +TARGET = "ti-fs-stub-firmware-am62x-gp-signed.bin" + +do_install() { + install -d ${D}${base_libdir}/firmware/ti-sysfw/ + install -m 0644 ${S}/ti-sysfw/${TARGET} ${D}${base_libdir}/firmware/ti-sysfw/${TARGET} +} + +FILES_${PN} = "${base_libdir}/firmware" + diff --git a/recipes-kernel/linux/kernel-rdepends.inc b/recipes-kernel/linux/kernel-rdepends.inc index 522d9674..17aee4c2 100644 --- a/recipes-kernel/linux/kernel-rdepends.inc +++ b/recipes-kernel/linux/kernel-rdepends.inc @@ -45,3 +45,7 @@ RDEPENDS_${KERNEL_PACKAGE_NAME}-base_append_j7 = " vxd-dec-fw" # Add run-time dependency for Chips&Media Wave521 firmware to the rootfs RDEPENDS_${KERNEL_PACKAGE_NAME}-base_append_j721s2-evm = " cnm-wave-fw" + +# Add run-time dependency for TIFS Low Power Module stub +RDEPENDS_${KERNEL_PACKAGE_NAME}-base_append_am62xx-evm = " tifs-lpm-stub" +