summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAaron Durbin <adurbin@chromium.org>2015-04-21 15:18:22 -0500
committerAaron Durbin <adurbin@chromium.org>2015-04-22 15:38:30 +0200
commit69cb2c2b5e4c4e4db005dc57249c0d15cadea5ee (patch)
tree952d017902077f8394e8a5ad8907b7f284f5df70
parent13d45aa92877bec1ff10221b3af85d57b59adf6d (diff)
downloadcoreboot-69cb2c2b5e4c4e4db005dc57249c0d15cadea5ee.tar.xz
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 <adurbin@chromium.org> Reviewed-on: http://review.coreboot.org/9934 Tested-by: build bot (Jenkins) Reviewed-by: Edward O'Callaghan <edward.ocallaghan@koparo.com> Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
-rw-r--r--src/lib/loaders/Makefile.inc1
-rw-r--r--src/lib/loaders/cbfs_romstage_loader.c31
-rw-r--r--src/lib/loaders/load_and_run_romstage.c32
3 files changed, 57 insertions, 7 deletions
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 <cbfs.h>
+#include <program_loading.h>
+
+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 <program_loading.h>
#include <timestamp.h>
+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();
+ for (i = 0; i < ARRAY_SIZE(loaders); i++) {
+ const struct prog_loader_ops *ops;
+
+ 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);
}
- timestamp_add_now(TS_END_COPYROM);
- prog_run(&romstage);
+ if (IS_ENABLED(CONFIG_BOOTBLOCK_CONSOLE))
+ die("Couldn't load romstage.\n");
+ halt();
}