From: Gerald Teschl <gerald@esi.ac.at>
To: linux-sound@vger.kernel.org
Subject: [PATCH] opl3sa2 isapnp activation fix
Date: Thu, 06 Jun 2002 18:44:06 +0000 [thread overview]
Message-ID: <marc-linux-sound-102338984907534@msgid-missing> (raw)
In-Reply-To: <marc-linux-sound-102320707510070@msgid-missing>
Now here is the new version of the opl3sa2 activation fix. This new
version now adds the fix as a quirk to isapnp. Zwane Mwaikambo and
I have discussed this very carefully and we now both agree that this
is the best way to fix the problem. The patch is against 2.4.19-pre10
and requires the previous isapnp_dma0.patch.
It now loops over all acceptable dma_resources and only allows dma0
if dma channel 0 is the ONLY value accepted by the card. In
addition, it also fixes the problem that a card would not get
deactivated upon removal of the module.
Gerald
opl3sa2_dma0.patch:
-------------------------
--- linux.orig/drivers/pnp/quirks.c Thu Jun 6 18:04:43 2002
+++ linux/drivers/pnp/quirks.c Thu Jun 6 18:07:25 2002
@@ -102,6 +102,28 @@
return;
}
+extern int isapnp_allow_dma0;
+static void __init quirk_opl3sax_resources(struct pci_dev *dev)
+{
+ /* This really isn't a device quirk but isapnp core code
+ * doesn't allow a DMA channel of 0, afflicted card is an
+ * OPL3Sax where x=4.
+ */
+ struct isapnp_resources *res;
+ int max;
+ res = (struct isapnp_resources *)dev->sysdata;
+ max = res->dma->map;
+ for (res = res->alt; res; res = res->alt) {
+ if (res->dma->map > max)
+ max = res->dma->map;
+ }
+ if (max = 1 && isapnp_allow_dma0 = -1) {
+ printk(KERN_INFO "isapnp: opl3sa4 quirk: Allowing dma 0.\n");
+ isapnp_allow_dma0 = 1;
+ }
+ return;
+}
+
/*
* ISAPnP Quirks
* Cards or devices that need some tweaking due to broken hardware
@@ -133,6 +155,8 @@
quirk_sb16audio_resources },
{ ISAPNP_VENDOR('C','T','L'), ISAPNP_DEVICE(0x0045),
quirk_sb16audio_resources },
+ { ISAPNP_VENDOR('Y','M','H'), ISAPNP_DEVICE(0x0021),
+ quirk_opl3sax_resources },
{ 0 }
};
--- linux.orig/drivers/sound/opl3sa2.c Thu Jun 6 18:04:44 2002
+++ linux/drivers/sound/opl3sa2.c Thu Jun 6 18:07:25 2002
@@ -57,6 +57,7 @@
* (Jan 7, 2001)
* Zwane Mwaikambo Added PM support. (Dec 4 2001)
* Zwane Mwaikambo Code, data structure cleanups. (Feb 15 2002)
+ * Gerald Teschl ISA PnP activate fix. (Jun 02 2002)
*
*/
@@ -873,10 +874,11 @@
}
else {
if(dev->activate(dev) < 0) {
- printk(KERN_WARNING PFX "ISA PnP activate failed\n");
+ printk(KERN_WARNING PFX "ISA PnP activate failed.\n");
opl3sa2_state[card].activated = 0;
return -ENODEV;
}
+ opl3sa2_state[card].activated = 1;
printk(KERN_DEBUG
PFX "Activated ISA PnP card %d (active=%d)\n",
next prev parent reply other threads:[~2002-06-06 18:44 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-06-04 15:42 [PATCH] opl3sa2 isapnp activation fix Zwane Mwaikambo
2002-06-04 15:48 ` Zwane Mwaikambo
2002-06-04 15:57 ` Gerald Teschl
2002-06-04 17:33 ` Zwane Mwaikambo
2002-06-04 17:34 ` Gerald Teschl
2002-06-05 15:05 ` Gerald Teschl
2002-06-06 18:44 ` Gerald Teschl [this message]
-- strict thread matches above, loose matches on Subject: below --
2002-06-04 15:57 Gerald Teschl
2002-06-04 15:42 ` Zwane Mwaikambo
2002-06-04 15:55 ` Zwane Mwaikambo
2002-06-04 17:34 ` Gerald Teschl
2002-06-04 17:33 ` Zwane Mwaikambo
2002-06-04 15:48 ` Zwane Mwaikambo
[not found] <Pine.LNX.4.44.0206051414200.26634-100000@netfinity.realnet.co.sz>
2002-06-05 15:05 ` Gerald Teschl
2002-06-06 18:44 Gerald Teschl
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=marc-linux-sound-102338984907534@msgid-missing \
--to=gerald@esi.ac.at \
--cc=linux-sound@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.