[OSS] Add CXT48 to modem black list in ac97

From: Herbert Xu <herbert@gondor.apana.org.au>
To: sud@latinsud.com (SuD Alex)
Cc: linux-kernel@vger.kernel.org, akpm@osdl.org
Subject: [OSS] Add CXT48 to modem black list in ac97
Date: Tue, 05 Apr 2005 22:32:18 +1000	[thread overview]
Message-ID: <E1DInEA-0006md-00@gondolin.me.apana.org.au> (raw)
In-Reply-To: <42507F12.6070009@latinsud.com>

SuD Alex <sud@latinsud.com> wrote:
>
> Googling i found that  jgarzik already got a patch for this 
> (ac97_codec.c:158):
> +    {0x43585430, "CXT48",            &default_ops,        
> AC97_DELUDED_MODEM },

Yes, until we manage to copy what ALSA does in this case, this
is what we'll have to do.

The following patch works around the misdetection of the CXT48
codec as a modem by the OSS ac97 driver.

Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au> 

We should fix the crash itself too.  Andrew, I'll send you some
a couple of patches on that later.

BTW Alex, if you have the time please determine whether ALSA
works properly on your machine.

Cheers,
-- 
Visit Openswan at http://www.openswan.org/
Email: Herbert Xu ~{PmV>HI~} <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
--
===== sound/oss/ac97_codec.c 1.24 vs edited =====

--- 1.24/sound/oss/ac97_codec.c	2005-03-08 15:41:36 +11:00
+++ edited/sound/oss/ac97_codec.c	2005-04-05 22:27:11 +10:00
@@ -155,6 +155,7 @@
 	{0x43525931, "Cirrus Logic CS4299 rev A", &crystal_digital_ops},
 	{0x43525933, "Cirrus Logic CS4299 rev C", &crystal_digital_ops},
 	{0x43525934, "Cirrus Logic CS4299 rev D", &crystal_digital_ops},
+	{0x43585430, "CXT48",			&default_ops,		AC97_DELUDED_MODEM },
 	{0x43585442, "CXT66",			&default_ops,		AC97_DELUDED_MODEM },
 	{0x44543031, "Diamond Technology DT0893", &default_ops},
 	{0x45838308, "ESS Allegro ES1988",	&null_ops},
