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 <marshalldawson3rd@gmail.com>
Change-Id: I0ff7935004b3d1ac5204d3ef575cfa98116a57fa
Reviewed-on: https://review.coreboot.org/c/coreboot/+/33989
Reviewed-by: Martin Roth <martinroth@google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
This commit is contained in:
Marshall Dawson 2019-06-25 16:06:12 -06:00 committed by Martin Roth
parent 48c5d29cde
commit c17cc63e48
3 changed files with 8 additions and 19 deletions

View File

@ -69,6 +69,7 @@
#define ACPI_GPE0_STS (ACPI_GPE0_BLK + 0x00) /* 4 bytes */ #define ACPI_GPE0_STS (ACPI_GPE0_BLK + 0x00) /* 4 bytes */
#define ACPI_GPE0_EN (ACPI_GPE0_BLK + 0x04) /* 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 ACPI_PM_TMR_BLK (PICASSO_ACPI_IO_BASE + 0x18) /* 4 bytes */
#define NCP_ERR 0xf0
#define SMB_BASE_ADDR 0xb00 #define SMB_BASE_ADDR 0xb00
#define PM2_INDEX 0xcd0 #define PM2_INDEX 0xcd0
#define PM2_DATA 0xcd1 #define PM2_DATA 0xcd1

View File

@ -279,6 +279,9 @@
#define RST_CMD BIT(2) #define RST_CMD BIT(2)
#define SYS_RST BIT(1) #define SYS_RST BIT(1)
/* IO 0xf0 NCP Error */
#define NCP_WARM_BOOT BIT(7) /* Write-once */
struct picasso_aoac { struct picasso_aoac {
int enable; int enable;
int status; int status;

View File

@ -25,32 +25,18 @@
void set_warm_reset_flag(void) void set_warm_reset_flag(void)
{ {
u32 htic; uint8_t ncp = inw(NCP_ERR);
htic = pci_read_config32(SOC_HT_DEV, HT_INIT_CONTROL);
htic |= HTIC_COLD_RST_DET; outb(NCP_ERR, ncp | NCP_WARM_BOOT);
pci_write_config32(SOC_HT_DEV, HT_INIT_CONTROL, htic);
} }
int is_warm_reset(void) int is_warm_reset(void)
{ {
u32 htic; return !!(inb(NCP_ERR) & NCP_WARM_BOOT);
htic = pci_read_config32(SOC_HT_DEV, HT_INIT_CONTROL);
return !!(htic & HTIC_COLD_RST_DET);
}
/* Clear bits 5, 9 & 10, used to signal the reset type */
static void clear_bios_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);
} }
void do_cold_reset(void) void do_cold_reset(void)
{ {
clear_bios_reset();
/* De-assert and then assert all PwrGood signals on CF9 reset. */ /* De-assert and then assert all PwrGood signals on CF9 reset. */
pm_write16(PWR_RESET_CFG, pm_read16(PWR_RESET_CFG) | pm_write16(PWR_RESET_CFG, pm_read16(PWR_RESET_CFG) |
TOGGLE_ALL_PWR_GOOD); TOGGLE_ALL_PWR_GOOD);
@ -60,7 +46,6 @@ void do_cold_reset(void)
void do_warm_reset(void) void do_warm_reset(void)
{ {
set_warm_reset_flag(); set_warm_reset_flag();
clear_bios_reset();
/* Assert reset signals only. */ /* Assert reset signals only. */
outb(RST_CMD | SYS_RST, SYS_RESET); outb(RST_CMD | SYS_RST, SYS_RESET);