On Thu, 2019-12-05 at 19:32 +0100, Anatolij Gustschin wrote:
On Thu, 5 Dec 2019 18:15:49 +0100 Anatolij Gustschin agust@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@beckhoff.com Beckhoff Automation GmbH & Co. KG | Managing Director: Dipl. Phys. Hans Beckhoff Registered office: Verl, Germany | Register court: Guetersloh HRA 7075