Skip to content

fix: add zstd package in essential hooks#112

Merged
RadxaYuntian merged 1 commit intoRadxaOS-SDK:mainfrom
imguoguo:install-zstd-in-essential-hooks
Oct 24, 2025
Merged

fix: add zstd package in essential hooks#112
RadxaYuntian merged 1 commit intoRadxaOS-SDK:mainfrom
imguoguo:install-zstd-in-essential-hooks

Conversation

@imguoguo
Copy link
Contributor

@github-actions
Copy link
Contributor

No linked issues found. Please add the corresponding issues in the pull request description.
Use GitHub automation to close the issue when a PR is merged

@RadxaYuntian
Copy link
Contributor

CI挂了,你确认一下dkms日志是为什么报错
可以用--debug参数来避免rsdk编译失败后不会留下rootfs内容

@imguoguo
Copy link
Contributor Author

CI挂了,你确认一下dkms日志是为什么报错 可以用--debug参数来避免rsdk编译失败后不会留下rootfs内容

我检查了一下日志:

make[2]: *** No rule to make target '/var/lib/dkms/radxa-overlays/0.2.6/build/arch/arm64/boot/dts/qcom/overlays/qcs6490-radxa-dragon-q6a-cam1-imx577.dtbo', needed by '/var/lib/dkms/radxa-overlays/0.2.6/build/arch/arm64/boot/dts/qcom/overlays/'.  Stop.
make[1]: *** [scripts/Makefile.build:500: /var/lib/dkms/radxa-overlays/0.2.6/build/arch/arm64/boot/dts/qcom/overlays] Error 2
make[1]: *** Waiting for unfinished jobs....

猜测是旧版内核构建到没受支持的 dtso 文件导致的?github action test 的时候给测试的 yaml 加了 CONFIG_ARCH_QCOM=n 来成功构建了,但是构建镜像时不会使用这个参数,会继续尝试构建 qcom 的 dtbo

@RadxaYuntian
Copy link
Contributor

看看那些内核是不是有开CONFIG_ARCH_QCOM。也许overlay那边还需要加上内核版本的条件编译?

@imguoguo
Copy link
Contributor Author

看看那些内核是不是有开CONFIG_ARCH_QCOM。也许overlay那边还需要加上内核版本的条件编译?

我检查了一下 Radxa-zero-2pro 的内核是有开 CONFIG_ARCH_QCOM 的,另外再仔细检查了一下,猜测是:对于新版内核来说,我们有 dts 复制到 dtso 的操作,所以新版内核可以直接用 dtso 构建出 dtbo ,旧版内核没有 dtso 的支持,脚本也没有 dtso 复制到 dts ,所以no rule to make target。

同时,dtso 的支持是在这个 commit 被引入的: https://github.com/torvalds/linux/blob/363547d2191cbc32ca954ba75d72908712398ff2/scripts/Makefile.lib#L411

@RadxaYuntian
Copy link
Contributor

是的,所以看看怎么在makefile里面检查linux内核版本

@RadxaYuntian RadxaYuntian added this pull request to the merge queue Oct 24, 2025
Merged via the queue into RadxaOS-SDK:main with commit 3d2f79e Oct 24, 2025
46 of 54 checks passed
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.

2 participants