Messages in this thread | | | Date | Thu, 25 Sep 2008 08:22:41 -0400 | From | Josh Boyer <> | Subject | Re: Fix PCI in Holly device tree |
| |
On Thu, Sep 25, 2008 at 08:11:29AM -0400, Josh Boyer wrote: >On Thu, 25 Sep 2008 12:39:04 +1000 >David Gibson <david@gibson.dropbear.id.au> wrote: > >> The PCI bridge on the Holly board is current incorrectly represented >> in the device tree. The current device tree node for the PCI bridge >> sits under the tsi-bridge node. That's not obviously wrong, but the >> PCI bridge translated some PCI spaces into CPU address ranges which >> were not translated by the tsi-bridge node. >> >> We used to get away with this problem because the PCI bridge discovery >> code was also buggy, assuming incorrectly that PCI host bridge nodes >> were always directly under the root bus and treating the translated >> addresses as raw CPU addresses, rather than parent bus addresses. >> This has since been fixed, breaking Holly. >> >> This could be fixed by adding extra translations to the tsi-bridge >> node, but this patch instead moves the Holly PCI bridge out of the >> tsi-bridge node to the root bus. This makes the tsi-bridge node >> represent only the built-in IO devices in the bridge, with a >> more-or-less contiguous address range. This is the same convention >> used on Freescale SoC chips, where the "soc" node represents only the >> IMMR region, and the PCI and other bus bridges are separate nodes >> under the root bus. >> >> Signed-off-by: David Gibson <david@gibson.dropbear.id.au> > >Acked-by: Josh Boyer <jwboyer@linux.vnet.ibm.com> > >Paul, I can include this in my 'next' branch if you aren't opposed. >I'll have another set of patches going in there today/tomorrow.
Er... on second thought, this actually fixes a regression on Holly. So I'll amend my offer to put it in my 'next' branch to be contingent on you not wanting to get it into 2.6.27 this late.
josh
| |