[PATCH] powerpc: Add MPC837x PCIE controller RC mode support
Li Li
r64360 at freescale.com
Thu Jan 3 19:31:06 EST 2008
On Thu, 2008-01-03 at 16:14 +0800, Arnd Bergmann wrote:
> On Thursday 03 January 2008, Li Li wrote:
> > > * The detection method for the primary bus is somewhat fragile,
> > > because
> > > we depend on the order of the nodes in the device tree, which is
> not
> > > specified to have a meaning. /Maybe/ there could be a property in
> > > (at most) one of the PCI host bridge nodes saying that this
> specific
> > > bus
> > > is the primary one.
> > >
> >
> > Put this primary in code is more save than in dts.
> > The dts lacks error checking function. If provide error dts such as
> two
> > pci hosts both has primary property will occur chaos.
>
> It's easy enough to just panic() if you find more than one primary
> bus,
> since the information from the device tree is static enough that one
> will know how to fix this after seeing the error.
>
It is just a case. what about if do not find primary pci host.
The current code can prevent this potential error and make system
robust.
- Tony
More information about the Linuxppc-dev
mailing list