ALSA: mips: Use SNDRV_PCM_INFO_SYNC_APPLPTR info flag
authorTakashi Iwai <tiwai@suse.de>
Sun, 2 Sep 2018 08:22:37 +0000 (10:22 +0200)
committerTakashi Iwai <tiwai@suse.de>
Tue, 4 Sep 2018 10:13:27 +0000 (12:13 +0200)
The recently introduced PCM info flag assures the call of ack ops at
each applptr change, and this is mandatory for the indirect PCM
helpers.

Also, with the proper ack callback, we need no longer prefill at
trigger start.  The relevant code can be killed.

Reviewed-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/mips/hal2.c

index c8904e732aaa5f1dbb365ca66bf742e616c25b13..2bd1cb7489ff3ea8dfb4e3ab8ad89cb9d553f8e5 100644 (file)
@@ -500,7 +500,8 @@ static const struct snd_pcm_hardware hal2_pcm_hw = {
        .info = (SNDRV_PCM_INFO_MMAP |
                 SNDRV_PCM_INFO_MMAP_VALID |
                 SNDRV_PCM_INFO_INTERLEAVED |
-                SNDRV_PCM_INFO_BLOCK_TRANSFER),
+                SNDRV_PCM_INFO_BLOCK_TRANSFER |
+                SNDRV_PCM_INFO_SYNC_APPLPTR),
        .formats =          SNDRV_PCM_FMTBIT_S16_BE,
        .rates =            SNDRV_PCM_RATE_8000_48000,
        .rate_min =         8000,
@@ -577,7 +578,6 @@ static int hal2_playback_trigger(struct snd_pcm_substream *substream, int cmd)
        case SNDRV_PCM_TRIGGER_START:
                hal2->dac.pcm_indirect.hw_io = hal2->dac.buffer_dma;
                hal2->dac.pcm_indirect.hw_data = 0;
-               substream->ops->ack(substream);
                hal2_start_dac(hal2);
                break;
        case SNDRV_PCM_TRIGGER_STOP: