All of lore.kernel.org
 help / color / mirror / Atom feed
From: Rene Herman <rene.herman@gmail.com>
To: Rask Ingemann Lambertsen <rask@sygehus.dk>
Cc: Adam Belay <ambx1@neo.rr.com>, alsa-devel@alsa-project.org
Subject: Re: [PATCH] PnP: Protocol for Media Vision Jazz16	sound cards
Date: Thu, 26 Apr 2007 14:55:03 +0200	[thread overview]
Message-ID: <4630A127.1030908@gmail.com> (raw)
In-Reply-To: <20070423185221.GA3157@sygehus.dk>

On 04/23/2007 08:52 PM, Rask Ingemann Lambertsen wrote:

> On Sun, Apr 22, 2007 at 06:27:44PM +0200, Rene Herman wrote:

>> The Intel 430TX and 440BX (which also uses the PIIX4 if I'm not
>> mistaken) were hugely popular chipsets in the later generation of
>> boards that still had ISA slots and given that noone will deperately
>> want the MPU-401 IRQ assigned; how would you feel about the attached?
>> Just adds the preffered options minus the MPU IRQ as the first
>> acceptable. Works for me...
> 
> Fine with me. Maybe the MPU IRQ should be omitted alltogether? You tell 
> me.

"Using the hardware to its fullest potential" should probably be called the 
preferred option. I mean, what's the fun of playing with this old hardware 
if we're not going to use it fully? :)

So as submitted (or nothing at all but if we suppose the driver will be 
seeing practical use, guarding against that PIIX4 seems best) is nicest I feel.

> Aren't you having problems loading the module after a resource allocation
> failure? I find that I need this additional patch to be able to retry:

I've had to remove the jazz16 again for a bit and I haven't tested an 
allocation failure yet. That looks to be an "obvious fix" though. Now all we 
need is a PnP maintainer...

Rene.

  reply	other threads:[~2007-04-26 12:57 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20070412195523.GC5324@sygehus.dk>
2007-04-20 17:27 ` [PATCH] PnP: Protocol for Media Vision Jazz16 sound cards Rene Herman
2007-04-20 20:00   ` Rask Ingemann Lambertsen
2007-04-22 16:27     ` Rene Herman
2007-04-23 18:52       ` Rask Ingemann Lambertsen
2007-04-26 12:55         ` Rene Herman [this message]
2007-04-26 13:00           ` Takashi Iwai
2007-04-26 15:11             ` 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=4630A127.1030908@gmail.com \
    --to=rene.herman@gmail.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=ambx1@neo.rr.com \
    --cc=rask@sygehus.dk \
    /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.