diff options
| author | Richard Spiegel <richard.spiegel@amd.corp-partner.google.com> | 2018-08-23 13:46:34 -0700 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2018-08-27 15:59:12 +0000 |
| commit | 4c7349cf85e744a60977f11104b9be4d43c46bc7 (patch) | |
| tree | 6a31df2cb623dd0040c825f59e6dc1484de4007e /src/vendorcode | |
| parent | 837e3387e5b6d4ea5823775087d23f9fb9e41f5f (diff) | |
vendorcode/amd/pi/00670F00/Lib/AmdLib.c: Remove IdsErrorStop
Function IdsErrorStop() is only used within AmdLib.c function
LibAmdMsrRead(), which in turn is only used once within PspBaseLib.c and
three times inside AmdLib.c, all with well defined MSR addresses.
IdsErrorStop() is used as a trap if MSR address is 0 or 0xFFFFFFFF, which
clearly it's not. Therefore it can be safely removed from AmdLib.c.
BUG=b:112885948
TEST=Build grunt
Change-Id: I47ffcbd4fbae28b6d711a340f0ac3f3b007e8e4f
Signed-off-by: Richard Spiegel <richard.spiegel@silverbackltd.com>
Reviewed-on: https://review.coreboot.org/28295
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Marshall Dawson <marshalldawson3rd@gmail.com>
Diffstat (limited to 'src/vendorcode')
| -rw-r--r-- | src/vendorcode/amd/pi/00670F00/Lib/amdlib.c | 27 |
1 files changed, 0 insertions, 27 deletions
diff --git a/src/vendorcode/amd/pi/00670F00/Lib/amdlib.c b/src/vendorcode/amd/pi/00670F00/Lib/amdlib.c index b03beee287..8a4113f9b2 100644 --- a/src/vendorcode/amd/pi/00670F00/Lib/amdlib.c +++ b/src/vendorcode/amd/pi/00670F00/Lib/amdlib.c @@ -266,9 +266,6 @@ LibAmdMsrRead ( IN OUT AMD_CONFIG_PARAMS *ConfigPtr ) { - if ((MsrAddress == 0xFFFFFFFF) || (MsrAddress == 0x00000000)) { - IdsErrorStop(MsrAddress); - } *Value = __readmsr (MsrAddress); } @@ -574,27 +571,3 @@ GetPciMmioAddress ( } return MmioIsEnabled; } - -BOOLEAN -IdsErrorStop ( - IN UINT32 FileCode - ) -{ - struct POST { - UINT16 deadlo; - UINT32 messagelo; - UINT16 deadhi; - UINT32 messagehi; - } post = {0xDEAD, FileCode, 0xDEAD, FileCode}; - UINT16 offset = 0; - UINT16 j; - - while(1) { - offset %= sizeof(struct POST) / 2; - WriteIo16(80, *((UINT16 *)&post)+offset); - ++offset; - for (j=0; j<250; ++j) { - ReadIo8(80); - } - } -} |