Merge pull request #2565 from lucianogdittgen/oelint-cleanup-8-recipe-metadata-4

oelint: Recipe metadata and variable-order cleanups

5 files changed, 64 insertions, 41 deletions

diff --git a/recipes-multimedia/alsa/imx-alsa-plugins_git.bb b/recipes-multimedia/alsa/imx-alsa-plugins_git.bb
index fced1cb66..2cd9124fc 100644
--- a/recipes-multimedia/alsa/imx-alsa-plugins_git.bb
+++ b/recipes-multimedia/alsa/imx-alsa-plugins_git.bb
@@ -2,21 +2,17 @@
 # Copyright 2017-2026 NXP
 # Released under the MIT license (see COPYING.MIT for the terms)
 
-DESCRIPTION = "Freescale alsa-lib plugins"
+SUMMARY = "ALSA-lib plugins for i.MX SoCs"
+DESCRIPTION = "Freescale/NXP ALSA-lib plugins for i.MX SoCs, including the \
+               software PDM (pulse-density modulation) capture plugin."
 HOMEPAGE = "https://github.com/nxp-imx/imx-alsa-plugins"
-LICENSE = "GPL-2.0-only"
 SECTION = "multimedia"
+LICENSE = "GPL-2.0-only"
+LIC_FILES_CHKSUM = "file://LICENSE;md5=94d55d512a9ba36caa9b7df079bae19f"
 DEPENDS = "alsa-lib"
 
-# For backwards compatibility
+# Provide the old fsl-alsa-plugins name for backwards compatibility
 PROVIDES += "fsl-alsa-plugins"
-RREPLACES:${PN} = "fsl-alsa-plugins"
-RPROVIDES:${PN} = "fsl-alsa-plugins"
-RCONFLICTS:${PN} = "fsl-alsa-plugins"
-
-LIC_FILES_CHKSUM = "file://LICENSE;md5=94d55d512a9ba36caa9b7df079bae19f"
-
-inherit autotools pkgconfig use-imx-headers
 
 PV = "1.0.26+${SRCPV}"
 
@@ -25,6 +21,8 @@ IMXALSA_SRC ?= "git://github.com/nxp-imx/imx-alsa-plugins.git;protocol=https"
 SRCBRANCH = "MM_04.10.03_2512_L6.18.2"
 SRCREV = "18eb79b6cf13fb988de7d4eef5b22eb60f8e2533"
 
+inherit autotools pkgconfig use-imx-headers
+
 CFLAGS:append = " -I${STAGING_INCDIR_IMX}"
 
 PACKAGECONFIG ??= "${PACKAGECONFIG_SWPDM}"
@@ -33,10 +31,18 @@ PACKAGECONFIG_SWPDM:mx8-nxp-bsp = "swpdm"
 
 PACKAGECONFIG[swpdm] = "--enable-swpdm,--disable-swpdm,imx-sw-pdm"
 
+# ALSA plugins install versioned-looking .so modules into ${libdir}/alsa-lib;
+# these are runtime plugin objects (not dev symlinks), so dev-so must be skipped.
+# nooelint: oelint.vars.insaneskip
 INSANE_SKIP:${PN} = "dev-so"
 
 FILES:${PN} += "${libdir}/alsa-lib/libasound_*.so"
 FILES:${PN}-dbg += "${libdir}/alsa-lib/.debug"
 FILES:${PN}-dev += "${libdir}/alsa-lib/*.la"
 
+# Runtime backwards compatibility with the old fsl-alsa-plugins name
+RREPLACES:${PN} = "fsl-alsa-plugins"
+RPROVIDES:${PN} = "fsl-alsa-plugins"
+RCONFLICTS:${PN} = "fsl-alsa-plugins"
+
 COMPATIBLE_MACHINE = "(imx-nxp-bsp)"
diff --git a/recipes-multimedia/gstreamer/gstreamer1.0-vaapi_1.26.6.bb b/recipes-multimedia/gstreamer/gstreamer1.0-vaapi_1.26.6.bb
index 26cac0a7d..d1ed15f87 100644
--- a/recipes-multimedia/gstreamer/gstreamer1.0-vaapi_1.26.6.bb
+++ b/recipes-multimedia/gstreamer/gstreamer1.0-vaapi_1.26.6.bb
@@ -1,20 +1,21 @@
 SUMMARY = "VA-API support to GStreamer"
-HOMEPAGE = "https://gstreamer.freedesktop.org/"
 DESCRIPTION = "gstreamer-vaapi consists of a collection of VA-API \
                based plugins for GStreamer and helper libraries: `vaapidecode', \
                `vaapiconvert', and `vaapisink'."
+HOMEPAGE = "https://gstreamer.freedesktop.org/"
+SECTION = "multimedia"
 
 REALPN = "gstreamer-vaapi"
 
 LICENSE = "LGPL-2.1-or-later"
 LIC_FILES_CHKSUM = "file://COPYING.LIB;md5=4fbd65380cdd255951079008b364516c"
+DEPENDS = "gstreamer1.0 gstreamer1.0-plugins-bad gstreamer1.0-plugins-base libva"
 
 SRC_URI = "https://gstreamer.freedesktop.org/src/${REALPN}/${REALPN}-${PV}.tar.xz"
 
 SRC_URI[sha256sum] = "d87c57244cecbd17bb030b698dcb67a66225de639f7c5b837391c4a8e5477667"
 
 S = "${UNPACKDIR}/${REALPN}-${PV}"
-DEPENDS = "gstreamer1.0 gstreamer1.0-plugins-bad gstreamer1.0-plugins-base libva"
 
 inherit meson pkgconfig features_check upstream-version-is-even
 
@@ -26,8 +27,6 @@ EXTRA_OEMESON += "\
     -Dtests=enabled \
 "
 
-PACKAGES =+ "${PN}-tests"
-
 # OpenGL packageconfig factored out to make it easy for distros
 # and BSP layers to pick either glx, egl, or no GL. By default,
 # try detecting X11 first, and if found (with OpenGL), use GLX,
@@ -47,7 +46,9 @@ PACKAGECONFIG[glx] = "-Dglx=enabled,-Dglx=disabled,virtual/libgl"
 PACKAGECONFIG[wayland] = "-Dwayland=enabled,-Dwayland=disabled,wayland-native wayland wayland-protocols"
 PACKAGECONFIG[x11] = "-Dx11=enabled,-Dx11=disabled,virtual/libx11 libxrandr libxrender"
 
+PACKAGES =+ "${PN}-tests"
+
 FILES:${PN} += "${libdir}/gstreamer-*/*.so"
 FILES:${PN}-dbg += "${libdir}/gstreamer-*/.debug"
 FILES:${PN}-dev += "${libdir}/gstreamer-*/*.a"
-FILES:${PN}-tests = "${bindir}/*"
+FILES:${PN}-tests += "${bindir}/*"
diff --git a/recipes-multimedia/gstreamer/imx-gst1.0-plugin_git.bb b/recipes-multimedia/gstreamer/imx-gst1.0-plugin_git.bb
index 486049d7b..aa398df0c 100644
--- a/recipes-multimedia/gstreamer/imx-gst1.0-plugin_git.bb
+++ b/recipes-multimedia/gstreamer/imx-gst1.0-plugin_git.bb
@@ -2,19 +2,22 @@
 # Copyright 2017-2025 NXP
 # Copyright (C) 2012-2015 O.S. Systems Software LTDA.
 # Released under the MIT license (see COPYING.MIT for the terms)
-DESCRIPTION = "Gstreamer freescale plugins"
+SUMMARY = "GStreamer 1.0 plugins for i.MX"
+DESCRIPTION = "Freescale/NXP GStreamer 1.0 plugins providing hardware-accelerated \
+               multimedia (audio/video codec, capture and sink) elements for \
+               i.MX SoCs."
 HOMEPAGE = "https://github.com/nxp-imx/imx-gst1.0-plugin"
 SECTION = "multimedia"
 LICENSE = "GPL-2.0-only & LGPL-2.0-only & LGPL-2.1-only"
 LIC_FILES_CHKSUM = "file://LICENSE.txt;md5=fbc093901857fcd118f065f900982c24"
 
 DEPENDS = "\
+    ${DEPENDS_IMXGPU} \
     gstreamer1.0 \
-    gstreamer1.0-plugins-base \
     gstreamer1.0-plugins-bad \
+    gstreamer1.0-plugins-base \
     imx-codec \
     imx-parser \
-    ${DEPENDS_IMXGPU} \
     libinput \
 "
 DEPENDS:append:mx6-nxp-bsp = " imx-lib"
@@ -31,11 +34,6 @@ DEPENDS_IMX_OPENCL_CONVERTER:mx6-nxp-bsp = ""
 DEPENDS_IMX_OPENCL_CONVERTER:mx7-nxp-bsp = ""
 DEPENDS_IMX_OPENCL_CONVERTER:mx8mm-nxp-bsp = ""
 
-# For backwards compatibility
-RREPLACES:${PN} = "gst1.0-fsl-plugin"
-RPROVIDES:${PN} = "gst1.0-fsl-plugin"
-RCONFLICTS:${PN} = "gst1.0-fsl-plugin"
-
 PV = "4.10.3+git${SRCPV}"
 
 SRC_URI = "${IMXGST_SRC};branch=${SRCBRANCH}"
@@ -60,17 +58,6 @@ EXTRA_OEMESON = "-Dplatform=${PLATFORM} \
                  -Dc_args="${CFLAGS} -I${STAGING_INCDIR_IMX}" \
 "
 
-PACKAGES =+ "${PN}-tools ${PN}-libgstfsl"
-
-# Add codec list that the beep plugin run-time depended
-BEEP_RDEPENDS = "imx-codec-aac imx-codec-mp3 imx-codec-oggvorbis"
-RDEPENDS:${PN} += "imx-parser ${BEEP_RDEPENDS} gstreamer1.0-plugins-good-id3demux "
-RDEPENDS:${PN}:append:mx8qm-nxp-bsp = " imx-dsp"
-RDEPENDS:${PN}:append:mx8qxp-nxp-bsp = " imx-dsp"
-RDEPENDS:${PN}:append:mx8dx-nxp-bsp = " imx-dsp"
-RDEPENDS:${PN}:append:mx8mp-nxp-bsp = " imx-dsp"
-RDEPENDS:${PN}:append:mx8ulp-nxp-bsp = " imx-dsp"
-
 PACKAGECONFIG ?= ""
 
 # FIXME: Add all features
@@ -82,13 +69,34 @@ MSDEPENDS = "imx-msparser imx-mscodec"
 PACKAGECONFIG[wma10dec] = ",,${MSDEPENDS},${MSDEPENDS}"
 PACKAGECONFIG[wma8enc] = ",,${MSDEPENDS},${MSDEPENDS}"
 
-FILES:${PN} = "${libdir}/gstreamer-1.0/*.so ${datadir}"
+PACKAGES =+ "${PN}-tools ${PN}-libgstfsl"
 
+# Deliberately replace the default main-package file list so the (unversioned)
+# GStreamer plugin .so files land in ${PN} instead of ${PN}-dev.
+# nooelint: oelint.var.filesoverride
+FILES:${PN} = "${libdir}/gstreamer-1.0/*.so ${datadir}"
 FILES:${PN}-dbg += "${libdir}/gstreamer-1.0/.debug"
 FILES:${PN}-dev += "${libdir}/gstreamer-1.0/*.la ${libdir}/pkgconfig/*.pc"
 FILES:${PN}-tools += "${bindir}/* ${libdir}/librecorder_engine-1.0${SOLIBS}"
-FILES:${PN}-libgstfsl = "${libdir}/libgstfsl-1.0${SOLIBS}"
+FILES:${PN}-libgstfsl += "${libdir}/libgstfsl-1.0${SOLIBS}"
+
+# Add codec list that the beep plugin run-time depended
+BEEP_RDEPENDS = "imx-codec-aac imx-codec-mp3 imx-codec-oggvorbis"
+RDEPENDS:${PN} += "${BEEP_RDEPENDS} gstreamer1.0-plugins-good-id3demux imx-parser"
+RDEPENDS:${PN}:append:mx8qm-nxp-bsp = " imx-dsp"
+RDEPENDS:${PN}:append:mx8qxp-nxp-bsp = " imx-dsp"
+RDEPENDS:${PN}:append:mx8dx-nxp-bsp = " imx-dsp"
+RDEPENDS:${PN}:append:mx8mp-nxp-bsp = " imx-dsp"
+RDEPENDS:${PN}:append:mx8ulp-nxp-bsp = " imx-dsp"
 
+# The plugins pull in codec/parser build dependencies conditionally through
+# PACKAGECONFIG, so the static build-deps QA check cannot see them all.
+# nooelint: oelint.vars.insaneskip
 INSANE_SKIP:${PN} = "build-deps"
 
+# Runtime backwards compatibility with the old gst1.0-fsl-plugin name
+RREPLACES:${PN} = "gst1.0-fsl-plugin"
+RPROVIDES:${PN} = "gst1.0-fsl-plugin"
+RCONFLICTS:${PN} = "gst1.0-fsl-plugin"
+
 COMPATIBLE_MACHINE = "(imx-nxp-bsp)"
diff --git a/recipes-multimedia/imx-codec/imx-codec_4.10.0.bb b/recipes-multimedia/imx-codec/imx-codec_4.10.0.bb
index 1c6e8081c..cac6d9c5e 100644
--- a/recipes-multimedia/imx-codec/imx-codec_4.10.0.bb
+++ b/recipes-multimedia/imx-codec/imx-codec_4.10.0.bb
@@ -60,6 +60,7 @@ python __set_insane_skip() {
         else:
             d.setVar("INSANE_SKIP:%s" % p, "ldflags textrel")
 }
+__set_insane_skip[doc] = "Set DEBIAN_NOAUTONAME and INSANE_SKIP for the prebuilt codec plugin packages."
 
 do_package_qa[prefuncs] += "__set_insane_skip"
 
@@ -82,6 +83,7 @@ python __split_libfslcodec_plugins() {
             d.setVar('RPROVIDES:%s' % pkg, ' libfslcodec')
             d.setVar('RCONFLICTS:%s' % pkg, ' libfslcodec')
 }
+__split_libfslcodec_plugins[doc] = "Split the codec plugin .so files into per-codec packages."
 
 python __set_metapkg_rdepends() {
     # Allow addition of all codecs in a image; useful specially for
@@ -91,9 +93,12 @@ python __set_metapkg_rdepends() {
                         codec_pkgs)
     d.appendVar('RDEPENDS:imx-codec-meta', ' ' + ' '.join(codec_pkgs))
 }
+__set_metapkg_rdepends[doc] = "Make the -meta package RDEPEND on all split codec packages."
 
 PACKAGESPLITFUNCS =+ "__split_libfslcodec_plugins __set_metapkg_rdepends"
 
+PACKAGE_ARCH = "${MACHINE_SOCARCH}"
+
 # We need to ensure we don't have '-src' package overrided
 PACKAGE_DEBUG_SPLIT_STYLE = 'debug-without-src'
 
@@ -106,7 +111,9 @@ PACKAGES += "${PN}-meta ${PN}-test-bin ${PN}-test-source"
 ALLOW_EMPTY:${PN} = "1"
 ALLOW_EMPTY:${PN}-meta = "1"
 
-# Ensure we get warnings if we miss something
+# Reset PN's default FILES to empty so any unpackaged file is flagged as a
+# warning instead of silently landing in the main package.
+# nooelint: oelint.var.filesoverride
 FILES:${PN} = ""
 
 FILES:${PN}-dev += "${libdir}/imx-mm/*/*${SOLIBSDEV} \
@@ -122,5 +129,4 @@ FILES:${PN}-oggvorbis += "${libdir}/imx-mm/audio-codec/wrap/lib_vorbisd_wrap_arm
 FILES:${PN}-nb += "${libdir}/imx-mm/audio-codec/wrap/lib_nbamrd_wrap_arm*_elinux.so.*"
 FILES:${PN}-wb += "${libdir}/imx-mm/audio-codec/wrap/lib_wbamrd_wrap_arm*_elinux.so.*"
 
-PACKAGE_ARCH = "${MACHINE_SOCARCH}"
 COMPATIBLE_MACHINE = "(imx-nxp-bsp)"
diff --git a/recipes-multimedia/tinycompress/tinycompress_1.2.5.bb b/recipes-multimedia/tinycompress/tinycompress_1.2.5.bb
index bffe15c87..5187c6127 100644
--- a/recipes-multimedia/tinycompress/tinycompress_1.2.5.bb
+++ b/recipes-multimedia/tinycompress/tinycompress_1.2.5.bb
@@ -1,5 +1,9 @@
-DESCRIPTION = "A library to handle compressed formats like MP3 etc."
+SUMMARY = "ALSA compress-offload API library"
+DESCRIPTION = "A userspace library implementing the ALSA compress-offload API, \
+               used to stream hardware-compressed audio formats such as MP3 to \
+               the DSP without decoding them on the CPU."
 HOMEPAGE = "https://github.com/alsa-project/tinycompress"
+SECTION = "multimedia"
 LICENSE = "LGPL-2.1-only | BSD-3-Clause"
 LIC_FILES_CHKSUM = "file://COPYING;md5=cf9105c1a2d4405cbe04bbe3367373a0"
 DEPENDS = "alsa-lib"
@@ -13,5 +17,3 @@ SRC_URI = "git://github.com/alsa-project/tinycompress.git;protocol=https;branch=
 SRCREV = "f3ba6e5c2126f2fb07e3d890f990d50c3e204e67"
 
 EXTRA_OECONF:append = " --enable-pcm"
-
-inherit autotools pkgconfig