On 4/24/17 4:54 AM, Takashi Iwai wrote:
On Mon, 24 Apr 2017 11:52:44 +0200, Vinod Koul wrote:
On Mon, Apr 24, 2017 at 11:43:47AM +0200, Takashi Iwai wrote:
On Mon, 24 Apr 2017 11:12:14 +0200, Vinod Koul wrote:
On Mon, Apr 24, 2017 at 11:00:45AM +0200, Takashi Iwai wrote:
Removing ignore_suspend makes the PM succeeds. But it hits some other ugly kernel bugs.
Okay have you marked .nonatomic = true for the machine DAIs?
Ah that's it. The patch below seems fixing the PM and the nonatomic problems. I'm not sure about the nonatomic flag for the compress stream, though.
Well we dont have upstream decoders so it wont be used in this case.
Also I fiddled only with FE. Do we need the same flags for BE? The others don't look setting like that, so I left so.
I dont remember if BE needs or not FE should suffice.
OK then I leave it as is.
When I submit the fix, I should put Cc to stable, and wonder which version we assure the nonatomic ops in SST driver. Did the code base support nonatomic ops from the beginning?
can we take this opportunity to align all drivers? The .nonatomic=true is set in all drivers for the BE, except for cht_bsw_max98090_ti.c It's either needed for all or not needed for all...
thanks,
Takashi
thanks,
Takashi
sound/soc/intel/boards/bytcr_rt5640.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
--- a/sound/soc/intel/boards/bytcr_rt5640.c +++ b/sound/soc/intel/boards/bytcr_rt5640.c @@ -646,7 +646,7 @@ static struct snd_soc_dai_link byt_rt564 .codec_dai_name = "snd-soc-dummy-dai", .codec_name = "snd-soc-dummy", .platform_name = "sst-mfld-platform",
.ignore_suspend = 1,
.nonatomic = true,@@ -659,7 +659,6 @@ static struct snd_soc_dai_link byt_rt564 .codec_dai_name = "snd-soc-dummy-dai", .codec_name = "snd-soc-dummy", .platform_name = "sst-mfld-platform",
.ignore_suspend = 1,@@ -672,6 +671,7 @@ static struct snd_soc_dai_link byt_rt564 .codec_dai_name = "snd-soc-dummy-dai", .codec_name = "snd-soc-dummy", .platform_name = "sst-mfld-platform",
.nonatomic = true,-- ~Vinod
Alsa-devel mailing list Alsa-devel@alsa-project.org http://mailman.alsa-project.org/mailman/listinfo/alsa-devel