Felix Held has uploaded this change for review.

View Change

soc/amd/common/block/acpimmio/print_reset_status: extend bit name table

Bit 23 in the PM_RST_STATUS register is called LtReset on Stoneyridge
and ShutdownMsg on Picasso/Cezanne/Sabrina. Bit 30 is reserved on
Stoneyridge and defined as SdpParityErr on the newer SoCs. Bit 31 is
only defined for Sabrina. Since the default value of undefined bits is 0
it isn't a problem to have descriptions for reserved reset status bits
on some SoCs.

Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Change-Id: I0782116d327fcad3817a10eb237ac6c8294846b3
---
M src/soc/amd/common/block/acpimmio/print_reset_status.c
1 file changed, 3 insertions(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/24/61624/1
diff --git a/src/soc/amd/common/block/acpimmio/print_reset_status.c b/src/soc/amd/common/block/acpimmio/print_reset_status.c
index 309401e..5ff0412 100644
--- a/src/soc/amd/common/block/acpimmio/print_reset_status.c
+++ b/src/soc/amd/common/block/acpimmio/print_reset_status.c
@@ -41,13 +41,15 @@
[20] = "DoFullReset",
[21] = "SleepReset",
[22] = "KbReset",
- [23] = "LtReset",
+ [23] = "LtReset/ShutdownMsg",
[24] = "FailBootRst",
[25] = "WatchdogIssueReset",
[26] = "RemoteResetFromASF",
[27] = "SyncFlood",
[28] = "HangReset",
[29] = "EcWatchdogRst",
+ [30] = "SdpParityErr",
+ [31] = "SwSyncFloodFlag",
};

printk(BIOS_DEBUG, "PMxC0 STATUS: 0x%x ", pmxc0_status);

To view, visit change 61624. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I0782116d327fcad3817a10eb237ac6c8294846b3
Gerrit-Change-Number: 61624
Gerrit-PatchSet: 1
Gerrit-Owner: Felix Held <felix-coreboot@felixheld.de>
Gerrit-MessageType: newchange