Merge pull request #7228 from ARMmbed/release-candidate

Release candidate for mbed-os-5.9.1

Author: Cruz Monrreal

.astylerc

@@ -16,7 +16,6 @@ convert-tabs
 
 # Indent switches and cases
 indent-switches
-indent-cases
 
 # Remove spaces in and around parentheses
 unpad-paren
@@ -32,3 +31,7 @@ align-reference=name
 # Attach { for classes and namespaces
 attach-namespaces
 attach-classes
+
+# Extend longer lines, define maximum 120 value. This results in aligned code,
+# otherwise the lines are broken and not consistent 
+max-continuation-indent=120

.travis.yml

@@ -260,3 +260,9 @@ matrix:
       script:
         # Check that no changes after clone. This check that .gitattributes is used right way.
         - git diff --exit-code
+
+    - env:
+        - NAME=licence_check
+      script:
+        - echo 'Checking that there is no GPL licence text in code'
+        - ! git grep -q --ignore-case "gnu general public";

TESTS/lorawan/loraradio/README.md

@@ -0,0 +1,17 @@
+# Description
+
+This document describes how to run LoRaRadio API tests.
+
+## Configuring target HW
+
+Before starting to test, the target HW must be configured for the test. This can be done by either modifying the application configuration `json` file (if using currently supported Semtech SX1272 or SX1276 radios) or implementing driver construction into test source.
+
+The default mbed_app.json file provides configuration for some already supported HWs.
+
+## Running tests
+
+You can use the following command to run tests:
+
+`mbed test -n mbed-os-tests-lorawan-loraradio -m TARGET -t GCC_ARM --app-config mbed-os/TESTS/lorawan/loraradio/template_mbed_app.txt`
+
+Replace TARGET with the target device.

TESTS/lorawan/loraradio/main.cpp

@@ -0,0 +1,282 @@
+/* mbed Microcontroller Library
+ * Copyright (c) 2017 ARM Limited
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+#include "utest.h"
+#include "unity.h"
+#include "greentea-client/test_env.h"
+
+#include "Semaphore.h"
+
+#include "mbed_trace.h"
+#define TRACE_GROUP "RTST"
+
+#include "LoRaRadio.h"
+
+#define SX1272   0xFF
+#define SX1276   0xEE
+
+#if (MBED_CONF_APP_LORA_RADIO == SX1272)
+    #include "SX1272_LoRaRadio.h"
+#elif (MBED_CONF_APP_LORA_RADIO == SX1276)
+    #include "SX1276_LoRaRadio.h"
+#else
+    #error [NOT_SUPPORTED] Requires parameters from application config file.
+#endif
+
+
+using namespace utest::v1;
+
+static LoRaRadio* radio = NULL;
+rtos::Semaphore event_sem(0);
+
+enum event_t
+{
+    EV_NONE,
+    EV_TX_DONE,
+    EV_TX_TIMEOUT,
+    EV_RX_DONE,
+    EV_RX_TIMEOUT,
+    EV_RX_ERROR,
+};
+static volatile event_t received_event;
+
+
+static void tx_done()
+{
+    wait_ms(2);
+    TEST_ASSERT_EQUAL(EV_NONE, received_event);
+    received_event = EV_TX_DONE;
+    TEST_ASSERT_EQUAL(osOK, event_sem.release());
+}
+
+static void tx_timeout()
+{
+    wait_ms(2);
+    TEST_ASSERT_EQUAL(EV_NONE, received_event);
+    received_event = EV_TX_TIMEOUT;
+    TEST_ASSERT_EQUAL(osOK, event_sem.release());
+}
+
+static void rx_done(const uint8_t *payload, uint16_t size, int16_t rssi, int8_t snr)
+{
+    wait_ms(2);
+    TEST_ASSERT_EQUAL(EV_NONE, received_event);
+    received_event = EV_RX_DONE;
+    TEST_ASSERT_EQUAL(osOK, event_sem.release());
+}
+
+static void rx_timeout()
+{
+    wait_ms(2);
+    TEST_ASSERT_EQUAL(EV_NONE, received_event);
+    received_event = EV_RX_TIMEOUT;
+    TEST_ASSERT_EQUAL(osOK, event_sem.release());
+}
+
+static void rx_error()
+{
+    wait_ms(2);
+    TEST_ASSERT_EQUAL(EV_NONE, received_event);
+    received_event = EV_RX_ERROR;
+    TEST_ASSERT_EQUAL(osOK, event_sem.release());
+}
+
+static radio_events radio_callbacks =
+{
+    .tx_done    = tx_done,
+    .tx_timeout = tx_timeout,
+    .rx_done    = rx_done,
+    .rx_timeout = rx_timeout,
+    .rx_error   = rx_error
+};
+
+
+void test_random()
+{
+    const uint32_t rand1 = radio->random();
+    const uint32_t rand2 = radio->random();
+    TEST_ASSERT_NOT_EQUAL(rand1, rand2);
+}
+
+void test_set_tx_config()
+{
+    uint8_t buffer[] = {0};
+
+    TEST_ASSERT_EQUAL(RF_IDLE, radio->get_status());
+
+    radio->set_tx_config(MODEM_LORA, 13, 0,
+                         0, 7,
+                         1, 8,
+                         false, true, false,
+                         0, false, 100);
+    radio->send(buffer, sizeof(buffer));
+
+    TEST_ASSERT_EQUAL(RF_TX_RUNNING, radio->get_status());
+
+    TEST_ASSERT_EQUAL(1, event_sem.wait(1000));
+    TEST_ASSERT_EQUAL(EV_TX_DONE, received_event);
+    received_event = EV_NONE;
+}
+
+void test_set_rx_config()
+{
+    TEST_ASSERT_EQUAL(RF_IDLE, radio->get_status());
+
+    radio->set_rx_config(MODEM_LORA, 0,     // modem, bandwidth,
+                         7, 1,              // datarate, coderate,
+                         0, 8,              // bandwidth_afc, preamble_len,
+                         24, false,         // symb_timeout, fix_len,
+                         0,                 // payload_len,
+                         false, false, 0,   // crc_on, freq_hop_on, hop_period,
+                         true, false);      // iq_inverted, rx_continuous
+    radio->receive(100);
+
+    TEST_ASSERT_EQUAL(RF_RX_RUNNING, radio->get_status());
+
+    TEST_ASSERT_EQUAL(1, event_sem.wait(1000));
+
+    // Nobody was sending to us so timeout is expected.
+    TEST_ASSERT_EQUAL(EV_RX_TIMEOUT, received_event);
+    received_event = EV_NONE;
+}
+
+void test_time_on_air()
+{
+    radio->set_rx_config(MODEM_LORA, 0,
+                         7, 1,
+                         0, 8,
+                         24, false,
+                         0,
+                         false, false, 0,
+                         true, false);
+    TEST_ASSERT_EQUAL(52, radio->time_on_air(MODEM_LORA, 20));
+
+    radio->set_tx_config(MODEM_LORA, 13, 0,
+                         0, 7,
+                         1, 8,
+                         false, true, false,
+                         0, false, 100);
+    TEST_ASSERT_EQUAL(72, radio->time_on_air(MODEM_LORA, 32));
+
+    // TODO: Add FSK tests
+}
+
+void test_perform_carrier_sense()
+{
+    TEST_ASSERT_TRUE(radio->perform_carrier_sense(MODEM_FSK, 865000000, -20, 1));
+    TEST_ASSERT_TRUE(radio->perform_carrier_sense(MODEM_LORA, 865000000, -20, 1));
+}
+
+void test_check_rf_frequency()
+{
+    // Test EU868 frequency
+    TEST_ASSERT_TRUE(radio->check_rf_frequency(865000000));
+}
+
+// Test setup
+utest::v1::status_t test_setup(const size_t number_of_cases) {
+    GREENTEA_SETUP(20, "default_auto");
+
+    mbed_trace_init();
+
+    return verbose_test_setup_handler(number_of_cases);
+}
+
+utest::v1::status_t case_setup_handler(const Case *const source, const size_t index_of_case)
+{
+#if (MBED_CONF_APP_LORA_RADIO == SX1272)
+
+    radio = new SX1272_LoRaRadio(MBED_CONF_APP_LORA_SPI_MOSI,
+                                 MBED_CONF_APP_LORA_SPI_MISO,
+                                 MBED_CONF_APP_LORA_SPI_SCLK,
+                                 MBED_CONF_APP_LORA_CS,
+                                 MBED_CONF_APP_LORA_RESET,
+                                 MBED_CONF_APP_LORA_DIO0,
+                                 MBED_CONF_APP_LORA_DIO1,
+                                 MBED_CONF_APP_LORA_DIO2,
+                                 MBED_CONF_APP_LORA_DIO3,
+                                 MBED_CONF_APP_LORA_DIO4,
+                                 MBED_CONF_APP_LORA_DIO5,
+                                 MBED_CONF_APP_LORA_RF_SWITCH_CTL1,
+                                 MBED_CONF_APP_LORA_RF_SWITCH_CTL2,
+                                 MBED_CONF_APP_LORA_TXCTL,
+                                 MBED_CONF_APP_LORA_RXCTL,
+                                 MBED_CONF_APP_LORA_ANT_SWITCH,
+                                 MBED_CONF_APP_LORA_PWR_AMP_CTL,
+                                 MBED_CONF_APP_LORA_TCXO);
+
+#elif (MBED_CONF_APP_LORA_RADIO == SX1276)
+
+    radio = new SX1276_LoRaRadio(MBED_CONF_APP_LORA_SPI_MOSI,
+                                 MBED_CONF_APP_LORA_SPI_MISO,
+                                 MBED_CONF_APP_LORA_SPI_SCLK,
+                                 MBED_CONF_APP_LORA_CS,
+                                 MBED_CONF_APP_LORA_RESET,
+                                 MBED_CONF_APP_LORA_DIO0,
+                                 MBED_CONF_APP_LORA_DIO1,
+                                 MBED_CONF_APP_LORA_DIO2,
+                                 MBED_CONF_APP_LORA_DIO3,
+                                 MBED_CONF_APP_LORA_DIO4,
+                                 MBED_CONF_APP_LORA_DIO5,
+                                 MBED_CONF_APP_LORA_RF_SWITCH_CTL1,
+                                 MBED_CONF_APP_LORA_RF_SWITCH_CTL2,
+                                 MBED_CONF_APP_LORA_TXCTL,
+                                 MBED_CONF_APP_LORA_RXCTL,
+                                 MBED_CONF_APP_LORA_ANT_SWITCH,
+                                 MBED_CONF_APP_LORA_PWR_AMP_CTL,
+                                 MBED_CONF_APP_LORA_TCXO);
+
+#else
+    #error [NOT_SUPPORTED] Unknown LoRa radio specified (SX1272,SX1276 are valid)
+#endif
+
+    TEST_ASSERT(radio);
+    radio->init_radio(&radio_callbacks);
+    radio->sleep();
+
+    return greentea_case_setup_handler(source, index_of_case);
+}
+
+utest::v1::status_t case_teardown_handler(const Case *const source, const size_t passed, const size_t failed, const failure_t reason)
+{
+    radio->sleep();
+
+#if (MBED_CONF_APP_LORA_RADIO == SX1272)
+    delete static_cast<SX1272_LoRaRadio*>(radio);
+#elif (MBED_CONF_APP_LORA_RADIO == SX1276)
+    delete static_cast<SX1276_LoRaRadio*>(radio);
+#else
+    #error [NOT_SUPPORTED] Unknown LoRa radio specified (SX1272,SX1276 are valid)
+#endif
+    radio = NULL;
+
+    return greentea_case_teardown_handler(source, passed, failed, reason);
+}
+
+const Case cases[] = {
+    Case("Test random",        case_setup_handler, test_random,        case_teardown_handler),
+    Case("Test set_tx_config", case_setup_handler, test_set_tx_config, case_teardown_handler),
+    Case("Test set_rx_config", case_setup_handler, test_set_rx_config, case_teardown_handler),
+    Case("Test time_on_air",   case_setup_handler, test_time_on_air,   case_teardown_handler),
+    Case("Test perform_carrier_sense", case_setup_handler, test_perform_carrier_sense, case_teardown_handler),
+    Case("Test check_rf_frequency",    case_setup_handler, test_check_rf_frequency,    case_teardown_handler),
+};
+
+Specification specification(test_setup, cases);
+
+int main() {
+    return !Harness::run(specification);
+}