lkml.org 
[lkml]   [2005]   [Nov]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [Alsa-devel] Re: + v4l-720-alsa-support-for-saa7134-that-should-work-fix.patch added to -mm tree
From
Date
Andrew,

Em Seg, 2005-11-07 às 11:48 -0800, Andrew Morton escreveu:
> Takashi Iwai <tiwai@suse.de> wrote:
> >
> > > 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.
>
> Thanks, guys.
>
> Despite all that, I am inclined to merge this patch into 2.6.15. Because
> it's in the middle of a 192-patch series and we do need to get the v4l tree
> synced up.
>
> Mauro, if we do that, do you think the above points can be addressed
> inside the next four weeks or so?
We are, in fact already working on these. We are about to send you a
newer patch addressing most of the points that Takashi and Lee showed.
You may apply the patches. If they noticed some weird stuff at the
newer patchset, we will have enough time for fix it.
>
> -
> 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/
>
Cheers,
Mauro.

-
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: 2005-11-07 21:38    [from the cache]
©2003-2014 Jasper Spaans. Advertise on this site