[PATCH] video: add opt out option for VIDEO_BBP selection

Tom Rini trini at konsulko.com
Tue Feb 4 19:08:17 CET 2020


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 at 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!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200204/2b6be4f7/attachment.sig>


More information about the U-Boot mailing list