From 1f9102634519da060845e1d94521b73e7b76cda1 Mon Sep 17 00:00:00 2001 From: Luciano Dittgen Date: Wed, 15 Jul 2026 09:03:04 -0300 Subject: qoriq-atf: Order PACKAGECONFIG and document findings per oelint Move the PACKAGECONFIG block up beside the fetch variables so it precedes PACKAGE_ARCH (oelint.var.order.PACKAGECONFIG). Accept two findings inline: PV:append must join without a leading space to remain a valid version string (oelint.vars.inconspaces false positive), and the arm-cot/secure consistency check needs a parse-time anonymous python (oelint.task.noanonpython). No functional change. Signed-off-by: Luciano Dittgen --- recipes-bsp/atf/qoriq-atf_2.12.bb | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) (limited to 'recipes-bsp') diff --git a/recipes-bsp/atf/qoriq-atf_2.12.bb b/recipes-bsp/atf/qoriq-atf_2.12.bb index c6a7ee252..f78de99f7 100644 --- a/recipes-bsp/atf/qoriq-atf_2.12.bb +++ b/recipes-bsp/atf/qoriq-atf_2.12.bb @@ -5,6 +5,8 @@ inherit deploy DEPENDS += "bc-native openssl openssl-native qoriq-cst-native rcw u-boot u-boot-mkimage-native" do_compile[depends] += "u-boot:do_deploy rcw:do_deploy uefi:do_deploy" +# Version suffix must join without a leading space to stay a valid PV. +# nooelint: oelint.vars.inconspaces PV:append = "+${SRCPV}" SRC_URI += "git://github.com/ARMmbed/mbedtls;protocol=https;nobranch=1;destsuffix=${S}/mbedtls;name=mbedtls \ @@ -14,6 +16,11 @@ SRCREV_mbedtls = "0795874acdf887290b2571b193cafd3c4041a708" SRCREV_ddr = "fbc036b88acb6c06ffed02c898cbae9856ec75ba" SRCREV_FORMAT = "atf" +PACKAGECONFIG ??= "\ + ${@bb.utils.filter('COMBINED_FEATURES', 'optee', d)} \ +" +PACKAGECONFIG[optee] = ",,optee-os-qoriq" + COMPATIBLE_MACHINE = "(qoriq)" PACKAGE_ARCH = "${MACHINE_ARCH}" @@ -68,11 +75,8 @@ EXTRA_OEMAKE += "\ ${@bb.utils.contains('DISTRO_FEATURES', 'fuse', 'fip_fuse FUSE_PROG=1 FUSE_PROV_FILE=fuse_scr.bin', '', d)} \ " -PACKAGECONFIG ??= "\ - ${@bb.utils.filter('COMBINED_FEATURES', 'optee', d)} \ -" -PACKAGECONFIG[optee] = ",,optee-os-qoriq" - +# Parse-time validation that arm-cot implies secure; needs an anonymous python. +# nooelint: oelint.task.noanonpython python() { if bb.utils.contains("DISTRO_FEATURES", "arm-cot", True, False, d): if not bb.utils.contains("DISTRO_FEATURES", "secure", True, False, d): -- cgit v1.2.3-54-g00ecf