-
Notifications
You must be signed in to change notification settings - Fork 7.6k
samples: enable build tests for the ch32v006evt #90690
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
base: main
Are you sure you want to change the base?
Conversation
Note that Twister should fail at the moment, but then pass once #90438 is merged and I rebase. |
Hmm, Twister ran: https://github.com/zephyrproject-rtos/zephyr/actions/runs/15291782992/job/43012659289?pr=90690 but the results don't include the ch32v006evt, but do pass for boards like |
Twister only runs on |
Thanks. Still sounds worthwhile then, and the developer can do a |
I'm fine with the change but the commit message is off -- it should simply refer to the fact you're lowering the min RAM requirements |
The ch32v006evt has 8 KiB of RAM which is enough for many of the basic samples but, by default, Twister assumes that a sample needs 16 KiB of RAM. Mark some of the samples as needing 8 KiB of RAM so the ch32v006evt gets better CI coverage. Signed-off-by: Michael Hope <[email protected]>
|
The ch32v006evt has 8 KiB of RAM which is enough for many of the basic samples but, by default, Twister assumes that a sample needs 16 KiB of RAM.
Mark some of the samples as needing 8 KiB of RAM so the ch32v006evt gets better CI coverage.