diff options
author | Barry Song <barry.song@analog.com> | 2009-09-23 11:51:04 -0400 |
---|---|---|
committer | Mark Brown <broonie@opensource.wolfsonmicro.com> | 2009-09-23 12:08:25 -0400 |
commit | 766df6d98f9c28dfc6f72c23a010819719e4c3e0 (patch) | |
tree | eda3e965c72c6d417f81e67ece66855a82ae9160 /sound/soc/blackfin | |
parent | e0274b0a300e58cad19ab347566c68762bb22d2c (diff) |
ASoC: Blackfin I2S: use dai state rather than local counter
Since the active field of the dai already tells us the stream activity,
the local counter variable is redundant and can be replaced.
Signed-off-by: Barry Song <barry.song@analog.com>
Signed-off-by: Mike Frysinger <vapier@gentoo.org>
Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Diffstat (limited to 'sound/soc/blackfin')
-rw-r--r-- | sound/soc/blackfin/bf5xx-i2s.c | 15 |
1 files changed, 1 insertions, 14 deletions
diff --git a/sound/soc/blackfin/bf5xx-i2s.c b/sound/soc/blackfin/bf5xx-i2s.c index 1e9d161c76c4..fe2c35ddcbf4 100644 --- a/sound/soc/blackfin/bf5xx-i2s.c +++ b/sound/soc/blackfin/bf5xx-i2s.c | |||
@@ -49,7 +49,6 @@ struct bf5xx_i2s_port { | |||
49 | u16 rcr1; | 49 | u16 rcr1; |
50 | u16 tcr2; | 50 | u16 tcr2; |
51 | u16 rcr2; | 51 | u16 rcr2; |
52 | int counter; | ||
53 | int configured; | 52 | int configured; |
54 | }; | 53 | }; |
55 | 54 | ||
@@ -133,16 +132,6 @@ static int bf5xx_i2s_set_dai_fmt(struct snd_soc_dai *cpu_dai, | |||
133 | return ret; | 132 | return ret; |
134 | } | 133 | } |
135 | 134 | ||
136 | static int bf5xx_i2s_startup(struct snd_pcm_substream *substream, | ||
137 | struct snd_soc_dai *dai) | ||
138 | { | ||
139 | pr_debug("%s enter\n", __func__); | ||
140 | |||
141 | /*this counter is used for counting how many pcm streams are opened*/ | ||
142 | bf5xx_i2s.counter++; | ||
143 | return 0; | ||
144 | } | ||
145 | |||
146 | static int bf5xx_i2s_hw_params(struct snd_pcm_substream *substream, | 135 | static int bf5xx_i2s_hw_params(struct snd_pcm_substream *substream, |
147 | struct snd_pcm_hw_params *params, | 136 | struct snd_pcm_hw_params *params, |
148 | struct snd_soc_dai *dai) | 137 | struct snd_soc_dai *dai) |
@@ -201,9 +190,8 @@ static void bf5xx_i2s_shutdown(struct snd_pcm_substream *substream, | |||
201 | struct snd_soc_dai *dai) | 190 | struct snd_soc_dai *dai) |
202 | { | 191 | { |
203 | pr_debug("%s enter\n", __func__); | 192 | pr_debug("%s enter\n", __func__); |
204 | bf5xx_i2s.counter--; | ||
205 | /* No active stream, SPORT is allowed to be configured again. */ | 193 | /* No active stream, SPORT is allowed to be configured again. */ |
206 | if (!bf5xx_i2s.counter) | 194 | if (!dai->active) |
207 | bf5xx_i2s.configured = 0; | 195 | bf5xx_i2s.configured = 0; |
208 | } | 196 | } |
209 | 197 | ||
@@ -284,7 +272,6 @@ static int bf5xx_i2s_resume(struct snd_soc_dai *dai) | |||
284 | SNDRV_PCM_FMTBIT_S32_LE) | 272 | SNDRV_PCM_FMTBIT_S32_LE) |
285 | 273 | ||
286 | static struct snd_soc_dai_ops bf5xx_i2s_dai_ops = { | 274 | static struct snd_soc_dai_ops bf5xx_i2s_dai_ops = { |
287 | .startup = bf5xx_i2s_startup, | ||
288 | .shutdown = bf5xx_i2s_shutdown, | 275 | .shutdown = bf5xx_i2s_shutdown, |
289 | .hw_params = bf5xx_i2s_hw_params, | 276 | .hw_params = bf5xx_i2s_hw_params, |
290 | .set_fmt = bf5xx_i2s_set_dai_fmt, | 277 | .set_fmt = bf5xx_i2s_set_dai_fmt, |