On Thu, Dec 20, 2018 at 06:48:08PM +0530, C, Ramalingam wrote: > > On 12/19/2018 8:51 PM, Daniel Vetter wrote: > > Indeed, I overlooked that. Maybe highlight it a bit more with a separate > > > > if (!CONFIG_ENABLED(MEI_HDCP)) > > return false; > > > > so it stick out more in the previous patch. Currently it's a bit burried. > > > > With that + the init ordering fixed: > > > > Reviewed-by: Daniel Vetter<daniel.vetter@xxxxxxxx> > > Sure. Thank you. > > > > > There's no need to split the patch out anymore, since without the mei > > patches you can't enable that Kconfig option and hence no bisect issues. > > Still Kconfig and makefile is added at 22nd patch but component support in mei_hdcp is added at 35th patch. > So even now this chunk needs to be kept after the component support addition. IS_ENABLED(CONFIG_FOO) will just always evaluate to false if the Kconfig symbol isn't defined yet. Kconfig works by injection additional #define contstants into the build. So current ordering is safe. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
