Date: Mon, 07 Nov 2005 16:54:54 +0100
Message-ID: <s5h4q6opi5t.wl%tiwai@suse.de>
From: Takashi Iwai <tiwai@suse.de>
To: Lee Revell <rlrevell@joe-job.com>
Cc: Mauro Carvalho Chehab <mchehab@brturbo.com.br>,
       alsa-devel@lists.sourceforge.net, Andrew Morton <akpm@osdl.org>,
       linux-kernel@vger.kernel.org, nshmyrev@yandex.ru, v4l@cerqueira.org
Subject: Re: [Alsa-devel] Re: + v4l-720-alsa-support-for-saa7134-that-should-work-fix.patch added to -mm tree
In-Reply-To: <1131377615.8383.9.camel@mindpipe>
References: <200511060743.jA67hpZa018948@shell0.pdx.osdl.net>
	<20051106001249.48d3ade0.akpm@osdl.org>
	<1131301995.13599.5.camel@mindpipe>
	<1131344803.10094.8.camel@localhost>
	<1131377615.8383.9.camel@mindpipe>
User-Agent: Wanderlust/2.12.0 (Your Wildest Dreams) SEMI/1.14.6 (Maruoka)
 FLIM/1.14.7 (=?ISO-8859-4?Q?Sanj=F2?=) APEL/10.6 MULE XEmacs/21.5 (beta18)
 (chestnut) (+CVS-20041021) (i386-suse-linux)
MIME-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka")
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 8bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1789
Lines: 52

At Mon, 07 Nov 2005 10:33:35 -0500,
Lee Revell wrote:
> 
> On Mon, 2005-11-07 at 04:26 -0200, Mauro Carvalho Chehab wrote:
> > Lee,
> > 
> > Em Dom, 2005-11-06 ?s 13:33 -0500, Lee Revell escreveu:
> > > On Sun, 2005-11-06 at 00:12 -0800, Andrew Morton wrote:
> > > > Well that didn't work.  The problem is that
> > > > drivers/media/video/saa7134/saa7134-alsa.c doesn't appear to be wired
> > > > up into the build system - it simply doesn't get compiled.
> > > > 
> > > > Please send a fix against next -mm? 
> > > 
> > > Also please send all ALSA related patches to
> > > alsa-devel@lists.sourceforge.net for review.
> > 
> > 	I'm sending you enclosed saa7134-alsa patch. To make easier to
> > understand, I've merged all stuff. This is highly dependent of the other
> > saa7134 parts, since PCI stuff are common to both video and audio
> > funcion on this device.
> > 	This is meant to replace saa7134-oss (after more tests) that,
> > currently, is part of saa7134 module.
> 
> OK, a brief review:
> 
>  - Why couldn't you use ALSA's DMA API?

This is OK, IMO.  Basically, it's up to the driver.


>  - The DMA must be stopped and started in the trigger callback, not the
> prepare callback.
> 
>  - If this device lacks a volume control alsa-lib can emulate it in
> software, just create a proper /usr/share/alsa/cards/your_card.conf
> file.
> 
>  - By ALSA convention the acceptable formats, sample rates, etc should
> be directly defined in the snd_pcm_hardware_t structure.
> 
>  - dev->oss needs to go.

Agreed about the rest.


Takashi
-
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/