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

Jonas Mark (BT-FIR/ENG1) Mark.Jonas at de.bosch.com
Wed Feb 6 14:30:37 UTC 2019


Hi Tomas,

> > I conclude that no upstream board is actually affected. They would only
> be affecte if they would use SPLASH_STORAGE_FIT.
> 
> You are right, indeed upstream boards should be ok. However, some
> other non upstream boards I know of would be affected.
> 
> So my initial question still stands, could existing implementation be kept
> as fallback?
> 
> +	if (!splash_file)
> +		splash_file = location->name;
> 
> location->name has worked fine for those cases, so it would be nice to
> keep that option intact.

I think that the .name field of struct splash_location is actually
intended to hold a human readable name for the storage itself. That the
code so far uses the storage location name synonymously with the splash
screen name is unfortunate. That's why we decided to fix this. I am
convinced that the patch actually makes it easier for others to start
reading splash screens from a FIT image.

Why do I think so? Because the behavior is now consistent to
what splash_load_fs() does. That is, if there is an environment variable
"splashfile" the splash screen name is taken from there. Else, the fixed
default name "splash.bmp" is used.

Regards,
Mark


Building Technologies, Panel Software Fire (BT-FIR/ENG1) 
Bosch Sicherheitssysteme GmbH | Postfach 11 11 | 85626 Grasbrunn | GERMANY | www.boschsecurity.com

Sitz: Stuttgart, Registergericht: Amtsgericht Stuttgart HRB 23118 
Aufsichtsratsvorsitzender: Christian Fischer; Geschäftsführung: Tanja Rückert, Andreas Bartz, Thomas Quante, Bernhard Schuster


More information about the U-Boot mailing list