diff mbox

[v2] ASoC: Intel: Atom: fix regression on compress DAI

Message ID 1454438989-15003-1-git-send-email-pierre-louis.bossart@linux.intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Pierre-Louis Bossart Feb. 2, 2016, 6:49 p.m. UTC
Commit a106804 ("ASoC: compress: Fix compress device direction check")
added a dependency on the compress-cpu-dai channel_min field
which was removed earlier by commit 77095796
("ASoC: Intel: Atom: clean-up compressed DAI definition")
as part of the baytrail cleanups.

The net result was a regression at probe on all Atom platforms
with no sound card created.

Fix by adding explicit initialization for channel_min to 1
for the compress-cpu-dai.

Reported-by: Tobias Mädel <alsa-devel@tbspace.de>
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
---
This should be applied before 4.5 is released

 sound/soc/intel/atom/sst-mfld-platform-pcm.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Vinod Koul Feb. 3, 2016, 2:23 a.m. UTC | #1
On Tue, Feb 02, 2016 at 12:49:49PM -0600, Pierre-Louis Bossart wrote:
> Commit a106804 ("ASoC: compress: Fix compress device direction check")
> added a dependency on the compress-cpu-dai channel_min field
> which was removed earlier by commit 77095796
> ("ASoC: Intel: Atom: clean-up compressed DAI definition")
> as part of the baytrail cleanups.
> 
> The net result was a regression at probe on all Atom platforms
> with no sound card created.
> 
> Fix by adding explicit initialization for channel_min to 1
> for the compress-cpu-dai.
> 
> Reported-by: Tobias Mädel <alsa-devel@tbspace.de>
> Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>

Acked-by: Vinod Koul <vinod.koul@intel.com>
diff mbox

Patch

diff --git a/sound/soc/intel/atom/sst-mfld-platform-pcm.c b/sound/soc/intel/atom/sst-mfld-platform-pcm.c
index 55c33dc..52ed434 100644
--- a/sound/soc/intel/atom/sst-mfld-platform-pcm.c
+++ b/sound/soc/intel/atom/sst-mfld-platform-pcm.c
@@ -528,6 +528,7 @@  static struct snd_soc_dai_driver sst_platform_dai[] = {
 	.ops = &sst_compr_dai_ops,
 	.playback = {
 		.stream_name = "Compress Playback",
+		.channels_min = 1,
 	},
 },
 /* BE CPU  Dais */