ASoC: STI: Fix reader substream pointer set

arnopo · broonie · commit 3c9d3f1bc2de · 2017-03-24T18:44:07.000Z
reader-&gt;substream is used in IRQ handler for error case but is never set.
Set value to pcm substream on DAI startup and clean it on dai shutdown.

Signed-off-by: Arnaud Pouliquen &lt;arnaud.pouliquen@st.com&gt;
Signed-off-by: Mark Brown &lt;broonie@kernel.org&gt;
diff --git a/sound/soc/sti/uniperif_reader.c b/sound/soc/sti/uniperif_reader.c
@@ -349,6 +349,8 @@ static int uni_reader_startup(struct snd_pcm_substream *substream,
 	struct uniperif *reader = priv->dai_data.uni;
 	int ret;
 
+	reader->substream = substream;
+
 	if (!UNIPERIF_TYPE_IS_TDM(reader))
 		return 0;
 
@@ -378,6 +380,7 @@ static void uni_reader_shutdown(struct snd_pcm_substream *substream,
 		/* Stop the reader */
 		uni_reader_stop(reader);
 	}
+	reader->substream = NULL;
 }
 
 static const struct snd_soc_dai_ops uni_reader_dai_ops = {

Original file line number	Diff line number	Diff line change
`@@ -349,6 +349,8 @@ static int uni_reader_startup(struct snd_pcm_substream *substream,`
`349`	`349`	`struct uniperif *reader = priv->dai_data.uni;`
`350`	`350`	`int ret;`
`351`	`351`
	`352`	`+ reader->substream = substream;`
	`353`	`+`
`352`	`354`	`if (!UNIPERIF_TYPE_IS_TDM(reader))`
`353`	`355`	`return 0;`
`354`	`356`
`@@ -378,6 +380,7 @@ static void uni_reader_shutdown(struct snd_pcm_substream *substream,`
`378`	`380`	`/* Stop the reader */`
`379`	`381`	`uni_reader_stop(reader);`
`380`	`382`	`}`
	`383`	`+ reader->substream = NULL;`
`381`	`384`	`}`
`382`	`385`
`383`	`386`	`static const struct snd_soc_dai_ops uni_reader_dai_ops = {`