lkml.org 
[lkml]   [2007]   [Jul]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [alsa-devel] [PATCH] ALSA AICA sound on SEGA Dreamcast - fix behaviour in poor resource conditions
On 06/07/07, Takashi Iwai <tiwai@suse.de> wrote:
> 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.
>

The shutdown is all done in the close() now, partly because
snd_aicapcm_pcm_trigger cannot sleep and close() can. Is there a
pressing reason to put code in snd_aicapcm_pcm_trigger?
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2007-07-06 11:49    [W:0.107 / U:0.200 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site