commit f085184fe5399b3659b98a8fae04ebb8c7f54927
author Mingyang Sun <mingyang.sun@arm.com> Tue May 11 11:44:19 2021 +0800
committer Ken Liu <ken.liu@arm.com> Wed May 19 04:54:29 2021 +0200
tree aaddbab1a084dcd3ecf36a6d80061d4e4fb5a34c
parent 69ee5f694aa34ded27009a479d7a9362a3a92f22

SPM: Fix warning of variable length data

Fix the warnings from armclang of variable length data not placed
at the end of struct.

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

secure_fw/include/load/partition_defs.h

diff --git a/secure_fw/include/load/partition_defs.h b/secure_fw/include/load/partition_defs.h
index 014f833..74d04d0 100644
--- a/secure_fw/include/load/partition_defs.h
+++ b/secure_fw/include/load/partition_defs.h
@@ -36,7 +36,12 @@
 #define SPM_PART_FLAG_PSA_ROT                   (1U << 8)
 #define SPM_PART_FLAG_IPC                       (1U << 9)
 
-/* Common partition structure type */
+/*
+ * Common partition structure type, the extendable data is right after it.
+ * Extendable data has different size for each partition, and must be 4-byte
+ * aligned. It includes: stack and heap position, dependencies, services and
+ * assets data.
+ */
 struct partition_static_info_t {
     uint32_t        psa_ff_ver;         /* Encode the version with magic    */
     uint32_t        pid;                /* Partition ID                     */
@@ -47,7 +52,6 @@
     uint32_t        ndeps;              /* Dependency number                */
     uint32_t        nservices;          /* Service number                   */
     uint32_t        nassets;            /* Asset numbers                    */
-    uintptr_t       vars[];             /* Struct extendable indicator      */
 } __attribute__((aligned(4)));
 
 #endif /* __PARTITION_DEFS_H__ */

secure_fw/include/load/partition_static_load.h

diff --git a/secure_fw/include/load/partition_static_load.h b/secure_fw/include/load/partition_static_load.h
index 1e7b4ca..f2f9c3d 100644
--- a/secure_fw/include/load/partition_static_load.h
+++ b/secure_fw/include/load/partition_static_load.h
@@ -36,7 +36,7 @@
      (ps_ptr)->nassets * sizeof(struct asset_desc_t))
 
 #define STATIC_INF_DEPS(ps_ptr)                                         \
-    ((uintptr_t)(ps_ptr)->vars + STATIC_INFO_EXT_LENGTH * sizeof(uintptr_t))
+    ((uintptr_t)(ps_ptr + 1) + STATIC_INFO_EXT_LENGTH * sizeof(uintptr_t))
 #define STATIC_INF_SERVICE(ps_ptr)                                      \
     ((uintptr_t)STATIC_INF_DEPS(ps_ptr) + (ps_ptr)->ndeps * sizeof(uint32_t))
 #define STATIC_INF_ASSET(ps_ptr)                                        \