All of lore.kernel.org
 help / color / mirror / Atom feed
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

  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 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.