-
-
Notifications
You must be signed in to change notification settings - Fork 2.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
rockchip64: fix rocks0 patch breaking current compilation #7815
Conversation
Yes, this patch really shreds the upstream device tree apart. You should probably only keep the audio related Also look like you should drop |
@paolosabatino |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've nothing against reorganizing the .dts files.
I am very much a neophyte in the wacky ways of device trees.
Please revise this patch to work with the current mainline.
Be sure to @brentr in your comment so I see it. I'll test try to test it that evening, hopefully before it gets stale.
This ready? |
No. It no longer applies atop the current head. I just verified that the current head builds a working image. |
Also verified that RockPI-S image works when build from the main branch. |
Oh, that was manual intervention of some other problem and i forgot to bump version of the package. If released image has package v1.0 and repository has package version v1.0, it will reinstall. But package version on the repo could have older kernel ... this anomaly will be fixed asap. Ignore this upgrade - if build works, lets merge, versioning is unrelated problem. |
@brentr @igorpecovnik I tried to compile on my local branch and it correctly applies on top of current (kernel 6.12.14): and the kernel builds fine too. I rebased my local branch to the latest main branch and now the patch does not apply anymore, with some hunks failing. It clashes with this other rk3308 patch: https://github.com/armbian/build/blob/main/patch/kernel/archive/rockchip64-6.12/board-rocks0-0001-Revert-arm64-dts-rockchip-Fix-sdmmc-access-on-rk3308.patch This evening I can fix the clashing, if the timing is ok for you @igorpecovnik. Nonetheless, the patch in this branch requires a major overhaul to correctly integrate the mainline dts, and not tearing it apart like now. I could lend a hand to do it after the 25.02 release. |
Timing is OK, we are still in the 2025/2 release window. |
97a4aaa
to
5e1b1a3
Compare
@igorpecovnik ok, this should be good enough to go for 25.02 |
changes have been accomodated to build against latest mainline sources
@brentr I "dismissed" the change request from your part since it was blocking the PR from merge and we're a bit tight with timing. Nothing personal, it's just I made the changes and now everything compiles fine against mainline kernel. Please write here if you got any other issue. |
@paolosabatino @igorpecovnik |
Description
A patch that reworks rocks0 board has been reworked as well to compile on kernel 6.12.13.
@brentr I just did a quick rework of the patch, but cannot test if it is resulting in a working device or not. Perhaps it would be better, in the future, to rewrite the patch to fit the changes on top of the mainline kernel device tree?
How Has This Been Tested?
Checklist: