[PATCH v3 11/12] ARM: mvebu: Relocate Armada 370 PCIe device tree nodes
Arnd Bergmann
arnd at arndb.de
Wed Jun 19 02:29:35 EST 2013
On Tuesday 18 June 2013, Ezequiel Garcia wrote:
> + ranges =
> + <0x82000000 0 0x40000 0xffff0001 0x40000 0 0x00002000
> + 0x82000000 0 0x80000 0xffff0001 0x80000 0 0x00002000
> + 0x82000000 0 0xe0000000 0xffff0002 0 0 0x08000000
> + 0x81000000 0 0 0xffff0002 0x8000000 0 0x00100000>;
To clarify my earlier comment, I think it would be nicer to write this as
ranges =
<0x82000000 0 0x40000 0xffff0001 0x40000 0 0x00002000
0x82000000 0 0x80000 0xffff0001 0x80000 0 0x00002000
0x82000000 1 0 MBUS_ID(0x12, 0x34) 0 1 0
0x82000000 2 0 MBUS_ID(0x13, 0x34) 0 1 0
0x81000000 1 0 MBUS_ID(0x12, 0x35) 0 0 0x10000;
0x81000000 2 0 MBUS_ID(0x13, 0x35) 0 0 0x10000>;
The MBUS_ID numbers above are made up since I don't know them, but this way you can
describe how the entire 4GB MMIO address space of the PCI bus is mapped into the
MBUS address space.
> + pcie at 1,0 {
> + device_type = "pci";
> + assigned-addresses = <0x82000800 0 0x40000 0 0x2000>;
> + reg = <0x0800 0 0 0 0>;
> + #address-cells = <3>;
> + #size-cells = <2>;
> + #interrupt-cells = <1>;
> + ranges;
Then you do a ranges property for each port with the high-order
address word equal to the port number:
ranges = <0x82000000 1 0 0x82000000 0 0 1 0
0x81000000 1 0 0x81000000 0 0 0 0x10000>;
> + interrupt-map-mask = <0 0 0 0>;
> + interrupt-map = <0 0 0 0 &mpic 58>;
> + marvell,pcie-port = <0>;
> + marvell,pcie-lane = <0>;
> + clocks = <&gateclk 5>;
> + status = "disabled";
> + pcie at 2,0 {
> + device_type = "pci";
> + assigned-addresses = <0x82002800 0 0x80000 0 0x2000>;
> + reg = <0x1000 0 0 0 0>;
> + #address-cells = <3>;
> + #size-cells = <2>;
> + #interrupt-cells = <1>;
> + ranges;
ranges = <0x82000000 2 0 0x82000000 0 0 1 0
0x81000000 2 0 0x81000000 0 0 0 0x10000>;
> + interrupt-map-mask = <0 0 0 0>;
> + interrupt-map = <0 0 0 0 &mpic 62>;
> + marvell,pcie-port = <1>;
> + marvell,pcie-lane = <0>;
> + clocks = <&gateclk 9>;
> + status = "disabled";
> + };
Does this make sense?
Arnd
More information about the devicetree-discuss
mailing list