Hi Sergei,
(2014/06/25 2:59), Sergei Shtylyov wrote:
> From: Ben Dooks <ben.dooks@xxxxxxxxxxxxxxx>
>
> Add device nodes for the R8A7790 internal PCI bridge devices.
>
> Signed-off-by: Ben Dooks <ben.dooks@xxxxxxxxxxxxxxx>
> Reviewed-by: Ian Molton <ian.molton@xxxxxxxxxxxxxxx>
> [Sergei: added several properties to the PCI bridge nodes]
> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@xxxxxxxxxxxxxxxxxx>
Thank you for the patch!
I tested this patch on my lager board and a usb memory, and it works.
Tested-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@xxxxxxxxxxx>
Best regards,
Yoshihiro Shimoda
> ---
> Changes in version 5:
> - added "device_type" and "ranges" properties to the PCI bridge nodes;
> - removed "0x" prefix from zero values in the "reg" properties.
>
> Changes in version 4:
> - refreshed the patch.
>
> Changes in version 3:
> - added interrupt-related properties to the PCI bridge nodes;
> - refreshed the patch.
>
> Changes in version 2:
> - reworded summary (fixing typo) and changelog;
> - removed extra spaces before {;
> - refreshed the patch.
>
> arch/arm/boot/dts/r8a7790.dtsi | 60 +++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 60 insertions(+)
>
> Index: renesas/arch/arm/boot/dts/r8a7790.dtsi
> ===================================================================
> --- renesas.orig/arch/arm/boot/dts/r8a7790.dtsi
> +++ renesas/arch/arm/boot/dts/r8a7790.dtsi
> @@ -930,6 +930,66 @@
> status = "disabled";
> };
>
> + pci0: pci@ee090000 {
> + compatible = "renesas,pci-r8a7790";
> + device_type = "pci";
> + clocks = <&mstp7_clks R8A7790_CLK_EHCI>;
> + reg = <0 0xee090000 0 0xc00>,
> + <0 0xee080000 0 0x1100>;
> + interrupts = <0 108 IRQ_TYPE_LEVEL_HIGH>;
> + status = "disabled";
> +
> + bus-range = <0 0>;
> + #address-cells = <3>;
> + #size-cells = <2>;
> + #interrupt-cells = <1>;
> + ranges = <0x02000000 0 0xee080000 0 0xee080000 0 0x00010000>;
> + interrupt-map-mask = <0xff00 0 0 0x7>;
> + interrupt-map = <0x0000 0 0 1 &gic 0 108 IRQ_TYPE_LEVEL_HIGH
> + 0x0800 0 0 1 &gic 0 108 IRQ_TYPE_LEVEL_HIGH
> + 0x1000 0 0 2 &gic 0 108 IRQ_TYPE_LEVEL_HIGH>;
> + };
> +
> + pci1: pci@ee0b0000 {
> + compatible = "renesas,pci-r8a7790";
> + device_type = "pci";
> + clocks = <&mstp7_clks R8A7790_CLK_EHCI>;
> + reg = <0 0xee0b0000 0 0xc00>,
> + <0 0xee0a0000 0 0x1100>;
> + interrupts = <0 112 IRQ_TYPE_LEVEL_HIGH>;
> + status = "disabled";
> +
> + bus-range = <1 1>;
> + #address-cells = <3>;
> + #size-cells = <2>;
> + #interrupt-cells = <1>;
> + ranges = <0x02000000 0 0xee0a0000 0 0xee0a0000 0 0x00010000>;
> + interrupt-map-mask = <0xff00 0 0 0x7>;
> + interrupt-map = <0x0000 0 0 1 &gic 0 112 IRQ_TYPE_LEVEL_HIGH
> + 0x0800 0 0 1 &gic 0 112 IRQ_TYPE_LEVEL_HIGH
> + 0x1000 0 0 2 &gic 0 112 IRQ_TYPE_LEVEL_HIGH>;
> + };
> +
> + pci2: pci@ee0d0000 {
> + compatible = "renesas,pci-r8a7790";
> + device_type = "pci";
> + clocks = <&mstp7_clks R8A7790_CLK_EHCI>;
> + reg = <0 0xee0d0000 0 0xc00>,
> + <0 0xee0c0000 0 0x1100>;
> + interrupts = <0 113 IRQ_TYPE_LEVEL_HIGH>;
> + status = "disabled";
> +
> + bus-range = <2 2>;
> + #address-cells = <3>;
> + #size-cells = <2>;
> + #interrupt-cells = <1>;
> + ranges = <0x02000000 0 0xee0c0000 0 0xee0c0000 0 0x00010000>;
> + interrupt-map-mask = <0xff00 0 0 0x7>;
> + interrupt-map = <0x0000 0 0 1 &gic 0 113 IRQ_TYPE_LEVEL_HIGH
> + 0x0800 0 0 1 &gic 0 113 IRQ_TYPE_LEVEL_HIGH
> + 0x1000 0 0 2 &gic 0 113 IRQ_TYPE_LEVEL_HIGH>;
> + };
> +
> pciec: pcie@fe000000 {
> compatible = "renesas,pcie-r8a7790";
> reg = <0 0xfe000000 0 0x80000>;
> --
> To unsubscribe from this list: send the line "unsubscribe devicetree" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
