On Fri, Nov 01, 2013 at 11:56:33PM +0300, Sergei Shtylyov wrote: > Hello. > > On 11/01/2013 10:45 PM, Matt Porter wrote: > > >Adds USB OTG/PHY and clock support to BCM281xx and enables > >UDC support on the bcm11351-brt and bcm28155-ap boards. > > >Signed-off-by: Matt Porter <matt.porter@xxxxxxxxxx> > >Reviewed-by: Markus Mayer <markus.mayer@xxxxxxxxxx> > >Reviewed-by: Tim Kryger <tim.kryger@xxxxxxxxxx> > >--- > > arch/arm/boot/dts/bcm11351-brt.dts | 6 ++++++ > > arch/arm/boot/dts/bcm11351.dtsi | 18 ++++++++++++++++++ > > arch/arm/boot/dts/bcm28155-ap.dts | 8 ++++++++ > > 3 files changed, 32 insertions(+) > > [...] > >diff --git a/arch/arm/boot/dts/bcm11351.dtsi b/arch/arm/boot/dts/bcm11351.dtsi > >index 0755f43..247f9fd 100644 > >--- a/arch/arm/boot/dts/bcm11351.dtsi > >+++ b/arch/arm/boot/dts/bcm11351.dtsi > >@@ -284,4 +284,22 @@ > > #clock-cells = <0>; > > }; > > }; > >+ > >+ usbotg: usbotg@3f120000 { > > According to ePAPR spec [1], the node name should be "usb@3f120000". Will address in v3. > >+ compatible = "snps,dwc2"; > >+ reg = <0x3f120000 0x10000>; > >+ interrupts = <GIC_SPI 47 IRQ_TYPE_LEVEL_HIGH>; > >+ clocks = <&usb_otg_ahb_clk>; > >+ clock-names = "otg"; > >+ phys = <&usbphy>; > >+ phy-names = "usb2-phy"; > >+ status = "disabled"; > >+ }; > >+ > >+ usbphy: usbphy@3f130000 { > > This one should probably be named "usb-phy@3f130000", just like > "ethernet-phy" from the ePAPR spec. Yes, agreed that will follow the same naming convention. I'll roll this in v3. Thanks for the review. -Matt > >+ compatible = "brcm,kona-usb2-phy"; > >+ reg = <0x3f130000 0x28>; > >+ #phy-cells = <0>; > >+ status = "disabled"; > >+ }; > > }; > > [1] http://www.power.org/resources/downloads/Power_ePAPR_APPROVED_v1.0.pdf > > WBR, Sergei > -- 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