On Thu, May 27, 2010 at 07:45:16AM -0500, Kumar Gala wrote: > > On May 27, 2010, at 6:20 AM, Wolfgang Denk wrote: > > > Dear Kumar Gala, > > > > In message <5f58de0b-6ef8-4ed6-a1a8-c0e37c853...@kernel.crashing.org> you > > wrote: > >> > >> This is my fault. However not sure what to do about it since we'd break > >> compatibility with kernel .dts to clean this up. > >> > >> 99% of the u-boot code should match the HW docs. In this one place I > >> tried to "rename" things such that it made sense. The pci aliases in > >> the .dts are in order of address (so whatever HW controller is @ 0x8000 > >> would be "pci0", 0x9000 - "pci1", etc.) > > > > This doesn't seem to be the case in U-Boot; here we see: > > > > #define CONFIG_SYS_PCIE3_ADDR (CONFIG_SYS_CCSRBAR+0x8000) > > #define CONFIG_SYS_PCIE2_ADDR (CONFIG_SYS_CCSRBAR+0x9000) > > #define CONFIG_SYS_PCIE1_ADDR (CONFIG_SYS_CCSRBAR+0xa000) > > > > i. e. the highest number is at the lowest address?? > > Correct, that is matching FSL HW docs numbering/naming. > > in the .dts the alias: > * "pci0" is @ 0x8000 - FSL HW calls it PCIE3 > * "pci1" is @ 0x9000 - FSL HW calls it PCIE2 > * "pci2" is @ 0xa000 - FSL HW calls it PCIE1
This is why the dts files should live with u-boot, not Linux... but if we can't change the dts file, we could ignore the aliases and look up PCI controllers by address. -Scott _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot