All of lore.kernel.org
 help / color / mirror / Atom feed
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: Wed, 04 Jul 2007 17:55:39 +0900	[thread overview]
Message-ID: <468B608B.50803@gmail.com> (raw)
In-Reply-To: <468A5532.9020600@genesi-usa.com>

Matt Sealey wrote:
> Well, let's put the class code as "native" since the chip is made "native",
> in a platform file somewhere. Then, we can have a configuration option in
> the platform code which allows users to choose whether the IDE configuration
> is reworked to steer to a single IRQ or two IRQs. That way they can choose
> which driver they are using (they cannot use both, and compiling in both is
> stupid, so..)
> 
> Then pata_via can just work. It does seem easier now to do this in the
> platform support thing. As long as it only affects brand new kernels and
> is only enabled when libata pata_via is enabled (we can make it depend on
> it) then it should work fine, although having it in some other config
> page seems pretty quirky. I don't know how you would suggest to do it..
> have CONFIG_FIX_PEGASOS_IDE in drivers/ata/Kconfig and code in
> arch/powerpc/platforms/chrp which recognises it? This patch idea has to
> get past two teams now, though.. and modifying chip registers that early
> in CHRP initialisation seems to be quirky in itself (I don't think PCI
> is up yet) :D

I forgot about the PCI resource fix up done for legacy hosts.  I think
making the host legacy is the best way to take here considering that -
no change for both ide and libata, just some fix up in platform code.
ATA native/legacy thing doesn't mean much.  It's just how the resources
are allocated.  Is there any specific reason to use native mode?

-- 
tejun

  reply	other threads:[~2007-07-04  8:55 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
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 [this message]
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=468B608B.50803@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 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.