storage: minor fixes

minor fixes

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

Author: simensrostad

app/overlay-memfault.conf

@@ -19,6 +19,9 @@ CONFIG_MEMFAULT_COREDUMP_COLLECT_BSS_REGIONS=y
 CONFIG_MEMFAULT_COREDUMP_FULL_THREAD_STACKS=y
 ## Disable metric to remove error due to missing File System mount point
 CONFIG_MEMFAULT_FS_BYTES_FREE_METRIC=n
+## Disable location metrics due to not being properly implemented for external handling of location
+## requests when using the location library.
+CONFIG_MEMFAULT_NCS_LOCATION_METRICS=n
 
 # Certificate management
 CONFIG_MEMFAULT_ROOT_CERT_STORAGE_NRF9160_MODEM=y
@@ -27,7 +30,6 @@ 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

app/src/main.c

@@ -440,7 +440,8 @@ static void sampling_begin_common(struct main_state *state_object)
 	uint32_t time_elapsed = current_time - state_object->sample_start_time;
 
 	if ((state_object->sample_start_time > 0) &&
-	    (time_elapsed < state_object->sample_interval_sec)) {
+	    (time_elapsed < state_object->sample_interval_sec) &&
+	    (state_object->chan != &BUTTON_CHAN)) {
 		LOG_DBG("Too soon to start sampling, time_elapsed: %d, interval: %d",
 			time_elapsed, state_object->sample_interval_sec);
 

app/src/modules/storage/Kconfig.storage

@@ -41,7 +41,7 @@ config APP_STORAGE_MAX_RECORDS_PER_TYPE
 
 config APP_STORAGE_BATCH_BUFFER_SIZE
 	int "Storage batch buffer size in bytes"
-	default 256
+	default 1024
 	help
 	  Size of the internal buffer for batch data access.
 	  This buffer is used to transfer data from the storage backend
@@ -95,6 +95,7 @@ config APP_STORAGE_SHELL
 
 config APP_STORAGE_SHELL_STATS
 	bool "Enable storage shell stats commands"
+	default y if APP_STORAGE_SHELL
 	help
 	  Enable shell commands for displaying statistics about the storage module.
 	  This includes information about stored records, types, and memory usage.

app/src/modules/storage/storage.c

@@ -938,8 +938,7 @@ static void state_buffer_pipe_active_run(void *o)
 			break;
 
 		default:
-			LOG_DBG("Ignoring message type: %d", msg->type);
-
+			/* Don't care */
 			break;
 		}
 	}

app/src/modules/storage/storage_shell.c

@@ -165,4 +165,4 @@ SHELL_STATIC_SUBCMD_SET_CREATE(storage_sub_cmds,
 	SHELL_SUBCMD_SET_END
 );
 
-SHELL_CMD_REGISTER(storage, &storage_sub_cmds, "Storage module commands", NULL);
+SHELL_CMD_REGISTER(att_storage, &storage_sub_cmds, "Storage module commands", NULL);