Seems like enum:s are 7-bits on arduino

Author: nicklasb

components/robusto/include/robusto_media_def.h

@@ -54,27 +54,27 @@ extern "C"
 
 /* Note: When adding a media type, update the conductor client.*/
 
-typedef enum e_media_type
+typedef enum  e_media_type
 {
-    robusto_mt_none = 0, 
+    robusto_mt_none = 0U, 
 
     /* Wireless */
-    robusto_mt_ble = 1,
-    robusto_mt_espnow = 2,
-    robusto_mt_lora = 4, 
+    robusto_mt_ble = 1U,
+    robusto_mt_espnow = 2U,
+    robusto_mt_lora = 4U, 
 
     /* Wired */
-    robusto_mt_i2c = 8,
+    robusto_mt_i2c = 8U,
 
-    robusto_mt_canbus = 16, 
+    robusto_mt_canbus = 16U, 
     /* TODO: To be implemented 
     
-    robusto_mt_umts = 32, // TODO: Only implemented as a gateway service. 
-    robusto_mt_wifi = 64,    // TODO: On what level should UMTS and wifi be implemented? Neither are direct and both use IP adressing
+    robusto_mt_umts = 32U, // TODO: Only implemented as a gateway service. 
+    robusto_mt_wifi = 64U,    // TODO: On what level should UMTS and wifi be implemented? Neither are direct and both use IP adressing
     // TTL? Onewire? Probably this should be implemented in some way. Or onewire. (perhaps separate for testing connections)
     */
     /* Special types */
-    robusto_mt_mock = 128
+    robusto_mt_mock = 128U
 } e_media_type;
 
 typedef uint8_t robusto_media_types;

components/robusto/network/src/peer/robusto_peer.c

@@ -325,7 +325,8 @@ robusto_media_t * get_media_info(robusto_peer_t * peer, e_media_type media_type)
     }
     #endif       
 
-    ROB_LOGE(peer_log_prefix, "Invalid media type (%hhu) in get_media_info for peer %s", media_type, peer->name);
+    ROB_LOGE(peer_log_prefix, "Invalid media type (%u) in get_media_info for peer %s", media_type, peer->name);
+    r_delay(1000);
     return NULL;
 }
 
@@ -339,6 +340,7 @@ robusto_media_t * get_media_info(robusto_peer_t * peer, e_media_type media_type)
  */
 float score_peer(robusto_peer_t *peer, e_media_type media_type, int data_length)
 {
+
     robusto_media_t *curr_info = get_media_info(peer, media_type);
 
     ROB_LOGD(peer_log_prefix, "Mt: %s, peer: %s ss: %"PRIu32", rs: %"PRIu32", sf: %"PRIu32", rf: %"PRIu32" ", 

components/robusto/network/src/peer/robusto_presentation.c

@@ -80,14 +80,14 @@ rob_ret_val_t robusto_send_presentation(robusto_peer_t *peer, robusto_media_type
     ROB_LOGD(presentation_log_prefix, ">> Presentation to send:");
     rob_log_bit_mesh(ROB_LOG_DEBUG, presentation_log_prefix, msg, msg_len);
     queue_state *q_state = robusto_malloc(sizeof(queue_state));
+    e_media_type supported_mt = get_host_supported_media_types();
     rob_ret_val_t ret_val_flag;
-    for (e_media_type media_type = 1; media_type < 256; media_type = media_type * 2)
+    for (uint16_t media_type = 1; media_type < 25; media_type = media_type * 2)
     {
-        if ((media_types & media_type) != media_type)
+        if ((!(media_types & media_type) || !(supported_mt & media_type)))
         {
             continue;
         }
-
         robusto_media_t *info = get_media_info(peer, media_type);
         /* Send presentation:
          * no receipt as a reply requires know outgoing id, which is only available after presentation is parsed

components/robusto/network/src/qos/qos_heartbeat.c

@@ -149,7 +149,7 @@ void peer_heartbeat(robusto_peer_t *peer)
         uint64_t last_heartbeat = curr_time - HEARTBEAT_IDLE_MARGIN_MS;
     }
 
-    for (e_media_type media_type = 1; media_type < 256; media_type = media_type * 2)
+    for (uint16_t media_type = 1; media_type < 256; media_type = media_type * 2)
     {
         if ((peer->supported_media_types & media_type) != media_type)
         {

components/robusto/network/src/qos/qos_scoring.c

@@ -120,11 +120,13 @@ void add_to_history(robusto_media_t *stats, bool sending, rob_ret_val_t result)
 uint32_t robusto_calc_suitability(e_media_type media_type, uint32_t payloadSize) {
     int phc;
     switch (media_type) {
+        #if CONFIG_ROBUSTO_SUPPORTS_CANBUS
         case robusto_mt_canbus:
         // CAN bus is great with small volumes, but quickly start losing out.
             phc = (payloadSize - CANBUS_MESSAGE_OFFSET <= 8) ? 100 : 80 - ((payloadSize - CANBUS_MESSAGE_OFFSET - 8) / 10);
         // TODO: Shoud stop showing the data length with a lot of data that won't be used in many cases, especially like the 20 bytes here
             break;
+        #endif
         case robusto_mt_espnow:
             phc = 100; // ESP-NOW is quite flexible
             break;
@@ -136,6 +138,7 @@ uint32_t robusto_calc_suitability(e_media_type media_type, uint32_t payloadSize)
             phc = 80 - (payloadSize - ROBUSTO_CRC_LENGTH / 10); // Decrease with size
             break;
         default:
+            ROB_LOGE(scoring_log_prefix, "robusto_calc_suitability: Unsupported media type: %s", media_type_to_str(media_type));
             phc = 50; // Default case
     }
     if (phc > 100) phc = 100; // Ensure not to exceed 100
@@ -231,7 +234,7 @@ void update_score(robusto_peer_t *peer, e_media_type media_type)
 void peer_scoring(robusto_peer_t *peer)
 {
 
-    for (e_media_type media_type = 1; media_type < 256; media_type = media_type * 2)
+    for (uint16_t media_type = 1; media_type < 256; media_type = media_type * 2)
     {
         if ((peer->supported_media_types & media_type) != media_type)
         {

components/robusto/network/src/qos/qos_state.c

@@ -290,7 +290,7 @@ rob_ret_val_t check_peer(robusto_peer_t *peer)
         return ROB_OK;
     }
     // Loop all media types and check each
-    for (e_media_type media_type = 1; media_type < 256; media_type = media_type * 2)
+    for (uint16_t media_type = 1; media_type < 256; media_type = media_type * 2)
     {
         if (
             !(

platformio.ini

@@ -35,50 +35,6 @@ extra_scripts =
   ${env.extra_scripts}
   test/test-coverage.py
 
-
-[env:TTGO-LoRa32-v1]
-; Used with the The TTGO LORA32 SX1278 board
-
-platform = espressif32@6.5.0 
-framework = espidf
-;platform_packages = platformio/framework-espidf@^3.50000.0
-board = ttgo-lora32-v1
-
-monitor_port = /dev/cu.wchusbserial531C0101801
-upload_port = /dev/cu.wchusbserial531C0101801
-test_port = /dev/cu.wchusbserial531C0101801
-;debug_test = *
-monitor_speed = 115200
-monitor_filters = esp32_exception_decoder
-;lib_extra_dirs = test
-; Private: MAC 58bf250541e0
-lib_deps = 
-  ${env.lib_deps}
-  adafruit/Adafruit SSD1306@^2.5.7
-
-; Better partioning
-;board_build.partitions = min_spiffs.csv
-
-[env:TTGO-T-Beam]
-; Used with the TTGO T-BEAM SX1262 TCXO board
-platform = espressif32@6.5.0 
-framework = espidf
-board = ttgo-t-beam
-monitor_port = /dev/cu.wchusbserial54FC0473971
-upload_port = /dev/cu.wchusbserial54FC0473971
-test_port = /dev/cu.wchusbserial54FC0473971
-
-;monitor_port = /dev/cu.wchusbserial537E0890921
-;upload_port = /dev/cu.wchusbserial537E0890921
-;test_port = /dev/cu.wchusbserial537E0890921
-board_build.partitions = min_spiffs.csv
-debug_test = *
-;monitor_port = /dev/cu.wchusbserial537E0890921
-;upload_port = /dev/cu.wchusbserial537E0890921
-monitor_speed = 115200
-monitor_filters = esp32_exception_decoder
-
-
 [env:Devkit_v4_no_network]
 ; Used with the The ESP32 devkit board
 platform = espressif32@6.5.0
@@ -96,3 +52,47 @@ monitor_filters = esp32_exception_decoder
 board_build.partitions = min_spiffs.csv
 
 
+
+; --------------------------
+; The RaspberryPi Pico uses picotool for uploading.
+; Note: There is nothing that identifies the board, so having multiple
+; boards might be a problem. TODO: Check how to specify this.
+[env:RaspberryPi_Pico]
+#platform = raspberrypi
+platform = https://github.com/maxgerhardt/platform-raspberrypi.git
+board = pico
+framework = arduino
+lib_extra_dirs = test, components
+lib_compat_mode = off
+test_framework= unity
+#test_framework= custom
+board_build.core = earlephilhower
+upload_protocol = picotool
+monitor_speed = 115200
+monitor_port = /dev/cu.usbmodem141101
+monitor_raw = yes
+test_port = /dev/cu.usbmodem141101
+;test_port = /Volumes/RPI-RP2
+;upload_port = /dev/cu.usbmodem141101
+;upload_port = /Volumes/RPI-RP2
+lib_deps = 
+  ${env.lib_deps}
+  Wire
+  ;https://github.com/nicklasb/RadioLib.git
+build_flags =
+    -DUNITY_OUTPUT_CHAR=robusto_test_log
+;board_build.hwids = [0x2E8A,0x00C0]
+;board_build.hwids = [0x2E8A,0x000A]
+; monitor_port = /dev/cu.usbmodem1423301
+; upload_port = /dev/cu.usbmodem1423201
+; test_port = /dev/cu.usbmodem1423201
+
+;upload_port = /dev/cu.wchusbserial1423201
+;test_port = /dev/cu.wchusbserial1423201
+
+; Do not re-enable these
+;lib_deps = feilipu/FreeRTOS
+;We will have to add build flags fo
+;build_flags =
+;  -DportUSE_WDTO=WDTO_15MS
+;/dev/cu.usbmodem4E8D368338001