[alsa-devel] [PATCH] STAC92xx : add support for missing features
tiwai at suse.de
Tue Aug 28 12:52:06 CEST 2007
At Tue, 28 Aug 2007 12:45:21 +0200,
> At Tue, 28 Aug 2007 04:54:29 +0300,
> Maxim Levitsky wrote:
> > Hi,
> > As promised, those are patches that extend patch_sigmatel.c driver to add
> > missing functionality
> Thanks for the patches!
> > They all are runtime tested with my STAC9227, and datasheet tested with others
> > Note that I did my best to ensure that even if a codec doesn't support a
> > feature (and I don't know sbout that), maximum impact will be a new,
> > non-working mixer control
> > STAC927x are almost exactly the same as STAC922x, but have DAC4
> > 1) Center/LFE swap :
> > It can be sometimes useful to swap those channels, since they are on same
> > line.
> > I add that control only in case Center/LFE DAC is there, and supports L/R
> > swap, it should be very safe.
> > 2) Analog loopback:
> > Added for 922x, 927x, and for 9204/5 which has a different verb, that offers
> > same analog loopback
> > 3) Volume Knob:
> > Added for 9204/5, 9220/1/3, 922x 92-7x
> > I made it a master volume
> > 4) Fixed support for codecs that have 2 or 3 ADC:
> > I added separate mixer controls for all 2/3 ADCs, and made PCM capture device
> > sub-streams count to depend on actual number of ADCs
> > I am looking for your comments,
> > Best regards,
> > Maxim Levitsky
> All patches except 3 look almost good to me. I'll give review back to
> each post. For the next version, could you give the changelog text
> for each patch (not only the subject line) together with your
> sign-off? Then I can merge to HG tree.
> About 3 (volume-knob), we'd need a bit more checks and tests. Does
> the volume knob widget work always as a mater control regardless
> whetehr the real hardware component exists or not? I.e. can we use it
> Tobin, Matt, could you test these patches, especially patch 3, with
> your hardwares? Anyone who has STAC92xx hardwares are welcome for
> testing it, too!
Just to be sure: the posted patch number seems confusing.
The "patch 4/5" is the volume-knob one although it's listed as 3 in
More information about the Alsa-devel