Skip to content

Commit 771e0fa

Browse files
gminnjtguggedal
authored andcommitted
memfault: fix kconfig style issues
Three Kconfig style issues were caught by a style script shared in zephyrproject-rtos/zephyr#94780. This PR addresses 2/3 of these: - MEMFAULT_NCS_STACK_METRICS - incorrect indent of help text - MEMFAULT_CDR_ENABLE – default n bool. This is a redefinition of a symbol in the Memfault module to override the default y and select another Kconfig: - There is no clear reason to make it default y. Code and RAM space added is reasonable: +460B flash, +224B RAM when building samples/debug/memfault, and can always be opted out of by the user - The #define in the extra platform config is no longer needed, so the select statement is unnecessary. The Memfault SDK now reads the Kconfig and sets this define automatically. Re-running the style script confirmed these two issues were fixed by these changes. The third failure (MEMFAULT_HTTP_USES_MBEDTLS - default n bool) will be fixed in a follow-up PR after the next Memfault module release, which will include this default in the original definition. Signed-off-by: Gillian Minnehan <[email protected]>
1 parent 82637e0 commit 771e0fa

File tree

2 files changed

+1
-12
lines changed

2 files changed

+1
-12
lines changed

modules/memfault-firmware-sdk/Kconfig

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -144,9 +144,7 @@ config MEMFAULT_NCS_STACK_METRICS
144144
default y
145145
help
146146
Collect metrics for unused stack space for selected stacks.
147-
Currently the following stacks' unused space is monitored:
148-
- connection_poll_thread, used by the cloud libraries for
149-
nRF Cloud, AWS IoT and Azure IoT Hub
147+
Currently the connection_poll_thread, BT RX, and BT TX threads are supported.
150148

151149
config MEMFAULT_NCS_LTE_METRICS
152150
bool "Collect LTE metrics"
@@ -253,11 +251,6 @@ config MEMFAULT_NCS_INTERNAL_FLASH_BACKED_COREDUMP
253251
help
254252
Use internal flash to store coredump data
255253

256-
config MEMFAULT_CDR_ENABLE
257-
bool "Custom Data Recording"
258-
select MEMFAULT_PLATFORM_EXTRA_CONFIG_FILE
259-
default n
260-
261254
if MEMFAULT_NCS_INTERNAL_FLASH_BACKED_COREDUMP
262255
config MEMFAULT_NCS_FLASH_REGION_SIZE
263256
hex

modules/memfault-firmware-sdk/config/memfault_platform_config_extra.h

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -10,10 +10,6 @@
1010
* "<NCS folder>/modules/lib/memfault-firmware-sdk/components/include/memfault/default_config.h"
1111
*/
1212

13-
#ifdef CONFIG_MEMFAULT_CDR_ENABLE
14-
#define MEMFAULT_CDR_ENABLE 1
15-
#endif
16-
1713
#ifdef CONFIG_MEMFAULT_NCS_ETB_CAPTURE
1814
#define MEMFAULT_PLATFORM_FAULT_HANDLER_CUSTOM 1
1915
#endif

0 commit comments

Comments
 (0)