[coreboot-gerrit] Change in coreboot[master]: amd/pi/00670F00 and 660F01: Add dummy structure to AGESA.h

Marshall Dawson (Code Review) gerrit at coreboot.org
Fri Apr 14 22:54:42 CEST 2017


Marshall Dawson has uploaded a new change for review. ( https://review.coreboot.org/19297 )

Change subject: amd/pi/00670F00 and 660F01: Add dummy structure to AGESA.h
......................................................................

amd/pi/00670F00 and 660F01: Add dummy structure to AGESA.h

Duplicate a structure from Mullins (00730F01) to allow Stoney
and Carrizo to build with common source.

Change-Id: Iba0853556e1d55039b8c45f8dbec2e5958d833fa
Signed-off-by: Marshall Dawson <marshalldawson3rd at gmail.com>
---
M src/vendorcode/amd/pi/00660F01/AGESA.h
M src/vendorcode/amd/pi/00670F00/AGESA.h
2 files changed, 20 insertions(+), 0 deletions(-)


  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/97/19297/1

diff --git a/src/vendorcode/amd/pi/00660F01/AGESA.h b/src/vendorcode/amd/pi/00660F01/AGESA.h
index 5f0e3f9..032ac76 100644
--- a/src/vendorcode/amd/pi/00660F01/AGESA.h
+++ b/src/vendorcode/amd/pi/00660F01/AGESA.h
@@ -3911,5 +3911,15 @@
 
 #define IDS_NUM_EXT_NV_ITEM (AGESA_IDS_EXT_ID_END - AGESA_IDS_EXT_ID_START + 1)
 
+/* Dummy structure to allow building with generic code, not used in CZ or ST. */
+typedef struct {
+     OUT   UINT32 Signature;           ///< "ASTR" for AMD Suspend-To-RAM
+     OUT   UINT16 Version;             ///< S3 Params version number
+  IN OUT   UINT32 Flags;               ///< Indicates operation
+  IN OUT   VOID   *NvStorage;          ///< Pointer to memory critical save state data
+  IN OUT   UINT32 NvStorageSize;       ///< Size in bytes of the NvStorage region
+  IN OUT   VOID   *VolatileStorage;    ///< Pointer to remaining AMD save state data
+  IN OUT   UINT32 VolatileStorageSize; ///< Size in bytes of the VolatileStorage region
+} AMD_S3_PARAMS;
 
 #endif // _AGESA_H_
diff --git a/src/vendorcode/amd/pi/00670F00/AGESA.h b/src/vendorcode/amd/pi/00670F00/AGESA.h
index ee9c629..002c942 100644
--- a/src/vendorcode/amd/pi/00670F00/AGESA.h
+++ b/src/vendorcode/amd/pi/00670F00/AGESA.h
@@ -3958,5 +3958,15 @@
 
 #define IDS_NUM_EXT_NV_ITEM (AGESA_IDS_EXT_ID_END - AGESA_IDS_EXT_ID_START + 1)
 
+/* Dummy structure to allow building with generic code, not used in CZ or ST. */
+typedef struct {
+     OUT   UINT32 Signature;           ///< "ASTR" for AMD Suspend-To-RAM
+     OUT   UINT16 Version;             ///< S3 Params version number
+  IN OUT   UINT32 Flags;               ///< Indicates operation
+  IN OUT   VOID   *NvStorage;          ///< Pointer to memory critical save state data
+  IN OUT   UINT32 NvStorageSize;       ///< Size in bytes of the NvStorage region
+  IN OUT   VOID   *VolatileStorage;    ///< Pointer to remaining AMD save state data
+  IN OUT   UINT32 VolatileStorageSize; ///< Size in bytes of the VolatileStorage region
+} AMD_S3_PARAMS;
 
 #endif // _AGESA_H_

-- 
To view, visit https://review.coreboot.org/19297
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Iba0853556e1d55039b8c45f8dbec2e5958d833fa
Gerrit-PatchSet: 1
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Owner: Marshall Dawson <marshalldawson3rd at gmail.com>



More information about the coreboot-gerrit mailing list