From: Klaus Reimer <k@ailis.de>
To: Bill Nottingham <notting@redhat.com>, linux-kernel@vger.kernel.org
Subject: Re: opl3sa2 in 2.4.2 on Toshiba Tecra 8000
Date: Thu, 29 Mar 2001 21:57:12 +0200 [thread overview]
Message-ID: <01032921571206.00483@neo> (raw)
In-Reply-To: <01032910124007.00454@neo> <20010329133927.A9950@devserv.devel.redhat.com> <01032921132002.00483@neo>
In-Reply-To: <01032921132002.00483@neo>
Hi,
> > Hm, OK, then never mind. :) I don't have an opl3sa2 here to test
> > how well the current driver works.
> I have the feeling that there is going something wrong with the parameters.
> I modified the opl3sa2 driver and manually set the hw_config->io_base
> variable to 0x538 and now THIS part of the sound card initialization is
> working. But now it says that there is an I/O conflict with MSS. Maybe this
> parameter is also 0x0 and not 0x530 as I specified with the mss_io
> parameter... I will investigate further...
Hm... I found this somewhere near line 920 in opl3sa2.c. I am not a kernel
hacker and this is first time I took a look into a kernel source code but
this first "if" statement is not looking right to me. The initialization of
the cfg[card] struct is what I need to be executed, but it is never executed
because the variable "io" is never -1. I have removed the io == -1 condition
from the first if-statement and now the driver is working. But it's still not
the same quality as the one in kernel 2.2.17: I have no access to the mixer
settings "bass" and "treble". But better than nothing and better than the
8Bit-Soundblaster emulation. I hope this is working better in the next
release.
if(!isapnp && io == -1 ) {
if(io == -1 || irq == -1 || dma == -1 ||
dma2 == -1 || mss_io == -1) {
printk(KERN_ERR
"opl3sa2: io, mss_io, irq, dma, [...]
return -EINVAL;
}
cfg[card].io_base = io;
cfg[card].irq = 0;
cfg[card].dma = -1;
cfg[card].dma2 = -1;
[....]
--
Bye, K
[a735 47ec d87b 1f15 c1e9 53d3 aa03 6173 a723 e391]
(Finger k@ailis.de to get public key)
next prev parent reply other threads:[~2001-03-29 20:01 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2001-03-29 8:12 opl3sa2 in 2.4.2 on Toshiba Tecra 8000 Klaus Reimer
2001-03-29 11:57 ` Jens Taprogge
2001-03-29 15:47 ` Bill Nottingham
2001-03-29 16:19 ` Klaus Reimer
2001-03-29 16:25 ` Bill Nottingham
2001-03-29 18:20 ` Klaus Reimer
2001-03-29 18:39 ` Bill Nottingham
2001-03-29 19:13 ` Klaus Reimer
2001-03-29 19:57 ` Klaus Reimer [this message]
-- strict thread matches above, loose matches on Subject: below --
2001-03-30 10:39 arobertson
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=01032921571206.00483@neo \
--to=k@ailis.de \
--cc=linux-kernel@vger.kernel.org \
--cc=notting@redhat.com \
/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.