On Tue, Feb 04, 2020 at 06:19:39PM +0100, Anatolij Gustschin wrote:
On Tue, 4 Feb 2020 11:49:59 -0500 Tom Rini trini@konsulko.com wrote: ...
config VIDEO_BPP32 bool "Support 32-bit-per-pixel displays" depends on DM_VIDEO
- default y if X86
- default y if DM_VIDEO && !VIDEO_BPP_OPT_OUT help Support drawing text and bitmaps onto a 32-bit-per-pixel display. Enabling this will include code to support this display. Without
I think this is more complex than it needs to be, honestly. The old behavior was that all of these options were essentially "default y" but it also wasn't possible to remove them (in some cases?). Now that we can let boards pick what they want, I think we should just go back to the old behavior of in essence "default y" (and they already have depends on DM_VIDEO). Thanks!
you mean we can go with default y if !VIDEO_BPP_OPT_OUT
I mean we don't need VIDEO_BPP_OPT_OUT as a symbol. All of the VIDEO_BPPx options should just be default y and boards can disable as desired. Thanks!