stage_cache: use cbmem init hooks

Instead of having the chipset code make the approrpiate
calls at the appropriate places use the cbmem init hooks
to take the appropriate action. That way no chipset code
needs to be changed in order to support the external
stage cache.

Change-Id: If74e6155ae86646bde02b2e1b550ade92b8ba9bb
Signed-off-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-on: http://review.coreboot.org/10481
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
This commit is contained in:
Aaron Durbin 2015-06-09 13:55:51 -05:00
parent 41607a4682
commit 42e6856436
7 changed files with 31 additions and 57 deletions

View File

@ -35,7 +35,6 @@
#include <cbfs.h> #include <cbfs.h>
#include <romstage_handoff.h> #include <romstage_handoff.h>
#include <reset.h> #include <reset.h>
#include <stage_cache.h>
#include <vendorcode/google/chromeos/chromeos.h> #include <vendorcode/google/chromeos/chromeos.h>
#if CONFIG_EC_GOOGLE_CHROMEEC #if CONFIG_EC_GOOGLE_CHROMEEC
#include <ec/google/chromeec/ec.h> #include <ec/google/chromeec/ec.h>
@ -256,17 +255,13 @@ void romstage_common(const struct romstage_params *params)
if (!wake_from_s3) { if (!wake_from_s3) {
cbmem_initialize_empty(); cbmem_initialize_empty();
stage_cache_create_empty();
/* Save data returned from MRC on non-S3 resumes. */ /* Save data returned from MRC on non-S3 resumes. */
save_mrc_data(params->pei_data); save_mrc_data(params->pei_data);
} else { } else if (cbmem_initialize()) {
stage_cache_recover(); #if CONFIG_HAVE_ACPI_RESUME
if (cbmem_initialize()) { /* Failed S3 resume, reset to come up cleanly */
#if CONFIG_HAVE_ACPI_RESUME reset_system();
/* Failed S3 resume, reset to come up cleanly */ #endif
reset_system();
#endif
}
} }
handoff = romstage_handoff_find_or_add(); handoff = romstage_handoff_find_or_add();

View File

@ -29,10 +29,6 @@ enum {
STAGE_REFCODE, STAGE_REFCODE,
}; };
/* Create an empty stage cache. */
void stage_cache_create_empty(void);
/* Recover existing stage cache. */
void stage_cache_recover(void);
/* Cache the loaded stage provided according to the parameters. */ /* Cache the loaded stage provided according to the parameters. */
void stage_cache_add(int stage_id, struct prog *stage); void stage_cache_add(int stage_id, struct prog *stage);
/* Load the cached stage at given location returning the stage entry point. */ /* Load the cached stage at given location returning the stage entry point. */

View File

@ -22,11 +22,6 @@
#include <stage_cache.h> #include <stage_cache.h>
#include <string.h> #include <string.h>
/* Provide empty implementations by default. */
void __attribute__((weak)) stage_cache_create_empty(void) {}
void __attribute__((weak)) stage_cache_recover(void) {}
/* Stage cache uses cbmem. */ /* Stage cache uses cbmem. */
void stage_cache_add(int stage_id, struct prog *stage) void stage_cache_add(int stage_id, struct prog *stage)
{ {

View File

@ -33,7 +33,7 @@ static inline struct imd *imd_get(void)
return car_get_var_ptr(&imd_stage_cache); return car_get_var_ptr(&imd_stage_cache);
} }
void stage_cache_create_empty(void) static void stage_cache_create_empty(void)
{ {
struct imd *imd; struct imd *imd;
void *base; void *base;
@ -49,7 +49,7 @@ void stage_cache_create_empty(void)
printk(BIOS_DEBUG, "Could not limit stage cache size.\n"); printk(BIOS_DEBUG, "Could not limit stage cache size.\n");
} }
void stage_cache_recover(void) static void stage_cache_recover(void)
{ {
struct imd *imd; struct imd *imd;
void *base; void *base;
@ -120,10 +120,13 @@ void stage_cache_load_stage(int stage_id, struct prog *stage)
prog_set_entry(stage, (void *)(uintptr_t)meta->entry_addr, NULL); prog_set_entry(stage, (void *)(uintptr_t)meta->entry_addr, NULL);
} }
#if ENV_RAMSTAGE static void stage_cache_setup(int is_recovery)
static void recover_sc(void *unused)
{ {
stage_cache_recover(); if (is_recovery)
stage_cache_recover();
else
stage_cache_create_empty();
} }
BOOT_STATE_INIT_ENTRY(BS_PRE_DEVICE, BS_ON_ENTRY, recover_sc, NULL);
#endif ROMSTAGE_CBMEM_INIT_HOOK(stage_cache_setup)
RAMSTAGE_CBMEM_INIT_HOOK(stage_cache_setup)

View File

@ -25,7 +25,6 @@
#include <console/console.h> #include <console/console.h>
#include <device/pci_def.h> #include <device/pci_def.h>
#include <halt.h> #include <halt.h>
#include <stage_cache.h>
#include <soc/gpio.h> #include <soc/gpio.h>
#include <soc/intel/common/mrc_cache.h> #include <soc/intel/common/mrc_cache.h>
#include <soc/iomap.h> #include <soc/iomap.h>
@ -169,16 +168,12 @@ void raminit(struct mrc_params *mp, int prev_sleep_state)
if (prev_sleep_state != 3) { if (prev_sleep_state != 3) {
cbmem_initialize_empty(); cbmem_initialize_empty();
stage_cache_create_empty(); } else if (cbmem_initialize()) {
} else { #if CONFIG_HAVE_ACPI_RESUME
stage_cache_recover(); printk(BIOS_DEBUG, "Failed to recover CBMEM in S3 resume.\n");
if (cbmem_initialize()) { /* Failed S3 resume, reset to come up cleanly */
#if CONFIG_HAVE_ACPI_RESUME reset_system();
printk(BIOS_DEBUG, "Failed to recover CBMEM in S3 resume.\n"); #endif
/* Failed S3 resume, reset to come up cleanly */
reset_system();
#endif
}
} }
printk(BIOS_DEBUG, "MRC Wrapper returned %d\n", ret); printk(BIOS_DEBUG, "MRC Wrapper returned %d\n", ret);

View File

@ -25,7 +25,6 @@
#include <console/console.h> #include <console/console.h>
#include <device/pci_def.h> #include <device/pci_def.h>
#include <halt.h> #include <halt.h>
#include <stage_cache.h>
#include <soc/gpio.h> #include <soc/gpio.h>
#include <soc/intel/common/mrc_cache.h> #include <soc/intel/common/mrc_cache.h>
#include <soc/iomap.h> #include <soc/iomap.h>
@ -169,16 +168,12 @@ void raminit(struct mrc_params *mp, int prev_sleep_state)
if (prev_sleep_state != 3) { if (prev_sleep_state != 3) {
cbmem_initialize_empty(); cbmem_initialize_empty();
stage_cache_create_empty(); } else if (cbmem_initialize()) {
} else { #if CONFIG_HAVE_ACPI_RESUME
stage_cache_recover(); printk(BIOS_DEBUG, "Failed to recover CBMEM in S3 resume.\n");
if (cbmem_initialize()) { /* Failed S3 resume, reset to come up cleanly */
#if CONFIG_HAVE_ACPI_RESUME reset_system();
printk(BIOS_DEBUG, "Failed to recover CBMEM in S3 resume.\n"); #endif
/* Failed S3 resume, reset to come up cleanly */
reset_system();
#endif
}
} }
printk(BIOS_DEBUG, "MRC Wrapper returned %d\n", ret); printk(BIOS_DEBUG, "MRC Wrapper returned %d\n", ret);

View File

@ -29,7 +29,6 @@
#include <ec/google/chromeec/ec.h> #include <ec/google/chromeec/ec.h>
#include <ec/google/chromeec/ec_commands.h> #include <ec/google/chromeec/ec_commands.h>
#endif #endif
#include <stage_cache.h>
#include <vendorcode/google/chromeos/chromeos.h> #include <vendorcode/google/chromeos/chromeos.h>
#include <soc/intel/common/mrc_cache.h> #include <soc/intel/common/mrc_cache.h>
#include <soc/iomap.h> #include <soc/iomap.h>
@ -111,16 +110,12 @@ void raminit(struct pei_data *pei_data)
if (pei_data->boot_mode != SLEEP_STATE_S3) { if (pei_data->boot_mode != SLEEP_STATE_S3) {
cbmem_initialize_empty(); cbmem_initialize_empty();
stage_cache_create_empty(); } else if (cbmem_initialize()) {
} else {
stage_cache_recover();
if (cbmem_initialize()) {
#if CONFIG_HAVE_ACPI_RESUME #if CONFIG_HAVE_ACPI_RESUME
printk(BIOS_DEBUG, "Failed to recover CBMEM in S3 resume.\n"); printk(BIOS_DEBUG, "Failed to recover CBMEM in S3 resume.\n");
/* Failed S3 resume, reset to come up cleanly */ /* Failed S3 resume, reset to come up cleanly */
reset_system(); reset_system();
#endif #endif
}
} }
printk(BIOS_DEBUG, "MRC data at %p %d bytes\n", pei_data->data_to_save, printk(BIOS_DEBUG, "MRC data at %p %d bytes\n", pei_data->data_to_save,