From: Andrey Panin <pazke@orbita.don.sitek.net>
To: Jeff Garzik <jgarzik@mandrakesoft.com>
Cc: linux-kernel@vger.kernel.org
Subject: Re: [PATCH] media/radio [check_region() removal... ]
Date: Thu, 9 Nov 2000 16:06:52 +0300 [thread overview]
Message-ID: <20001109160652.A1953@debian> (raw)
In-Reply-To: <Pine.LNX.4.21.0011090056470.22998-200000@tricky> <3A09EC3A.82324C57@mandrakesoft.com>
In-Reply-To: <3A09EC3A.82324C57@mandrakesoft.com>; from jgarzik@mandrakesoft.com on Wed, Nov 08, 2000 at 07:13:46PM -0500
On Wed, Nov 08, 2000 at 07:13:46PM -0500, Jeff Garzik wrote:
Hi all,
>
> Finally, a word to you, Alan, and others doing request_region work: it
> is more informative to pass the device name (minor, etc.) into
> request_region. Ditto for request_irq. Many (most, except net?)
> drivers use board/chip name instead of registered interface name. If
> you can use the interface name for request_region or request_irq, use
> it... it allows differentiation between multiple boards of the same
> type. That's especially when looking at ISA regions in /proc/ioports,
> or interrupt counts in /proc/interrupts.
>
two question about this:
1) how about drivers requesting 2 (or more) irq for one device ?
AFAIK some PowerMac net drivers do it (bmac.c for example).
2) i found that some net drivers (3c527.c, sk_mca.c) use io region and
don't call request_region() at all. Should they be fixed ?
Best regards,
Andrey
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/
next prev parent reply other threads:[~2000-11-09 13:05 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2000-11-09 0:03 [PATCH] media/radio [check_region() removal... ] Bartlomiej Zolnierkiewicz
2000-11-09 0:13 ` Jeff Garzik
2000-11-09 0:29 ` Bartlomiej Zolnierkiewicz
2000-11-09 4:02 ` Jeff Garzik
2000-11-09 13:06 ` Andrey Panin [this message]
2000-11-09 13:08 ` Jeff Garzik
2000-11-09 13:37 ` Alan Cox
2000-11-09 13:50 ` Jeff Garzik
-- strict thread matches above, loose matches on Subject: below --
2000-11-09 15:36 Russell Kroll
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=20001109160652.A1953@debian \
--to=pazke@orbita.don.sitek.net \
--cc=jgarzik@mandrakesoft.com \
--cc=linux-kernel@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.