16 Dec
2017
16 Dec
'17
7:10 a.m.
Hi,
I am outside so can't use mutt. Sorry for that.
This comment is going to be replaced in the 2nd set anyway because the whole function will be replaced.
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.
Thank you for your effort Nicolin
On Dec 15, 2017 20:43, "Timur Tabi" timur@tabi.org wrote:
On 12/13/17 5:18 PM, Nicolin Chen wrote:
* We are running on a SoC which does not support online SSI* reconfiguration, so we have to enable all necessary flags atonce
* even if we do not use them later (capture and playbackconfiguration)
* Online configuration is not supported* Enable or Disable all necessary bits at once
This is an example of a bad change, IMHO. The original was written in elegant prose. The new version is just two short sentences.