Merge pull request #17554 from chrysn-pull-requests/rust-gcoap-add-saul

examples/rust-gcoap: Add SAUL

examples/rust-gcoap/Cargo.toml

@@ -23,7 +23,7 @@ riot-wrappers = { version = "^0.8", features = [ "set_panic_handler", "panic_han
 
 coap-message-demos = { git = "https://gitlab.com/chrysn/coap-message-demos/", default-features = false }
 coap-handler-implementations = "0.5"
-riot-coap-handler-demos = { git = "https://gitlab.com/etonomy/riot-module-examples/", features = [ "vfs" ] }
+riot-coap-handler-demos = { git = "https://gitlab.com/etonomy/riot-module-examples/", features = [ "vfs", "saul" ] }
 
 # While currently this exmple does not use any RIOT modules implemented in
 # Rust, that may change; it is best practice for any RIOT application that has

examples/rust-gcoap/Makefile

@@ -19,9 +19,13 @@ USEMODULE += ztimer_usec
 USEMODULE += ztimer_msec
 USEMODULE += ztimer_sec
 
+# for the "vfs" feature of riot-coap-handler-demos (and vfs.c)
 USEMODULE += vfs
 USEMODULE += constfs
 
+# for the "saul" feature of riot-coap-handler-demos
+USEMODULE += saul_default
+
 # Comment this out to disable code in RIOT that does safety checking
 # which is not needed in a production environment but helps in the
 # development process:
@@ -30,10 +34,13 @@ DEVELHELP ?= 1
 # Change this to 0 show compiler invocation lines by default:
 QUIET ?= 1
 
-# Add 3k extra stack: The Rust examples take more of it than gcoap expects,
+# Add 8k extra stack: The Rust examples take more of it than gcoap expects,
 # presumably because the example use the standard library's sting formatting
 # instead of one of the more optimized formatters.
-CFLAGS += -DGCOAP_STACK_SIZE='(THREAD_STACKSIZE_DEFAULT+DEBUG_EXTRA_STACKSIZE+sizeof(coap_pkt_t)+1024)'
+CFLAGS += -DGCOAP_STACK_SIZE='(THREAD_STACKSIZE_DEFAULT+DEBUG_EXTRA_STACKSIZE+sizeof(coap_pkt_t)+8192)'
+# This thread needs some more stack for printing the addresses, once more being
+# hit by string formatting.
+CFLAGS += -DTHREAD_STACKSIZE_MAIN='(THREAD_STACKSIZE_DEFAULT+THREAD_EXTRA_STACKSIZE_PRINTF+2048)'
 
 # The name of crate (as per Cargo.toml package name, but with '-' replaced with '_')
 APPLICATION_RUST_MODULE = rust_gcoap

examples/rust-gcoap/Makefile.ci

@@ -1,12 +1,26 @@
 BOARD_INSUFFICIENT_MEMORY := \
     airfy-beacon \
+    b-l072z-lrwan1 \
     blackpill-stm32f103c8 \
+    blackpill-stm32f103cb \
     bluepill-stm32f030c8 \
     bluepill-stm32f103c8 \
+    bluepill-stm32f103cb \
     calliope-mini \
+    cc1350-launchpad \
+    cc2650-launchpad \
+    cc2650stk \
+    e104-bt5010a-tb \
+    e104-bt5011a-tb \
+    gd32vf103c-start \
+    hifive1 \
+    hifive1b \
     i-nucleo-lrwan1 \
     im880b \
+    lsn50 \
+    maple-mini \
     microbit \
+    microbit-v2 \
     nrf51dongle \
     nrf6310 \
     nucleo-c031c6 \
@@ -15,19 +29,22 @@ BOARD_INSUFFICIENT_MEMORY := \
     nucleo-f042k6 \
     nucleo-f070rb \
     nucleo-f072rb \
+    nucleo-f103rb \
     nucleo-f302r8 \
     nucleo-f303k8 \
     nucleo-f334r8 \
     nucleo-l011k4 \
     nucleo-l031k6 \
     nucleo-l053r8 \
+    nucleo-l073rz \
+    olimexino-stm32 \
+    opencm904 \
     samd10-xmini \
-    saml10-xpro \
-    saml11-xpro \
     seeedstudio-gd32 \
     sipeed-longan-nano \
     sipeed-longan-nano-tft \
     slstk3400a \
+    spark-core \
     stk3200 \
     stm32f030f4-demo \
     stm32f0discovery \

examples/rust-gcoap/src/lib.rs

@@ -24,6 +24,7 @@ fn main() {
     let handler = coap_message_demos::full_application_tree(None)
         .below(&["ps"], riot_coap_handler_demos::ps::ps_tree())
         .below(&["vfs"], riot_coap_handler_demos::vfs::vfs("/const"))
+        .below(&["saul"], riot_coap_handler_demos::saul::SaulHandler::new(&["saul"]))
         .with_wkc()
         ;
     let mut handler = riot_wrappers::coap_handler::v0_2::GcoapHandler(handler);