aboutsummaryrefslogtreecommitdiffstats
path: root/sound/soc/s3c24xx
diff options
context:
space:
mode:
authorWerner Almesberger <werner@openmoko.org>2008-04-14 08:26:44 -0400
committerJaroslav Kysela <perex@perex.cz>2008-05-19 11:55:10 -0400
commit33e5b22285f63ede858c00456f3ffbc2ea79d6cf (patch)
tree67ec525d50071e56cbaffdef3a0ea7dfe0b2ef57 /sound/soc/s3c24xx
parenta65f0568f6cc8433877fb71dd7d36b551854b0bc (diff)
[ALSA] soc - Fix s3c24xx-i2s LR sync while timer ticks are disabled
When timer ticks are disabled when calling sound/soc/s3c24xx/s3c24xx-i2s.c:s3c24xx_snd_lrsync and the LR signal never happens, we loop forever. This has been observed in the following call chain: snd_pcm_common_ioctl1 -> snd_pcm_action_lock_irq -> snd_pcm_action_single -> snd_pcm_do_resume -> soc_pcm_trigger -> s3c24xx_i2s_trigger The patch below changes the timeout mechanism to use udelay, which doesn't need timer ticks. Signed-off-by: Werner Almesberger <werner@openmoko.org> Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com> Signed-off-by: Takashi Iwai <tiwai@suse.de> Signed-off-by: Jaroslav Kysela <perex@perex.cz>
Diffstat (limited to 'sound/soc/s3c24xx')
-rw-r--r--sound/soc/s3c24xx/s3c24xx-i2s.c5
1 files changed, 3 insertions, 2 deletions
diff --git a/sound/soc/s3c24xx/s3c24xx-i2s.c b/sound/soc/s3c24xx/s3c24xx-i2s.c
index ddf87246c77b..4c52f7946d9e 100644
--- a/sound/soc/s3c24xx/s3c24xx-i2s.c
+++ b/sound/soc/s3c24xx/s3c24xx-i2s.c
@@ -175,7 +175,7 @@ static void s3c24xx_snd_rxctrl(int on)
175static int s3c24xx_snd_lrsync(void) 175static int s3c24xx_snd_lrsync(void)
176{ 176{
177 u32 iiscon; 177 u32 iiscon;
178 unsigned long timeout = jiffies + msecs_to_jiffies(5); 178 int timeout = 50; /* 5ms */
179 179
180 DBG("Entered %s\n", __func__); 180 DBG("Entered %s\n", __func__);
181 181
@@ -184,8 +184,9 @@ static int s3c24xx_snd_lrsync(void)
184 if (iiscon & S3C2410_IISCON_LRINDEX) 184 if (iiscon & S3C2410_IISCON_LRINDEX)
185 break; 185 break;
186 186
187 if (time_after(jiffies, timeout)) 187 if (!timeout--)
188 return -ETIMEDOUT; 188 return -ETIMEDOUT;
189 udelay(100);
189 } 190 }
190 191
191 return 0; 192 return 0;