From 3bfd7cc61e73439a2b1ac2d85faa7aaa988969ed Mon Sep 17 00:00:00 2001 From: Timothy Pearson Date: Sun, 1 Nov 2015 02:13:17 -0600 Subject: drivers/pc80: Rework normal / fallback selector code Per IRC and Gerrit discussion, the normal / fallback selector code is a rather weak spot in coreboot, and did not function correctly for certain use cases. Rework the selector to more clearly indicate proper operation, and also remove dead code. Also tentatively abandon use of RTC bit 385; a follow-up patch will remove said bit from all affected mainboards. The correct operation of the fallback code selector approximates that of a power line recloser, with a user option to attempt normal boot that can be cleared by firmware, but never set by firmware. Additionally, if cleared by user, the fallback path should always be used on the next reboot. Change-Id: I753ae9f0710c524875a85354ac2547df0c305569 Signed-off-by: Timothy Pearson Reviewed-on: http://review.coreboot.org/12289 Reviewed-by: Nico Huber Tested-by: build bot (Jenkins) Reviewed-by: Ronald G. Minnich --- src/Kconfig | 3 +++ 1 file changed, 3 insertions(+) (limited to 'src/Kconfig') diff --git a/src/Kconfig b/src/Kconfig index 79d5994bb2..865f7f5c5b 100644 --- a/src/Kconfig +++ b/src/Kconfig @@ -268,9 +268,12 @@ config BOOTBLOCK_SOURCE config SKIP_MAX_REBOOT_CNT_CLEAR bool "Do not clear reboot count after successful boot" default n + depends on BOOTBLOCK_NORMAL help Do not clear the reboot count immediately after successful boot. Set to allow the payload to control normal/fallback image recovery. + Note that it is the responsibility of the payload to reset the + normal boot bit to 1 after each successsful boot. config UPDATE_IMAGE bool "Update existing coreboot.rom image" -- cgit v1.2.3