[coreboot-gerrit] Change in coreboot[master]: soc/amd/common/block/acpimmio/print_reset_status: extend bit name table

[Felix Held (Code Review)]

Felix Held has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/61624 )

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

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 
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);