[media] tw686x: audio: Prevent hw param changes while busy
authorEzequiel Garcia <ezequiel@vanguardiasur.com.ar>
Sat, 4 Jun 2016 23:47:20 +0000 (20:47 -0300)
committerMauro Carvalho Chehab <mchehab@s-opensource.com>
Tue, 28 Jun 2016 10:53:05 +0000 (07:53 -0300)
Audio hw params are shared across all DMA channels,
so if the user changes any of these while any DMA channel is
enabled, it will impact the enabled channels, potentially causing
serious instability issues.

This commit avoids such situation, by preventing any hw param
change (on any DMA channel) if any other DMA audio channel is capturing.

Signed-off-by: Ezequiel Garcia <ezequiel@vanguardiasur.com.ar>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
drivers/media/pci/tw686x/tw686x-audio.c
drivers/media/pci/tw686x/tw686x.h

index 987a2266352544afdca4e4e2c3b42c41e08cde0f..96e444c4917363b67524436081113182d47b0cd7 100644 (file)
@@ -94,10 +94,8 @@ static int tw686x_pcm_hw_free(struct snd_pcm_substream *ss)
 
 /*
  * Audio parameters are global and shared among all
- * capture channels. The driver makes no effort to prevent
- * any modifications. User is free change the audio rate,
- * or period size, thus changing parameters for all capture
- * sub-devices.
+ * capture channels. The driver prevents changes to
+ * the parameters if any audio channel is capturing.
  */
 static const struct snd_pcm_hardware tw686x_capture_hw = {
        .info                   = (SNDRV_PCM_INFO_MMAP |
@@ -154,6 +152,14 @@ static int tw686x_pcm_prepare(struct snd_pcm_substream *ss)
        int i;
 
        spin_lock_irqsave(&dev->lock, flags);
+       /*
+        * Given the audio parameters are global (i.e. shared across
+        * DMA channels), we need to check new params are allowed.
+        */
+       if (((dev->audio_rate != rt->rate) ||
+            (dev->period_size != period_size)) && dev->audio_enabled)
+               goto err_audio_busy;
+
        tw686x_disable_channel(dev, AUDIO_CHANNEL_OFFSET + ac->ch);
        spin_unlock_irqrestore(&dev->lock, flags);
 
@@ -210,6 +216,10 @@ static int tw686x_pcm_prepare(struct snd_pcm_substream *ss)
        spin_unlock_irqrestore(&ac->lock, flags);
 
        return 0;
+
+err_audio_busy:
+       spin_unlock_irqrestore(&dev->lock, flags);
+       return -EBUSY;
 }
 
 static int tw686x_pcm_trigger(struct snd_pcm_substream *ss, int cmd)
@@ -223,6 +233,7 @@ static int tw686x_pcm_trigger(struct snd_pcm_substream *ss, int cmd)
        case SNDRV_PCM_TRIGGER_START:
                if (ac->curr_bufs[0] && ac->curr_bufs[1]) {
                        spin_lock_irqsave(&dev->lock, flags);
+                       dev->audio_enabled = 1;
                        tw686x_enable_channel(dev,
                                AUDIO_CHANNEL_OFFSET + ac->ch);
                        spin_unlock_irqrestore(&dev->lock, flags);
@@ -235,6 +246,7 @@ static int tw686x_pcm_trigger(struct snd_pcm_substream *ss, int cmd)
                break;
        case SNDRV_PCM_TRIGGER_STOP:
                spin_lock_irqsave(&dev->lock, flags);
+               dev->audio_enabled = 0;
                tw686x_disable_channel(dev, AUDIO_CHANNEL_OFFSET + ac->ch);
                spin_unlock_irqrestore(&dev->lock, flags);
 
index 4acb90543174ec37c679be5b9ae169f7f13239c0..35d7bc94f78fb9d2769a5e84aab836972d15e13f 100644 (file)
@@ -141,6 +141,7 @@ struct tw686x_dev {
        /* Per-device audio parameters */
        int audio_rate;
        int period_size;
+       int audio_enabled;
 
        struct timer_list dma_delay_timer;
        u32 pending_dma_en; /* must be protected by lock */