spl: fix entry_point equal to load_addr
authorGiulio Benetti <giulio.benetti@benettiengineering.com>
Fri, 10 Jan 2020 14:46:51 +0000 (15:46 +0100)
committerStefano Babic <sbabic@denx.de>
Tue, 14 Jan 2020 21:53:59 +0000 (22:53 +0100)
At the moment entry_point is set to image_get_load(header) that sets it
to "load address" instead of "entry point", assuming entry_point is
equal to load_addr, but it's not true. Then load_addr is set to
"entry_point - header_size", but this is wrong too since load_addr is
not an entry point.

So use image_get_ep() for entry_point assignment and image_get_load()
for load_addr assignment.

Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
common/spl/spl.c

index c1fce62..19085ad 100644 (file)
@@ -284,9 +284,9 @@ int spl_parse_image_header(struct spl_image_info *spl_image,
                        spl_image->entry_point = image_get_ep(header);
                        spl_image->size = image_get_data_size(header);
                } else {
-                       spl_image->entry_point = image_get_load(header);
+                       spl_image->entry_point = image_get_ep(header);
                        /* Load including the header */
-                       spl_image->load_addr = spl_image->entry_point -
+                       spl_image->load_addr = image_get_load(header) -
                                header_size;
                        spl_image->size = image_get_data_size(header) +
                                header_size;