From 29467e04fbc15f53001c0c95227db34160b776e7 Mon Sep 17 00:00:00 2001 From: Kristofer Jonsson Date: Fri, 26 Nov 2021 16:10:43 +0100 Subject: Fixing compiler warnings Adjusting toolchain files which compiler warnings to enable. Fixing compiler warnings reported by Arm Clang and Arm GCC. Change-Id: I715e875904ffd7ecfe994d3093cdf066373981b1 --- applications/freertos/main.cpp | 6 +++--- applications/message_handler/main.cpp | 2 ++ applications/message_handler/message_handler.cpp | 2 +- applications/trustzone_inference/secure/main_secure.cpp | 4 ++-- 4 files changed, 8 insertions(+), 6 deletions(-) (limited to 'applications') diff --git a/applications/freertos/main.cpp b/applications/freertos/main.cpp index ae4bc38..0f4009e 100644 --- a/applications/freertos/main.cpp +++ b/applications/freertos/main.cpp @@ -164,7 +164,7 @@ void ethosu_semaphore_give(void *sem) { if (xPortIsInsideInterrupt()) { ret = xSemaphoreGiveFromISR(handle, NULL); if (ret != pdTRUE) { - printf("Error: Failed to give semaphore from ISR. ret - 0x%08x\n", ret); + printf("Error: Failed to give semaphore from ISR. ret - 0x%08lx\n", ret); } } else { ret = xSemaphoreGive(handle); @@ -210,7 +210,7 @@ void inferenceProcessTask(void *pvParameters) { void inferenceSenderTask(void *pvParameters) { int ret = 0; - QueueHandle_t inferenceProcessQueue = reinterpret_cast(pvParameters); + QueueHandle_t _inferenceProcessQueue = reinterpret_cast(pvParameters); xInferenceJob jobs[NUM_JOBS_PER_TASK]; // Create queue for response messages @@ -227,7 +227,7 @@ void inferenceSenderTask(void *pvParameters) { job->responseQueue = senderQueue; // Send job printf("inferenceSenderTask: Sending inference job: job=%p, name=%s\n", job, job->name.c_str()); - if (xQueueSend(inferenceProcessQueue, &job, portMAX_DELAY) != pdPASS) { + if (xQueueSend(_inferenceProcessQueue, &job, portMAX_DELAY) != pdPASS) { printf("Error: inferenceSenderTask failed in send to Q.\n"); exit(1); } diff --git a/applications/message_handler/main.cpp b/applications/message_handler/main.cpp index 9f7c7cc..9b36f84 100644 --- a/applications/message_handler/main.cpp +++ b/applications/message_handler/main.cpp @@ -128,9 +128,11 @@ void ethosu_semaphore_give(void *sem) { namespace { +#ifdef MHU_IRQ void mailboxIrqHandler() { mailbox.handleMessage(); } +#endif void inferenceTask(void *pvParameters) { printf("Starting inference task\n"); diff --git a/applications/message_handler/message_handler.cpp b/applications/message_handler/message_handler.cpp index 7401546..f9f7304 100644 --- a/applications/message_handler/message_handler.cpp +++ b/applications/message_handler/message_handler.cpp @@ -375,7 +375,7 @@ void OutgoingMessageHandler::sendErrorRsp(ethosu_core_msg_err &error) { } void OutgoingMessageHandler::readCapabilties(ethosu_core_msg_capabilities_rsp &rsp) { - rsp = {0}; + rsp = {}; #ifdef ETHOSU struct ethosu_driver_version version; diff --git a/applications/trustzone_inference/secure/main_secure.cpp b/applications/trustzone_inference/secure/main_secure.cpp index df22929..984f19a 100644 --- a/applications/trustzone_inference/secure/main_secure.cpp +++ b/applications/trustzone_inference/secure/main_secure.cpp @@ -87,7 +87,7 @@ static int setup_sram0_mpc(const uint32_t baseaddr_s, /* Secure base address */ const struct mpc_sie_dev_cfg_t mpc_dev_cfg = {SRAM0_MPC}; /* MPC device data */ - struct mpc_sie_dev_data_t mpc_dev_data = {0}; + struct mpc_sie_dev_data_t mpc_dev_data = {}; /* MPC device itself */ struct mpc_sie_dev_t mpc_dev = {&mpc_dev_cfg, &mpc_dev_data}; @@ -154,7 +154,7 @@ static int setup_bram_mpc(const uint32_t baseaddr_s, /* Secure base address */ const struct mpc_sie_dev_cfg_t mpc_dev_cfg = {BRAM_MPC}; /* MPC device data */ - struct mpc_sie_dev_data_t mpc_dev_data = {0}; + struct mpc_sie_dev_data_t mpc_dev_data = {}; /* MPC device itself */ struct mpc_sie_dev_t mpc_dev = {&mpc_dev_cfg, &mpc_dev_data}; -- cgit v1.2.1