On 11/21/17 11:10 AM, Takashi Iwai wrote:
On Sat, 18 Nov 2017 10:25:28 +0100,
Takashi Iwai wrote:
On Sat, 18 Nov 2017 01:01:55 +0100,
Pierre-Louis Bossart wrote:
At the risk of being scolded for the third time in two days by
Linux overlords (no hard feelings), here's an attempt to clean
things up.
The first patch *should* implement what Linus, Takashi and Mark
tried to explain by email. There should be no functionality change
and could be merged if deemed ok.
The rest of the patch series does a more in-depth cleanup and should not
be merged without more testing (hence the RFC).
The 4th patch is really the most important one, there were nested
configs which made no sense to me. I don't know the history which led
to such complicated stuff but simpler is better.
The last 3 patches are just clean-ups of the machine driver configs,
for some reason there is no consistency in the settings so I tried to
apply common sense. There might be additional cleanup needed since I
don't really get why we need references to LPSS or DESIGNWARE for things
which are not visible to a machine driver, we should only depend on IC2 or
SPI in my opinion - depending on what the control interface is.
I tried to keep things to a minimum in each patch to make the reviews
easier, if people want them squashed that's fine by me.
I'll do some more testing on my side but I could use feedback. Thanks!
FYI, I've put these to a test branch, test/asoc-intel-kconfig, so that
0day bot can catch issues. Let's see.
No news is a good news, it seems that your patchset doesn't break
builds, at least. Let's go ahead!
There is additional testing being done at Intel on Skylake platforms,
let's see after the Turkey break if additional fixes are needed. I also
need to spend a bit more time on PCI platforms and SOF - those two were
the main source of comments.
Thanks for the feedback and comments everyone, much appreciated.