Skip to content

bundle camx & iris-vpu DMKS modules - #80

Draft
Christopher Obbard (obbardc) wants to merge 4 commits into
qcom/debian/latestfrom
wip/obbardc/more-dkms
Draft

bundle camx & iris-vpu DMKS modules#80
Christopher Obbard (obbardc) wants to merge 4 commits into
qcom/debian/latestfrom
wip/obbardc/more-dkms

Conversation

@obbardc

@obbardc Christopher Obbard (obbardc) commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Depends on:

Tested with local kernel build with camx-dkms with change above from qli-staging.

@obbardc
Christopher Obbard (obbardc) marked this pull request as ready for review August 28, 2026 16:02
@obbardc
Christopher Obbard (obbardc) marked this pull request as draft August 28, 2026 16:08
On failure the script printed tail -n 300 of make.log and nothing else, which
is the wrong end of the file for any module whose build helper keeps going
after a failure. Such a helper reports only at the very end, so a compile error
early on sits thousands of lines above the tail and never appears in the build
log, while the tail shows the routine output of whatever ran last and reads as
though nothing went wrong.

Print the whole log instead, prefixed with its path and length. The private
dkms tree is deleted on EXIT, so the inline copy is the only surviving record
in CI and there is nothing to be gained by truncating it.

Signed-off-by: Christopher Obbard <chris.obbard@oss.qualcomm.com>
…tely

BUILD_EXCLUSIVE_CONFIG may name several kernel options, space separated, all
of which must be set for dkms to build the module. The gate report tested the
whole string as a single symbol, which never matches, so every gate was
reported as unset:

    | CONFIG_ARCH_QCOM CONFIG_PM_DEVFREQ CONFIG_SYNC_FILE is NOT set

That is worse than saying nothing: it accuses the kernel config whatever the
real cause was, and it did exactly that while the actual failure was a broken
source tree. Both kgsl and iris-vpu declare multi-option gates, so the report
was wrong for every module that reached it.

Evaluate each option separately.

Signed-off-by: Christopher Obbard <chris.obbard@oss.qualcomm.com>
Add camx to debian/dkms-modules so the Qualcomm camx driver is built
against the kernel being packaged and shipped inside
linux-image-<KVER>-qcom and declare camx-dkms in Build-Depends.

Signed-off-by: Christopher Obbard <chris.obbard@oss.qualcomm.com>
Add iris-vpu to debian/dkms-modules so the Qualcomm video accelerator
driver is built against the kernel being packaged and shipped inside
linux-image-<KVER>-qcom and declare iris-vpu-dkms in Build-Depends.

Signed-off-by: Christopher Obbard <chris.obbard@oss.qualcomm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant