Re: [PATCH 00/10] soundwire: bus: fix race conditions, add suspend-resume

From: Pierre-Louis Bossart
Date: Tue Feb 25 2020 - 10:23:38 EST




On 2/25/20 4:27 AM, Vinod Koul wrote:
On 14-01-20, 18:08, Pierre-Louis Bossart wrote:
The existing mainline code for SoundWire does not handle critical race
conditions, and does not have any support for pm_runtime suspend or
clock-stop modes needed for e.g. jack detection or external VAD.

As suggested by Vinod, these patches for the bus are shared first -
with the risk that they are separated from their actual use in Intel
drivers, so reviewers might wonder why they are needed in the first
place.

For reference, the complete set of 90+ patches required for SoundWire
on Intel platforms is available here:

https://github.com/thesofproject/linux/pull/1692

These patches are not Intel-specific and are likely required for
e.g. Qualcomm-based implementations.

All the patches in this series were generated during the joint
Intel-Realtek validation effort on Intel reference designs and
form-factor devices. The support for the initialization_complete
signaling is already available in the Realtek codecs drivers merged in
the ASoC tree (rt700, rt711, rt1308, rt715)

Applied all, thanks

Thanks Vinod, I'll now prepare the update for the transition away from platform devices (minor update needed on the RFC already reviewed by Greg).