From mboxrd@z Thu Jan 1 00:00:00 1970 From: ray.jui@broadcom.com (Ray Jui) Date: Wed, 7 Dec 2016 09:19:36 -0800 Subject: [PATCH V2 2/5] ARM: BCM5301X: Specify USB controllers in DT In-Reply-To: <20161207075655.7396-2-zajec5@gmail.com> References: <20161207075655.7396-1-zajec5@gmail.com> <20161207075655.7396-2-zajec5@gmail.com> Message-ID: <64e06cd3-4834-c924-4345-ff88bd49236d@broadcom.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 12/6/2016 11:56 PM, Rafa? Mi?ecki wrote: > From: Rafa? Mi?ecki > > There are 3 separated controllers, one per USB /standard/. With PHY > drivers in place they can be simply supported with generic drivers. > > Signed-off-by: Rafa? Mi?ecki > --- > V2: Fix node names and reorder ehci with ohci. Thanks Ray! > --- > arch/arm/boot/dts/bcm5301x.dtsi | 33 ++++++++++++++++++++++++++++++++- > 1 file changed, 32 insertions(+), 1 deletion(-) > > diff --git a/arch/arm/boot/dts/bcm5301x.dtsi b/arch/arm/boot/dts/bcm5301x.dtsi > index f09a2bb..a4614c9 100644 > --- a/arch/arm/boot/dts/bcm5301x.dtsi > +++ b/arch/arm/boot/dts/bcm5301x.dtsi > @@ -248,8 +248,26 @@ > > #address-cells = <1>; > #size-cells = <1>; > + ranges; > > - phys = <&usb2_phy>; > + interrupt-parent = <&gic>; > + > + ehci: ehci at 21000 { > + #usb-cells = <0>; > + > + compatible = "generic-ehci"; > + reg = <0x00021000 0x1000>; > + interrupts = ; > + phys = <&usb2_phy>; > + }; > + > + ohci: ohci at 22000 { > + #usb-cells = <0>; > + > + compatible = "generic-ohci"; > + reg = <0x00022000 0x1000>; > + interrupts = ; > + }; > }; > > usb3: usb3 at 23000 { > @@ -257,6 +275,19 @@ > > #address-cells = <1>; > #size-cells = <1>; > + ranges; > + > + interrupt-parent = <&gic>; > + > + xhci: xhci at 23000 { > + #usb-cells = <0>; > + > + compatible = "generic-xhci"; > + reg = <0x00023000 0x1000>; > + interrupts = ; > + phys = <&usb3_phy>; > + phy-names = "usb"; > + }; > }; > > spi at 29000 { > Looks good to me! Thanks! Reviewed-by: Ray Jui