[U-Boot] [PATCH 07/11] video: Encapsulate font in video_font_data.h

Marek Vasut marex at denx.de
Sun Aug 11 19:29:46 CEST 2013


Dear Anatolij Gustschin,

> Hi,
> 
> On Tue, 30 Jul 2013 23:37:57 +0200
> 
> Marek Vasut <marex at denx.de> wrote:
> > This patch moves all the font configuration values into video_font_data.h
> > so they are all in the right place with the font. The video_font.h now
> > only includes video_font_data.h and will allow us to select and include
> > different font once more fonts are added.
> 
> this patch adds a warning:
> 
> Configuring for mcc200 board...
>    text	   data	    bss	    dec	    hex	filename
>  202480	  19924	 286272	 508676	  7c304	./u-boot
> /home/ag/u-boot/u-boot-video/include/video_font_data.h:16:22: warning:
> 'video_fontdata' defined but not used [-Wunused-variable]
> 
> and commit d3983ee85 did this video_font.h split. Any idea how we
> should handle it?

Dang. What about adding __maybe_ununsed for the video_fontdata ?

Best regards,
Marek Vasut


More information about the U-Boot mailing list