diff options
author | Steffen Trumtrar <s.trumtrar@pengutronix.de> | 2019-08-19 12:00:46 +0200 |
---|---|---|
committer | Sascha Hauer <s.hauer@pengutronix.de> | 2019-08-19 15:43:31 +0200 |
commit | da7695b6189747212c3d7ac9a67728677c089478 (patch) | |
tree | 4f64895f265548fc1213f9159525af57498ba442 /arch | |
parent | a3e62ffc7341254fd754886e560865556bc731c8 (diff) | |
download | barebox-da7695b6189747212c3d7ac9a67728677c089478.tar.gz barebox-da7695b6189747212c3d7ac9a67728677c089478.tar.xz |
ARM: socfpga: fix xload on arria10
Since the commit
5a1a5ed2537d7d12f851f3778707681d6c08d6e8
ARM: images: use piggydata
the loading mechanism in the arria10 xload is neither functional nor needed.
Now, barebox has/can be loaded like a normal image, so the filesize, that is
written to the barebox header, can be evaluated.
Signed-off-by: Steffen Trumtrar <s.trumtrar@pengutronix.de>
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Diffstat (limited to 'arch')
-rw-r--r-- | arch/arm/mach-socfpga/arria10-xload.c | 63 |
1 files changed, 12 insertions, 51 deletions
diff --git a/arch/arm/mach-socfpga/arria10-xload.c b/arch/arm/mach-socfpga/arria10-xload.c index 3622a3cd9c..6f137e300e 100644 --- a/arch/arm/mach-socfpga/arria10-xload.c +++ b/arch/arm/mach-socfpga/arria10-xload.c @@ -386,70 +386,31 @@ int arria10_load_fpga(int offset, int bitstream_size) return 0; } -static int image_size(void) -{ - uint32_t *image_end = (void *)__image_end; - uint32_t payload_len; - uint32_t pbl_len; - uint32_t arria10_header_len; - uint32_t sizep; - uint32_t arria10_crc; - - /* arria10 header is 512 byte */ - arria10_header_len = 512; - /* pbl is appended with 4 byte CRC for boot rom */ - arria10_crc = 4; - - /* The length of the PBL image */ - pbl_len = __image_end - _text; - - sizep = 4; - - /* The length of the payload is appended directly behind the PBL */ - payload_len = *(image_end); - - return pbl_len + arria10_header_len + sizep + arria10_crc + payload_len; -} - void arria10_start_image(int offset) { void *buf = (void *)0x0; - void *in_buf = (void *)SZ_1M; uint32_t start; - int size = 0; + unsigned int size; int ret; void __noreturn (*bb)(void); - uint32_t pbl_len = __image_end - _text; - uint32_t *image_end = (void *)__image_end; - uint32_t arria10_header_len; - uint32_t sizep; - uint32_t arria10_crc; - - size = image_size(); start = bootloader.first_sec + offset / SECTOR_SIZE; - ret = arria10_read_blocks(buf, start, ALIGN(size, SECTOR_SIZE)); - if (ret) { - puts_ll("Loading image failed\n"); + ret = arria10_read_blocks(buf, start, ALIGN(ARM_HEAD_SIZE, SECTOR_SIZE)); + if (ret) hang(); - } - - /* arria10 header is 512 byte */ - arria10_header_len = 512; - sizep = 4; - /* copy PBL */ - memcpy(in_buf, buf, pbl_len + sizep + arria10_header_len); - - /* pbl is appended with 4 byte CRC for boot rom */ - arria10_crc = 4; + if (is_barebox_arm_head(buf)) + size = *((unsigned int *)buf + ARM_HEAD_SIZE_OFFSET / + sizeof(unsigned int)); + else + hang(); - /* copy payload, skip the Arria10 CRC */ - memcpy(in_buf + pbl_len + sizep + arria10_header_len, - buf + pbl_len + sizep + arria10_header_len + arria10_crc, *(image_end)); + ret = arria10_read_blocks(buf, start, ALIGN(size, SECTOR_SIZE)); + if (ret) + hang(); - bb = in_buf; + bb = buf; bb(); |