From 3fca4ed45ea38081941a06531c114e463ef1a725 Mon Sep 17 00:00:00 2001 From: Patrick Rudolph Date: Fri, 10 Aug 2018 10:12:35 +0200 Subject: lib/fit: support booting initrd placed above 4GiB Store initrd offset as 64bit integer. Tested on Cavium CN81XX EVB: The initrd could be loaded when placed above 4GiB. Previously it failed to find the initrd. Change-Id: I5d1ae860ae4a4465546bc0ef89937d611d1e56ab Signed-off-by: Patrick Rudolph Reviewed-on: https://review.coreboot.org/28002 Tested-by: build bot (Jenkins) Reviewed-by: Aaron Durbin Reviewed-by: Philipp Deppenwiese --- src/include/device_tree.h | 1 + src/lib/device_tree.c | 16 ++++++++++++++++ src/lib/fit.c | 9 ++++----- 3 files changed, 21 insertions(+), 5 deletions(-) diff --git a/src/include/device_tree.h b/src/include/device_tree.h index 7f725e4d74..4c1a37c00c 100644 --- a/src/include/device_tree.h +++ b/src/include/device_tree.h @@ -158,6 +158,7 @@ void dt_add_bin_prop(struct device_tree_node *node, const char *name, void *data size_t size); void dt_add_string_prop(struct device_tree_node *node, const char *name, char *str); void dt_add_u32_prop(struct device_tree_node *node, const char *name, u32 val); +void dt_add_u64_prop(struct device_tree_node *node, const char *name, u64 val); void dt_add_reg_prop(struct device_tree_node *node, u64 *addrs, u64 *sizes, int count, u32 addr_cells, u32 size_cells); int dt_set_bin_prop_by_path(struct device_tree *tree, const char *path, diff --git a/src/lib/device_tree.c b/src/lib/device_tree.c index a905dbf893..89387ffdbf 100644 --- a/src/lib/device_tree.c +++ b/src/lib/device_tree.c @@ -881,6 +881,22 @@ void dt_add_u32_prop(struct device_tree_node *node, const char *name, u32 val) dt_add_bin_prop(node, name, val_ptr, sizeof(*val_ptr)); } +/* + * Add a 64-bit integer property to a node, or update it if it already exists. + * + * @param node The device tree node to add to. + * @param name The name of the new property. + * @param val The integer to be stored in the property. + */ +void dt_add_u64_prop(struct device_tree_node *node, const char *name, u64 val) +{ + u64 *val_ptr = malloc(sizeof(val)); + if (!val_ptr) + return; + *val_ptr = htobe64(val); + dt_add_bin_prop(node, name, val_ptr, sizeof(*val_ptr)); +} + /* * Add a 'reg' address list property to a node, or update it if it exists. * diff --git a/src/lib/fit.c b/src/lib/fit.c index fe8a82ef97..68f5bed2b8 100644 --- a/src/lib/fit.c +++ b/src/lib/fit.c @@ -212,12 +212,11 @@ void fit_add_ramdisk(struct device_tree *tree, void *ramdisk_addr, struct device_tree_node *node; node = dt_find_node(tree->root, path, NULL, NULL, 1); - /* Warning: this assumes the ramdisk is currently located below 4GiB. */ - u32 start = (uintptr_t)ramdisk_addr; - u32 end = start + ramdisk_size; + u64 start = (uintptr_t)ramdisk_addr; + u64 end = start + ramdisk_size; - dt_add_u32_prop(node, "linux,initrd-start", start); - dt_add_u32_prop(node, "linux,initrd-end", end); + dt_add_u64_prop(node, "linux,initrd-start", start); + dt_add_u64_prop(node, "linux,initrd-end", end); } static void update_reserve_map(uint64_t start, uint64_t end, -- cgit v1.2.3