ALSA: hda - Minor refactoring
authorTakashi Iwai <tiwai@suse.de>
Thu, 16 Apr 2015 05:53:08 +0000 (07:53 +0200)
committerTakashi Iwai <tiwai@suse.de>
Thu, 16 Apr 2015 06:47:28 +0000 (08:47 +0200)
Move the small portion of the common sequence in hda_intel.c and
hda_tegra.c into hda_controller.c.

Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/pci/hda/hda_controller.c
sound/pci/hda/hda_intel.c
sound/pci/hda/hda_tegra.c

index 21058b4..aaf01e8 100644 (file)
@@ -107,18 +107,22 @@ static int azx_pcm_hw_params(struct snd_pcm_substream *substream,
 {
        struct azx_pcm *apcm = snd_pcm_substream_chip(substream);
        struct azx *chip = apcm->chip;
+       struct azx_dev *azx_dev = get_azx_dev(substream);
        int ret;
 
-       dsp_lock(get_azx_dev(substream));
-       if (dsp_is_locked(get_azx_dev(substream))) {
+       dsp_lock(azx_dev);
+       if (dsp_is_locked(azx_dev)) {
                ret = -EBUSY;
                goto unlock;
        }
 
+       azx_dev->core.bufsize = 0;
+       azx_dev->core.period_bytes = 0;
+       azx_dev->core.format_val = 0;
        ret = chip->ops->substream_alloc_pages(chip, substream,
                                          params_buffer_bytes(hw_params));
 unlock:
-       dsp_unlock(get_azx_dev(substream));
+       dsp_unlock(azx_dev);
        return ret;
 }
 
index 9dff693..8a14718 100644 (file)
@@ -1743,9 +1743,6 @@ static int substream_alloc_pages(struct azx *chip,
        int ret;
 
        mark_runtime_wc(chip, azx_dev, substream, false);
-       azx_dev->core.bufsize = 0;
-       azx_dev->core.period_bytes = 0;
-       azx_dev->core.format_val = 0;
        ret = snd_pcm_lib_malloc_pages(substream, size);
        if (ret < 0)
                return ret;
index 397e182..d534980 100644 (file)
@@ -104,9 +104,6 @@ static int substream_alloc_pages(struct azx *chip,
 {
        struct azx_dev *azx_dev = get_azx_dev(substream);
 
-       azx_dev->core.bufsize = 0;
-       azx_dev->core.period_bytes = 0;
-       azx_dev->core.format_val = 0;
        return snd_pcm_lib_malloc_pages(substream, size);
 }