ARMmbed · cmonr · Oct 12, 2018 · Oct 30, 2018 · Oct 31, 2018 · Oct 31, 2018
diff --git a/.astyleignore b/.astyleignore
@@ -3,17 +3,22 @@ cmsis
 features/cryptocell
 features/mbedtls
 features/lwipstack/lwip
+features/lwipstack/lwip-sys
 rtos/TARGET_CORTEX/rtx4
 features/storage/filesystem/littlefs/littlefs/
 features/storage/filesystem/fat/ChaN
 features/storage/FEATURE_STORAGE
 features/frameworks
-features/FEATURE_BLE/targets
+features/FEATURE_BLE
 features/unsupported/
+features/netsocket/emac-drivers
 hal/storage_abstraction
-FEATURE_NANOSTACK/coap-service
-FEATURE_NANOSTACK/sal-stack-nanostack
+TESTS/mbed_hal/trng/pithy
+features/nanostack/coap-service
+features/nanostack/sal-stack-nanostack
 rtos/TARGET_CORTEX/rtx5
 TESTS/mbed_hal/trng/pithy
 targets
+components/802.15.4_RF
+components/wifi
 tools
diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
diff --git a/TESTS/mbed_drivers/flashiap/main.cpp b/TESTS/mbed_drivers/flashiap/main.cpp
@@ -55,9 +55,6 @@ void flashiap_program_test()
     TEST_ASSERT_TRUE(sector_size % page_size == 0);
     uint32_t prog_size = std::max(page_size, (uint32_t)8);
     uint8_t *data = new uint8_t[prog_size + 2];
-    for (uint32_t i = 0; i < prog_size + 2; i++) {
-        data[i] = i;
-    }
 
     // the one before the last sector in the system
     uint32_t address = (flash_device.get_flash_start() + flash_device.get_flash_size()) - (sector_size);
@@ -68,14 +65,27 @@ void flashiap_program_test()
     ret = flash_device.erase(address, sector_size);
     TEST_ASSERT_EQUAL_INT32(0, ret);
 
+    uint8_t erase_val = flash_device.get_erase_value();
+    memset(data, erase_val, prog_size);
+
+    uint8_t *data_flashed = new uint8_t[prog_size];
+    for (uint32_t i = 0; i < sector_size / prog_size; i++) {
+        uint32_t page_addr = address + i * prog_size;
+        ret = flash_device.read(data_flashed, page_addr, prog_size);
+        TEST_ASSERT_EQUAL_INT32(0, ret);
+        TEST_ASSERT_EQUAL_UINT8_ARRAY(data, data_flashed, prog_size);
+    }
+
+    for (uint32_t i = 0; i < prog_size + 2; i++) {
+        data[i] = i;
+    }
 
     for (uint32_t i = 0; i < sector_size / prog_size; i++) {
         uint32_t prog_addr = address + i * prog_size;
         ret = flash_device.program(data, prog_addr, prog_size);
         TEST_ASSERT_EQUAL_INT32(0, ret);
     }
 
-    uint8_t *data_flashed = new uint8_t[prog_size];
     for (uint32_t i = 0; i < sector_size / prog_size; i++) {
         uint32_t page_addr = address + i * prog_size;
         ret = flash_device.read(data_flashed, page_addr, prog_size);

diff --git a/TESTS/mbed_hal/minimum_requirements/main.cpp b/TESTS/mbed_hal/minimum_requirements/main.cpp
@@ -53,7 +53,7 @@ static void minimum_data_test()
 }
 
 
-utest::v1::status_t greentea_test_setup(const size_t number_of_cases) 
+utest::v1::status_t greentea_test_setup(const size_t number_of_cases)
 {
     GREENTEA_SETUP(30, "default_auto");
     return greentea_test_setup_handler(number_of_cases);
@@ -66,6 +66,7 @@ Case cases[] = {
 
 Specification specification(greentea_test_setup, cases, greentea_test_teardown_handler);
 
-int main() {
+int main()
+{
     Harness::run(specification);
 }
diff --git a/TESTS/mbed_hal/rtc_time/main.cpp b/TESTS/mbed_hal/rtc_time/main.cpp
@@ -169,13 +169,41 @@ void test_local_time_invalid_param()
     TEST_ASSERT_EQUAL(false, _rtc_localtime(1, NULL, RTC_4_YEAR_LEAP_YEAR_SUPPORT));
 }
 
+/* Test set_time() function called a few seconds apart.
+ *
+ * Given is set_time() function.
+ * When set_time() is used to set the system time two times.
+ * Then if the value returned from time() is always correct return true, otherwise return false.
+ */
+#define NEW_TIME 15
+void test_set_time_twice()
+{
+    time_t current_time;
+
+    /* Set the time to NEW_TIME and check it */
+    set_time(NEW_TIME);
+    current_time = time(NULL);
+    TEST_ASSERT_EQUAL (true, (current_time == NEW_TIME));
+
+    /* Wait 2 seconds */
+    wait_ms(2000);
+
+    /* set the time to NEW_TIME again and check it */
+    set_time(NEW_TIME);
+    current_time = time(NULL);
+    TEST_ASSERT_EQUAL (true, (current_time == NEW_TIME));
+}
+
 Case cases[] = {
     Case("test is leap year - RTC leap years full support", test_is_leap_year<RTC_FULL_LEAP_YEAR_SUPPORT>),
     Case("test is leap year - RTC leap years partial support", test_is_leap_year<RTC_4_YEAR_LEAP_YEAR_SUPPORT>),
     Case("test make time boundary values - RTC leap years full support", test_mk_time_boundary<RTC_FULL_LEAP_YEAR_SUPPORT>),
     Case("test make time boundary values - RTC leap years partial support", test_mk_time_boundary<RTC_4_YEAR_LEAP_YEAR_SUPPORT>),
     Case("test make time - invalid param", test_mk_time_invalid_param),
     Case("test local time - invalid param", test_local_time_invalid_param),
+#if DEVICE_RTC || DEVICE_LPTICKER
+    Case("test set_time twice", test_set_time_twice),
+#endif
 };
 
 utest::v1::status_t greentea_test_setup(const size_t number_of_cases)

diff --git a/TESTS/network/wifi/wifi_connect.cpp b/TESTS/network/wifi/wifi_connect.cpp
@@ -31,7 +31,7 @@ void wifi_connect(void)
 {
     WiFiInterface *wifi = get_interface();
 
-    char ssid[SSID_MAX_LEN+1] = MBED_CONF_APP_WIFI_UNSECURE_SSID;
+    char ssid[SSID_MAX_LEN + 1] = MBED_CONF_APP_WIFI_UNSECURE_SSID;
 
     TEST_ASSERT_EQUAL_INT(NSAPI_ERROR_OK, wifi->set_credentials(ssid, NULL));
     TEST_ASSERT_EQUAL_INT(NSAPI_ERROR_OK, wifi->connect());
@@ -42,7 +42,7 @@ void wifi_connect(void)
     TEST_ASSERT_EQUAL_INT(NSAPI_ERROR_OK, wifi->disconnect());
 
     // Driver is expected to cache the credentials
-    memset(ssid, 0, SSID_MAX_LEN+1);
+    memset(ssid, 0, SSID_MAX_LEN + 1);
 
     TEST_ASSERT_EQUAL_INT(NSAPI_ERROR_OK, wifi->connect());
     TEST_ASSERT_EQUAL_INT(NSAPI_ERROR_OK, wifi->disconnect());

diff --git a/TEST_APPS/device/socket_app/cmd_socket.cpp b/TEST_APPS/device/socket_app/cmd_socket.cpp
@@ -152,7 +152,7 @@ class SInfo {
     {
         assert(sock);
     }
-    SInfo(Socket* sock, bool delete_on_exit=true):
+    SInfo(Socket *sock, bool delete_on_exit = true):
         _id(id_count++),
         _sock(sock),
         _type(SInfo::OTHER),
@@ -1162,11 +1162,11 @@ static int cmd_socket(int argc, char *argv[])
                 cmd_printf("Invalid socket id\r\n");
                 return CMDLINE_RETCODE_FAIL;
             }
-            TCPSocket *new_sock = static_cast<TCPSocket*>(&new_info->socket());
-            nsapi_error_t ret = static_cast<TCPServer&>(info->socket()).accept(new_sock, &addr);
+            TCPSocket *new_sock = static_cast<TCPSocket *>(&new_info->socket());
+            nsapi_error_t ret = static_cast<TCPServer &>(info->socket()).accept(new_sock, &addr);
             if (ret == NSAPI_ERROR_OK) {
                 cmd_printf("TCPServer::accept() new socket sid: %d connection from %s port %d\r\n",
-                        new_info->id(), addr.get_ip_address(), addr.get_port());
+                           new_info->id(), addr.get_ip_address(), addr.get_port());
             }
             return handle_nsapi_error("TCPServer::accept()", ret);
         }

diff --git a/UNITTESTS/features/lorawan/loraphy/Test_LoRaPHY.cpp b/UNITTESTS/features/lorawan/loraphy/Test_LoRaPHY.cpp
@@ -27,7 +27,7 @@ class my_LoRaPHY : public LoRaPHY {
         phy_params.adr_ack_delay = 1;
     }
 
-    virtual ~my_LoRaPHY() 
+    virtual ~my_LoRaPHY()
     {
     }
 
@@ -40,19 +40,19 @@ class my_LoRaPHY : public LoRaPHY {
 class my_radio : public LoRaRadio {
 public:
 
-    virtual void init_radio(radio_events_t *events) 
+    virtual void init_radio(radio_events_t *events)
     {
     };
 
-    virtual void radio_reset() 
+    virtual void radio_reset()
     {
     };
 
-    virtual void sleep(void) 
+    virtual void sleep(void)
     {
     };
 
-    virtual void standby(void) 
+    virtual void standby(void)
     {
     };
 
@@ -62,31 +62,31 @@ class my_radio : public LoRaRadio {
                                uint16_t symb_timeout, bool fix_len,
                                uint8_t payload_len,
                                bool crc_on, bool freq_hop_on, uint8_t hop_period,
-                               bool iq_inverted, bool rx_continuous) 
+                               bool iq_inverted, bool rx_continuous)
     {
     };
 
     virtual void set_tx_config(radio_modems_t modem, int8_t power, uint32_t fdev,
                                uint32_t bandwidth, uint32_t datarate,
                                uint8_t coderate, uint16_t preamble_len,
                                bool fix_len, bool crc_on, bool freq_hop_on,
-                               uint8_t hop_period, bool iq_inverted, uint32_t timeout) 
+                               uint8_t hop_period, bool iq_inverted, uint32_t timeout)
     {
     };
 
-    virtual void send(uint8_t *buffer, uint8_t size) 
+    virtual void send(uint8_t *buffer, uint8_t size)
     {
     };
 
-    virtual void receive(void) 
+    virtual void receive(void)
     {
     };
 
-    virtual void set_channel(uint32_t freq) 
+    virtual void set_channel(uint32_t freq)
     {
     };
 
-    virtual uint32_t random(void) 
+    virtual uint32_t random(void)
     {
     };
 
@@ -95,15 +95,15 @@ class my_radio : public LoRaRadio {
         return uint8_value;
     };
 
-    virtual void set_max_payload_length(radio_modems_t modem, uint8_t max) 
+    virtual void set_max_payload_length(radio_modems_t modem, uint8_t max)
     {
     };
 
-    virtual void set_public_network(bool enable) 
+    virtual void set_public_network(bool enable)
     {
     };
 
-    virtual uint32_t time_on_air(radio_modems_t modem, uint8_t pkt_len) 
+    virtual uint32_t time_on_air(radio_modems_t modem, uint8_t pkt_len)
     {
     };
 
@@ -115,7 +115,7 @@ class my_radio : public LoRaRadio {
         return bool_value;
     };
 
-    virtual void start_cad(void) 
+    virtual void start_cad(void)
     {
     };
 
@@ -124,15 +124,15 @@ class my_radio : public LoRaRadio {
         return bool_value;
     };
 
-    virtual void set_tx_continuous_wave(uint32_t freq, int8_t power, uint16_t time) 
+    virtual void set_tx_continuous_wave(uint32_t freq, int8_t power, uint16_t time)
     {
     };
 
-    virtual void lock(void) 
+    virtual void lock(void)
     {
     };
 
-    virtual void unlock(void) 
+    virtual void unlock(void)
     {
     };
 
@@ -422,10 +422,10 @@ TEST_F(Test_LoRaPHY, compute_rx_win_params)
     list[0] = 125000;
     object->get_phy_params().bandwidths.table = list;
     uint8_t list2[1];
-    list2[0]= 12;
+    list2[0] = 12;
     object->get_phy_params().datarates.table = &list2;
     channel_params_t ch_lst[16];
-    memset(ch_lst, 0, sizeof(channel_params_t)*16);
+    memset(ch_lst, 0, sizeof(channel_params_t) * 16);
     ch_lst[0].band = 0;
     ch_lst[0].dr_range.fields.min = DR_0;
     ch_lst[0].dr_range.fields.max = DR_5;
@@ -513,7 +513,7 @@ TEST_F(Test_LoRaPHY, link_ADR_request)
 
     object->get_phy_params().max_channel_cnt = 16;
     channel_params_t li[16];
-    memset(li, 0, sizeof(channel_params_t)*16);
+    memset(li, 0, sizeof(channel_params_t) * 16);
     object->get_phy_params().channels.channel_list = li;
     li[0].frequency = 0;
     li[1].frequency = 5;
@@ -673,7 +673,7 @@ TEST_F(Test_LoRaPHY, set_next_channel)
     ch = 5;
     t1 = 16;
     t2 = 32;
-    memset(b, 0, sizeof(band_t)*4);
+    memset(b, 0, sizeof(band_t) * 4);
     object->get_phy_params().bands.size = 2;
     object->get_phy_params().bands.table = &b;
     b[0].off_time = 0;
@@ -767,7 +767,7 @@ TEST_F(Test_LoRaPHY, set_tx_cont_mode)
 
     cw_mode_params_t p;
     p.max_eirp = 0;
-    p.channel=0;
+    p.channel = 0;
     p.tx_power = -1;
     p.datarate = 0;
     p.antenna_gain = 1;

diff --git a/UNITTESTS/features/lorawan/lorawaninterface/Test_LoRaWANInterface.cpp b/UNITTESTS/features/lorawan/lorawaninterface/Test_LoRaWANInterface.cpp
@@ -44,9 +44,9 @@ class my_radio : public LoRaRadio {
                                uint8_t payload_len,
                                bool crc_on, bool freq_hop_on, uint8_t hop_period,
                                bool iq_inverted, bool rx_continuous)
-                               {
+    {
 
-                               };
+    };
 
     virtual void set_tx_config(radio_modems_t modem, int8_t power, uint32_t fdev,
                                uint32_t bandwidth, uint32_t datarate,

diff --git a/components/802.15.4_RF/atmel-rf-driver/atmel-rf-driver/NanostackRfPhyAtmel.h b/components/802.15.4_RF/atmel-rf-driver/atmel-rf-driver/NanostackRfPhyAtmel.h
@@ -58,8 +58,8 @@ class RFBits;
 class NanostackRfPhyAtmel : public NanostackRfPhy {
 public:
     NanostackRfPhyAtmel(PinName spi_mosi, PinName spi_miso,
-            PinName spi_sclk, PinName spi_cs,  PinName spi_rst, PinName spi_slp, PinName spi_irq,
-            PinName i2c_sda, PinName i2c_scl);
+                        PinName spi_sclk, PinName spi_cs,  PinName spi_rst, PinName spi_slp, PinName spi_irq,
+                        PinName i2c_sda, PinName i2c_scl);
     virtual ~NanostackRfPhyAtmel();
     virtual int8_t rf_register();
     virtual void rf_unregister();

diff --git a/components/802.15.4_RF/atmel-rf-driver/mbed_lib.json b/components/802.15.4_RF/atmel-rf-driver/mbed_lib.json
@@ -24,6 +24,10 @@
         "provide-default": {
             "help": "Provide default NanostackRfpy. [true/false]",
             "value": false
+        },
+        "irq-thread-stack-size":  {
+            "help": "The stack size of the Thread serving the Atmel RF interrupts",
+            "value": 1024
         }
     },
     "target_overrides": {