From c17cc63e481af7f981abbb9e2383b8447a2224d3 Mon Sep 17 00:00:00 2001 From: Marshall Dawson Date: Tue, 25 Jun 2019 16:06:12 -0600 Subject: src/amd/picasso: Update reset code Remove the scratch register indicators. Per AMD, AGESA no longer uses these. Use a new IO register to determine whether a warm reset should occur. Signed-off-by: Marshall Dawson Change-Id: I0ff7935004b3d1ac5204d3ef575cfa98116a57fa Reviewed-on: https://review.coreboot.org/c/coreboot/+/33989 Reviewed-by: Martin Roth Tested-by: build bot (Jenkins) --- src/soc/amd/picasso/include/soc/iomap.h | 1 + src/soc/amd/picasso/include/soc/southbridge.h | 3 +++ src/soc/amd/picasso/reset.c | 23 ++++------------------- 3 files changed, 8 insertions(+), 19 deletions(-) (limited to 'src/soc/amd') diff --git a/src/soc/amd/picasso/include/soc/iomap.h b/src/soc/amd/picasso/include/soc/iomap.h index 1d89fd7ec0..2870103e93 100644 --- a/src/soc/amd/picasso/include/soc/iomap.h +++ b/src/soc/amd/picasso/include/soc/iomap.h @@ -69,6 +69,7 @@ #define ACPI_GPE0_STS (ACPI_GPE0_BLK + 0x00) /* 4 bytes */ #define ACPI_GPE0_EN (ACPI_GPE0_BLK + 0x04) /* 4 bytes */ #define ACPI_PM_TMR_BLK (PICASSO_ACPI_IO_BASE + 0x18) /* 4 bytes */ +#define NCP_ERR 0xf0 #define SMB_BASE_ADDR 0xb00 #define PM2_INDEX 0xcd0 #define PM2_DATA 0xcd1 diff --git a/src/soc/amd/picasso/include/soc/southbridge.h b/src/soc/amd/picasso/include/soc/southbridge.h index 6fc37f009a..36880a14a3 100644 --- a/src/soc/amd/picasso/include/soc/southbridge.h +++ b/src/soc/amd/picasso/include/soc/southbridge.h @@ -279,6 +279,9 @@ #define RST_CMD BIT(2) #define SYS_RST BIT(1) +/* IO 0xf0 NCP Error */ +#define NCP_WARM_BOOT BIT(7) /* Write-once */ + struct picasso_aoac { int enable; int status; diff --git a/src/soc/amd/picasso/reset.c b/src/soc/amd/picasso/reset.c index ec5ee910d9..98410387fb 100644 --- a/src/soc/amd/picasso/reset.c +++ b/src/soc/amd/picasso/reset.c @@ -25,32 +25,18 @@ void set_warm_reset_flag(void) { - u32 htic; - htic = pci_read_config32(SOC_HT_DEV, HT_INIT_CONTROL); - htic |= HTIC_COLD_RST_DET; - pci_write_config32(SOC_HT_DEV, HT_INIT_CONTROL, htic); -} + uint8_t ncp = inw(NCP_ERR); -int is_warm_reset(void) -{ - u32 htic; - htic = pci_read_config32(SOC_HT_DEV, HT_INIT_CONTROL); - return !!(htic & HTIC_COLD_RST_DET); + outb(NCP_ERR, ncp | NCP_WARM_BOOT); } -/* Clear bits 5, 9 & 10, used to signal the reset type */ -static void clear_bios_reset(void) +int is_warm_reset(void) { - u32 htic; - htic = pci_read_config32(SOC_HT_DEV, HT_INIT_CONTROL); - htic &= ~HTIC_BIOSR_DETECT; - pci_write_config32(SOC_HT_DEV, HT_INIT_CONTROL, htic); + return !!(inb(NCP_ERR) & NCP_WARM_BOOT); } void do_cold_reset(void) { - clear_bios_reset(); - /* De-assert and then assert all PwrGood signals on CF9 reset. */ pm_write16(PWR_RESET_CFG, pm_read16(PWR_RESET_CFG) | TOGGLE_ALL_PWR_GOOD); @@ -60,7 +46,6 @@ void do_cold_reset(void) void do_warm_reset(void) { set_warm_reset_flag(); - clear_bios_reset(); /* Assert reset signals only. */ outb(RST_CMD | SYS_RST, SYS_RESET); -- cgit v1.2.3