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 _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
