From mboxrd@z Thu Jan 1 00:00:00 1970 From: Eugen.Hristev at microchip.com Date: Fri, 6 Dec 2019 13:44:17 +0000 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: <67e62590-06c8-7e55-075c-9e49af99ba6c@microchip.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 05.12.2019 19:15, Anatolij Gustschin 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. > > Signed-off-by: Anatolij Gustschin > --- Tested both patches on sama5d2_xplained board: Tested-by: Eugen Hristev