diff options
| author | Edward Hill <ecgh@chromium.org> | 2018-10-02 14:17:19 -0600 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2018-10-04 09:43:24 +0000 |
| commit | 917b400bc0d16a41af757e5ddf7ea76e29a00e3d (patch) | |
| tree | c484c10116309057b19099ea8db3d262823b79c7 /src/soc/amd/stoneyridge/southbridge.c | |
| parent | aeb7f0511bdfad83978e6c54ef9353a62a79f428 (diff) | |
amd/stoneyridge: Use BIOS_DEBUG to log PM1 and PMxC0 status
Use BIOS_DEBUG consistently to log PM1 and PMxC0 status registers
on boot. print_num_status_bits() was already using BIOS_DEBUG.
TEST=Inspect console for Grunt
BUG=b:110788201
Change-Id: If7da8c7c86e90a661338903ad05cc41e11f507d2
Signed-off-by: Edward Hill <ecgh@chromium.org>
Reviewed-on: https://review.coreboot.org/28885
Reviewed-by: Raul Rangel <rrangel@chromium.org>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/soc/amd/stoneyridge/southbridge.c')
| -rw-r--r-- | src/soc/amd/stoneyridge/southbridge.c | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/src/soc/amd/stoneyridge/southbridge.c b/src/soc/amd/stoneyridge/southbridge.c index b122f8311c..763ddd7481 100644 --- a/src/soc/amd/stoneyridge/southbridge.c +++ b/src/soc/amd/stoneyridge/southbridge.c @@ -667,7 +667,7 @@ static void sb_print_pmxc0_status(void) /* PMxC0 S5/Reset Status shows the source of previous reset. */ uint32_t pmxc0_status = pm_read32(PM_RST_STATUS); - static const char *const pmxc0_status_bits[] = { + static const char *const pmxc0_status_bits[32] = { [0] = "ThermalTrip", [1] = "FourSecondPwrBtn", [2] = "Shutdown", @@ -688,13 +688,12 @@ static void sb_print_pmxc0_status(void) [27] = "SyncFlood", [28] = "HangReset", [29] = "EcWatchdogRst", - [31] = "BIT31", }; - printk(BIOS_SPEW, "PMxC0 STATUS: 0x%x ", pmxc0_status); + printk(BIOS_DEBUG, "PMxC0 STATUS: 0x%x ", pmxc0_status); print_num_status_bits(ARRAY_SIZE(pmxc0_status_bits), pmxc0_status, pmxc0_status_bits); - printk(BIOS_SPEW, "\n"); + printk(BIOS_DEBUG, "\n"); } /* After console init */ @@ -763,7 +762,7 @@ static uint16_t reset_pm1_status(void) static uint16_t print_pm1_status(uint16_t pm1_sts) { - static const char *const pm1_sts_bits[] = { + static const char *const pm1_sts_bits[16] = { [0] = "TMROF", [4] = "BMSTATUS", [5] = "GBL", @@ -776,9 +775,9 @@ static uint16_t print_pm1_status(uint16_t pm1_sts) if (!pm1_sts) return 0; - printk(BIOS_SPEW, "PM1_STS: "); + printk(BIOS_DEBUG, "PM1_STS: "); print_num_status_bits(ARRAY_SIZE(pm1_sts_bits), pm1_sts, pm1_sts_bits); - printk(BIOS_SPEW, "\n"); + printk(BIOS_DEBUG, "\n"); return pm1_sts; } |