On 09-06-08 23:07, Krzysztof Helt wrote:
From: Krzysztof Helt krzysztof.h1@wp.pl
This patch adds support for WSS compatible Opti93x codec to the cs4231-lib.
Signed-off-by: Krzysztof Helt krzysztof.h1@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@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.
Rene.