commit fa5d460e5b28402e2f221d8c775d74cad059c98a
author Antonio de Angelis <Antonio.deAngelis@arm.com> Wed Jun 09 16:11:11 2021 +0100
committer Ken Liu <ken.liu@arm.com> Thu Jun 17 07:27:36 2021 +0200
tree a1004e82276a29df6483f0307bdce55bab76c155
parent 0c20bcc2d02f09dbc528a9a07cbc277e25ea043b

SPM: Use SPMLOG_ERRMSG instead of ERROR_MSG macro

The ERROR_MSG() macro is still used through the code
although is not defined anymore. Use the SPMLOG_ERRMSG
macro instead to provide error message prints.

Signed-off-by: Antonio de Angelis <antonio.deangelis@arm.com>
Change-Id: If23455e6f008b0ecc471579591c5d1e2e95cdccb

secure_fw/spm/cmsis_func/spm_func.c

diff --git a/secure_fw/spm/cmsis_func/spm_func.c b/secure_fw/spm/cmsis_func/spm_func.c
index 0967844..1c8edbc 100644
--- a/secure_fw/spm/cmsis_func/spm_func.c
+++ b/secure_fw/spm/cmsis_func/spm_func.c
@@ -762,7 +762,7 @@
 
         if ((id != TFM_SP_CORE_ID) || (tfm_secure_lock != 0)) {
             /* Invalid request during system initialization */
-            ERROR_MSG("Invalid service request during initialization!");
+            SPMLOG_ERRMSG("Invalid service request during initialization!\r\n");
             return TFM_ERROR_NOT_INITIALIZED;
         }
     }
@@ -806,7 +806,7 @@
 
     res = tfm_check_sfn_req_integrity(desc_ptr);
     if (res != TFM_SUCCESS) {
-        ERROR_MSG("Invalid service request!");
+        SPMLOG_ERRMSG("Invalid service request!\r\n");
         tfm_secure_api_error_handler();
     }
 
@@ -827,7 +827,7 @@
         tfm_spm_partition_set_state(
             desc_ptr->caller_part_idx, SPM_PARTITION_STATE_CLOSED);
         __enable_irq();
-        ERROR_MSG("Unauthorized service request!");
+        SPMLOG_ERRMSG("Unauthorized service request!\r\n");
         tfm_secure_api_error_handler();
     }
 
@@ -835,7 +835,7 @@
     if (res != TFM_SUCCESS) {
         /* FixMe: consider possible fault scenarios */
         __enable_irq();
-        ERROR_MSG("Failed to process service request!");
+        SPMLOG_ERRMSG("Failed to process service request!\r\n");
         tfm_secure_api_error_handler();
     }
 
@@ -877,7 +877,7 @@
         /* Unlock errors indicate ctx database corruption or unknown
          * anomalies. Halt execution
          */
-        ERROR_MSG("Secure API error during unlock!");
+        SPMLOG_ERRMSG("Secure API error during unlock!\r\n");
         tfm_secure_api_error_handler();
     }
     return (int32_t)res;
@@ -963,7 +963,7 @@
          * from Handler mode, which is not supported.
          * FixMe: error severity TBD
          */
-        ERROR_MSG("Service request SVC called with MSP active!");
+        SPMLOG_ERRMSG("Service request SVC called with MSP active!\r\n");
         tfm_secure_api_error_handler();
     }
 
@@ -988,7 +988,7 @@
 
     if (excReturn & EXC_RETURN_STACK_PROCESS) {
         /* FixMe: error severity TBD */
-        ERROR_MSG("Partition request SVC called with PSP active!");
+        SPMLOG_ERRMSG("Partition request SVC called with PSP active!\r\n");
         tfm_secure_api_error_handler();
     }
 
@@ -1048,7 +1048,7 @@
         /* Partition request SVC called with MSP active.
          * FixMe: error severity TBD
          */
-        ERROR_MSG("Partition request SVC called with MSP active!");
+        SPMLOG_ERRMSG("Partition request SVC called with MSP active!\r\n");
         tfm_secure_api_error_handler();
     }
 
@@ -1057,7 +1057,7 @@
         /* Unlock errors indicate ctx database corruption or unknown anomalies
          * Halt execution
          */
-        ERROR_MSG("Secure API error during unlock!");
+        SPMLOG_ERRMSG("Secure API error during unlock!\r\n");
         tfm_secure_api_error_handler();
     }