[U-Boot] [PATCH] cmd: bmp: manage centered display

Anatolij Gustschin agust at denx.de
Mon Dec 3 09:21:26 UTC 2018


Hi Patrick,

On Mon, 3 Dec 2018 08:45:57 +0000
Patrick DELAUNAY patrick.delaunay at st.com wrote:
...
> > > +#define BMP_ALIGN_CENTER	0x7FFF  
> > 
> > this already exists when including splash.h, I dropped it in v2.  
> 
> Ok, I don't see that splash.h is included in cmd/bmp.c.

In mainline U-Boot it is included, please see [1].

 
> In fact I hesitate to move this define in commun bmp include
> (in include/lcd.h with bmp_display prototype for exmaple) 
> to avoid a other double definition:
> 
> common/lcd.c:392 #define BMP_ALIGN_CENTER	0x7FFF
> drivers/video/video_bmp.c:143:#define BMP_ALIGN_CENTER	0x7fff
> include/splash.h:78:#define BMP_ALIGN_CENTER	0x7FFF

I've sent a patch cleaning up multiple defines, please see [2].

[1] http://git.denx.de/?p=u-boot.git;a=blob;f=cmd/bmp.c;h=02bdf48b4d422ced97dea7587f07d74c0aee6d6e;hb=HEAD#l20
[2] http://patchwork.ozlabs.org/patch/1006354

--
Anatolij


More information about the U-Boot mailing list