[U-Boot] [RFC PATCH 35/44] mkimage: Put FIT loading in function and tidy error handling

Marek Vasut marex at denx.de
Sat Jan 5 09:24:10 CET 2013


Dear Simon Glass,

> The fit_handle_file() function is quiet long

quite ;-)

> - split out the part that
> loads and checks a FIT into its own function. We will use this
> function for storing public keys into a destination FDT file.
> 
> The error handling is currently a bit repetitive - tidy it.
> 
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
>  tools/fit_image.c |   96
> +++++++++++++++++++++++++++++++--------------------- 1 files changed, 57
> insertions(+), 39 deletions(-)
> 
> diff --git a/tools/fit_image.c b/tools/fit_image.c
> index e0675d7..0f619a2 100644
> --- a/tools/fit_image.c
> +++ b/tools/fit_image.c
> @@ -47,6 +47,48 @@ static int fit_check_image_types (uint8_t type)
>  		return EXIT_FAILURE;
>  }
> 
> +int mmap_fdt(struct mkimage_params *params, const char *fname, void
> **blobp, +		struct stat *sbuf)
> +{
> +	void *ptr;
> +	int fd;
> +
> +	/* load FIT blob into memory */
> +	fd = open(fname, O_RDWR|O_BINARY);

Why is it RDWR even?

otherwise

Reviewed-by: Marek Vasut <marex at denx.de>

Best regards,
Marek Vasut


More information about the U-Boot mailing list