summaryrefslogtreecommitdiff
path: root/src/vendorcode/amd/agesa/f15tn/Lib
diff options
context:
space:
mode:
authorBruce Griffith <Bruce.Griffith@se-eng.com>2013-06-04 14:16:24 -0600
committerStefan Reinauer <stefan.reinauer@coreboot.org>2013-06-06 20:59:43 +0200
commit283ba7841581c5ffe64d1f0985cfa94be661b927 (patch)
tree4f4d60445daca61bc42febf488fa979f88a5f1a5 /src/vendorcode/amd/agesa/f15tn/Lib
parentaa588e027301063731b80e3e22a4ea3b1c0c08f1 (diff)
AGESA: Add "const" modifier to function parameters
Add CONST modifiers to read-only pass-by-reference function parameters in AGESA. This allows the use of "const" modifiers on the declaration of lookup tables that are pass-by-reference. These will be used to identify tables that are copied onto the HEAP but don't need to be. Change-Id: Ie1187a427804fddf47b935a110ad23931a3447a9 Signed-off-by: Bruce Griffith <Bruce.Griffith@se-eng.com> Reviewed-on: http://review.coreboot.org/3393 Tested-by: build bot (Jenkins) Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
Diffstat (limited to 'src/vendorcode/amd/agesa/f15tn/Lib')
-rw-r--r--src/vendorcode/amd/agesa/f15tn/Lib/amdlib.c60
-rw-r--r--src/vendorcode/amd/agesa/f15tn/Lib/amdlib.h44
2 files changed, 52 insertions, 52 deletions
diff --git a/src/vendorcode/amd/agesa/f15tn/Lib/amdlib.c b/src/vendorcode/amd/agesa/f15tn/Lib/amdlib.c
index 81ff635086..d468a0a803 100644
--- a/src/vendorcode/amd/agesa/f15tn/Lib/amdlib.c
+++ b/src/vendorcode/amd/agesa/f15tn/Lib/amdlib.c
@@ -111,8 +111,8 @@ VOID
STATIC
LibAmdGetDataFromPtr (
IN ACCESS_WIDTH AccessWidth,
- IN VOID *Data,
- IN VOID *DataMask,
+ IN CONST VOID *Data,
+ IN CONST VOID *DataMask,
OUT UINT32 *TemData,
OUT UINT32 *TempDataMask
);
@@ -549,7 +549,7 @@ VOID
LibAmdIoWrite (
IN ACCESS_WIDTH AccessWidth,
IN UINT16 IoAddress,
- IN VOID *Value,
+ IN CONST VOID *Value,
IN OUT AMD_CONFIG_PARAMS *StdHeader
)
{
@@ -587,8 +587,8 @@ VOID
LibAmdIoRMW (
IN ACCESS_WIDTH AccessWidth,
IN UINT16 IoAddress,
- IN VOID *Data,
- IN VOID *DataMask,
+ IN CONST VOID *Data,
+ IN CONST VOID *DataMask,
IN OUT AMD_CONFIG_PARAMS *StdHeader
)
{
@@ -619,8 +619,8 @@ VOID
LibAmdIoPoll (
IN ACCESS_WIDTH AccessWidth,
IN UINT16 IoAddress,
- IN VOID *Data,
- IN VOID *DataMask,
+ IN CONST VOID *Data,
+ IN CONST VOID *DataMask,
IN UINT64 Delay,
IN OUT AMD_CONFIG_PARAMS *StdHeader
)
@@ -686,7 +686,7 @@ VOID
LibAmdMemWrite (
IN ACCESS_WIDTH AccessWidth,
IN UINT64 MemAddress,
- IN VOID *Value,
+ IN CONST VOID *Value,
IN OUT AMD_CONFIG_PARAMS *StdHeader
)
{
@@ -724,8 +724,8 @@ VOID
LibAmdMemRMW (
IN ACCESS_WIDTH AccessWidth,
IN UINT64 MemAddress,
- IN VOID *Data,
- IN VOID *DataMask,
+ IN CONST VOID *Data,
+ IN CONST VOID *DataMask,
IN OUT AMD_CONFIG_PARAMS *StdHeader
)
{
@@ -756,8 +756,8 @@ VOID
LibAmdMemPoll (
IN ACCESS_WIDTH AccessWidth,
IN UINT64 MemAddress,
- IN VOID *Data,
- IN VOID *DataMask,
+ IN CONST VOID *Data,
+ IN CONST VOID *DataMask,
IN UINT64 Delay,
IN OUT AMD_CONFIG_PARAMS *StdHeader
)
@@ -837,7 +837,7 @@ VOID
LibAmdPciWrite (
IN ACCESS_WIDTH AccessWidth,
IN PCI_ADDR PciAddress,
- IN VOID *Value,
+ IN CONST VOID *Value,
IN OUT AMD_CONFIG_PARAMS *StdHeader
)
{
@@ -893,8 +893,8 @@ VOID
LibAmdPciRMW (
IN ACCESS_WIDTH AccessWidth,
IN PCI_ADDR PciAddress,
- IN VOID *Data,
- IN VOID *DataMask,
+ IN CONST VOID *Data,
+ IN CONST VOID *DataMask,
IN OUT AMD_CONFIG_PARAMS *StdHeader
)
{
@@ -925,8 +925,8 @@ VOID
LibAmdPciPoll (
IN ACCESS_WIDTH AccessWidth,
IN PCI_ADDR PciAddress,
- IN VOID *Data,
- IN VOID *DataMask,
+ IN CONST VOID *Data,
+ IN CONST VOID *DataMask,
IN UINT64 Delay,
IN OUT AMD_CONFIG_PARAMS *StdHeader
)
@@ -1025,7 +1025,7 @@ LibAmdPciWriteBits (
IN PCI_ADDR Address,
IN UINT8 Highbit,
IN UINT8 Lowbit,
- IN UINT32 *Value,
+ IN CONST UINT32 *Value,
IN AMD_CONFIG_PARAMS *StdHeader
)
{
@@ -1161,13 +1161,13 @@ LibAmdMemFill (
VOID
LibAmdMemCopy (
IN VOID *Destination,
- IN VOID *Source,
+ IN CONST VOID *Source,
IN UINTN CopyLength,
IN OUT AMD_CONFIG_PARAMS *StdHeader
)
{
UINT8 *Dest;
- UINT8 *SourcePtr;
+ CONST UINT8 *SourcePtr;
ASSERT (StdHeader != NULL);
Dest = Destination;
SourcePtr = Source;
@@ -1187,7 +1187,7 @@ LibAmdMemCopy (
*/
BOOLEAN
LibAmdVerifyImageChecksum (
- IN VOID *ImagePtr
+ IN CONST VOID *ImagePtr
)
{
// Assume ImagePtr points to the binary start ($AMD)
@@ -1224,19 +1224,19 @@ LibAmdVerifyImageChecksum (
* @retval NULL if image not found
* @retval pointer to image header
*/
-VOID *
+CONST VOID *
LibAmdLocateImage (
- IN VOID *StartAddress,
- IN VOID *EndAddress,
+ IN CONST VOID *StartAddress,
+ IN CONST VOID *EndAddress,
IN UINT32 Alignment,
- IN CHAR8 ModuleSignature[8]
+ IN CONST CHAR8 ModuleSignature[8]
)
{
- UINT8 *CurrentPtr;
+ CONST UINT8 *CurrentPtr;
AMD_MODULE_HEADER *ModuleHeaderPtr;
- UINT64 *SearchStr;
- UINT64 *InputStr;
+ CONST UINT64 *SearchStr;
+ CONST UINT64 *InputStr;
CurrentPtr = StartAddress;
InputStr = (UINT64 *)ModuleSignature;
@@ -1300,8 +1300,8 @@ VOID
STATIC
LibAmdGetDataFromPtr (
IN ACCESS_WIDTH AccessWidth,
- IN VOID *Data,
- IN VOID *DataMask,
+ IN CONST VOID *Data,
+ IN CONST VOID *DataMask,
OUT UINT32 *TemData,
OUT UINT32 *TempDataMask
)
diff --git a/src/vendorcode/amd/agesa/f15tn/Lib/amdlib.h b/src/vendorcode/amd/agesa/f15tn/Lib/amdlib.h
index d95d22ec8d..10ec9899a6 100644
--- a/src/vendorcode/amd/agesa/f15tn/Lib/amdlib.h
+++ b/src/vendorcode/amd/agesa/f15tn/Lib/amdlib.h
@@ -182,7 +182,7 @@ VOID
LibAmdIoWrite (
IN ACCESS_WIDTH AccessWidth,
IN UINT16 IoAddress,
- IN VOID *Value,
+ IN CONST VOID *Value,
IN OUT AMD_CONFIG_PARAMS *StdHeader
);
@@ -190,8 +190,8 @@ VOID
LibAmdIoRMW (
IN ACCESS_WIDTH AccessWidth,
IN UINT16 IoAddress,
- IN VOID *Data,
- IN VOID *DataMask,
+ IN CONST VOID *Data,
+ IN CONST VOID *DataMask,
IN OUT AMD_CONFIG_PARAMS *StdHeader
);
@@ -199,8 +199,8 @@ VOID
LibAmdIoPoll (
IN ACCESS_WIDTH AccessWidth,
IN UINT16 IoAddress,
- IN VOID *Data,
- IN VOID *DataMask,
+ IN CONST VOID *Data,
+ IN CONST VOID *DataMask,
IN UINT64 Delay,
IN OUT AMD_CONFIG_PARAMS *StdHeader
);
@@ -218,7 +218,7 @@ VOID
LibAmdMemWrite (
IN ACCESS_WIDTH AccessWidth,
IN UINT64 MemAddress,
- IN VOID *Value,
+ IN CONST VOID *Value,
IN OUT AMD_CONFIG_PARAMS *StdHeader
);
@@ -226,8 +226,8 @@ VOID
LibAmdMemRMW (
IN ACCESS_WIDTH AccessWidth,
IN UINT64 MemAddress,
- IN VOID *Data,
- IN VOID *DataMask,
+ IN CONST VOID *Data,
+ IN CONST VOID *DataMask,
IN OUT AMD_CONFIG_PARAMS *StdHeader
);
@@ -235,8 +235,8 @@ VOID
LibAmdMemPoll (
IN ACCESS_WIDTH AccessWidth,
IN UINT64 MemAddress,
- IN VOID *Data,
- IN VOID *DataMask,
+ IN CONST VOID *Data,
+ IN CONST VOID *DataMask,
IN UINT64 Delay,
IN OUT AMD_CONFIG_PARAMS *StdHeader
);
@@ -254,7 +254,7 @@ VOID
LibAmdPciWrite (
IN ACCESS_WIDTH AccessWidth,
IN PCI_ADDR PciAddress,
- IN VOID *Value,
+ IN CONST VOID *Value,
IN OUT AMD_CONFIG_PARAMS *StdHeader
);
@@ -262,8 +262,8 @@ VOID
LibAmdPciRMW (
IN ACCESS_WIDTH AccessWidth,
IN PCI_ADDR PciAddress,
- IN VOID *Data,
- IN VOID *DataMask,
+ IN CONST VOID *Data,
+ IN CONST VOID *DataMask,
IN OUT AMD_CONFIG_PARAMS *StdHeader
);
@@ -271,8 +271,8 @@ VOID
LibAmdPciPoll (
IN ACCESS_WIDTH AccessWidth,
IN PCI_ADDR PciAddress,
- IN VOID *Data,
- IN VOID *DataMask,
+ IN CONST VOID *Data,
+ IN CONST VOID *DataMask,
IN UINT64 Delay,
IN OUT AMD_CONFIG_PARAMS *StdHeader
);
@@ -291,7 +291,7 @@ LibAmdPciWriteBits (
IN PCI_ADDR Address,
IN UINT8 Highbit,
IN UINT8 Lowbit,
- IN UINT32 *Value,
+ IN CONST UINT32 *Value,
IN AMD_CONFIG_PARAMS *StdHeader
);
@@ -321,17 +321,17 @@ LibAmdMemFill (
VOID
LibAmdMemCopy (
IN VOID *Destination,
- IN VOID *Source,
+ IN CONST VOID *Source,
IN UINTN CopyLength,
IN OUT AMD_CONFIG_PARAMS *StdHeader
);
-VOID *
+CONST VOID *
LibAmdLocateImage (
- IN VOID *StartAddress,
- IN VOID *EndAddress,
+ IN CONST VOID *StartAddress,
+ IN CONST VOID *EndAddress,
IN UINT32 Alignment,
- IN CHAR8 ModuleSignature[8]
+ IN CONST CHAR8 ModuleSignature[8]
);
UINT32
@@ -341,7 +341,7 @@ LibAmdGetPackageType (
BOOLEAN
LibAmdVerifyImageChecksum (
- IN VOID *ImagePtr
+ IN CONST VOID *ImagePtr
);
UINT8