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

Steffen Dirkwinkel S.Dirkwinkel at beckhoff.com
Mon Dec 9 15:46:16 CET 2019


On Thu, 2019-12-05 at 19:32 +0100, Anatolij Gustschin wrote:
> 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.
>
>
> Thanks,
> Anatolij

Tested on mx53cx9020
Tested-by: Steffen Dirkwinkel <s.dirkwinkel at beckhoff.com>
Beckhoff Automation GmbH & Co. KG | Managing Director: Dipl. Phys. Hans Beckhoff
Registered office: Verl, Germany | Register court: Guetersloh HRA 7075




More information about the U-Boot mailing list