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
> > > [email protected] 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 [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
