lora: Support RISC-V build, update to latest libtock-c and move to examples

.gitmodules

@@ -4,9 +4,9 @@
 [submodule "lvgl/lvgl"]
 	path = lvgl/lvgl
 	url = https://github.com/littlevgl/lvgl.git
-[submodule "examples/lora/RadioLib"]
-	path = wip/lora/RadioLib
-	url = https://github.com/jgromes/RadioLib.git
 [submodule "libopenthread/openthread"]
 	path = libopenthread/openthread
 	url = https://github.com/openthread/openthread.git
+[submodule "libradio/RadioLib"]
+	path = libradio/RadioLib
+	url = https://github.com/jgromes/RadioLib.git

TockLibrary.mk

@@ -29,10 +29,6 @@ $(call check_defined, LIBNAME)
 $(call check_defined, $(LIBNAME)_DIR)
 $(call check_defined, $(LIBNAME)_SRCS)
 
-ifeq ($(strip $($(LIBNAME)_SRCS)),)
-  $(error Library "$(LIBNAME)" has no SRCS?)
-endif
-
 # directory for built output
 $(LIBNAME)_BUILDDIR := $($(LIBNAME)_DIR)/build
 

wip/lora/README.md → examples/lora/README.md

@@ -23,3 +23,14 @@ RadioLib.
 Note that the Makefiles will do this automatically when
 you run `make` in a subdirectory, but if you want to do
 it manually you can run the `build-RadioLib.sh` script.
+
+### Building
+
+To bulid any example just enter the directory and type
+
+```shell
+make
+```
+
+This will build RadioLib and then the example application for
+both ARM and RISC-V architectures.

examples/lora/sensor-receive/Makefile

@@ -0,0 +1,14 @@
+# Makefile for user application
+
+# Specify this directory relative to the current application.
+TOCK_USERLAND_BASE_DIR = ../../../
+
+# Which files to compile.
+CXX_SRCS := $(wildcard *.cc)
+
+# Use the libtock-c Make system
+include $(TOCK_USERLAND_BASE_DIR)/libradio/Makefile
+
+include $(TOCK_USERLAND_BASE_DIR)/libradio/Makefile.app
+
+include $(TOCK_USERLAND_BASE_DIR)/AppMakefile.mk

examples/lora/sensor-transmit/Makefile

@@ -0,0 +1,14 @@
+# Makefile for user application
+
+# Specify this directory relative to the current application.
+TOCK_USERLAND_BASE_DIR = ../../../
+
+# Which files to compile.
+CXX_SRCS := $(wildcard *.cc)
+
+# Use the libtock-c Make system
+include $(TOCK_USERLAND_BASE_DIR)/libradio/Makefile
+
+include $(TOCK_USERLAND_BASE_DIR)/libradio/Makefile.app
+
+include $(TOCK_USERLAND_BASE_DIR)/AppMakefile.mk

wip/lora/sensor-transmit/main.cc → examples/lora/sensor-transmit/main.cc

@@ -13,8 +13,8 @@
 #include "libtockHal.h"
 
 // Include some libtock-c helpers
-#include <sensors/humidity.h>
-#include <sensors/temperature.h>
+#include <libtock-sync/sensors/humidity.h>
+#include <libtock-sync/sensors/temperature.h>
 
 #define BUFFER_LEN 64
 
@@ -47,17 +47,17 @@ int main(void) {
   }
   printf("success!\r\n");
 
-  int temp      = 0;
-  unsigned humi = 0;
+  int temp = 0;
+  int humi = 0;
 
   // loop forever
   for ( ;;) {
     // Ensure there are no pending callbacks
     yield_no_wait();
 
     // Read some sensor data from the board
-    temperature_read_sync(&temp);
-    humidity_read_sync(&humi);
+    libtocksync_temperature_read(&temp);
+    libtocksync_humidity_read(&humi);
 
     snprintf(buffer, BUFFER_LEN, "Temp: %d, Hum: %u", temp, humi);
 

libradio/Makefile

@@ -0,0 +1,39 @@
+LIBNAME := RadioLib
+
+TOCK_USERLAND_BASE_DIR ?= ../
+
+$(LIBNAME)_DIR := $(TOCK_USERLAND_BASE_DIR)/libradio/$(LIBNAME)
+
+# We ideally want this:
+#     $(LIBNAME)_SRCS  :=                                     \
+#     $($(LIBNAME)_DIR)/src/Module.cpp                    \
+#     $(wildcard $($(LIBNAME)_DIR)/src/*.cpp)             \
+#     $(wildcard $($(LIBNAME)_DIR)/src/*/*.cpp)           \
+#     $(wildcard $($(LIBNAME)_DIR)/src/*/*/*.cpp)
+#
+# But due to this in the GCC arm binary in the CI:
+#
+# ```
+# (insn 179 178 46 8 (set (reg:SI 3 r3 [168])
+#         (mem/u/c:SI (plus:SI (reg:SI 12 ip [169])
+#                 (unspec:SI [
+#                         (symbol_ref/u:SI ("*.LC1") [flags 0x2])
+#                     ] UNSPEC_PIC_SYM)) [0  S4 A32])) "../../../RadioLib/src/modules/LR11x0/LR11x0.cpp":638:16 929 {*thumb1_movsi_insn}
+#      (expr_list:REG_EQUAL (symbol_ref/u:SI ("*.LC1") [flags 0x2])
+#         (nil)))
+# during RTL pass: postreload
+# ../../../RadioLib/src/modules/LR11x0/LR11x0.cpp:650:1: internal compiler error: in extract_constrain_insn, at recog.c:2195
+# ```
+#
+# we need to drop LR11x0
+$(LIBNAME)_SRCS  :=                                     \
+    $(wildcard $($(LIBNAME)_DIR)/src/*.cpp)             \
+    $(wildcard $($(LIBNAME)_DIR)/src/utils/*.cpp)       \
+    $(wildcard $($(LIBNAME)_DIR)/src/modules/C*/.cpp)   \
+    $(wildcard $($(LIBNAME)_DIR)/src/modules/LL*/*.cpp) \
+    $(wildcard $($(LIBNAME)_DIR)/src/modules/RF*/*.cpp) \
+    $(wildcard $($(LIBNAME)_DIR)/src/modules/S*/*.cpp)   \
+    $(wildcard $($(LIBNAME)_DIR)/src/modules/n*/*.cpp)   \
+    $(wildcard $($(LIBNAME)_DIR)/src/protocols/*/*.cpp) \
+
+include $(TOCK_USERLAND_BASE_DIR)/TockLibrary.mk

libradio/Makefile.app

@@ -0,0 +1,5 @@
+# Include the core RadioLib headers
+override CPPFLAGS += -isystem $(TOCK_USERLAND_BASE_DIR)/libradio/RadioLib/src
+
+# Include the Tock specific headers
+override CPPFLAGS += -isystem $(TOCK_USERLAND_BASE_DIR)/libradio/RadioLib/examples/NonArduino/Tock