[PATCH 0/5] video: Avoid #ifdef to increase build coverage

Simon Glass sjg at chromium.org
Sat Dec 21 02:10:32 CET 2019


A recent change to the video subsystem added more #ifdefs to the code but
these are not necessary. In fact we can often rely on the compiler to drop
dead code. Using the IS_ENABLED() macros with if() can produce the same
result while reducing the number of build combinations in the code.

This series removes most #ifdefs from the core video code. Code size for
wandaboard (for example) does not change.

It also includes a patch to enable 32bpp video on x86, since video is
currently broken.


Simon Glass (5):
  video: x86: Enable 32-bit graphics by default
  video: Avoid using #ifdef in video blitting code
  video: Avoid using #ifdef in console_rotate.c
  video: Avoid using #ifdef in vidconsole-uclass.c
  video: Avoid using #ifdef in video-uclass.c

 drivers/video/Kconfig             |   5 +-
 drivers/video/console_normal.c    | 125 ++++++------
 drivers/video/console_rotate.c    | 325 +++++++++++++++---------------
 drivers/video/vidconsole-uclass.c |  22 +-
 drivers/video/video-uclass.c      |  54 +++--
 5 files changed, 256 insertions(+), 275 deletions(-)

-- 
2.24.1.735.g03f4e72817-goog



More information about the U-Boot mailing list