At Thu, 5 Jul 2007 23:07:44 +0100, Adrian McMenamin wrote: > @@ -402,17 +396,10 @@ > static int snd_aicapcm_pcm_trigger(struct snd_pcm_substream > *substream, int cmd) > { > - struct snd_card_aica *dreamcastcard; > switch (cmd) { > case SNDRV_PCM_TRIGGER_START: > spu_begin_dma(substream); > break; > - case SNDRV_PCM_TRIGGER_STOP: > - dreamcastcard = substream->pcm->private_data; > - if (dreamcastcard->timer.data) > - del_timer(&dreamcastcard->timer); > - aica_chn_halt(); > - break; > default: > return -EINVAL; > } Is this a correct change? Then you'd have no control for stopping the stream. (BTW, your embedded patch was malformed...) thanks, Takashi _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel