mirrors/u-boot
0
0
Fork 0
mirror of https://github.com/u-boot/u-boot.git synced 2024-11-27 06:04:40 +08:00
Code Issues Packages Projects Releases Wiki Activity

tools: kwbimage: Add support for creating an image with no data

Browse Source 
This change add support for mkimage's -s option to kwbimage format. It will
create an kwbimage with empty data part of image (data part would contain
only required 32-bit checksum). mkimage's -s option is indicated by skipcpy
flag and it is basically in conflict with mkimage's -d (datafile) option.

"Empty" kwbimage with no data can still contain headers. For example it can
contain binary executable header which is copied by BootROM into L2SRAM.
This is useful for example for small images which can do not require DDR
RAM and can be run in L2SRAM (which do not require any initialization).

Signed-off-by: Pali Rohár <pali@kernel.org>
This commit is contained in:
Pali Rohár 2023-01-21 20:11:28 +01:00 committed by Stefan Roese
parent b07965b8a9
commit 3a521f0867
1 changed files with 25 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
tools/kwbimage.c
View File

@ -1887,7 +1887,9 @@ static void kwbimage_set_header(void *ptr, struct stat *sbuf, int ifd,
* Do not use sbuf->st_size as it contains size with padding. * Do not use sbuf->st_size as it contains size with padding.
* We need original image data size, so stat original file. * We need original image data size, so stat original file.
*/ */
if (stat(params->datafile, &s)) { if (params->skipcpy) {
s.st_size = 0;
} else if (stat(params->datafile, &s)) {
fprintf(stderr, "Could not stat data file %s: %s\n", fprintf(stderr, "Could not stat data file %s: %s\n",
params->datafile, strerror(errno)); params->datafile, strerror(errno));
exit(EXIT_FAILURE); exit(EXIT_FAILURE);
@ -2106,6 +2108,8 @@ static int kwbimage_verify_header(unsigned char *ptr, int image_size,
return 0; return 0;
} }
static int kwbimage_align_size(int bootfrom, int alloc_len, struct stat s);
static int kwbimage_generate(struct image_tool_params *params, static int kwbimage_generate(struct image_tool_params *params,
struct image_type_params *tparams) struct image_type_params *tparams)
{ {
@ -2124,7 +2128,9 @@ static int kwbimage_generate(struct image_tool_params *params,
exit(EXIT_FAILURE); exit(EXIT_FAILURE);
} }
if (stat(params->datafile, &s)) { if (params->skipcpy) {
s.st_size = 0;
} else if (stat(params->datafile, &s)) {
fprintf(stderr, "Could not stat data file %s: %s\n", fprintf(stderr, "Could not stat data file %s: %s\n",
params->datafile, strerror(errno)); params->datafile, strerror(errno));
exit(EXIT_FAILURE); exit(EXIT_FAILURE);
@ -2195,6 +2201,22 @@ static int kwbimage_generate(struct image_tool_params *params,
tparams->header_size = alloc_len; tparams->header_size = alloc_len;
tparams->hdr = hdr; tparams->hdr = hdr;
/*
* This function should return aligned size of the datafile.
* When skipcpy is set (datafile is skipped) then return value of this
* function is ignored, so we have to put required kwbimage aligning
* into the preallocated header size.
*/
if (params->skipcpy) {
tparams->header_size += kwbimage_align_size(bootfrom, alloc_len, s);
return 0;
} else {
return kwbimage_align_size(bootfrom, alloc_len, s);
}
}
static int kwbimage_align_size(int bootfrom, int alloc_len, struct stat s)
{
/* /*
* The resulting image needs to be 4-byte aligned. At least * The resulting image needs to be 4-byte aligned. At least
* the Marvell hdrparser tool complains if its unaligned. * the Marvell hdrparser tool complains if its unaligned.
@ -2510,7 +2532,7 @@ static int kwbimage_check_params(struct image_tool_params *params)
return 1; return 1;
} }
return (params->dflag && (params->fflag || params->lflag)) || return (params->dflag && (params->fflag || params->lflag || params->skipcpy)) ||
(params->fflag) || (params->fflag) ||
(params->lflag && (params->dflag || params->fflag)); (params->lflag && (params->dflag || params->fflag));
} }