sravan/system76-coreboot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fit: Add overlay support

Browse Source 
This patch adds support to boot FIT image configurations consisting of
a base device tree and one or more overlays. Since extracting the right
compatible string from overlay FDTs is problematic, we'll only support
this for FIT images that have the compatible string pulled out into the
config node.

This patch was adapted from depthcharge's http://crosreview.com/1555293

Change-Id: I0943f9a1869c9e416887c7ff16e33f7d91b74989
Signed-off-by: Julius Werner <jwerner@chromium.org>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/32873
Reviewed-by: Hung-Te Lin <hungte@chromium.org>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
This commit is contained in:
Julius Werner
2019-05-16 16:12:04 -07:00
committed by Patrick Georgi
parent fec4206299
commit 21b0b1adec
3 changed files with 55 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

39
src/lib/fit.c
View File

@@ -96,6 +96,31 @@ static struct fit_image_node *find_image(const char *name)
return NULL;
}
static struct fit_image_node *find_image_with_overlays(const char *name,
int bytes, struct list_node *prev)
{
struct fit_image_node *base = find_image(name);
if (!base)
return NULL;
int len = strnlen(name, bytes) + 1;
bytes -= len;
name += len;
while (bytes > 0) {
struct fit_overlay_chain *next = xzalloc(sizeof(*next));
next->overlay = find_image(name);
if (!next->overlay)
return NULL;
list_insert_after(&next->list_node, prev);
prev = &next->list_node;
len = strnlen(name, bytes) + 1;
bytes -= len;
name += len;
}
return base;
}
static void image_node(struct device_tree_node *node)
{
struct fit_image_node *image = xzalloc(sizeof(*image));
@@ -133,7 +158,8 @@ static void config_node(struct device_tree_node *node)
if (!strcmp("kernel", prop->prop.name))
config->kernel = find_image(prop->prop.data);
else if (!strcmp("fdt", prop->prop.name))
config->fdt = find_image(prop->prop.data);
config->fdt = find_image_with_overlays(prop->prop.data,
prop->prop.size, &config->overlays);
else if (!strcmp("ramdisk", prop->prop.name))
config->ramdisk = find_image(prop->prop.data);
else if (!strcmp("compatible", prop->prop.name))
@@ -408,6 +434,14 @@ static int fit_update_compat(struct fit_config_node *config)
return -1;
}
// FDT overlays are not supported in legacy FIT images.
if (config->overlays.next) {
printk(BIOS_ERR,
"ERROR: config %s has overlay but no compat!\n",
config->name);
return -1;
}
if (fdt_find_compat(fdt_blob, fdt_offset, &config->compat)) {
printk(BIOS_ERR,
"ERROR: Can't find compat string in FDT %s "
@@ -442,6 +476,7 @@ struct fit_config_node *fit_load(void *fit)
struct fit_image_node *image;
struct fit_config_node *config;
struct compat_string_entry *compat_node;
struct fit_overlay_chain *overlay_chain;
printk(BIOS_DEBUG, "FIT: Loading FIT from %p\n", fit);
@@ -505,6 +540,8 @@ struct fit_config_node *fit_load(void *fit)
}
printk(BIOS_DEBUG, ", kernel %s", config->kernel->name);
printk(BIOS_DEBUG, ", fdt %s", config->fdt->name);
list_for_each(overlay_chain, config->overlays, list_node)
printk(BIOS_DEBUG, " %s", overlay_chain->overlay->name);
if (config->ramdisk)
printk(BIOS_DEBUG, ", ramdisk %s",
config->ramdisk->name);