veyron: Trigger hard reset (via GPIO) if last reboot was caused by watchdog

Like Nyan, Veyron boards use a GPIO to reset the system so that we can
make the accompanying TPM reset secure and unforgeable. The normal
kernel reboot driver knows that, but the SoC-internal watchdog doesn't.

This patch implements a check for the global reset status register in
the early bootblock and triggers a hard_reset() when it matches "first
global watchdog reset" or "second global watchdog reset". Seems that
the difference between the two is is a choice controlled by
wdt_glb_srst_ctrl (unconfirmed), and we want this code to run in both
cases.

BRANCH=None
BUG=chrome-os-partner:33141
TEST=Run 'mem w 0xff800000 0x9' from the command line, watch how you end
up in recovery without this patch but can boot normally with it.

Change-Id: Ice79648831e1e97d22325711da9e82bbf6bf3c75
Signed-off-by: Stefan Reinauer <reinauer@chromium.org>
Original-Commit-Id: 5d7cb52b2c2dcb2fff0bf83fc168439dade4b1b7
Original-Change-Id: I2581bde84f0445c15896060544e9acb60de91c8c
Original-Signed-off-by: Julius Werner <jwerner@chromium.org>
Original-Reviewed-on: https://chromium-review.googlesource.com/231734
Original-Reviewed-by: David Hendricks <dhendrix@chromium.org>
Reviewed-on: http://review.coreboot.org/9629
Tested-by: build bot (Jenkins)
Reviewed-by: Marc Jones <marc.jones@se-eng.com>
This commit is contained in:
Julius Werner 2014-11-24 13:50:46 -08:00 committed by Stefan Reinauer
parent 5984aad9c8
commit 2460a5564f
6 changed files with 35 additions and 0 deletions

View File

@ -21,7 +21,9 @@
#include <arch/io.h> #include <arch/io.h>
#include <assert.h> #include <assert.h>
#include <bootblock_common.h> #include <bootblock_common.h>
#include <console/console.h>
#include <delay.h> #include <delay.h>
#include <reset.h>
#include <soc/clock.h> #include <soc/clock.h>
#include <soc/i2c.h> #include <soc/i2c.h>
#include <soc/grf.h> #include <soc/grf.h>
@ -55,6 +57,11 @@ void bootblock_mainboard_init(void)
udelay(100);/* Must wait for voltage to stabilize,2mV/us */ udelay(100);/* Must wait for voltage to stabilize,2mV/us */
rkclk_configure_cpu(); rkclk_configure_cpu();
if (rkclk_was_watchdog_reset()) {
printk(BIOS_INFO, "Last reset was watchdog... rebooting via GPIO!\n");
hard_reset();
}
/* i2c1 for tpm */ /* i2c1 for tpm */
writel(IOMUX_I2C1, &rk3288_grf->iomux_i2c1); writel(IOMUX_I2C1, &rk3288_grf->iomux_i2c1);

View File

@ -21,7 +21,9 @@
#include <arch/io.h> #include <arch/io.h>
#include <assert.h> #include <assert.h>
#include <bootblock_common.h> #include <bootblock_common.h>
#include <console/console.h>
#include <delay.h> #include <delay.h>
#include <reset.h>
#include <soc/clock.h> #include <soc/clock.h>
#include <soc/i2c.h> #include <soc/i2c.h>
#include <soc/grf.h> #include <soc/grf.h>
@ -55,6 +57,11 @@ void bootblock_mainboard_init(void)
udelay(100);/* Must wait for voltage to stabilize,2mV/us */ udelay(100);/* Must wait for voltage to stabilize,2mV/us */
rkclk_configure_cpu(); rkclk_configure_cpu();
if (rkclk_was_watchdog_reset()) {
printk(BIOS_INFO, "Last reset was watchdog... rebooting via GPIO!\n");
hard_reset();
}
/* i2c1 for tpm */ /* i2c1 for tpm */
writel(IOMUX_I2C1, &rk3288_grf->iomux_i2c1); writel(IOMUX_I2C1, &rk3288_grf->iomux_i2c1);

View File

@ -21,7 +21,9 @@
#include <arch/io.h> #include <arch/io.h>
#include <assert.h> #include <assert.h>
#include <bootblock_common.h> #include <bootblock_common.h>
#include <console/console.h>
#include <delay.h> #include <delay.h>
#include <reset.h>
#include <soc/clock.h> #include <soc/clock.h>
#include <soc/i2c.h> #include <soc/i2c.h>
#include <soc/grf.h> #include <soc/grf.h>
@ -55,6 +57,11 @@ void bootblock_mainboard_init(void)
udelay(100);/* Must wait for voltage to stabilize,2mV/us */ udelay(100);/* Must wait for voltage to stabilize,2mV/us */
rkclk_configure_cpu(); rkclk_configure_cpu();
if (rkclk_was_watchdog_reset()) {
printk(BIOS_INFO, "Last reset was watchdog... rebooting via GPIO!\n");
hard_reset();
}
/* i2c1 for tpm */ /* i2c1 for tpm */
writel(IOMUX_I2C1, &rk3288_grf->iomux_i2c1); writel(IOMUX_I2C1, &rk3288_grf->iomux_i2c1);

View File

@ -21,7 +21,9 @@
#include <arch/io.h> #include <arch/io.h>
#include <assert.h> #include <assert.h>
#include <bootblock_common.h> #include <bootblock_common.h>
#include <console/console.h>
#include <delay.h> #include <delay.h>
#include <reset.h>
#include <soc/clock.h> #include <soc/clock.h>
#include <soc/i2c.h> #include <soc/i2c.h>
#include <soc/grf.h> #include <soc/grf.h>
@ -55,6 +57,11 @@ void bootblock_mainboard_init(void)
udelay(100);/* Must wait for voltage to stabilize,2mV/us */ udelay(100);/* Must wait for voltage to stabilize,2mV/us */
rkclk_configure_cpu(); rkclk_configure_cpu();
if (rkclk_was_watchdog_reset()) {
printk(BIOS_INFO, "Last reset was watchdog... rebooting via GPIO!\n");
hard_reset();
}
/* i2c1 for tpm */ /* i2c1 for tpm */
writel(IOMUX_I2C1, &rk3288_grf->iomux_i2c1); writel(IOMUX_I2C1, &rk3288_grf->iomux_i2c1);

View File

@ -640,3 +640,9 @@ int rkclk_configure_vop_dclk(u32 vop_id, u32 dclk_hz)
} }
return 0; return 0;
} }
int rkclk_was_watchdog_reset(void)
{
/* Bits 5 and 4 are "second" and "first" global watchdog reset. */
return readl(&cru_ptr->cru_glb_rst_st) & 0x30;
}

View File

@ -49,4 +49,5 @@ void rkclk_configure_tsadc(unsigned int hz);
void rkclk_configure_vop_aclk(u32 vop_id, u32 aclk_hz); void rkclk_configure_vop_aclk(u32 vop_id, u32 aclk_hz);
int rkclk_configure_vop_dclk(u32 vop_id, u32 dclk_hz); int rkclk_configure_vop_dclk(u32 vop_id, u32 dclk_hz);
void rkclk_configure_edp(void); void rkclk_configure_edp(void);
int rkclk_was_watchdog_reset(void);
#endif /* __SOC_ROCKCHIP_RK3288_CLOCK_H__ */ #endif /* __SOC_ROCKCHIP_RK3288_CLOCK_H__ */