modules: Remove memfault module

Remove memfault module, all needed Memfault is now included in NCSs
integration.

Signed-off-by: Simen S. Røstad <simen.rostad@nordicsemi.no>

Author: simensrostad

app/CMakeLists.txt

@@ -24,12 +24,8 @@ add_subdirectory(src/modules/button)
 add_subdirectory_ifdef(CONFIG_APP_BATTERY src/modules/battery)
 add_subdirectory_ifdef(CONFIG_APP_ENVIRONMENTAL src/modules/environmental)
 add_subdirectory_ifdef(CONFIG_APP_LED src/modules/led)
-add_subdirectory_ifdef(CONFIG_APP_MEMFAULT src/modules/memfault)
 add_subdirectory_ifdef(CONFIG_APP_SHELL src/modules/shell)
 
-# Include Memfault configuration folder
-zephyr_include_directories(config)
-
 if (CONFIG_NRF_CLOUD_COAP_SEC_TAG GREATER_EQUAL 2147483648 AND CONFIG_NRF_CLOUD_COAP_SEC_TAG LESS_EQUAL 2147483667)
 	message(WARNING "CONFIG_NRF_CLOUD_COAP_SEC_TAG is set to a developer security tag. "
 			"TLS traffic can now be decrypted with Nordic tools. "

app/Kconfig

@@ -21,7 +21,6 @@ rsource "src/modules/led/Kconfig.led"
 rsource "src/modules/app/Kconfig.app"
 rsource "src/modules/fota/Kconfig.fota"
 rsource "src/modules/environmental/Kconfig.environmental"
-rsource "src/modules/memfault/Kconfig.memfault"
 rsource "src/modules/shell/Kconfig.shell"
 rsource "src/modules/button/Kconfig.button"
 

app/boards/thingy91x_nrf9151_ns.conf

@@ -15,6 +15,7 @@ CONFIG_APP_BATTERY=y
 
 # LED module
 CONFIG_APP_LED=y
+CONFIG_APP_LED_LOG_LEVEL_DBG=y
 
 # Fuel Gage
 CONFIG_NRF_FUEL_GAUGE=y
@@ -23,6 +24,7 @@ CONFIG_NPM1300_CHARGER=y
 # Wi-Fi
 CONFIG_WIFI=y
 CONFIG_WIFI_NRF70=y
+CONFIG_WIFI_NRF70_LOG_LEVEL_ERR=y
 CONFIG_WIFI_NRF70_SKIP_LOCAL_ADMIN_MAC=y
 CONFIG_NRF70_BH_WQ_STACK_SIZE=1024
 CONFIG_NRF70_IRQ_WQ_STACK_SIZE=1024

app/config/memfault_platform_config.h

@@ -1,19 +1,31 @@
 #
-# Copyright (c) 2024 Nordic Semiconductor
+# Copyright (c) 2025 Nordic Semiconductor
 #
 # SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
 #
 
-CONFIG_APP_MEMFAULT=y
-CONFIG_APP_MEMFAULT_LOG_LEVEL_DBG=y
-CONFIG_MEMFAULT_NCS_DEVICE_ID_IMEI=y
-CONFIG_MEMFAULT_NCS_PROJECT_KEY=""
-CONFIG_MEMFAULT_LOGGING_RAM_SIZE=4096
+CONFIG_MEMFAULT=y
 CONFIG_MEMFAULT_HEAP_STATS=y
-CONFIG_MEMFAULT_NCS_LOCATION_METRICS=y
-CONFIG_MEMFAULT_HTTP_DEDICATED_WORKQUEUE_STACK_SIZE=1560
+CONFIG_MEMFAULT_LOGGING_ENABLE=y
+CONFIG_MEMFAULT_LOGGING_RAM_SIZE=4096
 CONFIG_MEMFAULT_COREDUMP_FULL_THREAD_STACKS=y
+CONFIG_MEMFAULT_HTTP_DEDICATED_WORKQUEUE_STACK_SIZE=1560
+CONFIG_MEMFAULT_HTTP_PERIODIC_UPLOAD=y
+CONFIG_MEMFAULT_HTTP_ENABLE=y
 CONFIG_MEMFAULT_EVENT_STORAGE_SIZE=2048
 
+# Certificate management
+CONFIG_MEMFAULT_ROOT_CERT_STORAGE_NRF9160_MODEM=y
+CONFIG_MODEM_KEY_MGMT=y
+
+# Memfault NCS integration
+CONFIG_MEMFAULT_NCS_DEVICE_ID_IMEI=y
+CONFIG_MEMFAULT_NCS_PROJECT_KEY=""
+CONFIG_MEMFAULT_NCS_LOCATION_METRICS=y
+CONFIG_MEMFAULT_NCS_LTE_METRICS=y
+CONFIG_MEMFAULT_NCS_STACK_METRICS=y
+CONFIG_MEMFAULT_NCS_INTERNAL_FLASH_BACKED_COREDUMP=y
+CONFIG_MEMFAULT_NCS_POST_COREDUMP_ON_NETWORK_CONNECTED=y
+
 # Memfault depends on POSIX, disable unneeded POSIX features
 CONFIG_POSIX_FILE_SYSTEM=n

app/config/memfault_trace_reason_user_config.def

@@ -196,7 +196,6 @@ CONFIG_APP_FOTA_LOG_LEVEL_DBG=y
 CONFIG_APP_LOCATION_LOG_LEVEL_DBG=y
 CONFIG_APP_CLOUD_LOG_LEVEL_DBG=y
 CONFIG_APP_NETWORK_LOG_LEVEL_DBG=y
-CONFIG_APP_LED_LOG_LEVEL_DBG=y
 CONFIG_APP_SHELL_LOG_LEVEL_DBG=y
 CONFIG_APP_LOG_LEVEL_DBG=y
 

app/overlay-memfault.conf

@@ -12,20 +12,24 @@ tests:
     sysbuild: true
     integration_platforms:
       - thingy91x/nrf9151/ns
+      - nrf9151dk/nrf9151/ns
     platform_allow:
       - thingy91x/nrf9151/ns
+      - nrf9151dk/nrf9151/ns
+    tags: ci_build
+  app.build.memfault:
+    build_only: true
+    sysbuild: true
+    integration_platforms:
+      - thingy91x/nrf9151/ns
+      - nrf9151dk/nrf9151/ns
+    platform_allow:
+      - thingy91x/nrf9151/ns
+      - nrf9151dk/nrf9151/ns
+    extra_configs:
+      - CONFIG_MEMFAULT_NCS_PROJECT_KEY="SECRET"
+    extra_args: EXTRA_CONF_FILE="overlay-memfault.conf;overlay-etb.conf"
     tags: ci_build
-  # app.build.memfault:
-  #   build_only: true
-  #   sysbuild: true
-  #   integration_platforms:
-  #     - thingy91x/nrf9151/ns
-  #   platform_allow:
-  #     - thingy91x/nrf9151/ns
-  #   extra_configs:
-  #     - CONFIG_MEMFAULT_NCS_PROJECT_KEY="PROJECTKEY"
-  #   extra_args: EXTRA_CONF_FILE="overlay-memfault.conf;overlay-modemtrace-to-memfault.conf;overlay-etb.conf"
-  #   tags: ci_build
   # app.build.bootloader_update:
   #   build_only: true
   #   sysbuild: true

app/overlay-modemtrace-to-memfault.conf

@@ -15,12 +15,15 @@
 #include "message_channel.h"
 #include "button.h"
 #include "network.h"
-#include "environmental.h"
 #include "cloud_module.h"
 #include "fota.h"
 #include "location.h"
 #include "led.h"
 
+#if defined(CONFIG_APP_ENVIRONMENTAL)
+#include "environmental.h"
+#endif /* CONFIG_APP_ENVIRONMENTAL */
+
 #if defined(CONFIG_APP_BATTERY)
 #include "battery.h"
 #endif /* CONFIG_APP_BATTERY */

app/prj.conf

@@ -470,10 +470,6 @@ static void shadow_get(bool delta_only)
 		return;
 	} else if (err > 0) {
 		LOG_WRN("Cloud error: %d", err);
-
-		IF_ENABLED(CONFIG_MEMFAULT,
-			(MEMFAULT_TRACE_EVENT_WITH_STATUS(nrf_cloud_coap_shadow_get, err)));
-
 		return;
 	} else if (err) {
 		LOG_ERR("Failed to request shadow delta: %d", err);