update Golioth Zephyr SDK to v0.16.0

This is based on NCS v2.8.0

- Update async callbacks:
  - Remove golioth_response param
  - Add golioth_status and golioth_coap_rsp_code params
- Replace deprecated CONFIG_EVENTFD_MAX with CONFIG_ZVFS_OPEN_MAX
- Replace deprecated CONFIG_POSIX_MAX_FDS with CONFIG_ZVFS_OPEN_MAX
- Use CONFIG_MBEDTLS_LEGACY_CRYPTO_C to enable support for p-384 curves

Signed-off-by: Mike Szczys <[email protected]>

Author: szczys

boards/aludel_elixir_ns.conf

@@ -32,6 +32,12 @@ CONFIG_GOLIOTH_SAMPLE_NRF91_LTE_MONITOR=y
 CONFIG_MBEDTLS_KEY_EXCHANGE_DHE_PSK_ENABLED=n
 CONFIG_MBEDTLS_KEY_EXCHANGE_DHE_RSA_ENABLED=n
 
+# MbedTLS configuration to support p-384 curve. These options
+# enable using the MbedTLS built-in support for operations not
+# supported by the default nRF Oberon crypto backend
+CONFIG_NORDIC_SECURITY_BACKEND=n
+CONFIG_MBEDTLS_LEGACY_CRYPTO_C=y
+
 # Add Network Info Support
 CONFIG_MODEM_INFO=y
 

boards/aludel_mini_nrf9160_ns.conf

@@ -32,6 +32,12 @@ CONFIG_GOLIOTH_SAMPLE_NRF91_LTE_MONITOR=y
 CONFIG_MBEDTLS_KEY_EXCHANGE_DHE_PSK_ENABLED=n
 CONFIG_MBEDTLS_KEY_EXCHANGE_DHE_RSA_ENABLED=n
 
+# MbedTLS configuration to support p-384 curve. These options
+# enable using the MbedTLS built-in support for operations not
+# supported by the default nRF Oberon crypto backend
+CONFIG_NORDIC_SECURITY_BACKEND=n
+CONFIG_MBEDTLS_LEGACY_CRYPTO_C=y
+
 # Battery Support
 CONFIG_ALUDEL_BATTERY_MONITOR=y
 

prj.conf

@@ -16,15 +16,15 @@ CONFIG_GOLIOTH_STREAM=y
 CONFIG_GOLIOTH_SAMPLE_COMMON=y
 
 # Configure Golioth SDK dependencies
-CONFIG_EVENTFD_MAX=14
+CONFIG_ZVFS_EVENTFD_MAX=14
+CONFIG_ZVFS_OPEN_MAX=23
 CONFIG_LOG_PROCESS_THREAD_STACK_SIZE=1536
 CONFIG_MBEDTLS_ENABLE_HEAP=y
 CONFIG_MBEDTLS_HEAP_SIZE=10240
 CONFIG_MBEDTLS_SSL_IN_CONTENT_LEN=2048
 CONFIG_MBEDTLS_SSL_OUT_CONTENT_LEN=2048
 CONFIG_NETWORKING=y
 CONFIG_NET_IPV4=y
-CONFIG_POSIX_MAX_FDS=23
 
 # Application
 CONFIG_MAIN_STACK_SIZE=2048

src/app_sensors.c

@@ -27,12 +27,12 @@ static struct golioth_client *client;
 /* Add Sensor structs here */
 
 /* Callback for LightDB Stream */
-static void async_error_handler(struct golioth_client *client,
-				const struct golioth_response *response, const char *path,
+static void async_error_handler(struct golioth_client *client, enum golioth_status status,
+				const struct golioth_coap_rsp_code *coap_rsp_code, const char *path,
 				void *arg)
 {
-	if (response->status != GOLIOTH_OK) {
-		LOG_ERR("Async task failed: %d", response->status);
+	if (status != GOLIOTH_OK) {
+		LOG_ERR("Async task failed: %d", status);
 		return;
 	}
 }

src/app_state.c

@@ -24,12 +24,13 @@ uint32_t _example_int1 = 1;
 static struct golioth_client *client;
 
 static void async_handler(struct golioth_client *client,
-				       const struct golioth_response *response,
-				       const char *path,
-				       void *arg)
+			  enum golioth_status status,
+			  const struct golioth_coap_rsp_code *coap_rsp_code,
+			  const char *path,
+			  void *arg)
 {
-	if (response->status != GOLIOTH_OK) {
-		LOG_WRN("Failed to set state: %d", response->status);
+	if (status != GOLIOTH_OK) {
+		LOG_WRN("Failed to set state: %d", status);
 		return;
 	}
 
@@ -81,20 +82,16 @@ int app_state_update_actual(void)
 	return err;
 }
 
-static void app_state_desired_handler(struct golioth_client *client,
-				      const struct golioth_response *response,
-				      const char *path,
-				      const uint8_t *payload,
-				      size_t payload_size,
+static void app_state_desired_handler(struct golioth_client *client, enum golioth_status status,
+				      const struct golioth_coap_rsp_code *coap_rsp_code,
+				      const char *path, const uint8_t *payload, size_t payload_size,
 				      void *arg)
 {
 	int err = 0;
 	int ret;
 
-	if (response->status != GOLIOTH_OK) {
-		LOG_ERR("Failed to receive '%s' endpoint: %d",
-			APP_STATE_DESIRED_ENDP,
-			response->status);
+	if (status != GOLIOTH_OK) {
+		LOG_ERR("Failed to receive '%s' endpoint: %d", APP_STATE_DESIRED_ENDP, status);
 		return;
 	}
 

src/battery_monitor/battery.c

@@ -310,12 +310,12 @@ void log_battery_data(void)
 	LOG_INF("Battery measurement: voltage=%s, level=%s", get_batt_v_str(), get_batt_lvl_str());
 }
 
-static void async_error_handler(struct golioth_client *client,
-				const struct golioth_response *response, const char *path,
+static void async_error_handler(struct golioth_client *client, enum golioth_status status,
+				const struct golioth_coap_rsp_code *coap_rsp_code, const char *path,
 				void *arg)
 {
-	if (response->status != GOLIOTH_OK) {
-		LOG_ERR("Failed to stream battery data: %d", response->status);
+	if (status != GOLIOTH_OK) {
+		LOG_ERR("Failed to stream battery data: %d", status);
 		return;
 	}
 }

west.yml

@@ -7,7 +7,7 @@ manifest:
   projects:
     - name: golioth
       path: modules/lib/golioth-firmware-sdk
-      revision: v0.15.0
+      revision: v0.16.0
       url: https://github.com/golioth/golioth-firmware-sdk.git
       west-commands: scripts/west-commands.yml
       submodules: true