From mboxrd@z Thu Jan 1 00:00:00 1970 From: Anatolij Gustschin Date: Thu, 5 Dec 2019 19:32:45 +0100 Subject: [PATCH 2/2] video: make BPP and ANSI configs optional In-Reply-To: <20191205171549.15170-2-agust@denx.de> References: <20191205171549.15170-1-agust@denx.de> <20191205171549.15170-2-agust@denx.de> Message-ID: <20191205193245.6e0eadd1@crub> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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