On Tue, Feb 04, 2020 at 06:19:39PM +0100, Anatolij Gustschin wrote:
> On Tue, 4 Feb 2020 11:49:59 -0500
> Tom Rini [email protected] 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!

-- 
Tom

Attachment: signature.asc
Description: PGP signature

Reply via email to