[pulseaudio-discuss] [PATCH v2] alsa: Add support for sound cards with 4-channel input.

Arun Raghavan arun.raghavan at collabora.co.uk
Mon May 7 09:50:29 PDT 2012


On Mon, 2012-05-07 at 13:47 +0300, Tanu Kaskinen wrote:
> On Mon, 2012-05-07 at 09:46 +0530, Arun Raghavan wrote:
> > On Tue, 2012-05-01 at 20:09 +0300, Tanu Kaskinen wrote:
> > > Changes in v2:
> > >  - Call the mapping a generic 4-channel input mapping
> > >    instead of a 4-channel mic array mapping. The mapping
> > >    will be used also by sound cards that have two stereo
> > >    input jacks, so in those cases talking about mic arrays
> > >    is wrong.
> > >  - Added a comment about using the "hw" device name.
> > > 
> > > BugLink: https://bugs.freedesktop.org/show_bug.cgi?id=45813
> > > ---
> > >  src/modules/alsa/alsa-mixer.c                    |    1 +
> > >  src/modules/alsa/mixer/profile-sets/default.conf |    9 +++++++++
> > >  2 files changed, 10 insertions(+)
> > > 
> > > diff --git a/src/modules/alsa/alsa-mixer.c b/src/modules/alsa/alsa-mixer.c
> > > index abb12ee..8b54f75 100644
> > > --- a/src/modules/alsa/alsa-mixer.c
> > > +++ b/src/modules/alsa/alsa-mixer.c
> > > @@ -3888,6 +3888,7 @@ static int mapping_verify(pa_alsa_mapping *m, const pa_channel_map *bonus) {
> > >          { "analog-surround-61",     N_("Analog Surround 6.1") },
> > >          { "analog-surround-70",     N_("Analog Surround 7.0") },
> > >          { "analog-surround-71",     N_("Analog Surround 7.1") },
> > > +        { "analog-4-channel-input", N_("Analog 4-channel Input") },
> > 
> > Since this is limited to some specific hardware, shouldn't we set up
> > udev rules instead of always probing?
> 
> "All devices supporting 4-channel input" isn't very specific hardware
> IMHO, and I doubt udev can be used to query the amount of supported
> channels. But if you think that the new mapping really causes too much
> overhead for the common case, I can live with udev rules too (I think
> caching the probe results would be a better solution for the performance
> problems, but that's not available right now).

I was under the impression that it was primarily the PS3 Eye and Kinect
folks complaining about it. If 4 mic arrays are more common than just
that, go ahead and push it. :)

-- Arun



More information about the pulseaudio-discuss mailing list