From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: Segher Boessenkool <segher@kernel.crashing.org>
Cc: linuxppc-dev@ozlabs.org, David Woodhouse <dwmw2@infradead.org>,
alan@lxorguk.ukuu.org.uk, jgarzik@pobox.com
Subject: Re: [PATCH] Fix Maple PATA IRQ assignment.
Date: Tue, 02 Jan 2007 08:33:50 +1100 [thread overview]
Message-ID: <1167687230.23340.147.camel@localhost.localdomain> (raw)
In-Reply-To: <32106685eb2ecb2fb4adcf6e51f71482@kernel.crashing.org>
On Mon, 2007-01-01 at 21:44 +0100, Segher Boessenkool wrote:
> > I'm not sure you can switch it to native mode in sw... worth double
> > checking though. All boards based on 8111 on powerpc have this problem,
> > I think. What of the js20 and js21 with IBM firmware ?
>
> At least with SLOF, we use native mode.
>
> > Or does it have
> > the ide strapped to native mode ?
>
> It's not a strapping, it's a purely software thing -- see the PCIIDE
> specification.
I know the PCI IDE spec, but there are a number of chips out there that
cannot be changed without a strap. In this case, it's easy to test
though we'll need fixups in the device-tree to indicate the change in
routing, or a quirk.
The thing is, there -is- a cascaded 8259 in there, and that's where the
IRQ14 and IRQ15 end up in legacy mode (edge sensitive even) iirc, unless
they are aslo routed to the APIC directly, but in anyway it's hidden by
the firmware.
So we can have a quirk in the Maple code to reconfigure that all in
native mode. That will mean a bit more IRQ sharing since both IDE
channels will suddenly share PIRQA though... along with whatever else
uses that pin. Maybe not the best solution...
Or keep the hook and use it in libata. It makes sense I think in
general. I reckon other platforms might have good use of being able to
"remap" those IRQs 14 and 15 from IDE controllers in legacy mode. I've
seen plenty of cases where they are kept in that mode just to keep the
IRQs non-shared.
> Lots of controllers get the interrupt thing wrong though -- but
> AMD8111 isn't one of those.
Ben.
next prev parent reply other threads:[~2007-01-01 21:36 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-01-01 19:31 [PATCH] Fix Maple PATA IRQ assignment David Woodhouse
2007-01-01 20:10 ` Benjamin Herrenschmidt
2007-01-01 20:26 ` David Woodhouse
2007-01-01 21:23 ` Benjamin Herrenschmidt
2007-01-01 20:44 ` Segher Boessenkool
2007-01-01 21:33 ` Benjamin Herrenschmidt [this message]
2007-01-01 22:17 ` Segher Boessenkool
2007-01-01 20:42 ` Segher Boessenkool
2007-01-01 20:56 ` David Woodhouse
2007-01-01 21:22 ` Alan
2007-01-01 21:15 ` David Woodhouse
2007-01-01 21:34 ` Alan
2007-01-01 21:43 ` David Woodhouse
2007-01-02 0:17 ` Benjamin Herrenschmidt
2007-01-02 0:18 ` Benjamin Herrenschmidt
2007-01-26 22:28 ` Jeff Garzik
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=1167687230.23340.147.camel@localhost.localdomain \
--to=benh@kernel.crashing.org \
--cc=alan@lxorguk.ukuu.org.uk \
--cc=dwmw2@infradead.org \
--cc=jgarzik@pobox.com \
--cc=linuxppc-dev@ozlabs.org \
--cc=segher@kernel.crashing.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 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).