diff mbox

ARM64: dts: meson-gxbb-nexbox-a95x: Enable USB Nodes

Message ID 20171005132118.2150-1-peter@korsgaard.com (mailing list archive)
State New, archived
Headers show

Commit Message

Peter Korsgaard Oct. 5, 2017, 1:21 p.m. UTC 
Enable both gxbb USB controllers and add a 5V regulator for the OTG port
VBUS, similar to p20x.

Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
---
 .../boot/dts/amlogic/meson-gxbb-nexbox-a95x.dts    | 29 ++++++++++++++++++++++
 1 file changed, 29 insertions(+)

Comments

Neil Armstrong Oct. 5, 2017, 1:23 p.m. UTC | #1 
On 05/10/2017 15:21, Peter Korsgaard wrote:
> Enable both gxbb USB controllers and add a 5V regulator for the OTG port
> VBUS, similar to p20x.
> 
> Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
> ---
>  .../boot/dts/amlogic/meson-gxbb-nexbox-a95x.dts    | 29 ++++++++++++++++++++++
>  1 file changed, 29 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/amlogic/meson-gxbb-nexbox-a95x.dts b/arch/arm64/boot/dts/amlogic/meson-gxbb-nexbox-a95x.dts
> index 38dfdde5c147..60cd9c3b11c8 100644
> --- a/arch/arm64/boot/dts/amlogic/meson-gxbb-nexbox-a95x.dts
> +++ b/arch/arm64/boot/dts/amlogic/meson-gxbb-nexbox-a95x.dts
> @@ -88,6 +88,18 @@
>  		};
>  	};
>  
> +	usb_pwr: regulator-usb-pwrs {
> +		compatible = "regulator-fixed";
> +
> +		regulator-name = "USB_PWR";
> +
> +		regulator-min-microvolt = <5000000>;
> +		regulator-max-microvolt = <5000000>;
> +
> +		gpio = <&gpio GPIODV_24 GPIO_ACTIVE_HIGH>;
> +		enable-active-high;
> +	};
> +
>  	vddio_card: gpio-regulator {
>  		compatible = "regulator-gpio";
>  
> @@ -294,3 +306,20 @@
>  	pinctrl-0 = <&uart_ao_a_pins>;
>  	pinctrl-names = "default";
>  };
> +
> +&usb0_phy {
> +	status = "okay";
> +	phy-supply = <&usb_pwr>;
> +};
> +
> +&usb1_phy {
> +	status = "okay";
> +};
> +
> +&usb0 {
> +	status = "okay";
> +};
> +
> +&usb1 {
> +	status = "okay";
> +};
> 

Indeed, I forgot to add USB to this board,
Thanks

Acked-by: Neil Armstrong <narmstrong@baylibre.com>
Kevin Hilman Oct. 6, 2017, 11:10 p.m. UTC | #2 
Neil Armstrong <narmstrong@baylibre.com> writes:

> On 05/10/2017 15:21, Peter Korsgaard wrote:
>> Enable both gxbb USB controllers and add a 5V regulator for the OTG port
>> VBUS, similar to p20x.
>> 
>> Signed-off-by: Peter Korsgaard <peter@korsgaard.com>

[...]

> Indeed, I forgot to add USB to this board,
> Thanks
>
> Acked-by: Neil Armstrong <narmstrong@baylibre.com>

Applied to v4.15/dt64,

Thanks,

Kevin
diff mbox

Patch

diff --git a/arch/arm64/boot/dts/amlogic/meson-gxbb-nexbox-a95x.dts b/arch/arm64/boot/dts/amlogic/meson-gxbb-nexbox-a95x.dts
index 38dfdde5c147..60cd9c3b11c8 100644
--- a/arch/arm64/boot/dts/amlogic/meson-gxbb-nexbox-a95x.dts
+++ b/arch/arm64/boot/dts/amlogic/meson-gxbb-nexbox-a95x.dts
@@ -88,6 +88,18 @@ 
 		};
 	};
 
+	usb_pwr: regulator-usb-pwrs {
+		compatible = "regulator-fixed";
+
+		regulator-name = "USB_PWR";
+
+		regulator-min-microvolt = <5000000>;
+		regulator-max-microvolt = <5000000>;
+
+		gpio = <&gpio GPIODV_24 GPIO_ACTIVE_HIGH>;
+		enable-active-high;
+	};
+
 	vddio_card: gpio-regulator {
 		compatible = "regulator-gpio";
 
@@ -294,3 +306,20 @@ 
 	pinctrl-0 = <&uart_ao_a_pins>;
 	pinctrl-names = "default";
 };
+
+&usb0_phy {
+	status = "okay";
+	phy-supply = <&usb_pwr>;
+};
+
+&usb1_phy {
+	status = "okay";
+};
+
+&usb0 {
+	status = "okay";
+};
+
+&usb1 {
+	status = "okay";
+};