[PATCH 2/2] video: make BPP and ANSI configs optional

Anatolij Gustschin agust at denx.de
Thu Dec 5 19:32:45 CET 2019


On Thu,  5 Dec 2019 18:15:49 +0100
Anatolij Gustschin agust at denx.de wrote:

> Many boards do not use all selected framebuffer depth
> configurations, for such boards there is some unused
> code in video and console uclass routines. Make depth
> specific code optional to avoid dead code and slightly
> reduce binary size. Also make ANSI code optional for
> the same reason. When i.e. using only VIDEO_BPP16 the
> code size shrinks (below values when using gcc-7.3.0):
> 
>   $ ./tools/buildman/buildman -b video-wip -sS wandboard
>   ...
>   01: Merge git://git.denx.de/u-boot-sh
>   02: video: add guards around 16bpp/32bbp code
>   03: video: make BPP and ANSI configs optional
>          arm: (for 1/1 boards) all -776.0 bss -8.0 text -768.0
> 
> Also adjust defconfigs of DM_VIDEO enabled boards to select
> actually used BPP values.

Forgot to CC board maintainers. Maybe you could review or test these
two patches [1],[2] on your board to confirm that these don't break
frame buffer support.

[1] http://patchwork.ozlabs.org/patch/1204707
[2] http://patchwork.ozlabs.org/patch/1204704

Thanks,
Anatolij


More information about the U-Boot mailing list