Skip to content

Commit

Permalink
V4L/DVB (9742): em28xx-alsa: implement another locking schema
Browse files Browse the repository at this point in the history
Instead of using a spinlock, it is better to call the proper pcm stream
locking schema.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
  • Loading branch information
Mauro Carvalho Chehab committed Nov 24, 2008
1 parent 7a8f4cc commit 50f3beb
Showing 1 changed file with 24 additions and 13 deletions.
37 changes: 24 additions & 13 deletions drivers/media/video/em28xx/em28xx-audio.c
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,9 @@ static void em28xx_audio_isocirq(struct urb *urb)
struct em28xx *dev = urb->context;
int i;
unsigned int oldptr;
#ifdef NO_PCM_LOCK
unsigned long flags;
#endif
int period_elapsed = 0;
int status;
unsigned char *cp;
Expand All @@ -96,9 +98,26 @@ static void em28xx_audio_isocirq(struct urb *urb)
if (!length)
continue;

#ifdef NO_PCM_LOCK
spin_lock_irqsave(&dev->adev->slock, flags);

#endif
oldptr = dev->adev->hwptr_done_capture;
if (oldptr + length >= runtime->buffer_size) {
unsigned int cnt =
runtime->buffer_size - oldptr;
memcpy(runtime->dma_area + oldptr * stride, cp,
cnt * stride);
memcpy(runtime->dma_area, cp + cnt * stride,
length * stride - cnt * stride);
} else {
memcpy(runtime->dma_area + oldptr * stride, cp,
length * stride);
}

#ifndef NO_PCM_LOCK
snd_pcm_stream_lock(substream);
#endif

dev->adev->hwptr_done_capture += length;
if (dev->adev->hwptr_done_capture >=
runtime->buffer_size)
Expand All @@ -113,19 +132,11 @@ static void em28xx_audio_isocirq(struct urb *urb)
period_elapsed = 1;
}

#ifdef NO_PCM_LOCK
spin_unlock_irqrestore(&dev->adev->slock, flags);

if (oldptr + length >= runtime->buffer_size) {
unsigned int cnt =
runtime->buffer_size - oldptr;
memcpy(runtime->dma_area + oldptr * stride, cp,
cnt * stride);
memcpy(runtime->dma_area, cp + cnt * stride,
length * stride - cnt * stride);
} else {
memcpy(runtime->dma_area + oldptr * stride, cp,
length * stride);
}
#else
snd_pcm_stream_unlock(substream);
#endif
}
if (period_elapsed)
snd_pcm_period_elapsed(substream);
Expand Down

0 comments on commit 50f3beb

Please sign in to comment.