commit 45c6c204d509a2a04031744680060abe43bf49db
author Mingyang Sun <mingyang.sun@arm.com> Wed Sep 23 16:17:50 2020 +0800
committer Mingyang Sun <mingyang.sun@arm.com> Tue Oct 27 15:25:21 2020 +0800
tree 70e1758ea5864c33a4048d5a153d15e5e03961c6
parent 78458aa49c41492c08f7ed464eb2150fd5873ede

HAL: Fix hal status type definition

CHAR_MIN may be defined as unsigned by default on some systems,
and the value is 0. Use SCHAR_MIN to make sure it is negative.
Tthe HAL design document is also updated.

Change-Id: Ie74f261f704430d9bb2688960f6a8b938ef4c375
Signed-off-by: Mingyang Sun <mingyang.sun@arm.com>

docs/design_documents/hardware_abstraction_layer.rst

diff --git a/docs/design_documents/hardware_abstraction_layer.rst b/docs/design_documents/hardware_abstraction_layer.rst
index 6f862ef..ba3a1c2 100644
--- a/docs/design_documents/hardware_abstraction_layer.rst
+++ b/docs/design_documents/hardware_abstraction_layer.rst
@@ -128,7 +128,7 @@
 .. code-block:: c
 
   enum tfm_hal_status_t {
-      TFM_HAL_ERROR_MEM_FAULT = CHAR_MIN,
+      TFM_HAL_ERROR_MEM_FAULT = SCHAR_MIN,
       TFM_HAL_ERROR_MAX_VALUE,
       TFM_HAL_ERROR_BAD_STATE,
       TFM_HAL_ERROR_NOT_SUPPORTED,

platform/include/tfm_hal_defs.h

diff --git a/platform/include/tfm_hal_defs.h b/platform/include/tfm_hal_defs.h
index e099077..34a0a8e 100644
--- a/platform/include/tfm_hal_defs.h
+++ b/platform/include/tfm_hal_defs.h
@@ -12,7 +12,7 @@
 #include <stdint.h>
 
 enum tfm_hal_status_t {
-    TFM_HAL_ERROR_MEM_FAULT = CHAR_MIN,
+    TFM_HAL_ERROR_MEM_FAULT = SCHAR_MIN,
     TFM_HAL_ERROR_MAX_VALUE,
     TFM_HAL_ERROR_BAD_STATE,
     TFM_HAL_ERROR_NOT_SUPPORTED,