soc/amd/picasso: move chipset_handle_reset to common

The FSP integration code needs this function to be present. It's not
supposed to be called, but if it is, it'll print an error and call the
SoC's cold reset function.

Change-Id: I15f2622d9d9d0f22e3cf8e6283b578f5933b1a9f
Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/48537
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Marshall Dawson <marshalldawson3rd@gmail.com>
This commit is contained in:
Felix Held
2020-12-10 16:49:28 +01:00
parent 2f917e6cee
commit e04a18fc25
4 changed files with 20 additions and 11 deletions

View File

@@ -1,4 +1,5 @@
ifeq ($(CONFIG_SOC_AMD_COMMON),y) ifeq ($(CONFIG_SOC_AMD_COMMON),y)
subdirs-y += block subdirs-y += block
subdirs-y += fsp
subdirs-y += vboot subdirs-y += vboot
endif endif

View File

@@ -0,0 +1,4 @@
ifeq ($(CONFIG_PLATFORM_USES_FSP2_0),y)
romstage-y += fsp_reset.c
ramstage-y += fsp_reset.c
endif # CONFIG_PLATFORM_USES_FSP2_0

View File

@@ -0,0 +1,15 @@
/* SPDX-License-Identifier: GPL-2.0-only */
#include <amdblocks/reset.h>
#include <assert.h>
#include <console/console.h>
#include <fsp/util.h>
#include <stdint.h>
void chipset_handle_reset(uint32_t status)
{
printk(BIOS_ERR, "Error: unexpected call to %s(0x%08x). Doing cold reset.\n",
__func__, status);
BUG();
do_cold_reset();
}

View File

@@ -1,15 +1,12 @@
/* SPDX-License-Identifier: GPL-2.0-only */ /* SPDX-License-Identifier: GPL-2.0-only */
#include <arch/io.h> #include <arch/io.h>
#include <console/console.h>
#include <cf9_reset.h> #include <cf9_reset.h>
#include <reset.h> #include <reset.h>
#include <soc/reset.h> #include <soc/reset.h>
#include <soc/southbridge.h> #include <soc/southbridge.h>
#include <amdblocks/acpimmio.h> #include <amdblocks/acpimmio.h>
#include <amdblocks/reset.h> #include <amdblocks/reset.h>
#include <fsp/util.h>
#include <assert.h>
void set_warm_reset_flag(void) void set_warm_reset_flag(void)
{ {
@@ -44,11 +41,3 @@ void do_board_reset(void)
/* TODO: Would a warm_reset() suffice? */ /* TODO: Would a warm_reset() suffice? */
do_cold_reset(); do_cold_reset();
} }
void chipset_handle_reset(uint32_t status)
{
printk(BIOS_ERR, "Error: unexpected call to %s(0x%08x). Doing cold reset.\n",
__func__, status);
BUG();
do_cold_reset();
}