From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lee Revell Subject: Re: [Patch 1/1] V4L (926) Saa7134 alsa can only be autoloaded after saa7134 is active Date: Wed, 09 Nov 2005 17:59:03 -0500 Message-ID: <1131577144.8383.126.camel@mindpipe> References: <1131397121.6632.127.camel@localhost> <1131451671.2863.4.camel@frolic> <1131484155.4851.10.camel@frolic> Mime-Version: 1.0 Content-Type: text/plain Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1131484155.4851.10.camel@frolic> Sender: linux-kernel-owner@vger.kernel.org To: R C Cc: Takashi Iwai , mchehab@brturbo.com.br, linux-kernel@vger.kernel.org, akpm@osdl.org, video4linux-list@redhat.com, alsa-devel@lists.sourceforge.net, nshmyrev@yandex.ru List-Id: alsa-devel@alsa-project.org On Tue, 2005-11-08 at 21:09 +0000, R C wrote: > On Tue, 2005-11-08 at 14:20 +0100, Takashi Iwai wrote: > > > But saa7134_dma_stop() should be already called in trigger callback, > > which is called via snd_pcm_stop(). > > You're right. I wasn't aware pcm_stop() caused a trigger. > You may want to check the current version at > http://linuxtv.org/cgi-bin/viewcvs.cgi/v4l-kernel/linux/drivers/media/video/saa7134/saa7134-alsa.c?rev=1.21&root=v4l&view=auto Did you not read Takashi-san's ALSA driver guide? If not I'm impressed you made it work at all... http://www.alsa-project.org/~iwai/writing-an-alsa-driver/ Lee