[alsa-devel] [PATCH] STAC92xx : add support for missing features

[Takashi Iwai]

At Tue, 28 Aug 2007 12:45:21 +0200,
I wrote:
> 
> 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
> blindly?
> 
> 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
the above.


Takashi