At Thu, 29 Jan 2009 16:02:13 +0100, Krzysztof Helt wrote:
On Mon, 26 Jan 2009 18:33:15 +0100 Krzysztof Helt krzysztof.h1@poczta.fm wrote:
The problem is more complicated. The cs4236+ cards have two separate PNP bios ids! The problem here is that Linux kernel allows only one id per device. The ids are CS0010* for the CS0000* device and CS0110* for the CS0100* devices. The second device has only control port range (one io range).
I have prepared the patch which I tested on my Dell Latitude CPi laptop and now it properly discovers the CS4237B codec. Stanley, please test it as well.
Takashi,
Please take a look at the patch which unifies the cs4232 and the cs4236+ driver code. It should solve all problems with the Crystal PnP BIOS chip detection. At least, it solves this at my Dell laptop.
It's pending just because I'm waiting for the report from Stanley.
The patch itself looks sane to me. We'll need an additional change to remove snd-cs4236, too, though.
Regarding removal of cs4236 -- it'd be perhaps better to provide a module alias of snd-cs4236 to keep the old system happy. But it's just a minor issue.
Takashi