From 718c6faff4acf7a7716a1747fbfecdac462ef2c9 Mon Sep 17 00:00:00 2001 From: Nico Huber Date: Thu, 11 Oct 2018 22:54:25 +0200 Subject: reset: Finalize move to new API Move soft_reset() to `southbridge/amd/common/` it's only used for amdfam10 now. Drop hard_reset() for good. Change-Id: Ifdc5791160653c5578007f6c1b96015efe2b3e1e Signed-off-by: Nico Huber Reviewed-on: https://review.coreboot.org/29059 Tested-by: build bot (Jenkins) Reviewed-by: Aaron Durbin --- src/Kconfig | 7 ----- src/cpu/amd/family_10h-family_15h/init_cpus.c | 2 ++ src/include/reset.h | 10 ------- src/lib/reset.c | 35 ------------------------- src/mainboard/advansus/a785e-i/romstage.c | 2 +- src/mainboard/amd/bimini_fam10/romstage.c | 1 + src/mainboard/amd/mahogany_fam10/romstage.c | 1 + src/mainboard/amd/tilapia_fam10/romstage.c | 1 + src/mainboard/asus/kcma-d8/romstage.c | 2 +- src/mainboard/asus/kfsn4-dre/romstage.c | 2 +- src/mainboard/asus/kgpe-d16/romstage.c | 2 +- src/mainboard/asus/m4a78-em/romstage.c | 1 + src/mainboard/asus/m4a785-m/romstage.c | 1 + src/mainboard/asus/m5a88-v/romstage.c | 2 +- src/mainboard/avalue/eax-785e/romstage.c | 2 +- src/mainboard/gigabyte/ma785gm/romstage.c | 1 + src/mainboard/gigabyte/ma785gmt/romstage.c | 1 + src/mainboard/gigabyte/ma78gm/romstage.c | 1 + src/mainboard/hp/dl165_g6_fam10/romstage.c | 1 + src/mainboard/iei/kino-780am2-fam10/romstage.c | 1 + src/mainboard/jetway/pa78vm5/romstage.c | 1 + src/mainboard/msi/ms9652_fam10/romstage.c | 1 + src/mainboard/supermicro/h8dmr_fam10/romstage.c | 1 + src/mainboard/supermicro/h8qme_fam10/romstage.c | 1 + src/mainboard/supermicro/h8scm_fam10/romstage.c | 1 + src/mainboard/tyan/s2912_fam10/romstage.c | 1 + src/northbridge/amd/amdmct/mct_ddr3/mct_d.c | 2 +- src/security/vboot/Kconfig | 2 +- src/southbridge/amd/amd8111/early_ctrl.c | 1 + src/southbridge/amd/common/reset.h | 35 +++++++++++++++++++++++++ src/southbridge/amd/sb700/reset.c | 1 + src/southbridge/amd/sb800/early_setup.c | 1 + src/southbridge/amd/sr5650/early_setup.c | 2 +- src/southbridge/broadcom/bcm5785/early_setup.c | 1 + src/southbridge/nvidia/ck804/early_setup.c | 1 + src/southbridge/nvidia/ck804/early_setup_car.c | 1 + src/southbridge/nvidia/mcp55/early_ctrl.c | 1 + 37 files changed, 69 insertions(+), 61 deletions(-) create mode 100644 src/southbridge/amd/common/reset.h (limited to 'src') diff --git a/src/Kconfig b/src/Kconfig index 2777d266d8..0b97125c45 100644 --- a/src/Kconfig +++ b/src/Kconfig @@ -449,13 +449,6 @@ config RESUME_PATH_SAME_AS_BOOT same path as a regular boot. e.g. an x86 system runs from the reset vector at 0xfffffff0 on both resume and warm/cold boot. -config HAVE_HARD_RESET - bool - default n - help - This variable specifies whether a given board has a hard_reset - function, no matter if it's provided by board code or chipset code. - config HAVE_ROMSTAGE_CONSOLE_SPINLOCK bool depends on EARLY_CBMEM_INIT diff --git a/src/cpu/amd/family_10h-family_15h/init_cpus.c b/src/cpu/amd/family_10h-family_15h/init_cpus.c index 1247e60283..2ddbc43ee6 100644 --- a/src/cpu/amd/family_10h-family_15h/init_cpus.c +++ b/src/cpu/amd/family_10h-family_15h/init_cpus.c @@ -27,6 +27,8 @@ #include #include +#include + #if IS_ENABLED(CONFIG_SOUTHBRIDGE_AMD_SB700) #include #endif diff --git a/src/include/reset.h b/src/include/reset.h index 3eec193c84..5b5e1d30cc 100644 --- a/src/include/reset.h +++ b/src/include/reset.h @@ -39,14 +39,4 @@ __noreturn void board_reset(void); */ void do_board_reset(void); -/* Full board reset. Resets SoC and most/all board components (e.g. DRAM). */ -__noreturn void hard_reset(void); -/* Board reset. Resets SoC some board components (e.g. TPM but not DRAM). */ -__noreturn void soft_reset(void); - -/* Reset implementations. Implement these in SoC or mainboard code. Implement - at least hard_reset() if possible, others fall back to it if necessary. */ -void do_hard_reset(void); -void do_soft_reset(void); - #endif diff --git a/src/lib/reset.c b/src/lib/reset.c index 4b19e1e7e6..904776e91b 100644 --- a/src/lib/reset.c +++ b/src/lib/reset.c @@ -31,39 +31,4 @@ void do_board_reset(void) { printk(BIOS_CRIT, "No board_reset implementation, hanging...\n"); } -#else -/* - * Fall back to hard_reset() for a regression free transition. - * FIXME: Remove after everything is converted to board_reset(). - */ -__weak void do_board_reset(void) -{ - hard_reset(); -} #endif - -__noreturn static void __hard_reset(void) { - if (IS_ENABLED(CONFIG_HAVE_HARD_RESET)) - do_hard_reset(); - else - printk(BIOS_CRIT, "No hard_reset implementation, hanging...\n"); - halt(); -} - -/* Not all platforms implement all reset types. Fall back to hard_reset. */ -__weak void do_soft_reset(void) { __hard_reset(); } - -void hard_reset(void) -{ - printk(BIOS_INFO, "%s() called!\n", __func__); - dcache_clean_all(); - __hard_reset(); -} - -void soft_reset(void) -{ - printk(BIOS_INFO, "%s() called!\n", __func__); - dcache_clean_all(); - do_soft_reset(); - halt(); -} diff --git a/src/mainboard/advansus/a785e-i/romstage.c b/src/mainboard/advansus/a785e-i/romstage.c index 2a0a1d98a3..281a59aa99 100644 --- a/src/mainboard/advansus/a785e-i/romstage.c +++ b/src/mainboard/advansus/a785e-i/romstage.c @@ -40,6 +40,7 @@ #include #include #include +#include #include #include #include @@ -50,7 +51,6 @@ #include "resourcemap.c" #include "cpu/amd/quadcore/quadcore.c" #include "spd.h" -#include #define SERIAL_DEV PNP_DEV(0x2e, W83627HF_SP1) diff --git a/src/mainboard/amd/bimini_fam10/romstage.c b/src/mainboard/amd/bimini_fam10/romstage.c index f6bb70fe2c..51bc5d577b 100644 --- a/src/mainboard/amd/bimini_fam10/romstage.c +++ b/src/mainboard/amd/bimini_fam10/romstage.c @@ -36,6 +36,7 @@ #include #include #include +#include #include #include #include diff --git a/src/mainboard/amd/mahogany_fam10/romstage.c b/src/mainboard/amd/mahogany_fam10/romstage.c index 4a244eb376..e9f31112ff 100644 --- a/src/mainboard/amd/mahogany_fam10/romstage.c +++ b/src/mainboard/amd/mahogany_fam10/romstage.c @@ -40,6 +40,7 @@ #include #include #include +#include #include #include #include diff --git a/src/mainboard/amd/tilapia_fam10/romstage.c b/src/mainboard/amd/tilapia_fam10/romstage.c index 0e60fc5c75..fbb9238a72 100644 --- a/src/mainboard/amd/tilapia_fam10/romstage.c +++ b/src/mainboard/amd/tilapia_fam10/romstage.c @@ -38,6 +38,7 @@ #include #include #include +#include #include #include #include diff --git a/src/mainboard/asus/kcma-d8/romstage.c b/src/mainboard/asus/kcma-d8/romstage.c index cda416981f..ec76c45f6a 100644 --- a/src/mainboard/asus/kcma-d8/romstage.c +++ b/src/mainboard/asus/kcma-d8/romstage.c @@ -19,7 +19,6 @@ #include #include -#include #include #include #include @@ -39,6 +38,7 @@ #include #include #include +#include #include #include #include diff --git a/src/mainboard/asus/kfsn4-dre/romstage.c b/src/mainboard/asus/kfsn4-dre/romstage.c index a6fd8b2291..6045274692 100644 --- a/src/mainboard/asus/kfsn4-dre/romstage.c +++ b/src/mainboard/asus/kfsn4-dre/romstage.c @@ -22,7 +22,6 @@ #include #include -#include #include #include #include @@ -36,6 +35,7 @@ #include #include #include +#include #include #include #include diff --git a/src/mainboard/asus/kgpe-d16/romstage.c b/src/mainboard/asus/kgpe-d16/romstage.c index cb4a1ec904..9b72a79cfa 100644 --- a/src/mainboard/asus/kgpe-d16/romstage.c +++ b/src/mainboard/asus/kgpe-d16/romstage.c @@ -19,7 +19,6 @@ #include #include -#include #include #include #include @@ -39,6 +38,7 @@ #include #include #include +#include #include #include #include diff --git a/src/mainboard/asus/m4a78-em/romstage.c b/src/mainboard/asus/m4a78-em/romstage.c index 4320525db4..41845864f8 100644 --- a/src/mainboard/asus/m4a78-em/romstage.c +++ b/src/mainboard/asus/m4a78-em/romstage.c @@ -39,6 +39,7 @@ #include #include #include +#include #include #include #include diff --git a/src/mainboard/asus/m4a785-m/romstage.c b/src/mainboard/asus/m4a785-m/romstage.c index 44d8d51c00..d7538c8eef 100644 --- a/src/mainboard/asus/m4a785-m/romstage.c +++ b/src/mainboard/asus/m4a785-m/romstage.c @@ -40,6 +40,7 @@ #include #include #include +#include #include #include #include diff --git a/src/mainboard/asus/m5a88-v/romstage.c b/src/mainboard/asus/m5a88-v/romstage.c index 56e15418c7..512c08e34f 100644 --- a/src/mainboard/asus/m5a88-v/romstage.c +++ b/src/mainboard/asus/m5a88-v/romstage.c @@ -39,6 +39,7 @@ #include #include #include +#include #include #include #include @@ -48,7 +49,6 @@ #include #include "southbridge/amd/sb800/early_setup.c" #include "spd.h" -#include #include "resourcemap.c" #include "cpu/amd/quadcore/quadcore.c" diff --git a/src/mainboard/avalue/eax-785e/romstage.c b/src/mainboard/avalue/eax-785e/romstage.c index e9e865e28f..49352ca734 100644 --- a/src/mainboard/avalue/eax-785e/romstage.c +++ b/src/mainboard/avalue/eax-785e/romstage.c @@ -37,6 +37,7 @@ #include #include #include +#include #include #include #include @@ -44,7 +45,6 @@ #include #include #include "spd.h" -#include #include #include diff --git a/src/mainboard/gigabyte/ma785gm/romstage.c b/src/mainboard/gigabyte/ma785gm/romstage.c index 7b30452349..87bfaba5fd 100644 --- a/src/mainboard/gigabyte/ma785gm/romstage.c +++ b/src/mainboard/gigabyte/ma785gm/romstage.c @@ -36,6 +36,7 @@ #include #include #include +#include #include #include #include diff --git a/src/mainboard/gigabyte/ma785gmt/romstage.c b/src/mainboard/gigabyte/ma785gmt/romstage.c index fef31f74b5..0b9e07dab0 100644 --- a/src/mainboard/gigabyte/ma785gmt/romstage.c +++ b/src/mainboard/gigabyte/ma785gmt/romstage.c @@ -36,6 +36,7 @@ #include #include #include +#include #include #include #include diff --git a/src/mainboard/gigabyte/ma78gm/romstage.c b/src/mainboard/gigabyte/ma78gm/romstage.c index c09b170a2b..8dbdfb07c8 100644 --- a/src/mainboard/gigabyte/ma78gm/romstage.c +++ b/src/mainboard/gigabyte/ma78gm/romstage.c @@ -39,6 +39,7 @@ #include #include #include +#include #include #include #include diff --git a/src/mainboard/hp/dl165_g6_fam10/romstage.c b/src/mainboard/hp/dl165_g6_fam10/romstage.c index 6e65be23e9..5af01c176f 100644 --- a/src/mainboard/hp/dl165_g6_fam10/romstage.c +++ b/src/mainboard/hp/dl165_g6_fam10/romstage.c @@ -49,6 +49,7 @@ #include #include #include +#include #include "southbridge/broadcom/bcm5785/early_smbus.c" #include "southbridge/broadcom/bcm5785/early_setup.c" diff --git a/src/mainboard/iei/kino-780am2-fam10/romstage.c b/src/mainboard/iei/kino-780am2-fam10/romstage.c index b4612eed5e..815cf34d92 100644 --- a/src/mainboard/iei/kino-780am2-fam10/romstage.c +++ b/src/mainboard/iei/kino-780am2-fam10/romstage.c @@ -39,6 +39,7 @@ #include #include #include +#include #include #include #include diff --git a/src/mainboard/jetway/pa78vm5/romstage.c b/src/mainboard/jetway/pa78vm5/romstage.c index 28c3a5dea4..440d703ea5 100644 --- a/src/mainboard/jetway/pa78vm5/romstage.c +++ b/src/mainboard/jetway/pa78vm5/romstage.c @@ -40,6 +40,7 @@ #include #include #include +#include #include #include #include diff --git a/src/mainboard/msi/ms9652_fam10/romstage.c b/src/mainboard/msi/ms9652_fam10/romstage.c index 4555bf94c1..3d22439533 100644 --- a/src/mainboard/msi/ms9652_fam10/romstage.c +++ b/src/mainboard/msi/ms9652_fam10/romstage.c @@ -42,6 +42,7 @@ #include #include #include +#include #include #include "resourcemap.c" diff --git a/src/mainboard/supermicro/h8dmr_fam10/romstage.c b/src/mainboard/supermicro/h8dmr_fam10/romstage.c index 3e58848e61..3fb3ca09a0 100644 --- a/src/mainboard/supermicro/h8dmr_fam10/romstage.c +++ b/src/mainboard/supermicro/h8dmr_fam10/romstage.c @@ -37,6 +37,7 @@ #include #include #include +#include #include #include #include diff --git a/src/mainboard/supermicro/h8qme_fam10/romstage.c b/src/mainboard/supermicro/h8qme_fam10/romstage.c index e51cf19ce2..5ea0f64571 100644 --- a/src/mainboard/supermicro/h8qme_fam10/romstage.c +++ b/src/mainboard/supermicro/h8qme_fam10/romstage.c @@ -37,6 +37,7 @@ #include #include #include +#include #include #include #include diff --git a/src/mainboard/supermicro/h8scm_fam10/romstage.c b/src/mainboard/supermicro/h8scm_fam10/romstage.c index 045adaeff9..11da86ed68 100644 --- a/src/mainboard/supermicro/h8scm_fam10/romstage.c +++ b/src/mainboard/supermicro/h8scm_fam10/romstage.c @@ -37,6 +37,7 @@ #include #include #include +#include #include #include #include diff --git a/src/mainboard/tyan/s2912_fam10/romstage.c b/src/mainboard/tyan/s2912_fam10/romstage.c index 2d302f12d8..99d58f8d06 100644 --- a/src/mainboard/tyan/s2912_fam10/romstage.c +++ b/src/mainboard/tyan/s2912_fam10/romstage.c @@ -37,6 +37,7 @@ #include #include #include +#include #include #include #include diff --git a/src/northbridge/amd/amdmct/mct_ddr3/mct_d.c b/src/northbridge/amd/amdmct/mct_ddr3/mct_d.c index 27aaf1a54e..18f96c62a6 100644 --- a/src/northbridge/amd/amdmct/mct_ddr3/mct_d.c +++ b/src/northbridge/amd/amdmct/mct_ddr3/mct_d.c @@ -36,7 +36,7 @@ #include #include #include -#include +#include #include #include #include diff --git a/src/security/vboot/Kconfig b/src/security/vboot/Kconfig index af7578e630..77483d1a43 100644 --- a/src/security/vboot/Kconfig +++ b/src/security/vboot/Kconfig @@ -18,7 +18,7 @@ config VBOOT bool "Verify firmware with vboot." default n select VBOOT_MOCK_SECDATA if !TPM1 && !TPM2 - depends on HAVE_HARD_RESET || !MISSING_BOARD_RESET + depends on !MISSING_BOARD_RESET help Enabling VBOOT will use vboot to verify the components of the firmware (stages, payload, etc). diff --git a/src/southbridge/amd/amd8111/early_ctrl.c b/src/southbridge/amd/amd8111/early_ctrl.c index d04646cf85..ce29bf1e77 100644 --- a/src/southbridge/amd/amd8111/early_ctrl.c +++ b/src/southbridge/amd/amd8111/early_ctrl.c @@ -15,6 +15,7 @@ #include "amd8111.h" #include +#include unsigned get_sbdn(unsigned bus) { diff --git a/src/southbridge/amd/common/reset.h b/src/southbridge/amd/common/reset.h new file mode 100644 index 0000000000..ce101cb2dc --- /dev/null +++ b/src/southbridge/amd/common/reset.h @@ -0,0 +1,35 @@ +/* + * This file is part of the coreboot project. + * + * Copyright 2017 Google, Inc. + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; version 2 of the License. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + */ + +#ifndef _AMD_SB_RESET_H_ +#define _AMD_SB_RESET_H_ + +#include +#include +#include + +/* Implement the bare reset, e.g. write to cf9. */ +void do_soft_reset(void); + +/* Prepare for reset, run do_soft_reset(), halt. */ +static inline __noreturn void soft_reset(void) +{ + printk(BIOS_INFO, "%s() called!\n", __func__); + dcache_clean_all(); + do_soft_reset(); + halt(); +} + +#endif /* _AMD_SB_RESET_H_ */ diff --git a/src/southbridge/amd/sb700/reset.c b/src/southbridge/amd/sb700/reset.c index f5f7a2c2f3..4c9b0f4056 100644 --- a/src/southbridge/amd/sb700/reset.c +++ b/src/southbridge/amd/sb700/reset.c @@ -18,6 +18,7 @@ #include #include +#include #define HT_INIT_CONTROL 0x6C #define HTIC_BIOSR_Detect (1<<5) diff --git a/src/southbridge/amd/sb800/early_setup.c b/src/southbridge/amd/sb800/early_setup.c index 2ace9926c7..6835d993c4 100644 --- a/src/southbridge/amd/sb800/early_setup.c +++ b/src/southbridge/amd/sb800/early_setup.c @@ -19,6 +19,7 @@ #include #include #include +#include #include "sb800.h" #include "smbus.c" diff --git a/src/southbridge/amd/sr5650/early_setup.c b/src/southbridge/amd/sr5650/early_setup.c index 543cb0efbf..8b6f22a793 100644 --- a/src/southbridge/amd/sr5650/early_setup.c +++ b/src/southbridge/amd/sr5650/early_setup.c @@ -22,7 +22,7 @@ #include #include #include -#include +#include #include "sr5650.h" #include "cmn.h" diff --git a/src/southbridge/broadcom/bcm5785/early_setup.c b/src/southbridge/broadcom/bcm5785/early_setup.c index c2aa9bcb47..df7217c79b 100644 --- a/src/southbridge/broadcom/bcm5785/early_setup.c +++ b/src/southbridge/broadcom/bcm5785/early_setup.c @@ -15,6 +15,7 @@ */ #include +#include #include "bcm5785.h" static void bcm5785_enable_lpc(void) diff --git a/src/southbridge/nvidia/ck804/early_setup.c b/src/southbridge/nvidia/ck804/early_setup.c index 180f41a5fa..f2121f0e26 100644 --- a/src/southbridge/nvidia/ck804/early_setup.c +++ b/src/southbridge/nvidia/ck804/early_setup.c @@ -15,6 +15,7 @@ */ #include +#include #include "ck804.h" static int set_ht_link_ck804(u8 ht_c_num) diff --git a/src/southbridge/nvidia/ck804/early_setup_car.c b/src/southbridge/nvidia/ck804/early_setup_car.c index fbc2719310..266b97cfe6 100644 --- a/src/southbridge/nvidia/ck804/early_setup_car.c +++ b/src/southbridge/nvidia/ck804/early_setup_car.c @@ -16,6 +16,7 @@ * GNU General Public License for more details. */ +#include #include "ck804.h" /* Someone messed up and snuck in some K8-specific code */ diff --git a/src/southbridge/nvidia/mcp55/early_ctrl.c b/src/southbridge/nvidia/mcp55/early_ctrl.c index 66ceae29e0..c14224963e 100644 --- a/src/southbridge/nvidia/mcp55/early_ctrl.c +++ b/src/southbridge/nvidia/mcp55/early_ctrl.c @@ -19,6 +19,7 @@ #include #include #include +#include #include "mcp55.h" void do_soft_reset(void) -- cgit v1.2.3