libdrm: Use FILES += for drivers, document i.MX block per oelint

libdrm ships one sub-package per DRM driver. oelint flags every
FILES:${PN}-<driver> = as oelint.var.filesoverride; switch them to +=
(the vars are never otherwise set, so this is behaviourally identical)
and the check no longer fires.

The i.MX-specific vivante additions (PACKAGES/RRECOMMENDS/FILES/
PACKAGECONFIG :imxgpu overrides) are kept together as one block at the
end of the recipe for readability. That grouping trips the global
placement checks order.PACKAGES, order.FILES and order.PACKAGECONFIG,
which can only be cleared by scattering the block, so keep an inline
# nooelint for those three. FILES:${PN}-vivante also switches to += and
so no longer needs a filesoverride suppression.

No functional change.

Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>

1 files changed, 14 insertions, 10 deletions

diff --git a/recipes-graphics/drm/libdrm_2.4.127.imx.bb b/recipes-graphics/drm/libdrm_2.4.127.imx.bb
index 685e3f04a..de96d72e1 100644
--- a/recipes-graphics/drm/libdrm_2.4.127.imx.bb
+++ b/recipes-graphics/drm/libdrm_2.4.127.imx.bb
@@ -46,15 +46,15 @@ PACKAGES =+ "${PN}-tests ${PN}-drivers ${PN}-radeon ${PN}-nouveau ${PN}-omap \
              ${PN}-intel ${PN}-exynos ${PN}-freedreno ${PN}-amdgpu \
              ${PN}-etnaviv"
 
-FILES:${PN}-tests = "${bindir}/*"
-FILES:${PN}-radeon = "${libdir}/libdrm_radeon.so.*"
-FILES:${PN}-nouveau = "${libdir}/libdrm_nouveau.so.*"
-FILES:${PN}-omap = "${libdir}/libdrm_omap.so.*"
-FILES:${PN}-intel = "${libdir}/libdrm_intel.so.*"
-FILES:${PN}-exynos = "${libdir}/libdrm_exynos.so.*"
-FILES:${PN}-freedreno = "${libdir}/libdrm_freedreno.so.*"
-FILES:${PN}-amdgpu = "${libdir}/libdrm_amdgpu.so.* ${datadir}/${PN}/amdgpu.ids"
-FILES:${PN}-etnaviv = "${libdir}/libdrm_etnaviv.so.*"
+FILES:${PN}-tests += "${bindir}/*"
+FILES:${PN}-radeon += "${libdir}/libdrm_radeon.so.*"
+FILES:${PN}-nouveau += "${libdir}/libdrm_nouveau.so.*"
+FILES:${PN}-omap += "${libdir}/libdrm_omap.so.*"
+FILES:${PN}-intel += "${libdir}/libdrm_intel.so.*"
+FILES:${PN}-exynos += "${libdir}/libdrm_exynos.so.*"
+FILES:${PN}-freedreno += "${libdir}/libdrm_freedreno.so.*"
+FILES:${PN}-amdgpu += "${libdir}/libdrm_amdgpu.so.* ${datadir}/${PN}/amdgpu.ids"
+FILES:${PN}-etnaviv += "${libdir}/libdrm_etnaviv.so.*"
 
 RRECOMMENDS:${PN}-drivers = "${PN}-radeon ${PN}-nouveau ${PN}-omap ${PN}-intel \
                              ${PN}-exynos ${PN}-freedreno ${PN}-amdgpu \
@@ -62,9 +62,13 @@ RRECOMMENDS:${PN}-drivers = "${PN}-radeon ${PN}-nouveau ${PN}-omap ${PN}-intel \
 
 BBCLASSEXTEND = "native nativesdk"
 
+# The i.MX vivante driver additions are kept together as one block.
+# nooelint: oelint.var.order.PACKAGES
 PACKAGES:prepend:imxgpu = "${PN}-vivante "
 RRECOMMENDS:${PN}-drivers:append:imxgpu = " ${PN}-vivante"
-FILES:${PN}-vivante = "${libdir}/libdrm_vivante.so.*"
+# nooelint: oelint.var.order.FILES
+FILES:${PN}-vivante += "${libdir}/libdrm_vivante.so.*"
+# nooelint: oelint.var.order.PACKAGECONFIG
 PACKAGECONFIG:append:imxgpu = " vivante"
 PACKAGECONFIG[vivante] = "-Dvivante=true,-Dvivante=false"