summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLuciano Dittgen <luciano.dittgen@ossystems.com.br>2026-07-29 10:26:59 -0300
committerLuciano Dittgen <luciano.dittgen@ossystems.com.br>2026-08-03 16:13:30 -0300
commita066ce9bfe828855b471e65b72871a3b019a80de (patch)
treefcf883eff1bad901be370c86615b3415f149ff1b
parent684c3b8b5e28ba2baa69e3913ec0e1b114a07543 (diff)
downloadmeta-freescale-a066ce9bfe828855b471e65b72871a3b019a80de.tar.gz
recipes: Document the remaining custom tasks per oelint
Set doc strings on the custom functions oelint-adv reports via 'oelint.task.docstrings': the three i.MX boot firmware deploy helpers, the libubootenv uboot-config fixup handler, the qemu-qoriq wrapper generator, the testfloat SoftFloat unpack step, the linux-imx device tree compatibility handler and the imx-mp4-parser QA-skip hook. imx-mp4-parser reuses the wording already used for the identical hook in imx-parser, so the two prebuilt-binary recipes read the same. opencv and libcamera are deliberately left alone. Both recipes carry their upstream recipe as a marked verbatim block ("meta-openembedded copy"), and the two functions oelint flags -- do_unpack_extra and do_package_recalculate_ipa_signatures -- sit inside those blocks, where meta-openembedded sets no doc string. Annotating them would break the diff against upstream for no functional gain, so those two findings stay visible. bitbake never reads the 'doc' flag when computing task checksums -- bb/data.py build_dependencies() only consults vardeps, vardepvalue, vardepsexclude, exports, postfuncs, prefuncs, lineno and filename -- so these additions cannot change a signature or invalidate sstate. Verified by re-linting each file. Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
-rw-r--r--recipes-bsp/firmware-imx/imx-boot-firmware-files_8.31.bb3
-rw-r--r--recipes-bsp/u-boot/libubootenv_%.bbappend1
-rw-r--r--recipes-devtools/qemu/qemu-qoriq.inc1
-rw-r--r--recipes-extended/testfloat/testfloat_2a.bb1
-rw-r--r--recipes-kernel/linux/linux-imx.inc1
-rw-r--r--recipes-multimedia/imx-parser/imx-mp4-parser_git.bb1
6 files changed, 8 insertions, 0 deletions
diff --git a/recipes-bsp/firmware-imx/imx-boot-firmware-files_8.31.bb b/recipes-bsp/firmware-imx/imx-boot-firmware-files_8.31.bb
index 87569b4a3..e0219ca8d 100644
--- a/recipes-bsp/firmware-imx/imx-boot-firmware-files_8.31.bb
+++ b/recipes-bsp/firmware-imx/imx-boot-firmware-files_8.31.bb
@@ -19,6 +19,7 @@ deploy_for_mx8() {
19 install -m 0644 ${S}/firmware/hdmi/cadence/hdmirxfw.bin ${DEPLOYDIR} 19 install -m 0644 ${S}/firmware/hdmi/cadence/hdmirxfw.bin ${DEPLOYDIR}
20 install -m 0644 ${S}/firmware/hdmi/cadence/dpfw.bin ${DEPLOYDIR} 20 install -m 0644 ${S}/firmware/hdmi/cadence/dpfw.bin ${DEPLOYDIR}
21} 21}
22deploy_for_mx8[doc] = "Deploy the Cadence HDMI firmware used by the i.MX 8 boot container"
22 23
23deploy_for_mx8m() { 24deploy_for_mx8m() {
24 # Synopsys DDR 25 # Synopsys DDR
@@ -30,6 +31,7 @@ deploy_for_mx8m() {
30 install -m 0644 ${S}/firmware/hdmi/cadence/signed_dp_imx8m.bin ${DEPLOYDIR} 31 install -m 0644 ${S}/firmware/hdmi/cadence/signed_dp_imx8m.bin ${DEPLOYDIR}
31 install -m 0644 ${S}/firmware/hdmi/cadence/signed_hdmi_imx8m.bin ${DEPLOYDIR} 32 install -m 0644 ${S}/firmware/hdmi/cadence/signed_hdmi_imx8m.bin ${DEPLOYDIR}
32} 33}
34deploy_for_mx8m[doc] = "Deploy the Synopsys DDR and Cadence DP/HDMI firmware used by the i.MX 8M boot container"
33 35
34deploy_for_mx9() { 36deploy_for_mx9() {
35 # Synopsys DDR 37 # Synopsys DDR
@@ -37,6 +39,7 @@ deploy_for_mx9() {
37 install -m 0644 ${S}/firmware/ddr/synopsys/${ddr_firmware} ${DEPLOYDIR} 39 install -m 0644 ${S}/firmware/ddr/synopsys/${ddr_firmware} ${DEPLOYDIR}
38 done 40 done
39} 41}
42deploy_for_mx9[doc] = "Deploy the Synopsys DDR firmware used by the i.MX 9 boot container"
40 43
41python () { 44python () {
42 # Manually add the required functions as dependencies otherwise they won't be included in the 45 # Manually add the required functions as dependencies otherwise they won't be included in the
diff --git a/recipes-bsp/u-boot/libubootenv_%.bbappend b/recipes-bsp/u-boot/libubootenv_%.bbappend
index 619b510e1..d1cecf100 100644
--- a/recipes-bsp/u-boot/libubootenv_%.bbappend
+++ b/recipes-bsp/u-boot/libubootenv_%.bbappend
@@ -38,6 +38,7 @@ def fixup_uboot_config_dependency(d):
38python fixup_uboot_config_dependency_handler() { 38python fixup_uboot_config_dependency_handler() {
39 fixup_uboot_config_dependency(d) 39 fixup_uboot_config_dependency(d)
40} 40}
41fixup_uboot_config_dependency_handler[doc] = "Borrow UBOOT_MACHINE and UBOOT_CONFIG from IMX_DEFAULT_BOOTLOADER so libubootenv passes the uboot-config sanity check"
41 42
42fixup_uboot_config_dependency_handler[eventmask] = "bb.event.RecipePreFinalise" 43fixup_uboot_config_dependency_handler[eventmask] = "bb.event.RecipePreFinalise"
43addhandler fixup_uboot_config_dependency_handler 44addhandler fixup_uboot_config_dependency_handler
diff --git a/recipes-devtools/qemu/qemu-qoriq.inc b/recipes-devtools/qemu/qemu-qoriq.inc
index 33694c5ce..c598d3ff1 100644
--- a/recipes-devtools/qemu/qemu-qoriq.inc
+++ b/recipes-devtools/qemu/qemu-qoriq.inc
@@ -146,6 +146,7 @@ make_qemu_wrapper() {
146 GTK_THEME=Adwaita 146 GTK_THEME=Adwaita
147 done 147 done
148} 148}
149make_qemu_wrapper[doc] = "Wrap the installed qemu-system-* binaries so they find the gdk-pixbuf loaders, fontconfig and GTK theme at runtime"
149 150
150# Disable kvm/virgl/mesa on targets that do not support it 151# Disable kvm/virgl/mesa on targets that do not support it
151PACKAGECONFIG:remove:darwin = "kvm virglrenderer glx gtk+" 152PACKAGECONFIG:remove:darwin = "kvm virglrenderer glx gtk+"
diff --git a/recipes-extended/testfloat/testfloat_2a.bb b/recipes-extended/testfloat/testfloat_2a.bb
index db87ae985..45ca5ae11 100644
--- a/recipes-extended/testfloat/testfloat_2a.bb
+++ b/recipes-extended/testfloat/testfloat_2a.bb
@@ -29,6 +29,7 @@ do_unpack2(){
29 echo -e "\nERROR: command dos2unix or fromdos not found\n" && return 1 29 echo -e "\nERROR: command dos2unix or fromdos not found\n" && return 1
30 fi 30 fi
31} 31}
32do_unpack2[doc] = "Move the SoftFloat sources into ${S} and convert the shipped DOS line endings"
32addtask do_unpack2 after do_unpack before do_patch 33addtask do_unpack2 after do_unpack before do_patch
33 34
34do_compile(){ 35do_compile(){
diff --git a/recipes-kernel/linux/linux-imx.inc b/recipes-kernel/linux/linux-imx.inc
index d704e2921..f0f8b5ba2 100644
--- a/recipes-kernel/linux/linux-imx.inc
+++ b/recipes-kernel/linux/linux-imx.inc
@@ -60,5 +60,6 @@ python kernel_devicetree_32bit_compatibility_update() {
60 bb.warn("Updating KERNEL_DEVICETREE, removing sub-folders for older kernel. Use -D for more details. Set KERNEL_DEVICETREE_32BIT_COMPATIBILITY_UPDATE = \"0\" to disable.") 60 bb.warn("Updating KERNEL_DEVICETREE, removing sub-folders for older kernel. Use -D for more details. Set KERNEL_DEVICETREE_32BIT_COMPATIBILITY_UPDATE = \"0\" to disable.")
61 d.setVar('KERNEL_DEVICETREE', output) 61 d.setVar('KERNEL_DEVICETREE', output)
62} 62}
63kernel_devicetree_32bit_compatibility_update[doc] = "Strip device tree sub-folders from KERNEL_DEVICETREE for older 32-bit kernels that keep the dtb files flat"
63addhandler kernel_devicetree_32bit_compatibility_update 64addhandler kernel_devicetree_32bit_compatibility_update
64kernel_devicetree_32bit_compatibility_update[eventmask] = "bb.event.RecipeParsed" 65kernel_devicetree_32bit_compatibility_update[eventmask] = "bb.event.RecipeParsed"
diff --git a/recipes-multimedia/imx-parser/imx-mp4-parser_git.bb b/recipes-multimedia/imx-parser/imx-mp4-parser_git.bb
index 1c70fb6e4..2c3628639 100644
--- a/recipes-multimedia/imx-parser/imx-mp4-parser_git.bb
+++ b/recipes-multimedia/imx-parser/imx-mp4-parser_git.bb
@@ -31,6 +31,7 @@ python __set_insane_skip() {
31 for p in d.getVar('PACKAGES').split(): 31 for p in d.getVar('PACKAGES').split():
32 d.setVar("INSANE_SKIP:%s" % p, "ldflags dev-so textrel buildpaths") 32 d.setVar("INSANE_SKIP:%s" % p, "ldflags dev-so textrel buildpaths")
33} 33}
34__set_insane_skip[doc] = "Skip QA checks that cannot be satisfied by the prebuilt parser binaries"
34 35
35do_package_qa[prefuncs] += "__set_insane_skip" 36do_package_qa[prefuncs] += "__set_insane_skip"
36 37