Updating mbed-client-c and sal-stack-nanostack-eventloop

**mbed-client-c**

[Full Changelog](ARMmbed/mbed-client-c@v3.0.1...v3.0.2)

**New feature**

- Make resending flags configurable through Yotta and mbed-cli build systems
- Introducing Configuration management through Config file

**Closed issues:**

- IOTCLT-882 - Confirmable messages sending

**sal-stack-nanostack-eventloop**
[Full Changelog](PelionIoT/sal-stack-nanostack-eventloop@mbed-os-5.0-rc1...mbed-os.5.2-rc1)

**New features**

- Move configuration parameter resolution to separate header and allow user definable configuration header to be used.
- Port eventloop system timer to use new tick timer functions platform_tick_timer_register, platform_tick_timer_start and platform_tick_timer_stop.

Author: Yogesh Pande

features/FEATURE_COMMON_PAL/mbed-client-c/CHANGELOG.md

@@ -1,5 +1,131 @@
 # Change Log
 
+## [v3.0.2](https://github.com/ARMmbed/mbed-client-c/releases/tag/v3.0.2) (24-Sep-2016)
+[Full Changelog](https://github.com/ARMmbed/mbed-client-c/compare/v3.0.1...v3.0.2)
+
+**New feature**
+
+- Make resending flags configurable through Yotta and mbed-cli build systems
+- Introducing Configuration management through Config file
+
+**Closed issues:**
+
+- IOTCLT-882 - Confirmable messages sending
+
+**Merged pull requests:**
+
+commit 58f712b9f684bede1905ce35a1a39ebb257c049c (HEAD, tag: v3.0.2, origin/master, origin/HEAD, master)
+Author: Yogesh Pande <[email protected]>
+Date:   Sat Sep 24 15:04:36 2016 +0300
+
+    version v3.0.2
+
+commit 39d5e98025f3d8067cea85c4a06470e0e9374812
+Merge: cf4c119 31d6566
+Author: Yogesh Pande <[email protected]>
+Date:   Sat Sep 24 14:51:30 2016 +0300
+
+    Merge pull request #79 from ARMmbed/config-mechanism
+    
+    Introducing Configuration management through Config file
+
+commit 31d6566dddfe449e87819244027b847f53f53a96 (origin/config-mechanism)
+Author: Yogesh Pande <[email protected]>
+Date:   Sat Sep 24 12:53:47 2016 +0300
+
+    Fixing valgrind with unit tests
+
+commit a95a90a31e8f6123d14df84688b4e2e2f32e5507
+Author: Yogesh Pande <[email protected]>
+Date:   Sat Sep 24 12:00:44 2016 +0300
+
+    Fixing valgrind memory leaks for unit tests
+
+commit 248fc713a4303cb1f724d10f4aa6d68933bb5aed
+Author: Yogesh Pande <[email protected]>
+Date:   Sat Sep 24 11:32:01 2016 +0300
+
+    Fix for failing unit tests
+
+commit cec0980b4ccff9708cd0130fea936a69839739b8
+Merge: cf09b2b cf4c119
+Author: Yogesh Pande <[email protected]>
+Date:   Thu Sep 22 18:02:39 2016 +0300
+
+    Merge branch 'master' into config-mechanism
+
+commit cf4c1196520d64bf863b39efefc663ee60e58c61
+Merge: 330a16b f5f15cc
+Author: Antti Yli-Tokola <[email protected]>
+Date:   Thu Sep 22 12:55:33 2016 +0300
+
+    Merge pull request #80 from ARMmbed/revert-77-duplication
+    
+    Revert "Enable message duplication by default"
+
+commit f5f15ccce5fd719aa8d7de14a583d02fd0fd05b9 (origin/revert-77-duplication)
+Author: Antti Yli-Tokola <[email protected]>
+Date:   Thu Sep 22 12:30:50 2016 +0300
+
+    Revert "Enable message duplication by default"
+
+commit cf09b2b6233bf897a17bdb0102962947df87f4ae
+Author: Yogesh Pande <[email protected]>
+Date:   Thu Sep 22 09:57:41 2016 +0300
+
+    Fixed comments from PR#79
+    
+    This commit includes
+     - Fixing comments from PR-79
+
+commit 8fd7cc258168047c29aca27a659de448f45e0e6f
+Author: Yogesh Pande <[email protected]>
+Date:   Wed Sep 21 20:42:28 2016 +0300
+
+    Introducing Configuration management through Config file
+    
+    This commit includes
+     - Support for configuring various build time parameters throuh user supplied config file
+      that can be passed at compile time through Makefile system which will work for
+      non mbed-OS and non yotta build system as well.
+
+commit 330a16bbfc5dc597015c7bdd935b603282d7064e
+Merge: b7d909e e268081
+Author: Antti Yli-Tokola <[email protected]>
+Date:   Wed Sep 21 12:05:29 2016 +0300
+
+    Merge pull request #78 from ARMmbed/resend_flags
+    
+    Make resend flags configurable through Yotta and mbed-cli build sy…
+
+commit e268081bae1fa1db3b5b64dfb77d5b7a5bfed748
+Merge: dc16b29 b7d909e
+Author: Antti Yli-Tokola <[email protected]>
+Date:   Wed Sep 21 12:02:32 2016 +0300
+
+    Merge branch 'master' into resend_flags
+
+commit b7d909eca1c2259453dbe9c094b8a0c53d638fea
+Merge: 7f0eaea ad3b468
+Author: Antti Yli-Tokola <[email protected]>
+Date:   Wed Sep 21 12:02:15 2016 +0300
+
+    Merge pull request #77 from ARMmbed/duplication
+    
+    Enable message duplication by default
+
+commit dc16b299ab9ff4bd1eaa9d3ec120fa58180820c4
+Author: Antti Yli-Tokola <[email protected]>
+Date:   Mon Sep 19 14:35:31 2016 +0300
+
+    Make resending flags configurable through Yotta and mbed-cli build systems
+
+commit ad3b46881bb61dfc2510d13d373bcd1faad114f4
+Author: Antti Yli-Tokola <[email protected]>
+Date:   Fri Sep 16 08:58:38 2016 +0300
+
+    Enable message duplication by default
+
 ## [v3.0.1](https://github.com/ARMmbed/mbed-client-c/releases/tag/v3.0.1) (15-Sep-2016)
 [Full Changelog](https://github.com/ARMmbed/mbed-client-c/compare/mbed-os-5.0-rc1...v3.0.1)
 

features/FEATURE_COMMON_PAL/mbed-client-c/module.json

@@ -1,6 +1,6 @@
 {
   "name": "mbed-client-c",
-  "version": "3.0.1",
+  "version": "3.0.2",
   "description": "Nanostack NSDL and COAP library",
   "keywords": [
     "coap",

features/FEATURE_COMMON_PAL/mbed-client-c/nsdl-c/sn_config.h

@@ -0,0 +1,90 @@
+/*
+ * Copyright (c) 2016 ARM Limited. All rights reserved.
+ * SPDX-License-Identifier: Apache-2.0
+ * 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.
+ */
+
+#ifndef SN_CONFIG_H
+#define SN_CONFIG_H
+
+/**
+* \brief Configuration options (set of defines and values)
+*
+*  This lists set of compile-time options that needs to be used to enable
+*  or disable features selectively, and set the values for the mandatory
+*  parameters.
+*/
+
+/**
+ * \def SN_COAP_DUPLICATION_MAX_MSGS_COUNT
+ * \brief For Message duplication detection
+ * Init value for the maximum count of messages to be stored for duplication detection
+ * Setting of this value to 0 will disable duplication check, also reduce use of ROM memory
+ * Default is set to 1.
+ */
+#undef SN_COAP_DUPLICATION_MAX_MSGS_COUNT   /* 1 */
+
+/**
+ * \def SN_COAP_MAX_BLOCKWISE_PAYLOAD_SIZE
+ *
+ * \brief For Message blockwising
+ * Init value for the maximum payload size to be sent and received at one blockwise message
+ * Setting of this value to 0 will disable this feature, and also reduce use of ROM memory
+ * Note: This define is common for both received and sent Blockwise messages
+ */
+#undef SN_COAP_MAX_BLOCKWISE_PAYLOAD_SIZE   /* 0 */ // < Must be 2^x and x is at least 4. Suitable values: 0, 16, 32, 64, 128, 256, 512 and 1024
+
+/**
+ * \def COAP_DISABLE_OBS_FEATURE
+ *
+ * \brief Disables CoAP 'obs' sending feature
+ * as part of registration message, this might be
+ * needed to be enabled for some strict LWM2M server implementation.
+ * By default, this feature is disabled.
+ */
+#undef COAP_DISABLE_OBS_FEATURE
+
+/**
+ * \def SN_COAP_RESENDING_QUEUE_SIZE_MSGS
+ *
+ * \brief Sets the number of messages stored
+ * in the resending queue. Default is 2
+ */
+#undef SN_COAP_RESENDING_QUEUE_SIZE_MSGS    /* 2  */ // < Default re-sending queue size - defines how many messages can be stored. Setting this to 0 disables feature
+
+/**
+ * \def SN_COAP_RESENDING_QUEUE_SIZE_BYTES
+ *
+ * \brief Sets the size of the re-sending buffer.
+ * Setting this to 0 disables this feature.
+ * By default, this feature is disabled.
+ */
+#undef SN_COAP_RESENDING_QUEUE_SIZE_BYTES   /* 0  */ // Default re-sending queue size - defines size of the re-sending buffer. Setting this to 0 disables feature
+
+/**
+ * \def SN_COAP_MAX_INCOMING_MESSAGE_SIZE
+ *
+ * \brief Sets the maximum size (in bytes) that
+ * mbed Client will allow to be handled while
+ * receiving big payload in blockwise mode.
+ * Application can set this value based on their
+ * available storage capability.
+ * By default, maximum size is UINT16_MAX, 65535 bytes.
+ */
+#undef SN_COAP_MAX_INCOMING_MESSAGE_SIZE    /* UINT16_MAX */
+
+#ifdef MBED_CLIENT_USER_CONFIG_FILE
+#include MBED_CLIENT_USER_CONFIG_FILE
+#endif
+
+#endif // SN_CONFIG_H

features/FEATURE_COMMON_PAL/mbed-client-c/nsdl-c/sn_nsdl_lib.h

@@ -26,6 +26,7 @@
 
 #include "ns_list.h"
 #include "sn_coap_header_internal.h"
+#include "sn_config.h"
 
 #ifdef __cplusplus
 extern "C" {
@@ -41,8 +42,27 @@ struct sn_coap_hdr_;
 #define ENABLE_RESENDINGS                               1   /**< Enable / Disable resending from library in building */
 
 #define SN_COAP_RESENDING_MAX_COUNT                     3   /**< Default number of re-sendings  */
+
+#ifdef YOTTA_CFG_COAP_RESENDING_QUEUE_SIZE_MSGS
+#define SN_COAP_RESENDING_QUEUE_SIZE_MSGS YOTTA_CFG_COAP_RESENDING_QUEUE_SIZE_MSGS
+#elif defined MBED_CONF_MBED_CLIENT_SN_COAP_RESENDING_QUEUE_SIZE_MSGS
+#define SN_COAP_RESENDING_QUEUE_SIZE_MSGS MBED_CONF_MBED_CLIENT_SN_COAP_RESENDING_QUEUE_SIZE_MSGS
+#endif
+
+#ifndef SN_COAP_RESENDING_QUEUE_SIZE_MSGS
 #define SN_COAP_RESENDING_QUEUE_SIZE_MSGS               2   /**< Default re-sending queue size - defines how many messages can be stored. Setting this to 0 disables feature */
+#endif
+
+#ifdef YOTTA_CFG_COAP_RESENDING_QUEUE_SIZE_BYTES
+#define SN_COAP_RESENDING_QUEUE_SIZE_BYTES YOTTA_CFG_COAP_RESENDING_QUEUE_SIZE_BYTES
+#elif defined MBED_CONF_MBED_CLIENT_SN_COAP_RESENDING_QUEUE_SIZE_BYTES
+#define SN_COAP_RESENDING_QUEUE_SIZE_BYTES MBED_CONF_MBED_CLIENT_SN_COAP_RESENDING_QUEUE_SIZE_BYTES
+#endif
+
+#ifndef SN_COAP_RESENDING_QUEUE_SIZE_BYTES
 #define SN_COAP_RESENDING_QUEUE_SIZE_BYTES              0   /**< Default re-sending queue size - defines size of the re-sending buffer. Setting this to 0 disables feature */
+#endif
+
 #define DEFAULT_RESPONSE_TIMEOUT                        10  /**< Default re-sending timeout as seconds */
 
 /* These parameters sets maximum values application can set with API */
@@ -81,7 +101,6 @@ struct sn_coap_hdr_;
 
 /* Init value for the maximum payload size to be sent and received at one blockwise message                         */
 /* Setting of this value to 0 will disable this feature, and also reduce use of ROM memory                          */
-/* Note: Current Coap implementation supports Blockwise transfers specification version draft-ietf-core-block-03    */
 /* Note: This define is common for both received and sent Blockwise messages                                        */
 
 #ifdef YOTTA_CFG_COAP_MAX_BLOCKWISE_PAYLOAD_SIZE

features/FEATURE_COMMON_PAL/mbed-client-c/run_unit_tests.sh

@@ -29,6 +29,7 @@
 #include "sn_coap_protocol_internal.h"
 #include "sn_nsdl_lib.h"
 #include "sn_grs.h"
+#include "sn_config.h"
 #include "mbed-trace/mbed_trace.h"
 
 #define TRACE_GROUP "coap"

features/FEATURE_COMMON_PAL/mbed-client-c/run_unit_tests_with_valgrind.sh

@@ -27,6 +27,7 @@
 #include "sn_coap_parser_stub.h"
 #include "sn_coap_header_check_stub.h"
 
+
 int retCounter = 0;
 static coap_s *coap_handle = NULL;
 void myFree(void* addr);
@@ -87,9 +88,13 @@ TEST(libCoap_protocol, sn_coap_protocol_destroy)
     msg_ptr->send_msg_ptr->uri_path_len = 2;
 
     ns_list_add_to_end(&handle->linked_list_resent_msgs, msg_ptr);
+#if SN_COAP_DUPLICATION_MAX_MSGS_COUNT
     ns_list_init(&handle->linked_list_duplication_msgs);
+#endif
+#if SN_COAP_MAX_BLOCKWISE_PAYLOAD_SIZE
     ns_list_init(&handle->linked_list_blockwise_sent_msgs);
     ns_list_init(&handle->linked_list_blockwise_received_payloads);
+#endif
     CHECK( 0 == sn_coap_protocol_destroy(handle));
 }
 

features/FEATURE_COMMON_PAL/mbed-client-c/source/libCoap/src/include/sn_coap_protocol_internal.h

@@ -0,0 +1,13 @@
+#ifndef TEST_CONFIG_H
+#define TEST_CONFIG_H
+
+/**
+ * \def SN_COAP_DUPLICATION_MAX_MSGS_COUNT
+ * \brief For Message duplication detection
+ * Init value for the maximum count of messages to be stored for duplication detection
+ * Setting of this value to 0 will disable duplication check, also reduce use of ROM memory
+ * Default is set to 0.
+ */
+#define SN_COAP_DUPLICATION_MAX_MSGS_COUNT  1
+
+#endif 

features/FEATURE_COMMON_PAL/mbed-client-c/source/libNsdl/src/include/sn_grs.h

@@ -0,0 +1,32 @@
+# Change Log
+
+## [mbed-os.5.2-rc1](https://github.com/ARMmbed/sal-stack-nanostack-eventloop/releases/tag/mbed-os.5.2-rc1) (24-Sep-2016)
+[Full Changelog](https://github.com/ARMmbed/sal-stack-nanostack-eventloop/compare/mbed-os-5.0-rc1...mbed-os.5.2-rc1)
+
+**New features**
+
+- Move configuration parameter resolution to separate header and allow user definable configuration header to be used.
+- Port eventloop system timer to use new tick timer functions platform_tick_timer_register, platform_tick_timer_start and platform_tick_timer_stop.
+
+**Merged pull requests:**
+
+commit 283299dbd744c91cb30810ad263798842905c054 (HEAD, tag: mbed-os.5.2-rc1, origin/master, origin/HEAD, master)
+Author: Jaakko Kukkohovi <[email protected]>
+Date:   Fri Sep 23 10:05:55 2016 +0300
+
+    Change configuration method (#23)
+    
+    Move configuration parameter resolution to separate header and allow
+    user definable configuration header to be used.
+
+commit 269b00fba6a637c363f7d1690db0889118bca010
+Author: Jaakko Kukkohovi <[email protected]>
+Date:   Wed Sep 21 11:27:40 2016 +0300
+
+    Systimer using platform tick timer (#22)
+    
+    * Port eventloop system timer to use new tick timer functions platform_tick_timer_register, platform_tick_timer_start and platform_tick_timer_stop.Existing functionality is preserved by implementing these functions by default using the high resolution timer as before
+    * New configuration parameter use_platform_tick_timer disables the high resolution timer based tick timer implementation and allows platform to implement the tick timer functions.
+    * New configuration parameter exclude_highres_timer removes the high resolution timer from the build thus providing only eventloop.
+    * Minar support now implemented by providing platform tick timer using minar timers and implicitly using platform tick timers for eventloop when using minar
+