Skip to content

Add support for rock3b board - #204

Merged
lsf37 merged 1 commit into
seL4:masterfrom
JDuchniewicz:rock3b
Dec 9, 2025
Merged

lsf37 merged 1 commit into
seL4:masterfrom
JDuchniewicz:rock3b

Conversation

@JDuchniewicz

Copy link
Copy Markdown
Contributor

Adds support for Radxa Rock3b board. Based on the implementation of rockpro64 board.

@Indanz Indanz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just realised you copied the current rockpro64 timer driver, and probably the serial driver too I guess (haven't checked).

Please don't do that, just adapt the current drivers so it's suitable for both boards and use a symlink here to make it compile.

Comment thread libplatsupport/plat_include/rk3568/platsupport/plat/clock.h Outdated
Comment thread libplatsupport/plat_include/rk3568/platsupport/plat/timer.h Outdated
Comment thread libplatsupport/plat_include/rk3568/platsupport/plat/timer.h
Comment thread libplatsupport/src/plat/rk3568/ltimer.c Outdated
Comment thread libplatsupport/src/plat/rk3568/ltimer.c Outdated
Comment thread libplatsupport/src/plat/rk3568/timer.c Outdated
Comment thread libplatsupport/src/plat/rk3568/timer.c Outdated
Comment thread libplatsupport/src/plat/rk3568/timer.c Outdated
Comment thread libplatsupport/src/plat/rk3568/timer.c Outdated
Comment thread libplatsupport/src/plat/rk3568/timer.c Outdated

@Indanz Indanz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Getting in the right direction, but still unnecessary code duplication remaining.

Comment thread libplatsupport/plat_include/rockpro64/platsupport/plat/timer.h
Comment thread libplatsupport/src/mach/rockchip/ltimer.c Outdated
Comment thread libplatsupport/src/mach/rockchip/timer.c
Comment thread libplatsupport/mach_include/rockchip/platsupport/mach/timer.h Outdated
Comment thread libplatsupport/src/mach/rockchip/ltimer.c Outdated
Comment thread libplatsupport/src/mach/rockchip/ltimer.c Outdated
Comment thread libplatsupport/src/mach/rockchip/timer.c Outdated
Comment thread libplatsupport/src/mach/rockchip/timer.c
Comment thread libplatsupport/src/mach/rockchip/timer.c Outdated
@Indanz Indanz added the hw-test sel4test hardware builds + runs label Dec 1, 2025

@Indanz Indanz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please tidy up the commit history a bit.

Once that's done and the copyright mess is fixed, it's good to go.

Comment thread libplatsupport/mach_include/rockchip/platsupport/mach/timer.h Outdated
Comment thread libplatsupport/plat_include/rockpro64/platsupport/plat/timer.h Outdated
Comment thread libplatsupport/src/mach/rockchip/serial.c Outdated
Comment thread libplatsupport/src/mach/rockchip/timer.c Outdated
Signed-off-by: Jakub Duchniewicz <j.duchniewicz@unsw.edu.au>
@JDuchniewicz

Copy link
Copy Markdown
Contributor Author

@Indanz is there anything else I should do to get this merged? I see some failures in the HW runs but I fail to understand why they are happening (my code did not touch these platforms at all).

@Indanz

Indanz commented Dec 9, 2025

Copy link
Copy Markdown
Contributor

@Indanz is there anything else I should do to get this merged?

I don't think so. Just bring seL4 up-to-date so I can merge it I think. @lsf37, do you have anything?

I see some failures in the HW runs but I fail to understand why they are happening (my code did not touch these platforms at all).

They're unrelated. Our CI system is not the most stable at the moment, not sure what's going on, people messing around in the server room or whatever.

@lsf37

lsf37 commented Dec 9, 2025

Copy link
Copy Markdown
Member

All good to go from my side. The remaining zcu106 failure is the known flakiness of that particular board in the machine queue.

@lsf37
lsf37 merged commit 07a7e15 into seL4:master Dec 9, 2025
153 of 170 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hw-test sel4test hardware builds + runs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants