lupyuen commented on PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#issuecomment-2601488835
> Maybe the `Toolchain.defs` is not exported or used in kernel build?
If I disable `hello-rust-cargo`, then`rv-virt:knsh64` builds OK:
https://gist.github.com/lupyuen/6a09e4c0
no1wudi commented on PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#issuecomment-2601450323
Maybe the `Toolchain.defs` is not exported or used in kernel build?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
lupyuen commented on PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#issuecomment-2601236272
Cool! Lemme know if you have plans to support `rv-virt:knsh64`, it fails at
`make import` with Missing Target. Thanks :-)
https://gist.github.com/lupyuen/cc4ec45a9d1342a1b367d792284
lupyuen commented on PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#issuecomment-2601215701
@no1wudi Ah OK the hello_rust_cargo stack was full on nsh64:
https://gist.github.com/lupyuen/fba254be9ef64d3856c52e1a14a1b9ec#file-gistfile1-txt-L533
```text
dump_tasks:PID
no1wudi commented on PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#issuecomment-2601222732
> I followed your config and increased the hello_rust_cargo Stack Size from
2KB to 8KB, and it works! I'll add this to my article thanks :-)
Yes for 64 bit platform, the stack us
no1wudi commented on PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#issuecomment-2601187377
@lupyuen It's OK for me:
```
Executing task: qemu-system-riscv64 -semihosting -M virt,aclint=on -cpu
rv64 -smp 8 -bios none -nographic -kernel nuttx/nuttx
ABC
NuttShel
lupyuen commented on PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#issuecomment-2601165388
Hi @no1wudi: I'm writing an article to introduce Rust Std Library on NuttX.
Today `rv-virt:nsh` works fine, would you be supporting `nsh64` and `knsh64`?
`rv-virt:nsh64` crashes
xiaoxiang781216 merged PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...
no1wudi commented on code in PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#discussion_r1909884163
##
examples/rust/hello/hello/Cargo.toml:
##
@@ -0,0 +1,22 @@
+[package]
Review Comment:
OK
--
This is an automated message from the Apache Git Service.
To re
xiaoxiang781216 commented on code in PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#discussion_r1909882646
##
examples/rust/hello/hello/Cargo.toml:
##
@@ -0,0 +1,22 @@
+[package]
Review Comment:
but it is a demo in source, not 3rd library. only 3rd package nee
no1wudi commented on PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#issuecomment-2581547310
> > However, as of now, there is no automated way to conduct these tests
directly on the development board
>
> Sounds like a fun challenge :-)
>
> * [Daily Automated Testi
lupyuen commented on PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#issuecomment-2581541276
> However, as of now, there is no automated way to conduct these tests
directly on the development board
Sounds like a fun challenge :-)
- [Daily Automated Testing for Milk-V
no1wudi commented on PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#issuecomment-2581537794
@lvanasse
> Sorry I have other questions @no1wudi, what if we want to use `cargo test`
to run test on the program? Would it be at runtime that we maybe instead launch
the test?
no1wudi commented on code in PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#discussion_r1909632880
##
examples/rust/hello/hello/Cargo.toml:
##
@@ -0,0 +1,22 @@
+[package]
Review Comment:
The outer "hello" directory serves as the application entry point on the
xiaoxiang781216 commented on code in PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#discussion_r1909224651
##
examples/rust/hello/hello/Cargo.toml:
##
@@ -0,0 +1,22 @@
+[package]
Review Comment:
why not put into examples/rust/hello/Cargo.toml
--
This is an
lvanasse commented on PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#issuecomment-2580788830
Sorry I have other questions @no1wudi, what if we want to use `cargo test`
to run test on the program? Would it be at runtime that we maybe instead launch
the test?
And what ab
lvanasse commented on PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#issuecomment-2580618220
> > What is your opinion on the idea of restarting the development of the
NuttX API in the
[rustix](https://github.com/bytecodealliance/rustix/tree/nuttx) crate?
>
> @lvanasse
lupyuen commented on PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#issuecomment-2580547312
> What is your opinion on the idea of restarting the development of the
NuttX API in the
[rustix](https://github.com/bytecodealliance/rustix/tree/nuttx) crate?
@lvanasse rustix
no1wudi commented on PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#issuecomment-2580536405
> My gut feel is that we should do it the Zephyr way. But make it
POSIX-like. Thus we should drop `nuttx-embedded-hal` altogether :-)
Thanks for your information, personally, I a
lupyuen commented on PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#issuecomment-2580514424
Actually I'm not too clear if we should stick to the Official Rust Embedded
HAL for NuttX. Here's why...
To blink an LED in Rust Embedded HAL: We fetch the GPIO, then switch it o
lvanasse commented on PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#issuecomment-2580528764
> Actually I'm not too clear if we should stick to the Official Rust
Embedded HAL for NuttX. Here's why...
>
> To blink an LED in Rust Embedded HAL: We fetch the GPIO, then swit
no1wudi commented on PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#issuecomment-2580465975
> @no1wudi would you suggest use your https://github.com/no1wudi/nuttx.rs
repo? Or should it be the NuttX project to have a nuttx-rs repository?
This repository is quite old
lvanasse commented on PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#issuecomment-2580384598
> @lvanasse nuttx-embedded-hal is a very old crate, we might need to revamp
it :-)
That's fine :). Just wanted to know what would be the way forward for having
the NuttX API in
lupyuen commented on PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#issuecomment-2580361080
@lvanasse nuttx-embedded-hal is a very old crate, we might need to revamp it
:-)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
lvanasse commented on PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#issuecomment-2580347287
> If you need to use NuttX-specific system interfaces, such as SPI/I2C/LCD
drivers, you will need to create additional crates for those
@no1wudi would you suggest use your https://
no1wudi commented on PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#issuecomment-2580067803
> @no1wudi let's try this #2934 allow application only do register
Emm, #2934 seems don't work for me.
--
This is an automated message from the Apache Git Service.
To respond t
xuxin930 commented on PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#issuecomment-2579993331
> > Sorry @no1wudi why is `hello_rust_cargo` missing from the NuttX Image?
https://gist.github.com/lupyuen/b81596576e393be0abac2b7687740052#file-gistfile1-txt-L870
> > ```
> > $
no1wudi commented on code in PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#discussion_r1908535401
##
examples/hello_rust_cargo/CMakeLists.txt:
##
@@ -0,0 +1,63 @@
+#
##
+# apps/examples/
no1wudi commented on PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#issuecomment-2579342644
@lupyuen Rust build with cmake need this PR:
https://github.com/apache/nuttx-apps/pull/2478, but it has beed reverted:
https://github.com/apache/nuttx-apps/pull/2480
Let's wait
no1wudi commented on PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#issuecomment-2579167561
> Sorry @no1wudi why is `hello_rust_cargo` missing from the NuttX Image?
https://gist.github.com/lupyuen/b81596576e393be0abac2b7687740052#file-gistfile1-txt-L870
>
> ```
> $ c
no1wudi commented on PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#issuecomment-2579146693
@lupyuen Please try build with make for now, I'll look into the issue with
cmake first.
--
This is an automated message from the Apache Git Service.
To respond to the message, please
lupyuen commented on PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#issuecomment-2579136744
https://gist.github.com/lupyuen/b81596576e393be0abac2b7687740052#file-gistfile1-txt-L870
Sorry @no1wudi why is `hello_rust_cargo` missing from the NuttX Image?
```text
$ cmake
no1wudi commented on PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#issuecomment-2579119138
@lupyuen I had submitted a fix for this issue:
https://github.com/rust-lang/libc/pull/4222
For now, please modify the file, add a force cast locally
/home/luppy/.rustup/toolchai
lupyuen commented on PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#issuecomment-2579108920
@no1wudi Sorry did I miss something? This is for Ubuntu 24.04.1 LTS x86_64:
https://gist.github.com/lupyuen/b81596576e393be0abac2b7687740052
```bash
$ rustup toolchain install n
no1wudi commented on PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#issuecomment-2579071008
@lvanasse @lupyuen Please refer to this document to test it:
https://github.com/apache/nuttx/pull/15476 😄
--
This is an automated message from the Apache Git Service.
To respond to
no1wudi commented on PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#issuecomment-2579022602
@lvanasse Thank you for offering your help. Regarding your questions, here's
the current situation:
> Do we use a specific crate to interact with NuttX? For instance, how
should
lvanasse commented on PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#issuecomment-2577743328
Hi @no1wudi,
I'm really looking forward to this PR—great job! I’d be happy to help if you
need an extra hand, especially with documentation.
I do have a few questions, an
no1wudi commented on PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#issuecomment-2577267870
@lupyuen I'll perpare a doc for it soon 😄
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go
lupyuen commented on PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#issuecomment-2577242902
@no1wudi Thanks for the great job! What command shall I run to build this
for QEMU RISC-V 64-bit? Might be good to include some minimal docs for this
release thanks :-)
--
This is a
no1wudi commented on code in PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#discussion_r1906711613
##
examples/hello_rust_cargo/CMakeLists.txt:
##
@@ -0,0 +1,63 @@
+#
##
+# apps/examples/
no1wudi commented on code in PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#discussion_r1906714934
##
examples/hello_rust_cargo/Makefile:
##
@@ -0,0 +1,32 @@
+
+# apps/examples/hello_rust/
no1wudi commented on PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#issuecomment-2475494867
I'll continue this work next week
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the sp
acassis commented on PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#issuecomment-2468806319
ping @no1wudi
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To
cederom commented on code in PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#discussion_r1826583750
##
examples/hello_rust_cargo/CMakeLists.txt:
##
@@ -0,0 +1,63 @@
+#
##
+# apps/examples/
xiaoxiang781216 commented on code in PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#discussion_r1811774256
##
examples/hello_rust_cargo/CMakeLists.txt:
##
@@ -0,0 +1,63 @@
+#
##
+# apps/e
nuttxpr commented on PR #2487:
URL: https://github.com/apache/nuttx-apps/pull/2487#issuecomment-2430723684
[**\[Experimental Bot, please feedback
here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues)
## PR Requirements Review
**Meeting Requireme
46 matches
Mail list logo