<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<p><br>
</p>
<div class="moz-cite-prefix">On 12/19/2018 8:51 PM, Daniel Vetter
wrote:<br>
</div>
<blockquote type="cite"
cite="mid:20181219152114.GJ21184@phenom.ffwll.local">
<pre class="moz-quote-pre" wrap="">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 <a class="moz-txt-link-rfc2396E" href="mailto:daniel.vetter@ffwll.ch" moz-do-not-send="true"><daniel.vetter@ffwll.ch></a></pre>
</blockquote>
<pre>Sure. Thank you.</pre>
<blockquote type="cite"
cite="mid:20181219152114.GJ21184@phenom.ffwll.local">
<pre class="moz-quote-pre" wrap="">
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.</pre>
</blockquote>
<pre>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.
-Ram
</pre>
<blockquote type="cite"
cite="mid:20181219152114.GJ21184@phenom.ffwll.local">
<pre class="moz-quote-pre" wrap="">
Cheers, Daniel</pre>
</blockquote>
</body>
</html>