commit 5b64747890bb391808a3714474d6beba68c9329b
author Tamas Ban <tamas.ban@arm.com> Sat Jan 05 08:59:30 2019 +0000
committer Tamas Ban <tamas.ban@arm.com> Wed Feb 13 14:02:13 2019 +0100
tree f92bc45f30cb9ad2ffbb1e3bed3ba2584fbd83d5
parent b6b8056aa8861f9ae2faf4f0c5e84ff82929e34f

Attest: Refactor shared data TLV entry header

It was done to make easier the lookup of boot status
relatead entries, which belongs to the same SW module.

Change-Id: I753dc03a2373d1975245502288e10c507402b6d5
Signed-off-by: Tamas Ban <tamas.ban@arm.com>

bl2/src/boot_record.c

diff --git a/bl2/src/boot_record.c b/bl2/src/boot_record.c
index e11059a..b12b9e9 100644
--- a/bl2/src/boot_record.c
+++ b/bl2/src/boot_record.c
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2018, Arm Limited. All rights reserved.
+ * Copyright (c) 2018-2019, Arm Limited. All rights reserved.
  *
  * SPDX-License-Identifier: BSD-3-Clause
  *
@@ -35,7 +35,7 @@
 
 enum shared_memory_err_t
 boot_add_data_to_shared_area(uint8_t        major_type,
-                             uint8_t        minor_type,
+                             uint16_t       minor_type,
                              size_t         size,
                              const uint8_t *data)
 {
@@ -67,16 +67,15 @@
      */
     for(; offset < tlv_end; offset += tlv_entry.tlv_len) {
         tlv_entry = *((struct shared_data_tlv_entry *)offset);
-        if (tlv_entry.tlv_major_type == major_type &&
-            tlv_entry.tlv_minor_type == minor_type ) {
+        if (GET_MAJOR(tlv_entry.tlv_type) == major_type &&
+            GET_MINOR(tlv_entry.tlv_type) == minor_type) {
             return SHARED_MEMORY_OVERWRITE;
         }
     }
 
     /* Add TLV entry */
-    tlv_entry.tlv_major_type = major_type;
-    tlv_entry.tlv_minor_type = minor_type;
-    tlv_entry.tlv_len        = SHARED_DATA_ENTRY_SIZE(size);
+    tlv_entry.tlv_type = SET_TLV_TYPE(major_type, minor_type);
+    tlv_entry.tlv_len  = SHARED_DATA_ENTRY_SIZE(size);
 
     /* Verify overflow of shared area */
     if ((tlv_header->tlv_tot_len + tlv_entry.tlv_len) >