[U-Boot] [PATCH] dm: video: make ANSI escape sequence support optional

Rob Clark robdclark at gmail.com
Fri Oct 6 16:56:00 UTC 2017


On Fri, Oct 6, 2017 at 12:15 PM, Anatolij Gustschin <agust at denx.de> wrote:
> On Sat, 30 Sep 2017 10:19:17 +0200
> Anatolij Gustschin agust at denx.de wrote:
>
>> As mentioned in review comments for ANSI escape sequence
>> support patches, this should be optional to reduce code
>> size. Disable escape sequence support when CONFIG_VIDEO_ANSI
>> is not enabled.
>>
>> Signed-off-by: Anatolij Gustschin <agust at denx.de>
>> ---
>> This patch is based on basic ANSI escape seq. support series:
>> https://www.mail-archive.com/u-boot@lists.denx.de/msg263777.html
>>
>>  drivers/video/vidconsole-uclass.c | 6 ++++++
>>  include/video_console.h           | 4 ++++
>>  2 files changed, 10 insertions(+)
>
> Applied to u-boot-video/master.
>

Like I mentioned before, we shouldn't really need this patch..  It
only makes a trivial difference in size (which you could probably get
back by just adding #ifdef VIDEO_ANSI / #endif around the single case
statement in vidconsole_put_char())

   text  data  bss   dec  hex
   1937   224    0  2161  871  VIDEO_ANSI disabled with this patch
   2002   224    0  2226  8b2  VIDEO_ANSI disabled without this patch
   2698   224    0  2922  b6a  VIDEO_ANSI enabled

And it makes the code a lot uglier and removes at least compile-time
testing when VIDEO_ANSI is disabled.

So I think you should drop/revert this patch

BR,
-R


More information about the U-Boot mailing list