linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: Dan Malek <dan@embeddededge.com>
To: Tom Rini <trini@kernel.crashing.org>
Cc: Wolfgang Grandegger <wolfgang.grandegger@bluewin.ch>,
	Wolfgang Denk <wd@denx.de>,
	linuxppc-embedded@lists.linuxppc.org
Subject: Re: [PATCH and RFC] Remove request_8xxirq
Date: Mon, 24 Jun 2002 13:49:28 -0400	[thread overview]
Message-ID: <3D175BA8.6090206@embeddededge.com> (raw)
In-Reply-To: 20020624170055.GB3489@opus.bloom.county


Tom Rini wrote:

> Ah, which is why you just expanded things the way you did..  Which works
> for that context but still leaves the larger problem of request_irq()
> panic()'ing on unknown interrupt numbers.

All you need to do is modify request_irq() to check/map the numbers that
are OK for you to use on your system.  The problem with request_irq() is
it is easy to pass a number you think is fine from some legacy perspective,
but it messes up the SIU due to "numberspace" overlap.  To trap these and
print out a message saves lots of traffic on this mailing list :-)


> Well maybe Dan will get encouraged enough to try and properly fix
> request_irq() and break some of the legacy PC drivers in 2.5 again. :)

I can't break it until I dredge up some of the stuff we did way back in 2.1.xx
and get it to stick.  Unless we have the underlying mechanism to manage
an arbitrary, cascaded interrupt controller, all we have are processor and
platform specific hacks with functions that behave differently.  I guess
I can do it for PPC only, and then hope this time it gets generally
integrated entirely.  In this case, there will be a different function name,
as request_irq() will be left as some macro or simple function for
legacy compatibility.

Thanks.


	-- Dan


** Sent via the linuxppc-embedded mail list. See http://lists.linuxppc.org/

  reply	other threads:[~2002-06-24 17:49 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20020620214551.GI16052@opus.bloom.county>
2002-06-20 21:58 ` [PATCH and RFC] Remove request_8xxirq Wolfgang Denk
     [not found]   ` <20020620221016.GK16052@opus.bloom.county>
     [not found]     ` <3D12F667.30902@bluewin.ch>
2002-06-24 17:00       ` Tom Rini
2002-06-24 17:49         ` Dan Malek [this message]
2002-06-24 17:59           ` Tom Rini
     [not found]         ` <3D1793E1.3030509@bluewin.ch>
     [not found]           ` <20020624224100.GL3489@opus.bloom.county>
2002-06-25  0:03             ` Dan Malek
     [not found] <15635.16331.965632.877856@argo.ozlabs.ibm.com>
2002-06-23 16:54 ` Stephan Linke
2002-06-21  2:32 Andy Lowe
2002-06-21  4:35 ` Dan Malek
2002-06-21  4:54   ` Andy Lowe
     [not found] <3D124DF4.6060505@embeddededge.com>
2002-06-20 22:02 ` Wolfgang Denk
  -- strict thread matches above, loose matches on Subject: below --
2002-06-20 20:34 Tom Rini
2002-06-20 21:04 ` Dan Malek
2002-06-20 21:39   ` Tom Rini
2002-06-20 22:03     ` Dan Malek
2002-06-20 16:58       ` Benjamin Herrenschmidt
2002-06-20 22:34       ` Tom Rini
2002-06-20 21:16 ` Wolfgang Denk

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=3D175BA8.6090206@embeddededge.com \
    --to=dan@embeddededge.com \
    --cc=linuxppc-embedded@lists.linuxppc.org \
    --cc=trini@kernel.crashing.org \
    --cc=wd@denx.de \
    --cc=wolfgang.grandegger@bluewin.ch \
    /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;
as well as URLs for NNTP newsgroup(s).