From: "Clemens Ladisch" <cladisch@fastmail.net>
To: Rene Herman <rene.herman@gmail.com>
Cc: Krzysztof Helt <krzysztof.h1@gmail.com>,
ALSA devel <alsa-devel@alsa-project.org>
Subject: Re: [PATCH] opti931: additional check for OPL3 device
Date: Mon, 17 Sep 2007 12:39:53 +0200 [thread overview]
Message-ID: <1190025593.12774.1211014755@webmail.messagingengine.com> (raw)
In-Reply-To: <46EE5284.9070202@gmail.com>
Rene Herman wrote:
> On 09/17/2007 10:14 AM, Clemens Ladisch wrote:
> > Krzysztof Helt wrote:
> >> Should the OPL4 detection has the same fallback or just all Opti cards
> >> should search the FM chip at the range + 8 address?
> >
> > It appears a fallback shoudn't be needed because the chip always returns
> > a range suitable for an OPL4.
> >
> > Since an OPL3 range is never larger than 8 bytes, and OPL4 needs 16
> > bytes, we could add 8 to get the FM address _if_ the range is at least 16
> > bytes; thus we are safe even if a chip publishes an OPL3-only range.
>
> At least 16 won't do -- the OPTi chip provides a 12-byte range. More than 8
> would work, but just +8 is fine. I have all OPTi ISA-PnP chips (924, 925,
> 931, 933) and more aren't going to be produced. The 0x380 base is what it
> calls the OPL4Base and ALBase (AdLib) is at 0x388.
Are there chips that have both OPL4Base and ALBase?
> The chip can defer to external FM it seems, but even if there are OPTi/OPL4
> combo's out there in the wild (I sort of doubt it)
Why?
> the driver's just doing opl3 anyway. ie, still needs just the +8.
Only the 93x driver; the 92x driver has OPL4 support.
Regards,
Clemens
next prev parent reply other threads:[~2007-09-17 10:39 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-09-14 14:35 [PATCH] opti931: additional check for OPL3 device Krzysztof Helt
2007-09-14 14:52 ` Clemens Ladisch
2007-09-14 16:47 ` Krzysztof Helt
2007-09-17 8:14 ` Clemens Ladisch
2007-09-17 10:10 ` Rene Herman
2007-09-17 10:39 ` Clemens Ladisch [this message]
2007-09-17 12:26 ` Rene Herman
2007-09-17 12:26 ` Krzysztof Helt
2007-09-17 12:40 ` Rene Herman
2007-09-17 13:57 ` Rene Herman
2007-09-17 14:45 ` Krzysztof Helt
2007-09-17 15:09 ` Takashi Iwai
2007-09-17 23:04 ` Rene Herman
2007-09-14 16:33 ` Rene Herman
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=1190025593.12774.1211014755@webmail.messagingengine.com \
--to=cladisch@fastmail.net \
--cc=alsa-devel@alsa-project.org \
--cc=krzysztof.h1@gmail.com \
--cc=rene.herman@gmail.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox