summaryrefslogtreecommitdiffstats
path: root/src/include
diff options
context:
space:
mode:
authorJulius Werner <jwerner@chromium.org>2019-05-13 16:34:16 -0700
committerPatrick Georgi <pgeorgi@google.com>2019-06-21 09:21:01 +0000
commitb379f1964ec6b0a59b7d869154a8bc739eb55fc0 (patch)
tree87a7b8b6beb582bbc387a7e31877ce5a9591ca9f /src/include
parent735ddc930f77950e1f292c574067106cf38e2e87 (diff)
downloadcoreboot-b379f1964ec6b0a59b7d869154a8bc739eb55fc0.tar.gz
coreboot-b379f1964ec6b0a59b7d869154a8bc739eb55fc0.tar.bz2
coreboot-b379f1964ec6b0a59b7d869154a8bc739eb55fc0.zip
fit: Refactor config node handling
This patch makes some minor refactoring to the way the FIT parser handles config nodes. A lot of this code was written in the dawn age of depthcharge when its device tree library wasn't as well-stocked yet, so some of it can be rewritten nicer with more high-level primitives. There's no point in storing both the string name and the actual FDT node of a FIT image node separately, since the latter also contains the former, so remove that. Also eliminate code for the case of not having an FDT (which makes no sense), and move some more FDT validity/compat checking into fit_update_compat() (mostly in anticipation of later changes). This patch was adapted from depthcharge's http://crosreview.com/1553456 with a couple of modifications specific to coreboot's custom FIT loading code. Change-Id: Ia79e0fd0e1159c4aca64c453b82a0379b133350d Signed-off-by: Julius Werner <jwerner@chromium.org> Reviewed-on: https://review.coreboot.org/c/coreboot/+/32870 Reviewed-by: Hung-Te Lin <hungte@chromium.org> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/include')
-rw-r--r--src/include/fit.h11
1 files changed, 4 insertions, 7 deletions
diff --git a/src/include/fit.h b/src/include/fit.h
index 6e0667f6cf6c..758ee70c9caf 100644
--- a/src/include/fit.h
+++ b/src/include/fit.h
@@ -27,7 +27,7 @@
struct fit_image_node
{
const char *name;
- const void *data;
+ void *data;
uint32_t size;
int compression;
@@ -37,12 +37,9 @@ struct fit_image_node
struct fit_config_node
{
const char *name;
- const char *kernel;
- struct fit_image_node *kernel_node;
- const char *fdt;
- struct fit_image_node *fdt_node;
- const char *ramdisk;
- struct fit_image_node *ramdisk_node;
+ struct fit_image_node *kernel;
+ struct fit_image_node *fdt;
+ struct fit_image_node *ramdisk;
struct fdt_property compat;
int compat_rank;
int compat_pos;