From 69cb2c2b5e4c4e4db005dc57249c0d15cadea5ee Mon Sep 17 00:00:00 2001 From: Aaron Durbin Date: Tue, 21 Apr 2015 15:18:22 -0500 Subject: [PATCH] coreboot: add a place to choose romstage loader Instead of always loading romstage from cbfs provide a way, similar to ramstage and payload, for other program loaders to intervene. For now, only the cbfs loader is consulted. TEST=Booted to end of ramstage on qemu-armv7 Change-Id: I87c3e2e566d7a0723e775aa427de58af745ecdd5 Signed-off-by: Aaron Durbin Reviewed-on: http://review.coreboot.org/9934 Tested-by: build bot (Jenkins) Reviewed-by: Edward O'Callaghan Reviewed-by: Paul Menzel --- src/lib/loaders/Makefile.inc | 1 + src/lib/loaders/cbfs_romstage_loader.c | 31 ++++++++++++++++++++++ src/lib/loaders/load_and_run_romstage.c | 34 +++++++++++++++++++------ 3 files changed, 58 insertions(+), 8 deletions(-) create mode 100644 src/lib/loaders/cbfs_romstage_loader.c diff --git a/src/lib/loaders/Makefile.inc b/src/lib/loaders/Makefile.inc index 71d9c44507..fbc7f870a4 100644 --- a/src/lib/loaders/Makefile.inc +++ b/src/lib/loaders/Makefile.inc @@ -18,6 +18,7 @@ # bootblock-y += load_and_run_romstage.c +bootblock-y += cbfs_romstage_loader.c romstage-y += cbfs_ramstage_loader.c romstage-y += load_and_run_ramstage.c ramstage-y += cbfs_payload_loader.c diff --git a/src/lib/loaders/cbfs_romstage_loader.c b/src/lib/loaders/cbfs_romstage_loader.c new file mode 100644 index 0000000000..e5b42158f8 --- /dev/null +++ b/src/lib/loaders/cbfs_romstage_loader.c @@ -0,0 +1,31 @@ +/* + * This file is part of the coreboot project. + * + * Copyright 2015 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. + * + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software + * Foundation, Inc. + */ + +#include +#include + +static int cbfs_load_romstage(struct prog *romstage) +{ + return cbfs_load_prog_stage(CBFS_DEFAULT_MEDIA, romstage); +} + +const struct prog_loader_ops cbfs_romstage_loader = { + .name = "CBFS", + .prepare = cbfs_load_romstage, +}; diff --git a/src/lib/loaders/load_and_run_romstage.c b/src/lib/loaders/load_and_run_romstage.c index 9bd9603bab..1c2ed77abf 100644 --- a/src/lib/loaders/load_and_run_romstage.c +++ b/src/lib/loaders/load_and_run_romstage.c @@ -26,20 +26,38 @@ #include #include +extern const struct prog_loader_ops cbfs_romstage_loader; + +static const struct prog_loader_ops *loaders[] = { + &cbfs_romstage_loader, +}; + void run_romstage(void) { + int i; struct prog romstage = { .name = CONFIG_CBFS_PREFIX "/romstage", .type = PROG_ROMSTAGE, }; - timestamp_add_now(TS_START_COPYROM); - if (cbfs_load_prog_stage(CBFS_DEFAULT_MEDIA, &romstage) < 0) { - if (IS_ENABLED(CONFIG_BOOTBLOCK_CONSOLE)) - die("Couldn't load romstage.\n"); - halt(); - } - timestamp_add_now(TS_END_COPYROM); + for (i = 0; i < ARRAY_SIZE(loaders); i++) { + const struct prog_loader_ops *ops; - prog_run(&romstage); + ops = loaders[i]; + + printk(BIOS_DEBUG, "Trying %s romstage loader.\n", ops->name); + + timestamp_add_now(TS_START_COPYROM); + + if (ops->prepare(&romstage)) + continue; + + timestamp_add_now(TS_END_COPYROM); + + prog_run(&romstage); + } + + if (IS_ENABLED(CONFIG_BOOTBLOCK_CONSOLE)) + die("Couldn't load romstage.\n"); + halt(); }