suit: Logs correction

Misleading logs corrected.

Signed-off-by: Sylwester Konczyk <[email protected]>

Author: SylwesterKonczyk

subsys/mgmt/suitfu/src/suitfu_mgmt.c

@@ -24,14 +24,14 @@ LOG_MODULE_REGISTER(suitfu_mgmt, CONFIG_MGMT_SUITFU_LOG_LEVEL);
 
 #define SYSTEM_UPDATE_WORKER_STACK_SIZE 2048
 
-K_THREAD_STACK_DEFINE(system_update_stack_area, SYSTEM_UPDATE_WORKER_STACK_SIZE);
+static K_THREAD_STACK_DEFINE(system_update_stack_area, SYSTEM_UPDATE_WORKER_STACK_SIZE);
 
 struct system_update_work {
 	struct k_work_delayable work;
 	/* Other data to pass to the workqueue might go here */
 };
 
-struct k_work_q system_update_work_queue;
+static struct k_work_q system_update_work_queue;
 
 static void update_failure(void)
 {
@@ -40,7 +40,7 @@ static void update_failure(void)
 	}
 }
 
-static void schedule_system_update(struct k_work *item)
+static void schedule_envelope_processing(struct k_work *item)
 {
 	int ret = suit_dfu_candidate_preprocess();
 
@@ -85,8 +85,8 @@ int suitfu_mgmt_candidate_envelope_process(void)
 {
 	static struct system_update_work suw;
 
-	LOG_INF("Schedule system reboot");
-	k_work_init_delayable(&suw.work, schedule_system_update);
+	LOG_INF("Schedule envelope processing");
+	k_work_init_delayable(&suw.work, schedule_envelope_processing);
 
 	int ret = k_work_schedule_for_queue(&system_update_work_queue, &suw.work, K_NO_WAIT);
 

subsys/sdfw_services/services/suit_service/suit_ipuc.c

@@ -102,7 +102,7 @@ suit_plat_err_t suit_ipuc_get_info(size_t idx, struct zcbor_string *component_id
 	*role = rsp_data->SSF_SUIT_RSP_ARG(get_ipuc_info, role);
 	ssf_client_decode_done(rsp_pkt);
 
-#if defined(CONFIG_SUIT_LOG_LEVEL_INF) || defined(CONFIG_SUIT_LOG_LEVEL_DBG)
+#if defined(CONFIG_SSF_SUIT_SERVICE_LOG_LEVEL_INF) || defined(CONFIG_SSF_SUIT_SERVICE_LOG_LEVEL_DBG)
 	suit_component_type_t component_type = SUIT_COMPONENT_TYPE_UNSUPPORTED;
 
 	suit_plat_decode_component_type(component_id, &component_type);

subsys/suit/orchestrator_app/src/suit_orchestrator_app.c

@@ -6,6 +6,7 @@
 
 #include <dfu/suit_dfu.h>
 
+#include <zephyr/kernel.h>
 #include <zephyr/logging/log.h>
 #include <zephyr/logging/log_ctrl.h>
 
@@ -266,6 +267,20 @@ int suit_dfu_update_start(void)
 		update_regions_count++;
 	}
 #endif /* CONFIG_SUIT_CACHE_APP_IPUC_ID */
+#endif
+	for (size_t i = 0; i < update_regions_count; i++) {
+		LOG_INF("Update region %d/%d, 0x%08X, %d", i + 1, update_regions_count,
+			(uint32_t)update_candidate[i].mem, update_candidate[i].size);
+	}
+
+#if defined(CONFIG_SUIT_LOG_LEVEL_INF) || defined(CONFIG_SUIT_LOG_LEVEL_DBG)
+	/* Display empty line prior to reboot
+	 */
+	printk("\n");
+
+	/* Give a chance to display a log prior to reboot
+	 */
+	k_msleep(100);
 #endif
 
 	return suit_trigger_update(update_candidate, update_regions_count);