[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