linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: "Mark A. Greer" <mgreer@mvista.com>
Cc: linuxppc-dev <Linuxppc-dev@ozlabs.org>
Subject: Re: [PATCH 1/4] powerpc: mv64x60 - Use early_* PCI accessors for hotswap reg
Date: Thu, 17 Jan 2008 13:39:02 +1100	[thread overview]
Message-ID: <1200537542.20581.3.camel@pasglop> (raw)
In-Reply-To: <20080117004744.GC1917@mag.az.mvista.com>


On Wed, 2008-01-16 at 17:47 -0700, Mark A. Greer wrote:
> On Thu, Jan 17, 2008 at 09:48:59AM +1100, Benjamin Herrenschmidt wrote:
> > 
> > On Mon, 2008-01-14 at 15:51 -0700, Mark A. Greer wrote:
> > > From: Mark A. Greer <mgreer@mvista.com>
> > > 
> > > The mv64x60 Hotswap register is on the first hose of the mv64x60
> > > hostbridge.  To access it, manually find the hose structure and
> > > use the early_* PCI accessor routines because the hostbridge is
> > > normally hidden.
> > 
> > Can't we unhide the NB instead ?
> 
> Hi Ben.
> 
> Possibly but it may cause issues since many hostbridge have BARs
> that don't comply with the PCI spec which may get hosed when scanning
> the PCI bus. 

True. But it's nice to be able to see their config space.

I tend to use a header quirk to blank the BARs of the host bridge in
those cases. But that does mean that the sizing operation is done on
them (writing ff's etc...) which might be an issue in your case (or
not).

Ben.

      reply	other threads:[~2008-01-17  2:39 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-01-14 22:51 [PATCH 1/4] powerpc: mv64x60 - Use early_* PCI accessors for hotswap reg Mark A. Greer
2008-01-14 22:58 ` [PATCH 2/4] powerpc: mv64x60 - Exit when no hs_reg_valid property Mark A. Greer
2008-01-14 22:59 ` [PATCH 3/4] powerpc: Katana750i - Add DTS file Mark A. Greer
2008-01-14 23:34   ` David Gibson
2008-01-15 19:08     ` Mark A. Greer
2008-01-16  0:22       ` David Gibson
2008-01-16 20:48         ` Mark A. Greer
2008-01-17  1:06           ` David Gibson
2008-01-17  2:28             ` Mark A. Greer
2008-01-16 22:04   ` [PATCH 3/4 v2] " Mark A. Greer
2008-01-14 23:00 ` [PATCH 4/4] powerpc: Katana750i - Add platform support Mark A. Greer
2008-01-14 23:33   ` Stephen Rothwell
2008-01-15 17:36     ` Mark A. Greer
2008-01-16 22:12   ` [PATCH 4/4 v2] " Mark A. Greer
2008-01-16 23:27     ` Stephen Rothwell
2008-01-17  0:35       ` Mark A. Greer
2008-01-17  1:58     ` David Gibson
2008-01-14 23:19 ` [PATCH 1/4] powerpc: mv64x60 - Use early_* PCI accessors for hotswap reg Stephen Rothwell
2008-01-15 17:20   ` Mark A. Greer
2008-01-14 23:21 ` Stephen Rothwell
2008-01-15 17:31   ` Mark A. Greer
2008-01-16 22:00 ` [PATCH 1/4 v2] " Mark A. Greer
2008-01-16 22:48 ` [PATCH 1/4] " Benjamin Herrenschmidt
2008-01-17  0:47   ` Mark A. Greer
2008-01-17  2:39     ` Benjamin Herrenschmidt [this message]

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=1200537542.20581.3.camel@pasglop \
    --to=benh@kernel.crashing.org \
    --cc=Linuxppc-dev@ozlabs.org \
    --cc=mgreer@mvista.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).