Hi Sebastian, On 18/04/2014 13:23, Sebastian Hesselbarth wrote: > On 04/18/2014 12:22 PM, Gregory CLEMENT wrote: >> The Marvell Armada 38x SoCs contain two xHCI host. This commit adds >> the Device Tree description of those interfaces at the SoC level, and >> also enables the two USB3 ports on the Armada 385 DB platform and one >> USB3 port on the Armada 385 RD platforms. >> >> Signed-off-by: Gregory CLEMENT <gregory.clement@xxxxxxxxxxxxxxxxxx> >> --- >> arch/arm/boot/dts/armada-385-db.dts | 8 ++++++++ >> arch/arm/boot/dts/armada-385-rd.dts | 4 ++++ >> arch/arm/boot/dts/armada-38x.dtsi | 17 +++++++++++++++++ >> 3 files changed, 29 insertions(+) > [...] >> diff --git a/arch/arm/boot/dts/armada-38x.dtsi b/arch/arm/boot/dts/armada-38x.dtsi >> index a064f59da02d..4f117e66aa1a 100644 >> --- a/arch/arm/boot/dts/armada-38x.dtsi >> +++ b/arch/arm/boot/dts/armada-38x.dtsi >> @@ -355,6 +355,23 @@ >> clocks = <&coredivclk 0>; >> status = "disabled"; >> }; >> + >> + usb3@f0000 { >> + compatible = "marvell,xhci-armada-380"; >> + reg = <0xf0000 0x3fff>,<0xf4000 0x3fff>; >> + interrupts = <GIC_SPI 16 IRQ_TYPE_LEVEL_HIGH>; >> + clocks = <&gateclk 9>; > > Using generic PHY framework will allow you to reference the PHY nodes > with "usb-phy" here. > > BTW, you added a "marvell,armada-380-usb-utmi" in the patch for PHY init > but I can see no corresponding node added. Yes a commit was lost during one of my rebase! Thanks to git reflog, I found it. Thanks, Gregory > > Sebastian > >> + status = "disabled"; >> + }; >> + >> + usb3@f8000 { >> + compatible = "marvell,xhci-armada-380"; >> + reg = <0xf8000 0x3fff>,<0xfc000 0x3fff>; >> + interrupts = <GIC_SPI 17 IRQ_TYPE_LEVEL_HIGH>; >> + clocks = <&gateclk 10>; >> + status = "disabled"; >> + }; >> + >> }; >> }; >> >> > -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com -- 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