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

Hi Simon, On Fri, 20 Dec 2019 18:10:32 -0700 Simon Glass s...@chromium.org wrote: ... > 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

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

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