[coreboot-gerrit] Change in coreboot[master]: vendorcode/amd/cimx/sb*: Rename RSDP header

Marc Jones (Code Review) gerrit at coreboot.org
Thu Aug 23 18:36:38 CEST 2018


Marc Jones has uploaded this change for review. ( https://review.coreboot.org/28294


Change subject: vendorcode/amd/cimx/sb*: Rename RSDP header
......................................................................

vendorcode/amd/cimx/sb*: Rename RSDP header

Rename RSDP to RSDP_HEADER to match other AMD vendorcode and
to not pollute the namespace. We will use RSDP in a future patch.

Change-Id: I3b66135ae1732b86b5ebfcdc01a850a0d9d3eb50
Signed-off-by: Marc Jones <marcj303 at gmail.com>
---
M src/vendorcode/amd/cimx/sb800/ACPILIB.c
M src/vendorcode/amd/cimx/sb800/ACPILIB.h
M src/vendorcode/amd/cimx/sb900/AcpiLib.c
M src/vendorcode/amd/cimx/sb900/AcpiLib.h
4 files changed, 4 insertions(+), 4 deletions(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/94/28294/1

diff --git a/src/vendorcode/amd/cimx/sb800/ACPILIB.c b/src/vendorcode/amd/cimx/sb800/ACPILIB.c
index bc11209..e112fe8 100644
--- a/src/vendorcode/amd/cimx/sb800/ACPILIB.c
+++ b/src/vendorcode/amd/cimx/sb800/ACPILIB.c
@@ -63,7 +63,7 @@
   Rsdt = NULL;
   do {
     if ( *RsdPtr == Int32FromChar('R', 'S', 'D', ' ') && *(RsdPtr + 1) == Int32FromChar('P', 'T', 'R', ' ')) {
-      Rsdt = (UINT32*) (UINTN) ((RSDP*)RsdPtr)->RsdtAddress;
+      Rsdt = (UINT32*) (UINTN) ((RSDP_HEADER*)RsdPtr)->RsdtAddress;
       break;
     }
     RsdPtr += 4;
diff --git a/src/vendorcode/amd/cimx/sb800/ACPILIB.h b/src/vendorcode/amd/cimx/sb800/ACPILIB.h
index 73571f4..442043a 100644
--- a/src/vendorcode/amd/cimx/sb800/ACPILIB.h
+++ b/src/vendorcode/amd/cimx/sb800/ACPILIB.h
@@ -44,7 +44,7 @@
   unsigned long long  XsdtAddress;      /* physical address of XSDT */
   unsigned char  ExtendedChecksum;  /* chechsum of whole table */
   unsigned char  Reserved[3];
-} RSDP;
+} RSDP_HEADER;
 
 
 /**
diff --git a/src/vendorcode/amd/cimx/sb900/AcpiLib.c b/src/vendorcode/amd/cimx/sb900/AcpiLib.c
index 2b8a389..845b34d 100644
--- a/src/vendorcode/amd/cimx/sb900/AcpiLib.c
+++ b/src/vendorcode/amd/cimx/sb900/AcpiLib.c
@@ -61,7 +61,7 @@
 //    if ( *RsdPtr == ' DSR' && *(RsdPtr + 1) == ' RTP' ) {
 //    if ( (*RsdPtr == 0x52534420) && (*(RsdPtr + 1) == 0x50545220) ) {
     if ( (*RsdPtr == 0x20445352) && (*(RsdPtr + 1) == 0x20525450) ) {
-      Rsdt = (UINT32*) (UINTN) ((RSDP*)RsdPtr)->RsdtAddress;
+      Rsdt = (UINT32*) (UINTN) ((RSDP_HEADER*)RsdPtr)->RsdtAddress;
       break;
     }
     RsdPtr += 4;
diff --git a/src/vendorcode/amd/cimx/sb900/AcpiLib.h b/src/vendorcode/amd/cimx/sb900/AcpiLib.h
index 63913c2..c604023 100644
--- a/src/vendorcode/amd/cimx/sb900/AcpiLib.h
+++ b/src/vendorcode/amd/cimx/sb900/AcpiLib.h
@@ -40,7 +40,7 @@
   unsigned long long  XsdtAddress;      ///< physical address of XSDT
   unsigned char  ExtendedChecksum;  ///< chechsum of whole table
   unsigned char  Reserved[3];       ///< Reserved
-} RSDP;
+} RSDP_HEADER;
 
 
 /// DESCRIPTION_HEADER - ACPI common table header

-- 
To view, visit https://review.coreboot.org/28294
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I3b66135ae1732b86b5ebfcdc01a850a0d9d3eb50
Gerrit-Change-Number: 28294
Gerrit-PatchSet: 1
Gerrit-Owner: Marc Jones <marc at marcjonesconsulting.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20180823/40a37c0f/attachment.html>


More information about the coreboot-gerrit mailing list