Merge pull request #2575 from lucianogdittgen/oelint-cleanup-11-recipe-metadata-7

oelint: Recipe metadata, do_install and split-package FILES cleanups

11 files changed, 50 insertions, 23 deletions

diff --git a/recipes-bsp/firmware-imx/firmware-ele-imx_2.0.5.bb b/recipes-bsp/firmware-imx/firmware-ele-imx_2.0.5.bb
index 954c8385b..37f951adf 100644
--- a/recipes-bsp/firmware-imx/firmware-ele-imx_2.0.5.bb
+++ b/recipes-bsp/firmware-imx/firmware-ele-imx_2.0.5.bb
@@ -29,6 +29,8 @@ do_deploy () {
 }
 addtask deploy after do_install before do_build
 
+# Package the deployed ELE firmware blob explicitly (filesoverride).
+# nooelint: oelint.var.filesoverride
 FILES:${PN} = "${nonarch_base_libdir}/firmware"
 
 RREPLACES:${PN} = "firmware-sentinel"
diff --git a/recipes-bsp/firmware-imx/firmware-imx_8.31.bb b/recipes-bsp/firmware-imx/firmware-imx_8.31.bb
index 288fa13bc..623bac9e0 100644
--- a/recipes-bsp/firmware-imx/firmware-imx_8.31.bb
+++ b/recipes-bsp/firmware-imx/firmware-imx_8.31.bb
@@ -163,15 +163,15 @@ PACKAGES_DYNAMIC = "${PN}-vpu-* ${PN}-sdma-* ${PN}-easrc-* ${PN}-xcvr-* ${PN}-xu
 # files it makes no sense.
 PACKAGES = "${PN} ${PN}-epdc ${PN}-hdmi ${PN}-vpu-amphion ${PN}-vpu-coda980 ${PN}-vpu-wave511 ${PN}-vpu-wave"
 
-FILES:${PN}-epdc = "${nonarch_base_libdir}/firmware/imx/epdc/"
-FILES:${PN}-hdmi = "\
+FILES:${PN}-epdc += "${nonarch_base_libdir}/firmware/imx/epdc/"
+FILES:${PN}-hdmi += "\
     ${nonarch_base_libdir}/firmware/hdmitxfw.bin \
     ${nonarch_base_libdir}/firmware/hdmirxfw.bin \
     ${nonarch_base_libdir}/firmware/dpfw.bin \
 "
-FILES:${PN}-vpu-amphion = "${nonarch_base_libdir}/firmware/amphion/vpu/*"
-FILES:${PN}-vpu-coda980 = "${nonarch_base_libdir}/firmware/cnm/coda980_enc_fw.bin"
-FILES:${PN}-vpu-wave511 = "${nonarch_base_libdir}/firmware/cnm/wave511_dec_fw.bin"
-FILES:${PN}-vpu-wave = "${nonarch_base_libdir}/firmware/wave633c_codec_fw.bin"
+FILES:${PN}-vpu-amphion += "${nonarch_base_libdir}/firmware/amphion/vpu/*"
+FILES:${PN}-vpu-coda980 += "${nonarch_base_libdir}/firmware/cnm/coda980_enc_fw.bin"
+FILES:${PN}-vpu-wave511 += "${nonarch_base_libdir}/firmware/cnm/wave511_dec_fw.bin"
+FILES:${PN}-vpu-wave += "${nonarch_base_libdir}/firmware/wave633c_codec_fw.bin"
 
 COMPATIBLE_MACHINE = "(imx-generic-bsp)"
diff --git a/recipes-bsp/firmware-imx/firmware-nxp-wifi_1.1.bb b/recipes-bsp/firmware-imx/firmware-nxp-wifi_1.1.bb
index 2ef41529c..6043a1b9b 100644
--- a/recipes-bsp/firmware-imx/firmware-nxp-wifi_1.1.bb
+++ b/recipes-bsp/firmware-imx/firmware-nxp-wifi_1.1.bb
@@ -74,12 +74,15 @@ RDEPENDS:${PN}-all-usb = "\
     ${PN}-nxpiw610-usb \
 "
 
-FILES:${PN}-nxp-common = "\
+# The firmware is split into per-chip sub-packages; each block keeps its FILES
+# beside the matching R* vars for readability.
+# nooelint: oelint.var.order.FILES
+FILES:${PN}-nxp-common += "\
     ${nonarch_base_libdir}/firmware/nxp/wifi_mod_para.conf \
     ${nonarch_base_libdir}/firmware/nxp/helper_uart_3000000.bin \
 "
 
-FILES:${PN}-nxp8987-sdio = "\
+FILES:${PN}-nxp8987-sdio += "\
     ${nonarch_base_libdir}/firmware/nxp/*8987* \
 "
 RDEPENDS:${PN}-nxp8987-sdio += "${PN}-nxp-common"
@@ -87,11 +90,12 @@ RPROVIDES:${PN}-nxp8987-sdio = "linux-firmware-nxp8987-sdio"
 RREPLACES:${PN}-nxp8987-sdio = "linux-firmware-nxp8987-sdio"
 RCONFLICTS:${PN}-nxp8987-sdio = "linux-firmware-nxp8987-sdio"
 
-FILES:${PN}-nxp8997-sdio = "\
+# nooelint: oelint.var.order.FILES
+FILES:${PN}-nxp8997-sdio += "\
     ${nonarch_base_libdir}/firmware/mrvl/sdiouart8997_combo_v4.bin \
 "
 
-FILES:${PN}-nxp9098-common = "\
+FILES:${PN}-nxp9098-common += "\
     ${nonarch_base_libdir}/firmware/nxp/ed_mac_ctrl_V3_909x.conf \
     ${nonarch_base_libdir}/firmware/nxp/txpwrlimit_cfg_9098.conf \
     ${nonarch_base_libdir}/firmware/nxp/uart9098_bt_v1.bin \
@@ -101,7 +105,8 @@ RPROVIDES:${PN}-nxp9098-common = "linux-firmware-nxp9098-common"
 RREPLACES:${PN}-nxp9098-common = "linux-firmware-nxp9098-common"
 RCONFLICTS:${PN}-nxp9098-common = "linux-firmware-nxp9098-common"
 
-FILES:${PN}-nxp9098-pcie = "\
+# nooelint: oelint.var.order.FILES
+FILES:${PN}-nxp9098-pcie += "\
     ${nonarch_base_libdir}/firmware/nxp/pcie*9098* \
 "
 RDEPENDS:${PN}-nxp9098-pcie += "${PN}-nxp9098-common"
@@ -109,7 +114,8 @@ RPROVIDES:${PN}-nxp9098-pcie = "linux-firmware-nxp9098-pcie"
 RREPLACES:${PN}-nxp9098-pcie = "linux-firmware-nxp9098-pcie"
 RCONFLICTS:${PN}-nxp9098-pcie = "linux-firmware-nxp9098-pcie"
 
-FILES:${PN}-nxp9098-sdio = "\
+# nooelint: oelint.var.order.FILES
+FILES:${PN}-nxp9098-sdio += "\
     ${nonarch_base_libdir}/firmware/nxp/sd*9098* \
 "
 RDEPENDS:${PN}-nxp9098-sdio += "${PN}-nxp9098-common"
@@ -117,13 +123,15 @@ RPROVIDES:${PN}-nxp9098-sdio = "linux-firmware-nxp9098-sdio"
 RREPLACES:${PN}-nxp9098-sdio = "linux-firmware-nxp9098-sdio"
 RCONFLICTS:${PN}-nxp9098-sdio = "linux-firmware-nxp9098-sdio"
 
+# nooelint: oelint.var.order.FILES
 FILES:${PN}-nxpaw693-pcie += "\
     ${nonarch_base_libdir}/firmware/nxp/pcie*aw693* \
     ${nonarch_base_libdir}/firmware/nxp/uart*aw693* \
 "
 RDEPENDS:${PN}-nxpaw693-pcie += "${PN}-nxp-common"
 
-FILES:${PN}-nxpiw416-sdio = "\
+# nooelint: oelint.var.order.FILES
+FILES:${PN}-nxpiw416-sdio += "\
     ${nonarch_base_libdir}/firmware/mrvl/sdiouartiw416_combo_v0.bin \
     ${nonarch_base_libdir}/firmware/nxp/*iw416* \
 "
@@ -132,6 +140,7 @@ RPROVIDES:${PN}-nxpiw416-sdio = "linux-firmware-nxpiw416-sdio"
 RREPLACES:${PN}-nxpiw416-sdio = "linux-firmware-nxpiw416-sdio"
 RCONFLICTS:${PN}-nxpiw416-sdio = "linux-firmware-nxpiw416-sdio"
 
+# nooelint: oelint.var.order.FILES
 FILES:${PN}-nxpiw610-sdio += "\
     ${nonarch_base_libdir}/firmware/nxp/sd_iw610.bin.se \
     ${nonarch_base_libdir}/firmware/nxp/sduart_iw610.bin.se \
@@ -144,12 +153,14 @@ RPROVIDES:${PN}-nxpiw610-sdio = "linux-firmware-nxpiw610-sdio"
 RREPLACES:${PN}-nxpiw610-sdio = "linux-firmware-nxpiw610-sdio"
 RCONFLICTS:${PN}-nxpiw610-sdio = "linux-firmware-nxpiw610-sdio"
 
+# nooelint: oelint.var.order.FILES
 FILES:${PN}-nxpiw610-usb += "\
     ${nonarch_base_libdir}/firmware/nxp/usb*_iw610.bin.se \
 "
 RDEPENDS:${PN}-nxpiw610-usb += "${PN}-nxp-common"
 
-FILES:${PN}-nxpiw612-sdio = "\
+# nooelint: oelint.var.order.FILES
+FILES:${PN}-nxpiw612-sdio += "\
     ${nonarch_base_libdir}/firmware/nxp/sd_w61x_v1.bin.se \
     ${nonarch_base_libdir}/firmware/nxp/sduart_nw61x_*.bin.se \
     ${nonarch_base_libdir}/firmware/nxp/uartspi_n61x_*.bin.se \
diff --git a/recipes-bsp/firmware-imx/firmware-sof-imx_2.3.0.bb b/recipes-bsp/firmware-imx/firmware-sof-imx_2.3.0.bb
index 84e00a32c..b951b8580 100644
--- a/recipes-bsp/firmware-imx/firmware-sof-imx_2.3.0.bb
+++ b/recipes-bsp/firmware-imx/firmware-sof-imx_2.3.0.bb
@@ -21,4 +21,6 @@ do_install() {
     cp -r sof* ${D}${nonarch_base_libdir}/firmware/imx/
 }
 
+# Package the installed SOF firmware/topology tree explicitly (filesoverride).
+# nooelint: oelint.var.filesoverride
 FILES:${PN} = "${nonarch_base_libdir}/firmware/imx"
diff --git a/recipes-bsp/imx-lib/imx-lib_git.bb b/recipes-bsp/imx-lib/imx-lib_git.bb
index 837df109a..2abb0a3e0 100644
--- a/recipes-bsp/imx-lib/imx-lib_git.bb
+++ b/recipes-bsp/imx-lib/imx-lib_git.bb
@@ -2,10 +2,11 @@
 # Copyright (C) 2012-2018 O.S. Systems Software LTDA.
 # Copyright 2017 NXP
 
+SUMMARY = "i.MX platform-specific libraries"
 DESCRIPTION = "Platform specific libraries for imx platform"
 HOMEPAGE = "https://github.com/nxp-imx/imx-lib"
-LICENSE = "LGPL-2.1-only"
 SECTION = "multimedia"
+LICENSE = "LGPL-2.1-only"
 
 LIC_FILES_CHKSUM = "file://LICENSE;md5=fbc093901857fcd118f065f900982c24"
 
diff --git a/recipes-bsp/imx-seco/imx-seco_5.9.4.1.bb b/recipes-bsp/imx-seco/imx-seco_5.9.4.1.bb
index 0502495a2..025d32c29 100644
--- a/recipes-bsp/imx-seco/imx-seco_5.9.4.1.bb
+++ b/recipes-bsp/imx-seco/imx-seco_5.9.4.1.bb
@@ -2,6 +2,7 @@
 
 SUMMARY = "NXP i.MX SECO firmware"
 DESCRIPTION = "Firmware for i.MX Security Controller Subsystem"
+HOMEPAGE = "https://www.nxp.com/"
 SECTION = "base"
 LICENSE = "LicenseRef-Proprietary"
 LIC_FILES_CHKSUM = "file://COPYING;md5=ca53281cc0caa7e320d4945a896fb837"
@@ -28,5 +29,8 @@ do_deploy () {
     install -m 0644 ${S}/firmware/seco/${SECO_FIRMWARE_NAME} ${DEPLOYDIR}
 }
 
+# The prebuilt SECO firmware is deployed under /lib/firmware, so the main
+# package claims the root path explicitly (filesoverride).
+# nooelint: oelint.var.filesoverride
 FILES:${PN} = "/"
 COMPATIBLE_MACHINE = "(mx8qm-generic-bsp|mx8qxp-generic-bsp|mx8dxl-generic-bsp|mx8dx-generic-bsp)"
diff --git a/recipes-bsp/imx-uuc/imx-uuc_git.bb b/recipes-bsp/imx-uuc/imx-uuc_git.bb
index 446fb58e9..0c525e713 100644
--- a/recipes-bsp/imx-uuc/imx-uuc_git.bb
+++ b/recipes-bsp/imx-uuc/imx-uuc_git.bb
@@ -1,11 +1,12 @@
 # Copyright (C) 2016 Freescale Semiconductor
 # Copyright (C) 2017-2019,2024-2025 NXP
 SUMMARY = "A Daemon wait for NXP mfgtools host's command"
+DESCRIPTION = "Update Utility Client daemon that waits for commands from the NXP mfgtools host during manufacturing programming."
 HOMEPAGE = "https://github.com/nxp-imx/imx-uuc"
 SECTION = "base"
-DEPENDS = "dosfstools-native virtual/kernel"
 LICENSE = "GPL-2.0-only"
 LIC_FILES_CHKSUM = "file://LICENSE;md5=b234ee4d69f5fce4486a80fdaf4a4263"
+DEPENDS = "dosfstools-native virtual/kernel"
 
 PR = "r1"
 PV = "0.5.1+git${SRCPV}"
diff --git a/recipes-bsp/mc-utils/mc-utils_git.bb b/recipes-bsp/mc-utils/mc-utils_git.bb
index ff502afb4..a25e33855 100644
--- a/recipes-bsp/mc-utils/mc-utils_git.bb
+++ b/recipes-bsp/mc-utils/mc-utils_git.bb
@@ -1,3 +1,4 @@
+SUMMARY = "QorIQ DPAA Management Complex config utilities"
 DESCRIPTION = "The Management Complex (MC) is a key component of DPAA"
 HOMEPAGE = "https://github.com/nxp-qoriq/mc-utils"
 SECTION = "mc-utils"
@@ -38,7 +39,7 @@ do_deploy () {
 }
 addtask deploy after do_install
 
+PACKAGE_ARCH = "${MACHINE_ARCH}"
 PACKAGES += "${PN}-image"
 FILES:${PN}-image += "/boot"
-PACKAGE_ARCH = "${MACHINE_ARCH}"
 COMPATIBLE_MACHINE = "(qoriq-arm64)"
diff --git a/recipes-bsp/qe-ucode/qe-ucode_git.bb b/recipes-bsp/qe-ucode/qe-ucode_git.bb
index 78dc9c3f6..366bbdfc2 100644
--- a/recipes-bsp/qe-ucode/qe-ucode_git.bb
+++ b/recipes-bsp/qe-ucode/qe-ucode_git.bb
@@ -1,4 +1,5 @@
-DESCRIPTION = "qe microcode binary"
+SUMMARY = "QorIQ QE microcode"
+DESCRIPTION = "QUICC Engine (QE) microcode binary for NXP QorIQ SoCs"
 HOMEPAGE = "https://github.com/NXP/qoriq-qe-ucode"
 SECTION = "qe-ucode"
 LICENSE = "LicenseRef-NXP-Binary-EULA"
@@ -20,9 +21,10 @@ do_deploy () {
 }
 addtask deploy before do_build after do_install
 
+PACKAGE_ARCH = "${MACHINE_SOCARCH}"
+
 PACKAGES += "${PN}-image"
 FILES:${PN}-image += "/boot/*"
 
 COMPATIBLE_MACHINE = "(qoriq)"
-PACKAGE_ARCH = "${MACHINE_SOCARCH}"
 
diff --git a/recipes-bsp/rcw/rcw_git.bb b/recipes-bsp/rcw/rcw_git.bb
index dbcc42108..1e526b699 100644
--- a/recipes-bsp/rcw/rcw_git.bb
+++ b/recipes-bsp/rcw/rcw_git.bb
@@ -1,6 +1,7 @@
 SUMMARY = "Reset Configuration Word"
 DESCRIPTION = "Reset Configuration Word - hardware boot-time parameters for the QorIQ targets"
 HOMEPAGE = "https://github.com/nxp-qoriq/rcw"
+SECTION = "bsp"
 LICENSE = "BSD-3-Clause"
 LIC_FILES_CHKSUM = "file://LICENSE;md5=44a0d0fad189770cc022af4ac6262cbe"
 
@@ -35,8 +36,9 @@ do_deploy () {
 }
 addtask deploy after do_install
 
+PACKAGE_ARCH = "${MACHINE_ARCH}"
+
 PACKAGES += "${PN}-image"
 FILES:${PN}-image += "/boot"
 
 COMPATIBLE_MACHINE = "(qoriq)"
-PACKAGE_ARCH = "${MACHINE_ARCH}"
diff --git a/recipes-bsp/uefi/uefi_git.bb b/recipes-bsp/uefi/uefi_git.bb
index b71894bb9..9b81bd153 100644
--- a/recipes-bsp/uefi/uefi_git.bb
+++ b/recipes-bsp/uefi/uefi_git.bb
@@ -1,4 +1,5 @@
-DESCRIPTION = "Unified Extensible Firmware Interface"
+SUMMARY = "UEFI firmware for QorIQ platforms"
+DESCRIPTION = "Prebuilt UEFI (Unified Extensible Firmware Interface) firmware and GRUB binaries for NXP QorIQ platforms"
 HOMEPAGE = "https://github.com/NXP/qoriq-uefi-binary"
 SECTION = "bootloaders"
 LICENSE = "LicenseRef-NXP-Binary-EULA"
@@ -26,9 +27,9 @@ do_deploy () {
 }
 addtask deploy before do_build after do_install
 
+PACKAGE_ARCH = "${MACHINE_SOCARCH}"
+
 PACKAGES += "${PN}-image"
 FILES:${PN}-image += "/uefi/*"
 
-PACKAGE_ARCH = "${MACHINE_SOCARCH}"
-
 COMPATIBLE_MACHINE = "(qoriq)"