[U-Boot] [PATCH v3 1/2] splash: Use splashfile instead of location->name

Melin Tomas tomas.melin at vaisala.com
Fri Feb 8 12:21:05 UTC 2019


Hi,

On 2/8/19 11:51 AM, Mark Jonas wrote:
> diff --git a/common/splash_source.c b/common/splash_source.c
> index 62763b9..e1e73db 100644
> --- a/common/splash_source.c
> +++ b/common/splash_source.c
> @@ -303,6 +303,7 @@ static int splash_load_fit(struct splash_location *location, u32 bmp_load_addr)
>   {
>   	int res;
>   	int node_offset;
> +	const char *splash_file;
>   	int splash_offset;
>   	int splash_size;
>   	struct image_header *img_header;
> @@ -335,10 +336,15 @@ static int splash_load_fit(struct splash_location *location, u32 bmp_load_addr)
>   		return -EINVAL;
>   	}
>   
> -	node_offset = fit_image_get_node(fit_header, location->name);
> +	/* Get the splash image node */
> +	splash_file = env_get("splashfile");
> +	if (!splash_file)
> +		splash_file = SPLASH_SOURCE_DEFAULT_FILE_NAME;

As mentioned during latest review round, this default name should really 
IMO be something else than a file name since it's not a file, it's a FIT 
node. splash_file should also probably be splash_node or so.

I'm not even sure is this patch really needed, selecting an appropriate 
name for the splash_location.name, matching the splash image node should 
work equally.

BR,

Tomas




More information about the U-Boot mailing list