On Dec 31, 2008, at 11:41 AM, Anton Vorontsov wrote:

On Wed, Dec 31, 2008 at 10:48:07AM -0600, Kumar Gala wrote:

On Dec 30, 2008, at 2:19 PM, Anton Vorontsov wrote:



+               compatible = "fsl,mpc8315-pcie", "fsl,mpc8314-pcie";
+               reg = <0xe0009000 0x00001000 0xb0000000 0x01000000>;

the size on the 0xb0000000 seems wrong

This is how FSL U-Boots configure the cfg window, and I'm trying to
retain the compatibility. Anyway, now Linux maps only the first 4kB
of that area, so we don't actually care about this, but dts is still
correct for older u-boots. Newer U-Boots might want to correct this
value if they want to use other cfg window setup...

I'd rather not pollute the .dts with this

So you prefer to change the cfg window size to 0x1000?

I'd prefer to drop it (assuming that is reasonable).

Does the PCI-express OF draft spec cover describing cfg space

http://playground.sun.com/1275/proposals/New/516-it.txt

- k
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev

Reply via email to