Added new GenericFspMboxMessage_t interface to save NVDIMM keys

* Added struct AttributeSetter_t to be sent as a GenericFspMboxMessage_t
  message via a firmware request.
* Created 'sendAttributes(const vector<Attribute>&)' to pack the
  attribute data into the message to be sent.
* Moved AttributeTank::Attribute from the private area to the public
  area so it can be used outside of class AttributeTank.
* Added more documentation on how the process of adding and creating
  messages is done.

Change-Id: Id592b6460a6f1f12a75c01b5a131bcf492480d8b
RTC:208343
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/77951
Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com>
Reviewed-by: Matthew Raybuck <matthew.raybuck@ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>

1 files changed, 23 insertions, 23 deletions

diff --git a/src/include/usr/targeting/common/attributeTank.H b/src/include/usr/targeting/common/attributeTank.H
index a3f9afeab..ccb568889 100644
--- a/src/include/usr/targeting/common/attributeTank.H
+++ b/src/include/usr/targeting/common/attributeTank.H
@@ -5,7 +5,7 @@
 /*                                                                        */
 /* OpenPOWER HostBoot Project                                             */
 /*                                                                        */
-/* Contributors Listed Below - COPYRIGHT 2013,2017                        */
+/* Contributors Listed Below - COPYRIGHT 2013,2019                        */
 /* [+] International Business Machines Corp.                              */
 /*                                                                        */
 /*                                                                        */
@@ -146,6 +146,28 @@ namespace AttributeTank
 #ifndef STANDALONE_COMPILE
 
     /**
+     * @struct Attribute
+     *
+     * This structure defines a single attribute.
+     */
+    struct Attribute
+    {
+        /**
+         * @brief Constructor
+         */
+        Attribute();
+
+        /**
+         * @brief Destructor. Frees memory
+         */
+        ~Attribute();
+
+        // Public data
+        AttributeHeader iv_hdr;
+        uint8_t * iv_pVal;      // Pointer to attribute value
+    };
+
+    /**
      * @struct AttributeSerializedChunk
      *
      * This structure defines a chunk of memory for containing serialized
@@ -397,28 +419,6 @@ private:
     AttributeTank(const AttributeTank & i_right);
     AttributeTank & operator=(const AttributeTank & i_right);
 
-    /**
-     * @struct Attribute
-     *
-     * This structure defines a single attribute.
-     */
-    struct Attribute
-    {
-        /**
-         * @brief Constructor
-         */
-        Attribute();
-
-        /**
-         * @brief Destructor. Frees memory
-         */
-        ~Attribute();
-
-        // Public data
-        AttributeHeader iv_hdr;
-        uint8_t * iv_pVal;      // Pointer to attribute value
-    };
-
     // The attributes
     // Note: A possible performance boost could be to store the elements in a
     //       map, the key could be a sub-structure.