diff mbox

[v2] ASoC: cs4270: fix DAPM stream name mismatch

Message ID alpine.LNX.2.00.1610091322540.27835@acc-1004367-dev (mailing list archive)
State New, archived
Headers show

Commit Message

murray foster Oct. 9, 2016, 8:28 p.m. UTC 
Mismatching stream names in DAPM route and widget definitions are
causing compilation errors. Fixing these names allows the cs4270
driver to compile and function.

Signed-off-by: Murray Foster <mrafoster@gmail.com>
Acked-by: Paul Handrigan <Paul.Handrigan@cirrus.com>
---
 sound/soc/codecs/cs4270.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

Comments

Mark Brown Oct. 11, 2016, 1:23 p.m. UTC | #1 
On Sun, Oct 09, 2016 at 01:28:45PM -0700, murray foster wrote:

> Mismatching stream names in DAPM route and widget definitions are
> causing compilation errors. Fixing these names allows the cs4270
> driver to compile and function.

This commit log can't be right, we've got extensive build coverage and
there's no way build coverage could spot a mismatch between strings.
I'm assuming this happens at probe time?
diff mbox

Patch

diff --git a/sound/soc/codecs/cs4270.c b/sound/soc/codecs/cs4270.c
index e07807d..3670086 100644
--- a/sound/soc/codecs/cs4270.c
+++ b/sound/soc/codecs/cs4270.c
@@ -148,11 +148,11 @@  SND_SOC_DAPM_OUTPUT("AOUTR"),
 };

 static const struct snd_soc_dapm_route cs4270_dapm_routes[] = {
-	{ "Capture", NULL, "AINA" },
-	{ "Capture", NULL, "AINB" },
+	{ "Capture", NULL, "AINL" },
+	{ "Capture", NULL, "AINR" },

-	{ "AOUTA", NULL, "Playback" },
-	{ "AOUTB", NULL, "Playback" },
+	{ "AOUTL", NULL, "Playback" },
+	{ "AOUTR", NULL, "Playback" },
 };

 /**