diff options
| author | Luciano Dittgen <luciano.dittgen@ossystems.com.br> | 2026-07-29 10:26:59 -0300 |
|---|---|---|
| committer | Luciano Dittgen <luciano.dittgen@ossystems.com.br> | 2026-08-03 16:13:30 -0300 |
| commit | a066ce9bfe828855b471e65b72871a3b019a80de (patch) | |
| tree | fcf883eff1bad901be370c86615b3415f149ff1b | |
| parent | 684c3b8b5e28ba2baa69e3913ec0e1b114a07543 (diff) | |
| download | meta-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.bb | 3 | ||||
| -rw-r--r-- | recipes-bsp/u-boot/libubootenv_%.bbappend | 1 | ||||
| -rw-r--r-- | recipes-devtools/qemu/qemu-qoriq.inc | 1 | ||||
| -rw-r--r-- | recipes-extended/testfloat/testfloat_2a.bb | 1 | ||||
| -rw-r--r-- | recipes-kernel/linux/linux-imx.inc | 1 | ||||
| -rw-r--r-- | recipes-multimedia/imx-parser/imx-mp4-parser_git.bb | 1 |
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 | } |
| 22 | deploy_for_mx8[doc] = "Deploy the Cadence HDMI firmware used by the i.MX 8 boot container" | ||
| 22 | 23 | ||
| 23 | deploy_for_mx8m() { | 24 | deploy_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 | } |
| 34 | deploy_for_mx8m[doc] = "Deploy the Synopsys DDR and Cadence DP/HDMI firmware used by the i.MX 8M boot container" | ||
| 33 | 35 | ||
| 34 | deploy_for_mx9() { | 36 | deploy_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 | } |
| 42 | deploy_for_mx9[doc] = "Deploy the Synopsys DDR firmware used by the i.MX 9 boot container" | ||
| 40 | 43 | ||
| 41 | python () { | 44 | python () { |
| 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): | |||
| 38 | python fixup_uboot_config_dependency_handler() { | 38 | python fixup_uboot_config_dependency_handler() { |
| 39 | fixup_uboot_config_dependency(d) | 39 | fixup_uboot_config_dependency(d) |
| 40 | } | 40 | } |
| 41 | fixup_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 | ||
| 42 | fixup_uboot_config_dependency_handler[eventmask] = "bb.event.RecipePreFinalise" | 43 | fixup_uboot_config_dependency_handler[eventmask] = "bb.event.RecipePreFinalise" |
| 43 | addhandler fixup_uboot_config_dependency_handler | 44 | addhandler 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 | } |
| 149 | make_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 |
| 151 | PACKAGECONFIG:remove:darwin = "kvm virglrenderer glx gtk+" | 152 | PACKAGECONFIG: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 | } |
| 32 | do_unpack2[doc] = "Move the SoftFloat sources into ${S} and convert the shipped DOS line endings" | ||
| 32 | addtask do_unpack2 after do_unpack before do_patch | 33 | addtask do_unpack2 after do_unpack before do_patch |
| 33 | 34 | ||
| 34 | do_compile(){ | 35 | do_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 | } |
| 63 | kernel_devicetree_32bit_compatibility_update[doc] = "Strip device tree sub-folders from KERNEL_DEVICETREE for older 32-bit kernels that keep the dtb files flat" | ||
| 63 | addhandler kernel_devicetree_32bit_compatibility_update | 64 | addhandler kernel_devicetree_32bit_compatibility_update |
| 64 | kernel_devicetree_32bit_compatibility_update[eventmask] = "bb.event.RecipeParsed" | 65 | kernel_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 | ||
| 35 | do_package_qa[prefuncs] += "__set_insane_skip" | 36 | do_package_qa[prefuncs] += "__set_insane_skip" |
| 36 | 37 | ||
