[alsa-devel] [PATCH 1/2] opti93x: add support for Opti93x codec in cs4231-lib
rene.herman at keyaccess.nl
Tue Jun 10 01:21:43 CEST 2008
On 09-06-08 23:07, Krzysztof Helt wrote:
> From: Krzysztof Helt <krzysztof.h1 at wp.pl>
> This patch adds support for WSS compatible Opti93x
> codec to the cs4231-lib.
> Signed-off-by: Krzysztof Helt <krzysztof.h1 at wp.pl>
> This patch was sent some time ago but was not accepted
> due to regression of the driver on the opti 930 chip.
> The regression is fixed with already accepted patch, so
> it should work now.
Could you elaborate a bit? Yesterday's patch removing the QS1000 from
82C930 fixed the 930 staying silent but the regression this introduced
previously at least appeared to be a different one.
Post QS1000, 82C930 appeared to work fine, just didn't actually produce
a signal onto its analog out. Post CS4231, 82C930 wasn't just silent but
didn't even generate any interrupts anymore.
I didn't compare with your old code...
> The patch was prepared against the 2.6.26-rc5 kernel. Tested on Opti
> 930, 931 and 933.
Tested-by: Rene Herman <rene.herman at gmail.com>
also on 930, 931 and 933 for playback, capture on 930 and 931. So
whatever the answer to the above, it appears that problem is now also
fixed. Which is very good; this is a rather welcome unification/cleanup
as far as I'm concerned.
The mixer levels do seem different between the old and new driver. Does
that make sense? The new one appears to be a bit louder so there might
be an offset issue somewhere I suppose. I didn't compare that closely.
More information about the Alsa-devel