On 11/18/2021 2:05 PM, Denys Dmytriyenko wrote:
On Thu, Nov 18, 2021 at 07:57:50PM +0000, Yogesh Siraswar via 
lists.yoctoproject.org wrote:
From: Praneeth Bajjuri <[email protected]>

Add the firmware packaging recipe for the video decoder
for j721e.

Signed-off-by: Praneeth Bajjuri <[email protected]>
Signed-off-by: Yogesh Siraswar <[email protected]>
---

resend v3 as v4

Changes from v2:
1) Fixed the summary
2) Bumped the ti-linux-fw to pick the video decoder binary
3) Added the kernel dependency to build this recipe

Thanks for fixing the recipe.



  .../ti-img-encode-decode/vxd-dec-fw_git.bb    | 24 +++++++++++++++++++
  recipes-bsp/ti-linux-fw/ti-linux-fw.inc       |  5 ++--
  recipes-kernel/linux/kernel-rdepends.inc      |  3 +++
  3 files changed, 30 insertions(+), 2 deletions(-)
  create mode 100644 recipes-bsp/ti-img-encode-decode/vxd-dec-fw_git.bb

diff --git a/recipes-bsp/ti-img-encode-decode/vxd-dec-fw_git.bb 
b/recipes-bsp/ti-img-encode-decode/vxd-dec-fw_git.bb
new file mode 100644
index 00000000..cccda6bb
--- /dev/null
+++ b/recipes-bsp/ti-img-encode-decode/vxd-dec-fw_git.bb
@@ -0,0 +1,24 @@
+SUMMARY = "Video Decoding Firmware"
+LICENSE = "TI-IMG"

And meta-ti does not define such license:
https://git.yoctoproject.org/cgit/cgit.cgi/meta-ti/tree/licenses

This is based on similar patch that we have in thud, lets use the same license. Ref: https://git.yoctoproject.org/cgit/cgit.cgi/meta-ti/tree/recipes-bsp/ti-img-encode-decode/vxd-dec-fw_git.bb?h=thud&id=5d174a34a9d468fa11505c3b8d0ad9eda5fd0ba0



+LIC_FILES_CHKSUM = "file://LICENSE.ti-img;md5=84ca7278930db001870686ad997d6bb1"
+
+require recipes-bsp/ti-linux-fw/ti-linux-fw.inc
+
+PV = "${IMG_DEC_FW_VERSION}"
+PR = "${INC_PR}.0"
+
+CLEANBROKEN = "1"
+
+COMPATIBLE_MACHINE = "j7"
+
+PACKAGE_ARCH = "${MACHINE_ARCH}"
+
+S = "${WORKDIR}/git"
+TARGET = "pvdec_full_bin.fw"
+
+do_install() {
+        install -d ${D}${base_libdir}/firmware/ti-img
+        install -m 0644 ${S}/ti-img/${TARGET} 
${D}${base_libdir}/firmware/ti-img/${TARGET}
+}
+
+FILES_${PN} = "${base_libdir}/firmware"
diff --git a/recipes-bsp/ti-linux-fw/ti-linux-fw.inc 
b/recipes-bsp/ti-linux-fw/ti-linux-fw.inc
index c59120b8..f6d5b8d1 100644
--- a/recipes-bsp/ti-linux-fw/ti-linux-fw.inc
+++ b/recipes-bsp/ti-linux-fw/ti-linux-fw.inc
@@ -13,11 +13,12 @@ NETCP_SA_FW_VERSION = "1.0.0"
  NETCP_PA_FW_VERSION = "3.0.2.3"
  QMSS_PDSP_FW_VERSION = "1.0.0.9"
  PRUETH_FW_AM65X_VERSION = "08.00.00.20"
-PRUETH_FW_AM65X_SR2_VERSION = "02.02.09.06"
+PRUETH_FW_AM65X_SR2_VERSION = "02.02.09.07"
  GOODIX_FW_VERSION = "1.0.0.0"
  CADENCE_MHDP_FW_VERSION = "1.2.17"
+IMG_DEC_FW_VERSION = "1.0"
-TI_LINUX_FW_SRCREV ?= "95232a584175a8ac872adf477b2dbb9ca68ba063"
+TI_LINUX_FW_SRCREV ?= "c96a734a059889e571dcf30c0638157c690a6e6e"
  SRCREV = "${TI_LINUX_FW_SRCREV}"
BRANCH ?= "ti-linux-firmware"
diff --git a/recipes-kernel/linux/kernel-rdepends.inc 
b/recipes-kernel/linux/kernel-rdepends.inc
index 5e5241c9..76b7a508 100644
--- a/recipes-kernel/linux/kernel-rdepends.inc
+++ b/recipes-kernel/linux/kernel-rdepends.inc
@@ -39,3 +39,6 @@ RDEPENDS_${KERNEL_PACKAGE_NAME}-base_append_am64xx-evm = " 
prueth-fw-am65x-sr2"
# Add run-time dependency for Cadence MHDP firmware to the rootfs
  RDEPENDS_${KERNEL_PACKAGE_NAME}-base_append_j7 = " cadence-mhdp-fw"
+
+# Add run-time dependency for Video Decoding firmware to the rootfs
+RDEPENDS_${KERNEL_PACKAGE_NAME}-base_append_j7 = " vxd-dec-fw"
--
2.17.1

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#14171): 
https://lists.yoctoproject.org/g/meta-ti/message/14171
Mute This Topic: https://lists.yoctoproject.org/mt/87152165/21656
Group Owner: [email protected]
Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to