Fixed some config params

.gitignore

@@ -1,4 +1,16 @@
 *.pyc
 BUILD
 mbed_config.h
-mbed_app.json
+
+mbed-os
+Cayenne-LPP
+mbed-lora-radio-drv
+sensors/DHT
+sensors/HX711
+
+*.bin
+devices.txt
+
+# mbed vscode export generated files
+GettingStarted.html
+.vscode

README.md

@@ -13,12 +13,30 @@ mbed toolchain GCC_ARM
 mbed target NUCLEO_L073RZ
 # set-up the compiler path if needed
 mbed config -G GCC_ARM_PATH /opt/gcc-arm-none-eabi-7-2017-q4-major/bin
-cp mbed_app.json.sample mbed_app.json
+```
-# modify lora.device-eui, lora.application-eui and lora.application-key in mbed_app.json
+
+### Bootstrap mode
+
+This mode is used to initialize a mbed NVStore on the flash and store board-specific parameters.
+To compile the bootstrap binary you have to set "mode-bootstrap" to true into mbed_app.json and set
+lorawan EUIs and key.
+
+```
 mbed compile --profile profiles/release.json
 ```
 
-Then you can use stm32flash or - may be better - a stlink interface to copy the generated .bin file onto the RAK811.
+Then you can use flash the board and start it. You can check the serial console to see reporting.
+
+### Production mode
+
+Just reset "mode-boostrap" to false and compile it (you can delete BUILD folder from previous compilation) :
+
+```
+mbed compile --profile profiles/release.json
+```
+
+In this mode, lorawan parameters are ignored (because the program get it from the flash) but
+leave it as-is because they are used to determine the stored size ot the euis and key.
 
 ## Notes
 

main.cpp

@@ -6,6 +6,61 @@
 #include "mbed.h"
 #include "nvstore.h"
 
+#define NVKEY_LORA_DEVICE_EUI 0x01
+#define NVKEY_LORA_APP_EUI    0x02
+#define NVKEY_LORA_APP_KEY    0x03
+#define NVKEY_HX711_OFFSET    0x04
+#define NVKEY_HX711_SCALE     0x05
+
+#if MBED_CONF_APP_MODE_BOOTSTRAP
+
+const static uint8_t dev_eui[] = MBED_CONF_LORA_DEVICE_EUI;
+const static uint8_t app_eui[] = MBED_CONF_LORA_APPLICATION_EUI;
+const static uint8_t app_key[] = MBED_CONF_LORA_APPLICATION_KEY;
+
+int main(void) {
+    int rc;
+    uint16_t key;
+
+    NVStore &nvstore = NVStore::get_instance();
+
+    rc = nvstore.init();
+    if (rc != NVSTORE_SUCCESS) {
+        printf("Unable to init NVStore : %d\n", rc);
+        return -1;
+    }
+    rc = nvstore.reset();
+    if (rc != NVSTORE_SUCCESS) {
+        printf("Unable to reset NVStore : %d\n", rc);
+        return -1;
+    }
+    printf("NVStore initiated and reseted\n");
+
+    key = NVKEY_LORA_DEVICE_EUI;
+    if (nvstore.set(key, sizeof(dev_eui), dev_eui) != NVSTORE_SUCCESS) {
+        printf("Unable to write LoRa Device EUI\n");
+    }
+    key = NVKEY_LORA_APP_EUI;
+    if (nvstore.set(key, sizeof(app_eui), app_eui) != NVSTORE_SUCCESS) {
+        printf("Unable to write LoRa Application EUI\n");
+    }
+    key = NVKEY_LORA_APP_KEY;
+    if (nvstore.set(key, sizeof(app_key), app_key) != NVSTORE_SUCCESS) {
+        printf("Unable to write LoRa Application key\n");
+    }
+
+    while(1) {
+        printf("Bootstrap done\n");
+        wait(1);
+    }
+}
+
+#else // MBED_CONF_APP_MODE_BOOTSTRAP
+
+static uint8_t dev_eui[] = MBED_CONF_LORA_DEVICE_EUI;
+static uint8_t app_eui[] = MBED_CONF_LORA_APPLICATION_EUI;
+static uint8_t app_key[] = MBED_CONF_LORA_APPLICATION_KEY;
+
 #if MBED_CONF_APP_DHT_ENABLED
 #include "DHT.h"
 #endif
@@ -38,22 +93,28 @@ static CayenneLPP cayenne(51);
 
 #if MBED_CONF_APP_BATTERY_ENABLED
 AnalogIn bat(MBED_CONF_APP_BATTERY_ADC);
-AnalogIn vrefint(ADC_VREF);
+AnalogIn vref(ADC_VREF);
 
 static float battery_voltage() {
     float vbat = bat.read();
-    double vdd = (1.224f) / vrefint.read();
+    float vrefin = vref.read();
 
-    return vbat * MBED_CONF_APP_BATTERY_RATIO * vdd;
+    float vrefcal = (float)(*(uint16_t*)VREFINT_CAL_ADDR) * (1.0f / (float)0xFFF);
+    float vdda = 3.0f * vrefcal / vrefin;
+
+    return vbat * vdda * MBED_CONF_APP_BATTERY_RATIO;
 }
 
 static uint8_t battery_level() {
     float vbat = battery_voltage();
 
-    double ratio = 253.0f / (4.2f - 3.3f);
+    if (vbat >= MBED_CONF_APP_BATTERY_MAX) {
-    int value = ratio * (vbat - 3.4f) + 1.0f;
+        return 254;
-
+    } else if (vbat > MBED_CONF_APP_BATTERY_MIN && vbat < MBED_CONF_APP_BATTERY_MAX) {
-    return value < 1 ? 1 : (value > 254 ? 254 : value);
+        return ( 253 * ( vbat - MBED_CONF_APP_BATTERY_MIN ) ) / ( MBED_CONF_APP_BATTERY_MAX - MBED_CONF_APP_BATTERY_MIN ) + 1;
+    } else {
+        return 1;
+    }
 }
 #endif
 
@@ -67,35 +128,45 @@ HX711 loadcell(MBED_CONF_APP_HX711_DATA, MBED_CONF_APP_HX711_CLK);
 
 int main(void) {
     lorawan_status_t retcode;
-
+    lorawan_connect_t connect;
-#if NVSTORE_ENABLED
-    NVStore &nvstore = NVStore::get_instance();
     int rc;
     uint16_t key;
-    // Values read or written by NVStore need to be aligned to a uint32_t address (even if their sizes
+    uint16_t readsize;
-    // aren't)
-    uint32_t value;
-    // Initialize NVstore. Note that it can be skipped, as it is lazily called by all other APIs
-    rc = nvstore.init();
-    printf("Init NVStore. \n");
-    // Show NVStore size, maximum number of keys and area addresses and sizes
-    printf("NVStore size is %d.\n", nvstore.size());
-    printf("NVStore max number of keys is %d (out of %d possible ones in this flash configuration).\n",
-            nvstore.get_max_keys(), nvstore.get_max_possible_keys());
-    printf("NVStore areas:\n");
-    for (uint8_t area = 0; area < NVSTORE_NUM_AREAS; area++) {
-        uint32_t area_address;
-        size_t area_size;
-        nvstore.get_area_params(area, area_address, area_size);
-        printf("Area %d: address 0x%08lx, size %d (0x%x).\n", area, area_address, area_size, area_size);
-    }
-#else
-    printf("NVStore is disabled for this board\n");
-#endif
 
-while(1) {
+    // NVStore is a sigleton, get its instance
-    wait(1);
+    NVStore &nvstore = NVStore::get_instance();
-}
+
+    if (nvstore.init() != NVSTORE_SUCCESS) {
+        debug("\r\n NVStore initialization failed! \r\n");
+        return -1;
+    }
+
+    key = NVKEY_LORA_DEVICE_EUI;
+    rc = nvstore.get(key, sizeof(dev_eui), dev_eui, readsize); 
+    if (rc != NVSTORE_SUCCESS || readsize != sizeof(dev_eui)) {
+        debug("\r\n Failed to read LoRa Device EUI from NVStore! \r\n");
+        return -1;
+    }
+
+    key = NVKEY_LORA_APP_EUI;
+    rc = nvstore.get(key, sizeof(app_eui), app_eui, readsize);
+    if (rc != NVSTORE_SUCCESS || readsize != sizeof(app_eui)) {
+        debug("\r\n Failed to read LoRa Application EUI from NVStore! \r\n");
+        return -1;
+    }
+
+    key = NVKEY_LORA_APP_KEY;
+    rc =nvstore.get(key, sizeof(app_key), app_key, readsize);
+    if (rc != NVSTORE_SUCCESS || readsize != sizeof(app_key)) {
+        debug("\r\n Failed to read LoRa Application Key from NVStore! \r\n");
+        return -1;
+    }
+
+    connect.connect_type = LORAWAN_CONNECTION_OTAA;
+    connect.connection_u.otaa.app_eui = const_cast<uint8_t *>(app_eui);
+    connect.connection_u.otaa.dev_eui = const_cast<uint8_t *>(dev_eui);
+    connect.connection_u.otaa.app_key = const_cast<uint8_t *>(app_key);
+    connect.connection_u.otaa.nb_trials = MBED_CONF_LORA_NB_TRIALS;
 
 #if MBED_CONF_APP_HX711_ENABLED
     loadcell.powerDown();
@@ -133,7 +204,7 @@ while(1) {
 
     debug("\r\n Adaptive data  rate (ADR) - Enabled \r\n");
 
-    retcode = lorawan.connect();
+    retcode = lorawan.connect(connect);
 
     if(retcode == LORAWAN_STATUS_OK || retcode == LORAWAN_STATUS_CONNECT_IN_PROGRESS) {
     } else {
@@ -154,16 +225,30 @@ while(1) {
  */
 static void send_message() {
     int16_t retcode;
+    int eid;
 
-#if MBED_CONF_APP_BATTERY_ENABLED && MBED_CONF_APP_BATTERY_IN_LPP
+    // setup next transmission
-    cayenne.addAnalogInput(1, battery_voltage());
+    eid = ev_queue.call_in(MBED_CONF_APP_TX_TIMER, send_message);
-#endif
+
+    // check backoff time before sending new frame
+    if (MBED_CONF_LORA_DUTY_CYCLE_ON) {
+        lorawan_status_t status;
+        int backoff;
+
+        status = lorawan.get_backoff_metadata(backoff);
+        if (status == LORAWAN_STATUS_OK && backoff > 0) {
+            debug("\r\n %d ms backoff, delay next send \r\n", backoff);
+            ev_queue.cancel(eid);
+            ev_queue.call_in(backoff+MBED_CONF_APP_TX_TIMER, send_message);
+            return;
+        }
+    }
 
 #if MBED_CONF_APP_DHT_ENABLED
     int err = dht.read();
     if(err == DHT::SUCCESS) {
-        cayenne.addTemperature(1, dht.getTemperature());
+        cayenne.addTemperature(MBED_CONF_APP_DHT_TEMP_LPP_ID, dht.getTemperature());
-        cayenne.addRelativeHumidity(1, dht.getHumidity());
+        cayenne.addRelativeHumidity(MBED_CONF_APP_DHT_HUM_LPP_ID, dht.getHumidity());
     } else {
         debug("Error code : %d\r\n", err);
     }
@@ -172,11 +257,15 @@ static void send_message() {
 #if MBED_CONF_APP_HX711_ENABLED
     loadcell.powerUp();
     if(loadcell.waitReadyRetry(20, 100)) {
-        cayenne.addAnalogInput(2, loadcell.getUnits(5));
+        cayenne.addAnalogInput(MBED_CONF_APP_HX711_LPP_ID, loadcell.getUnits(5));
     }
     loadcell.powerDown();
 #endif
 
+#if MBED_CONF_APP_BATTERY_ENABLED && MBED_CONF_APP_BATTERY_IN_LPP
+    cayenne.addAnalogInput(MBED_CONF_APP_BATTERY_LPP_ID, battery_voltage());
+#endif
+
     // No data to send
     if(cayenne.getSize() == 0) {
         return;
@@ -223,13 +312,7 @@ static void lora_event_handler(lorawan_event_t event) {
     switch(event) {
         case CONNECTED:
             debug("\r\n Connection - Successful \r\n");
-            if(MBED_CONF_LORA_DUTY_CYCLE_ON) {
+            send_message();
-                send_message();
-            } else {
-                send_message();
-                ev_queue.call_every(MBED_CONF_APP_TX_TIMER, send_message);
-            }
-
             break;
         case DISCONNECTED:
             ev_queue.break_dispatch();
@@ -237,19 +320,12 @@ static void lora_event_handler(lorawan_event_t event) {
             break;
         case TX_DONE:
             debug("\r\n Message Sent to Network Server \r\n");
-            if(MBED_CONF_LORA_DUTY_CYCLE_ON) {
-                send_message();
-            }
             break;
         case TX_TIMEOUT:
         case TX_ERROR:
         case TX_CRYPTO_ERROR:
         case TX_SCHEDULING_ERROR:
             debug("\r\n Transmission Error - EventCode = %d \r\n", event);
-            // try again
-            if(MBED_CONF_LORA_DUTY_CYCLE_ON) {
-                send_message();
-            }
             break;
         case RX_DONE:
             debug("\r\n Received message from Network Server \r\n");
@@ -265,4 +341,6 @@ static void lora_event_handler(lorawan_event_t event) {
         default:
             MBED_ASSERT("Unknown Event");
     }
-}
+}
+
+#endif // MBED_APP_MODE_BOOTSTRAP

mbed_app.json

@@ -1,6 +1,7 @@
 {
     "config": {
-        "tx-timer":            { "value": 30000 },
+        "mode-bootstrap":      { "value": false },
+        "tx-timer":            { "value": 600000 },
         "lora-radio": {
             "help": "Which radio to use (options: SX1272,SX1276)",
             "value": "SX1276"
@@ -24,39 +25,45 @@
         "lora-ant-switch":     { "value": "NC" },
         "lora-pwr-amp-ctl":    { "value": "NC" },
         "lora-tcxo":           { "value": "NC" },
-        "battery-enabled":     { "value": "false" },
+        "battery-enabled":     { "value": false },
-        "battery-in-lpp":      { "value": "false" },
+        "battery-lpp-id":      { "value": 1 },
+        "battery-in-lpp":      { "value": false },
         "battery-adc":         { "value": "NC" },
+        "battery-min":         { "value": "3.20f" },
+        "battery-max":         { "value": "4.15f" },
         "battery-ratio":       { "value": "2.0f" },
-        "dht-enabled":         { "value": "false" },
+        "hx711-enabled":       { "value": false },
-        "dht-type":            { "value": "DHT::DHT22" },
+        "hx711-lpp-id":        { "value": 2 },
-        "dht-data":            { "value": "NC" },
-        "hx711-enabled":       { "value": "false" },
         "hx711-clk":           { "value": "NC" },
         "hx711-data":          { "value": "NC" },
         "hx711-scale":         { "value": "1.0f" },
-        "hx711-offset":        { "value": 0 }
+        "hx711-offset":        { "value": 0 },
+        "dht-enabled":         { "value": false },
+        "dht-temp-lpp-id":     { "value": 3 },
+        "dht-hum-lpp-id":      { "value": 4 },
+        "dht-type":            { "value": "DHT::DHT22" },
+        "dht-data":            { "value": "NC" }
     },
     "target_overrides": {
         "*": {
+            "mode-bootstrap": false,
+            "tx-timer": 600000,
             "platform.stdio-convert-newlines": true,
             "platform.stdio-baud-rate": 115200,
             "platform.default-serial-baud-rate": 115200,
             "lora.app-port": 3,
-            "lora.over-the-air-activation": true,
+            "lora.duty-cycle-on": true,
-            "lora.duty-cycle-on": false,
             "lora.phy": 0,
-            "lora.device-eui": "{ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 }",
+            "lora.device-eui": "{ 0xAB, 0xBA, 0xAB, 0xBA, 0xAB, 0xBA, 0xAB, 0xBA }",
-            "lora.application-eui": "{ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 }",
+            "lora.application-eui": "{ 0xAC, 0xCA, 0xAC, 0xCA, 0xAC, 0xCA, 0xAC, 0xCA }",
-            "lora.application-key": "{ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 }"
+            "lora.application-key": "{ 0xAD, 0xDA, 0xAD, 0xDA, 0xAD, 0xDA, 0xAD, 0xDA, 0xAD, 0xDA, 0xAD, 0xDA, 0xAD, 0xDA, 0xAD, 0xDA }"
         },
 
         "NUCLEO_L073RZ": {
             "main_stack_size":      1024,
             "target.lpticker_lptim": 0,
             "target.clock_source": "USE_PLL_HSI",
-            "lora.duty-cycle-on":  false,
+            "sx1272-lora-driver.radio-variant": "SX1272MB2XAS",
-            "tx-timer":            600000,
             "lora-radio":          "SX1272",
             "lora-spi-mosi":       "D11",
             "lora-spi-miso":       "D12",
@@ -73,26 +80,21 @@
             "lora-rf-switch-ctl2": "NC",
             "lora-txctl":          "NC",
             "lora-rxctl":          "NC",
-            "lora-ant-switch":     "A4",
+            "lora-ant-switch":     "NC",
             "lora-pwr-amp-ctl":    "NC",
             "lora-tcxo":           "NC",
             "battery-enabled":     true,
             "battery-in-lpp":      true,
             "battery-adc":         "A1",
             "battery-ratio":       "2.0f",
-            "dht-enabled":         true,
-            "dht-data":            "D8",
-            "dht-type":            "DHT::DHT22",
             "hx711-enabled":       true,
             "hx711-clk":           "D6",
             "hx711-data":          "D7",
             "hx711-scale":         "21000.0f",
             "hx711-offset":        59600,
-            "nvstore.max_keys": 20,
+            "dht-enabled":         true,
-            "nvstore.area_1_address": "0x0802e000",
+            "dht-data":            "D8",
-            "nvstore.area_1_size"   : "0x1000",
+            "dht-type":            "DHT::DHT22"
-            "nvstore.area_2_address": "0x0802f000",
-            "nvstore.area_2_size"   : "0x1000"
         },
 
         "MTB_RAK811": {

scripts/gen_images.sh

@@ -0,0 +1,47 @@
+#!/bin/bash
+
+# patterns
+DEVICEEUI=ABBAABBAABBAABBA
+APPEUI=ACCAACCAACCAACCA
+APPKEY=ADDAADDAADDAADDAADDAADDAADDAADDA
+
+if [ $# -ne 2 ]; then
+  echo "gen_images <master_binary> <devices_file>\n"
+  echo "devices file must be in the form :\n"
+  echo "dev1name:dev1eui:dev1appeui:dev1appkey"
+  echo "dev2name:dev2eui:dev2appeui:dev2appkey\n"
+  exit 1;
+fi
+
+SED=sed
+MASTER=$1
+DEVICES=$2
+
+if [[ $OSTYPE == "darwin"* ]]; then
+  if ! command -v gsed > /dev/null 2>&1; then
+    echo "You need gnu-sed on darwin system (available with homebrew)"
+    exit 1
+  fi
+  SED=gsed
+fi
+
+if [ ! -e $MASTER ]; then
+  echo "The master file $MASTER doesn't exists"
+  exit 1;
+fi
+
+if [ ! -e $DEVICES ]; then
+  echo "The devices file $DEVICES doesn't exists"
+  exit 1;
+fi
+
+for DEVICE in $(cat $DEVICES); do
+  name=$(echo $DEVICE | cut -d : -f 1)
+  deviceeui=$(echo $DEVICE | cut -d : -f 2)
+  appeui=$(echo $DEVICE | cut -d : -f 3)
+  appkey=$(echo $DEVICE | cut -d : -f 4)
+  echo "Generating device $name ($deviceeui)"
+  hexdump -ve '1/1 "%.2X"' $MASTER | $SED "s/${DEVICEEUI}/${deviceeui}/; s/${APPEUI}/${appeui}/; s/${APPKEY}/${appkey}/" | xxd -r -p - ${deviceeui}.bin
+done
+
+exit;