Skip to content

Commit

Permalink
dumpimage: replace the term "datafile" by "subimage"
Browse files Browse the repository at this point in the history
Signed-off-by: Guilherme Maciel Ferreira <[email protected]>
  • Loading branch information
guilhermeferreira authored and trini committed Jan 29, 2015
1 parent f41f5b7 commit 67f946c
Show file tree
Hide file tree
Showing 4 changed files with 19 additions and 18 deletions.
6 changes: 3 additions & 3 deletions tools/default_image.c
Original file line number Diff line number Diff line change
Expand Up @@ -117,7 +117,7 @@ static void image_set_header(void *ptr, struct stat *sbuf, int ifd,
image_set_hcrc(hdr, checksum);
}

static int image_extract_datafile(void *ptr, struct image_tool_params *params)
static int image_extract_subimage(void *ptr, struct image_tool_params *params)
{
const image_header_t *hdr = (const image_header_t *)ptr;
ulong file_data;
Expand All @@ -144,7 +144,7 @@ static int image_extract_datafile(void *ptr, struct image_tool_params *params)
}

/* save the "data file" into the file system */
return imagetool_save_datafile(params->outfile, file_data, file_len);
return imagetool_save_subimage(params->outfile, file_data, file_len);
}

/*
Expand All @@ -159,7 +159,7 @@ U_BOOT_IMAGE_TYPE(
image_verify_header,
image_print_contents,
image_set_header,
image_extract_datafile,
image_extract_subimage,
image_check_image_types,
NULL,
NULL
Expand Down
12 changes: 6 additions & 6 deletions tools/dumpimage.c
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ static struct image_tool_params params = {
};

/*
* dumpimage_extract_datafile -
* dumpimage_extract_subimage -
*
* It scans all registered image types,
* verifies image_header for each supported image type
Expand All @@ -28,7 +28,7 @@ static struct image_tool_params params = {
* returns negative if input image format does not match with any of
* supported image types
*/
static int dumpimage_extract_datafile(struct image_type_params *tparams,
static int dumpimage_extract_subimage(struct image_type_params *tparams,
void *ptr, struct stat *sbuf)
{
int retval = -1;
Expand All @@ -42,11 +42,11 @@ static int dumpimage_extract_datafile(struct image_type_params *tparams,
* Extract the file from the image
* if verify is successful
*/
if (tparams->extract_datafile) {
retval = tparams->extract_datafile(ptr, &params);
if (tparams->extract_subimage) {
retval = tparams->extract_subimage(ptr, &params);
} else {
fprintf(stderr,
"%s: extract_datafile undefined for %s\n",
"%s: extract_subimage undefined for %s\n",
params.cmdname, tparams->name);
return -2;
}
Expand Down Expand Up @@ -170,7 +170,7 @@ int main(int argc, char **argv)
* Extract the data files from within the matched
* image type. Returns the error code if not matched
*/
retval = dumpimage_extract_datafile(tparams, ptr,
retval = dumpimage_extract_subimage(tparams, ptr,
&sbuf);
} else {
/*
Expand Down
2 changes: 1 addition & 1 deletion tools/imagetool.c
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ int imagetool_verify_print_header(
return retval;
}

int imagetool_save_datafile(
int imagetool_save_subimage(
const char *file_name,
ulong file_data,
ulong file_len)
Expand Down
17 changes: 9 additions & 8 deletions tools/imagetool.h
Original file line number Diff line number Diff line change
Expand Up @@ -110,14 +110,15 @@ struct image_type_params {
void (*set_header) (void *, struct stat *, int,
struct image_tool_params *);
/*
* This function is used by the command to retrieve a data file from
* the image (i.e. dumpimage -i <image> -p <position> <data_file>).
* This function is used by the command to retrieve a component
* (sub-image) from the image (i.e. dumpimage -i <image> -p <position>
* <sub-image-name>).
* Thus the code to extract a file from an image must be put here.
*
* Returns 0 if the file was successfully retrieved from the image,
* or a negative value on error.
*/
int (*extract_datafile) (void *, struct image_tool_params *);
int (*extract_subimage)(void *, struct image_tool_params *);
/*
* Some image generation support for ex (default image type) supports
* more than one type_ids, this callback function is used to check
Expand Down Expand Up @@ -169,18 +170,18 @@ int imagetool_verify_print_header(
struct image_tool_params *params);

/**
* imagetool_save_datafile - store data into a file
* imagetool_save_subimage - store data into a file
* @file_name: name of the destination file
* @file_data: data to be written
* @file_len: the amount of data to store
*
* imagetool_save_datafile() store file_len bytes of data pointed by file_data
* imagetool_save_subimage() store file_len bytes of data pointed by file_data
* into the file name by file_name.
*
* returns:
* zero in case of success or a negative value if fail.
*/
int imagetool_save_datafile(
int imagetool_save_subimage(
const char *file_name,
ulong file_data,
ulong file_len);
Expand All @@ -202,7 +203,7 @@ void pbl_load_uboot(int fd, struct image_tool_params *mparams);
_verify_header, \
_print_header, \
_set_header, \
_extract_datafile, \
_extract_subimage, \
_check_image_type, \
_fflag_handle, \
_vrec_header \
Expand All @@ -215,7 +216,7 @@ void pbl_load_uboot(int fd, struct image_tool_params *mparams);
.verify_header = _verify_header, \
.print_header = _print_header, \
.set_header = _set_header, \
.extract_datafile = _extract_datafile, \
.extract_subimage = _extract_subimage, \
.check_image_type = _check_image_type, \
.fflag_handle = _fflag_handle, \
.vrec_header = _vrec_header \
Expand Down

0 comments on commit 67f946c

Please sign in to comment.