WIP HTTP on the MCXN947

Author: noahp

overlay-http.conf

@@ -21,6 +21,8 @@ CONFIG_MBEDTLS_SERVER_NAME_INDICATION=y
 CONFIG_TLS_CREDENTIALS=y
 CONFIG_DNS_SERVER_IP_ADDRESSES=y
 CONFIG_DNS_SERVER1="1.1.1.1"
+# TODO, the PEM certs fail to parse, I suspect due to OOM
+CONFIG_MEMFAULT_TLS_CERTS_USE_DER=y
 
 # not sure these help?
 # CONFIG_MBEDTLS_ENABLE_HEAP=y
@@ -32,7 +34,16 @@ CONFIG_MBEDTLS_ENABLE_HEAP=y
 CONFIG_MBEDTLS_HEAP_SIZE=34000
 CONFIG_MBEDTLS_SSL_MAX_CONTENT_LEN=4096
 
+# 1536 seems to be the minimum for the MCXN947 when we send memfault data
+# CONFIG_NET_BUF_DATA_SIZE=512
+CONFIG_NET_BUF_DATA_SIZE=1536
+
 # debug
 CONFIG_MBEDTLS_DEBUG=y
 CONFIG_MBEDTLS_LOG_LEVEL_DBG=y
 CONFIG_NET_SOCKETS_LOG_LEVEL_DBG=y
+CONFIG_NET_LOG=y
+CONFIG_NET_BUF_LOG=y
+CONFIG_NET_BUF_LOG_LEVEL_DBG=y
+# CONFIG_NET_BUF_SIMPLE_LOG=y
+# CONFIG_NET_L2_ETHERNET_LOG_LEVEL_DBG=y

prj.conf

@@ -21,6 +21,8 @@ CONFIG_LOG_PRINTK=n
 
 CONFIG_HEAP_MEM_POOL_SIZE=4096
 CONFIG_MAIN_STACK_SIZE=4096
+CONFIG_SHELL_STACK_SIZE=4096
+CONFIG_SYSTEM_WORKQUEUE_STACK_SIZE=2048
 
 CONFIG_ASSERT=y
 CONFIG_COMPILER_WARNINGS_AS_ERRORS=y

src/main.c

@@ -215,6 +215,8 @@ void k_sys_fatal_error_handler(unsigned int reason, const z_arch_esf_t *esf) {
 #include "memfault/ports/zephyr/http.h"
 static void initialize_net(void) {
 #if defined(CONFIG_NETWORKING)
+  memfault_zephyr_port_install_root_certs();
+
   struct net_if *iface = net_if_get_default();
 
   if (!iface) {

west.yml

@@ -7,9 +7,10 @@ manifest:
       # https://github.com/zephyrproject-rtos/zephyr/commit/e4e463af812e61bb5bfb4b1087d48e1dbcd9d8d5
       # And board support was added here:
       # https://github.com/zephyrproject-rtos/zephyr/commit/8fa1c6d9c6cbc7972f482b17584c48b4ea952e55
-      # Use a commit that's a little later, that seems more stable.
+      # And this PR as of 2024-03-23 has the ETH QoS driver added, required for
+      # the HTTP example:
       url: https://github.com/zephyrproject-rtos/zephyr
-      revision: 369e40b27bca1f08fefe23924f7dc82dd096c53f
+      revision: refs/pull/70637/head
 
       import:
         # Limit the Zephyr modules to the required set