| Message ID | 1385403367-18144-10-git-send-email-matt.porter@linaro.org |
|---|---|
| State | Superseded |
| Headers | show |
Hi, On Monday 25 November 2013 11:46 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@linaro.org> > Reviewed-by: Markus Mayer <markus.mayer@linaro.org> > Reviewed-by: Tim Kryger <tim.kryger@linaro.org> > --- > 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-brt.dts b/arch/arm/boot/dts/bcm11351-brt.dts > index 23cd16d..396b704 100644 > --- a/arch/arm/boot/dts/bcm11351-brt.dts > +++ b/arch/arm/boot/dts/bcm11351-brt.dts > @@ -44,5 +44,11 @@ > status = "okay"; > }; > > + usbotg: usb@3f120000 { > + status = "okay"; > + }; using &usbotg could have been better. I mean if you have something like &usbotg, you realise that the node has been created somewhere and we are just referencing it here to add more properties. Thanks Kishon
On Wed, Nov 27, 2013 at 02:27:19PM +0530, Kishon Vijay Abraham I wrote: > Hi, > > On Monday 25 November 2013 11:46 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@linaro.org> > > Reviewed-by: Markus Mayer <markus.mayer@linaro.org> > > Reviewed-by: Tim Kryger <tim.kryger@linaro.org> > > --- > > 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-brt.dts b/arch/arm/boot/dts/bcm11351-brt.dts > > index 23cd16d..396b704 100644 > > --- a/arch/arm/boot/dts/bcm11351-brt.dts > > +++ b/arch/arm/boot/dts/bcm11351-brt.dts > > @@ -44,5 +44,11 @@ > > status = "okay"; > > }; > > > > + usbotg: usb@3f120000 { > > + status = "okay"; > > + }; > > using &usbotg could have been better. I mean if you have something like > &usbotg, you realise that the node has been created somewhere and we are just > referencing it here to add more properties. That's normally my preference, but Christian owns this file and doesn't like that syntax. Instead, I'm conforming with the rest of the .dts entries. He and I discussed moving to &label references a while back and that's a no go. Thanks, Matt
diff --git a/arch/arm/boot/dts/bcm11351-brt.dts b/arch/arm/boot/dts/bcm11351-brt.dts index 23cd16d..396b704 100644 --- a/arch/arm/boot/dts/bcm11351-brt.dts +++ b/arch/arm/boot/dts/bcm11351-brt.dts @@ -44,5 +44,11 @@ status = "okay"; }; + usbotg: usb@3f120000 { + status = "okay"; + }; + usbphy: usb-phy@3f130000 { + status = "okay"; + }; }; diff --git a/arch/arm/boot/dts/bcm11351.dtsi b/arch/arm/boot/dts/bcm11351.dtsi index 1246885..868879a 100644 --- a/arch/arm/boot/dts/bcm11351.dtsi +++ b/arch/arm/boot/dts/bcm11351.dtsi @@ -243,4 +243,22 @@ #clock-cells = <0>; }; }; + + usbotg: usb@3f120000 { + 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 = "device"; + status = "disabled"; + }; + + usbphy: usb-phy@3f130000 { + compatible = "brcm,kona-usb2-phy"; + reg = <0x3f130000 0x28>; + #phy-cells = <0>; + status = "disabled"; + }; }; diff --git a/arch/arm/boot/dts/bcm28155-ap.dts b/arch/arm/boot/dts/bcm28155-ap.dts index 08e47c2..a3bc436 100644 --- a/arch/arm/boot/dts/bcm28155-ap.dts +++ b/arch/arm/boot/dts/bcm28155-ap.dts @@ -43,4 +43,12 @@ cd-gpios = <&gpio 14 0>; status = "okay"; }; + + usbotg: usb@3f120000 { + status = "okay"; + }; + + usbphy: usb-phy@3f130000 { + status = "okay"; + }; };