4 Jul
2017
4 Jul
'17
7:04 p.m.
On Tue, Jul 04, 2017 at 05:26:05PM +0200, Wolfgang Denk wrote:
Dear Rob,
In message 20170704131739.4399-1-robdclark@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