Skip to content
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

risc-v/qemu-rv: Add Build Config for leds64_zig #13186

Merged
merged 1 commit into from
Sep 5, 2024

Conversation

kassane
Copy link
Contributor

@kassane kassane commented Aug 26, 2024

Summary

This PR adds a new Build Config rv-virt:leds64_zig that builds the Zig App leds_zig for QEMU RISC-V 64-bit.

based on: #12862

Impact

N/A

Testing

# $PWD == nuttx_path
$ ./tools/configure.sh rv-virt:leds64_zig
$ qemu-system-riscv64 \
    -semihosting \
    -M virt,aclint=on \
    -cpu rv64 -smp 8 \
    -bios none \
    -kernel nuttx -nographic
ABCboard_userled: LED 1 set to 0
board_userled: LED 2 set to 0
board_userled: LED 3 set to 0

NuttShell (NSH) NuttX-12.6.0-RC1
nsh> leds_zig
leds_main: Starting the led_daemon
leds_main: led_daemon started

led_daemon (pid# 4): Running
led_daemon: Opening /dev/userleds
led_daemon: Supported LEDs 7
led_daemon: LED set 1
board_userled: LED 1 set to 1
board_userled: LED 2 set to 0
board_userled: LED 3 set to 0
nsh> led_daemon: LED set 0
board_userled: LED 1 set to 0
board_userled: LED 2 set to 0
board_userled: LED 3 set to 0
led_daemon: LED set 1
board_userled: LED 1 set to 1
board_userled: LED 2 set to 0
board_userled: LED 3 set to 0
# [...] SKIP

Reference

cc: @xiaoxiang781216

@xiaoxiang781216
Copy link
Contributor

@kassane we need fix #13139 (comment) to pass ci.

@kassane

This comment has been minimized.

@kassane

This comment was marked as resolved.

@simbit18
Copy link
Contributor

simbit18 commented Sep 3, 2024

@kassane please rebase now NuttX Docker Image has Zig.

This PR adds a new Build Config `rv-virt:leds64_zig` that builds the Zig App `leds_zig` for QEMU RISC-V 64-bit.
ref.: apache/nuttx-apps#2534
@kassane
Copy link
Contributor Author

kassane commented Sep 3, 2024

Finally ⚡

CI: https://github.com/apache/nuttx/actions/runs/10682497286/job/29608588514?pr=13186#step:7:451

====================================================================================
Configuration/Tool: rv-virt/leds64_zig
2024-09-03 12:41:50
------------------------------------------------------------------------------------
  Cleaning...
  Configuring...
  Building NuttX...
riscv-none-elf-ld: warning: /github/workspace/sources/nuttx/nuttx has a LOAD segment with RWX permissions
  Normalize rv-virt/leds64_zig

@simbit18
Copy link
Contributor

simbit18 commented Sep 4, 2024

@xiaoxiang781216 now zig is in NuttX image and PR is ok.

@xiaoxiang781216
Copy link
Contributor

@xiaoxiang781216 now zig is in NuttX image and PR is ok.

Thanks.

@xiaoxiang781216 xiaoxiang781216 merged commit 1936e68 into apache:master Sep 5, 2024
29 checks passed
@kassane kassane deleted the enable_leds-zig branch September 5, 2024 11:16
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.

3 participants