<div dir="auto"><div dir="auto" style="font-family:sans-serif">Hi,</div><div dir="auto" style="font-family:sans-serif"><br></div><span style="font-family:sans-serif">I am outside so can't use mutt. Sorry for that.</span><div dir="auto" style="font-family:sans-serif"><br></div><div dir="auto" style="font-family:sans-serif">This comment is going to be replaced in the 2nd set anyway because the whole function will be replaced.</div><div dir="auto" style="font-family:sans-serif"><br></div><div dir="auto" style="font-family:sans-serif">And please point out all comments that you think I need to rework. I am totally fine to do that. I don't think every single one is bad. And this patch has to go in as it also adds a lot of new comments.</div><div dir="auto" style="font-family:sans-serif"><br></div><div dir="auto" style="font-family:sans-serif">Thank you for your effort</div><div dir="auto" style="font-family:sans-serif">Nicolin</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Dec 15, 2017 20:43, "Timur Tabi" <<a href="mailto:timur@tabi.org">timur@tabi.org</a>> wrote:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="quoted-text">On 12/13/17 5:18 PM, Nicolin Chen wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
-        * We are running on a SoC which does not support online SSI<br>
-        * reconfiguration, so we have to enable all necessary flags at once<br>
-        * even if we do not use them later (capture and playback configuration)<br>
+        * Online configuration is not supported<br>
+        * Enable or Disable all necessary bits at once<br>
</blockquote>
<br></div>
This is an example of a bad change, IMHO.  The original was written in elegant prose.  The new version is just two short sentences.<br>
</blockquote></div><br></div>