[U-Boot] [PATCH] video: add config option to skip framebuffer clear

Tom Rini trini at konsulko.com
Tue Jul 4 17:04:25 UTC 2017


On Tue, Jul 04, 2017 at 05:26:05PM +0200, Wolfgang Denk wrote:
> Dear Rob,
> 
> In message <20170704131739.4399-1-robdclark at gmail.com> you wrote:
> >
> > +#ifndef CONFIG_SYS_NO_FB_CLEAR
> >  	video_clear();
> > +#endif
> 
> This should be CONFIG_NO_FB_CLEAR.  CONFIG_SYS_ should only be used
> for settings that are not optional, i. e. that are mandatory due to
> hardware properties or such.  In this case it is a decision of the
> user whether he wants to clear the FB or not, thus no _SYS.

Also, we want to avoid #ifndef CONFIG_NO..., so please re-work the logic
(and make the new option default y).  Thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170704/d1593c48/attachment.sig>


More information about the U-Boot mailing list