From: Tejun Heo <htejun@gmail.com>
To: Matt Sealey <matt@genesi-usa.com>
Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>, linux-ide@vger.kernel.org
Subject: Re: how to handle pata_via when controller not in fully-pci-native mode (two irqs?)
Date: Tue, 03 Jul 2007 16:33:31 +0900 [thread overview]
Message-ID: <4689FBCB.90608@gmail.com> (raw)
In-Reply-To: <467CF1E6.8000905@genesi-usa.com>
Matt Sealey wrote:
> I'm a litle confused here.
>
> Page 10 (4.2.1) so I just mask off bit 0 and bit 2 of the class prog-if byte and set
> it to "compatible" mode which will basically mean PCI mapped registers and
> two IRQs? This corresponds with the table on Page 7 (4.2.1.1)..
>
> Checked the Pegasos IDE class code and in the firmware it's set to 0x1018F so, I
> would assume the correct value is 0x1018A (knocking out the "native mode").
>
> Okay! Great. This looks easy. Just gotta decode PCI addresses in Forth now..
I think I wrote this in the bugzilla entry but my memory is fuzzy as
always, so I'll repeat it here. If you turn off the native bits,
libata-sff will use ATA_PRIMARY_CMD and its friends to determine IO
ports and ATA_PRIMARY_IRQ and friends to determine IRQs. PCI BAR is
only used to determine IO ports for BMDMA register.
ie. The legacy mode means "there's no standard way to configure the damn
thing. Just believe what the arch code says. However, BMDMA is
introduced after PCI so let's use PCI BAR for it." So, if you're gonna
make the controller legacy (which I think is the correct way), don't
forget to update those arch macros.
--
tejun
next prev parent reply other threads:[~2007-07-03 7:34 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-06-23 0:26 how to handle pata_via when controller not in fully-pci-native mode (two irqs?) Matt Sealey
2007-06-23 4:23 ` Jeff Garzik
2007-06-23 9:33 ` Alan Cox
2007-06-23 9:33 ` Matt Sealey
2007-06-23 9:53 ` Alan Cox
2007-06-23 10:11 ` Matt Sealey
2007-07-03 7:33 ` Tejun Heo [this message]
2007-07-03 8:11 ` Matt Sealey
2007-07-03 9:21 ` Tejun Heo
2007-07-03 12:44 ` Matt Sealey
2007-07-03 12:17 ` Alan Cox
2007-07-03 12:32 ` Matt Sealey
2007-07-03 13:38 ` Alan Cox
2007-07-03 13:41 ` Matt Sealey
2007-07-03 13:53 ` Jeff Garzik
2007-07-03 14:00 ` Matt Sealey
2007-07-03 15:08 ` Jeff Garzik
2007-07-03 13:53 ` Alan Cox
2007-07-03 13:54 ` Matt Sealey
2007-07-04 8:55 ` Tejun Heo
2007-07-04 19:08 ` Matt Sealey
2007-07-05 2:28 ` Tejun Heo
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=4689FBCB.90608@gmail.com \
--to=htejun@gmail.com \
--cc=alan@lxorguk.ukuu.org.uk \
--cc=linux-ide@vger.kernel.org \
--cc=matt@genesi-usa.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 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).