diff options
author | Gabe Black <gabeblack@google.com> | 2014-04-21 19:51:15 -0700 |
---|---|---|
committer | Marc Jones <marc.jones@se-eng.com> | 2014-12-17 20:49:23 +0100 |
commit | a0b02e7127b432125979753f886cbeecbe521652 (patch) | |
tree | 0a7ed001a58ce1ac1d7dc7efd4eff699c5c42d89 | |
parent | 5a056d30a1fd609994e0a9aa70f5788c68e3a785 (diff) | |
download | coreboot-a0b02e7127b432125979753f886cbeecbe521652.tar.xz |
nyan*: cbmem: Move the call to cbmemc_reinit.
The call was after the call to vboot_verify_firmware and so would only be
called when falling back to RO, aka recovery mode. This change moves it to
before vboot_verify_firmware so we'll always have the cbmem console.
BUG=None
TEST=Built and booted on nyan and verified that the cbmem console was the same
as the serial output. Built for big and blaze.
BRANCH=nyan
Original-Change-Id: I02d01110659689b08d32777dae384ac3e01b3b9f
Original-Signed-off-by: Gabe Black <gabeblack@google.com>
Original-Reviewed-on: https://chromium-review.googlesource.com/196158
Original-Reviewed-by: David Hendricks <dhendrix@chromium.org>
Original-Commit-Queue: Gabe Black <gabeblack@chromium.org>
Original-Tested-by: Gabe Black <gabeblack@chromium.org>
(cherry picked from commit d3e4a778e4a0f5ade7d633d8ce7e72ef06c44086)
Signed-off-by: Marc Jones <marc.jones@se-eng.com>
Change-Id: Id14a19a78bcb21cb0c4030c2e41195e491f690d5
Reviewed-on: http://review.coreboot.org/7777
Tested-by: build bot (Jenkins)
Reviewed-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
-rw-r--r-- | src/mainboard/google/nyan/romstage.c | 7 | ||||
-rw-r--r-- | src/mainboard/google/nyan_big/romstage.c | 7 | ||||
-rw-r--r-- | src/mainboard/google/nyan_blaze/romstage.c | 7 |
3 files changed, 12 insertions, 9 deletions
diff --git a/src/mainboard/google/nyan/romstage.c b/src/mainboard/google/nyan/romstage.c index 62cbd50fc2..b4e27330ac 100644 --- a/src/mainboard/google/nyan/romstage.c +++ b/src/mainboard/google/nyan/romstage.c @@ -213,6 +213,10 @@ static void __attribute__((noinline)) romstage(void) configure_ec_spi_bus(); configure_tpm_i2c_bus(); +#if CONFIG_CONSOLE_CBMEM + cbmemc_reinit(); +#endif + vboot_verify_firmware(romstage_handoff_find_or_add()); timestamp_add(TS_START_COPYRAM, timestamp_get()); @@ -220,9 +224,6 @@ static void __attribute__((noinline)) romstage(void) "fallback/coreboot_ram"); timestamp_add(TS_END_COPYRAM, timestamp_get()); -#if CONFIG_CONSOLE_CBMEM - cbmemc_reinit(); -#endif stage_exit(entry); } diff --git a/src/mainboard/google/nyan_big/romstage.c b/src/mainboard/google/nyan_big/romstage.c index 62cbd50fc2..b4e27330ac 100644 --- a/src/mainboard/google/nyan_big/romstage.c +++ b/src/mainboard/google/nyan_big/romstage.c @@ -213,6 +213,10 @@ static void __attribute__((noinline)) romstage(void) configure_ec_spi_bus(); configure_tpm_i2c_bus(); +#if CONFIG_CONSOLE_CBMEM + cbmemc_reinit(); +#endif + vboot_verify_firmware(romstage_handoff_find_or_add()); timestamp_add(TS_START_COPYRAM, timestamp_get()); @@ -220,9 +224,6 @@ static void __attribute__((noinline)) romstage(void) "fallback/coreboot_ram"); timestamp_add(TS_END_COPYRAM, timestamp_get()); -#if CONFIG_CONSOLE_CBMEM - cbmemc_reinit(); -#endif stage_exit(entry); } diff --git a/src/mainboard/google/nyan_blaze/romstage.c b/src/mainboard/google/nyan_blaze/romstage.c index 62cbd50fc2..b4e27330ac 100644 --- a/src/mainboard/google/nyan_blaze/romstage.c +++ b/src/mainboard/google/nyan_blaze/romstage.c @@ -213,6 +213,10 @@ static void __attribute__((noinline)) romstage(void) configure_ec_spi_bus(); configure_tpm_i2c_bus(); +#if CONFIG_CONSOLE_CBMEM + cbmemc_reinit(); +#endif + vboot_verify_firmware(romstage_handoff_find_or_add()); timestamp_add(TS_START_COPYRAM, timestamp_get()); @@ -220,9 +224,6 @@ static void __attribute__((noinline)) romstage(void) "fallback/coreboot_ram"); timestamp_add(TS_END_COPYRAM, timestamp_get()); -#if CONFIG_CONSOLE_CBMEM - cbmemc_reinit(); -#endif stage_exit(entry); } |