linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: Jon Loeliger <jdl@freescale.com>
Cc: linuxppc-dev list <linuxppc-dev@ozlabs.org>,
	Paul Mackerras <paulus@samba.org>,
	Wei Zhang <Wei.Zhang@freescale.com>
Subject: Re: [PATCH 3/5] Float the pci bus number on MPC8641HPCN board.
Date: Tue, 05 Jun 2007 08:24:54 +1000	[thread overview]
Message-ID: <1180995894.31677.77.camel@localhost.localdomain> (raw)
In-Reply-To: <1180983654.9632.28.camel@ld0161-tx32>

On Mon, 2007-06-04 at 14:00 -0500, Jon Loeliger wrote:
> On Fri, 2007-06-01 at 18:28, Benjamin Herrenschmidt wrote:
> > On Fri, 2007-06-01 at 14:58 -0500, Kumar Gala wrote:
> > > 
> > > So looking at the code it looks like pci_assign_all_buses should  
> > > handle numbering the busses properly when we scan the busses, so  
> > > what's the issue that causes that not to be correct? 
> > 
> > Yeah, that's what powermac uses... Any reason it wouldn't work ?
> > 
> > Ben.
> 
> Empirical testing says it doesn't work.

You may also need to tweak pcibios_assign_bus_offset to bump it to a
higher value. There are issues when renumbering bus numbers when during
the probe, a bus is renumbered to the same number as a not-yet probed
bridge on the same segment.

That is fixed on powermac by adding an artificial "offset" of 16 between
bus numbers of different domains via this offset variable.

Ben.

  reply	other threads:[~2007-06-04 22:25 UTC|newest]

Thread overview: 52+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-05-22  3:38 [PATCH 0/5] Add the support for MPC8641 silicon rev 2.0 and MPC8641HPCN board 2.0 Zhang Wei
2007-05-22  3:38 ` [PATCH 1/5] Remove the errata fix codes for MPC8641 silicon ver 1.0 which is end of life Zhang Wei
2007-05-22  3:38   ` [PATCH 2/5] Add uli1575 pci-bridge sector to MPC8641HPCN dts file Zhang Wei
2007-05-22  3:38     ` [PATCH 3/5] Float the pci bus number on MPC8641HPCN board Zhang Wei
2007-05-22  3:38       ` [PATCH 4/5] Set RC of mpc8641 to transparent bridge for transfer legacy I/O access Zhang Wei
2007-05-22  3:38         ` [PATCH 5/5] Set IDE in ULI1575 to not native mode Zhang Wei
2007-05-29 18:15           ` Jon Loeliger
2007-05-29 20:50           ` Kumar Gala
2007-05-30  2:25             ` Zhang Wei-r63237
2007-05-30 11:40               ` Benjamin Herrenschmidt
2007-05-31  2:00                 ` Zhang Wei-r63237
2007-05-30 11:18             ` Benjamin Herrenschmidt
2007-05-29 18:15         ` [PATCH 4/5] Set RC of mpc8641 to transparent bridge for transfer legacy I/O access Jon Loeliger
2007-05-22 17:59       ` [PATCH 3/5] Float the pci bus number on MPC8641HPCN board Wade Farnsworth
2007-05-23  3:35         ` Zhang Wei-r63237
2007-05-31 13:43           ` Kumar Gala
2007-06-01  2:01             ` Zhang Wei-r63237
2007-06-01  5:08               ` Kumar Gala
2007-06-01  6:28                 ` Paul Mackerras
2007-06-01  6:45                   ` Zhang Wei-r63237
2007-06-01  7:44                     ` Benjamin Herrenschmidt
2007-06-01  7:53                       ` Zhang Wei-r63237
2007-06-01  8:00                         ` Benjamin Herrenschmidt
2007-06-01 16:35                       ` Matt Sealey
2007-06-01 16:47                         ` Jon Loeliger
2007-06-01 22:41                           ` Benjamin Herrenschmidt
2007-06-01 22:41                         ` Benjamin Herrenschmidt
2007-06-02  8:36                           ` Segher Boessenkool
2007-06-02  8:55                             ` Benjamin Herrenschmidt
2007-06-01 19:58                     ` Kumar Gala
2007-06-01 23:28                       ` Benjamin Herrenschmidt
2007-06-04 19:00                         ` Jon Loeliger
2007-06-04 22:24                           ` Benjamin Herrenschmidt [this message]
2007-06-01  6:29                 ` Zhang Wei-r63237
2007-05-29 18:14       ` Jon Loeliger
2007-05-22 20:37     ` [PATCH 2/5] Add uli1575 pci-bridge sector to MPC8641HPCN dts file xianghua xiao
2007-05-23  3:38       ` Zhang Wei-r63237
2007-05-29 18:14     ` Jon Loeliger
2007-05-29 20:39     ` Kumar Gala
2007-05-30 10:02       ` Zhang Wei-r63237
2007-05-30 10:18         ` Segher Boessenkool
2007-05-30 11:07         ` Benjamin Herrenschmidt
2007-05-30 22:26       ` Wade Farnsworth
2007-05-29 18:13   ` [PATCH 1/5] Remove the errata fix codes for MPC8641 silicon ver 1.0 which is end of life Jon Loeliger
2007-05-22  3:53 ` [PATCH 0/5] Add the support for MPC8641 silicon rev 2.0 and MPC8641HPCN board 2.0 Kumar Gala
2007-05-22  4:46   ` Zhang Wei-r63237
2007-05-22 12:54     ` Kumar Gala
2007-05-22 16:28 ` Jon Loeliger
2007-05-22 17:47 ` Wade Farnsworth
2007-05-23  2:48   ` [PATCH 0/5] Add the support for MPC8641 silicon rev 2.0 andMPC8641HPCN " Zhang Wei-r63237
2007-05-23 22:37     ` Wade Farnsworth
2007-05-29 18:12 ` [PATCH 0/5] Add the support for MPC8641 silicon rev 2.0 and MPC8641HPCN " Jon Loeliger

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=1180995894.31677.77.camel@localhost.localdomain \
    --to=benh@kernel.crashing.org \
    --cc=Wei.Zhang@freescale.com \
    --cc=jdl@freescale.com \
    --cc=linuxppc-dev@ozlabs.org \
    --cc=paulus@samba.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).