[PATCH 1/2] ARM: dts: Enable USB hub on Arndale

[PATCH 1/2] ARM: dts: Enable USB hub on Arndale

[Mark Brown]

From: Mark Brown <broonie@linaro.org>

The Arndale has a SMSC USB3503 connected in hardware only mode like a PHY,
support it using the usb-nop-xceiv binding.

Note that due to a regrettable decision to use a regulator to represent
the reset signal this uses a fixed voltage regulator to do that, there
is a plan to use the reset controller binding once that is merged so it
does not seem worthwhile to fix the usb-nop-xceiv driver at this point.

Signed-off-by: Mark Brown <broonie@linaro.org>
Tested-by: Tushar Behera <tushar.behera@linaro.org>
---
 arch/arm/boot/dts/exynos5250-arndale.dts | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/arch/arm/boot/dts/exynos5250-arndale.dts b/arch/arm/boot/dts/exynos5250-arndale.dts
index 96d528d..2428ffd 100644
--- a/arch/arm/boot/dts/exynos5250-arndale.dts
+++ b/arch/arm/boot/dts/exynos5250-arndale.dts
@@ -539,4 +539,18 @@
 	rtc {
 		status = "okay";
 	};
+
+	usb_hub_bus {
+		compatible = "simple-bus";
+		#address-cells = <1>;
+		#size-cells = <0>;
+
+		// SMSC USB3503 connected in hardware only mode as a PHY
+		usb_hub: usb_hub {
+			compatible = "smsc,usb3503a";
+
+			reset-gpios = <&gpx3 5 1>;
+			connect-gpios = <&gpd1 7 1>;
+		};
+	};
 };
-- 
1.8.4.rc2

[PATCH 2/2] ARM: dts: Hook up internal PHY on Arndale

[Mark Brown]

From: Mark Brown <broonie@linaro.org>

While the Linux driver stack is capable of figuring this out for itself
document the fact that we really do use the internal PHY even with the
directly wired hub on the board to save anyone else having to work this
out for themselves.

Signed-off-by: Mark Brown <broonie@linaro.org>
---
 arch/arm/boot/dts/exynos5250-arndale.dts | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/arch/arm/boot/dts/exynos5250-arndale.dts b/arch/arm/boot/dts/exynos5250-arndale.dts
index 2428ffd..e477e23 100644
--- a/arch/arm/boot/dts/exynos5250-arndale.dts
+++ b/arch/arm/boot/dts/exynos5250-arndale.dts
@@ -553,4 +553,8 @@
 			connect-gpios = <&gpd1 7 1>;
 		};
 	};
+
+	usb@12110000 {
+		usb-phy = <&usb2_phy>;
+	};
 };
-- 
1.8.4.rc2

Re: [PATCH 1/2] ARM: dts: Enable USB hub on Arndale

[Kukjin Kim]

On 08/15/13 08:27, Mark Brown wrote:
> From: Mark Brown<broonie@linaro.org>
>
> The Arndale has a SMSC USB3503 connected in hardware only mode like a PHY,
> support it using the usb-nop-xceiv binding.
>
> Note that due to a regrettable decision to use a regulator to represent
> the reset signal this uses a fixed voltage regulator to do that, there
> is a plan to use the reset controller binding once that is merged so it
> does not seem worthwhile to fix the usb-nop-xceiv driver at this point.
>
> Signed-off-by: Mark Brown<broonie@linaro.org>
> Tested-by: Tushar Behera<tushar.behera@linaro.org>
> ---
>   arch/arm/boot/dts/exynos5250-arndale.dts | 14 ++++++++++++++
>   1 file changed, 14 insertions(+)
>
> diff --git a/arch/arm/boot/dts/exynos5250-arndale.dts b/arch/arm/boot/dts/exynos5250-arndale.dts
> index 96d528d..2428ffd 100644
> --- a/arch/arm/boot/dts/exynos5250-arndale.dts
> +++ b/arch/arm/boot/dts/exynos5250-arndale.dts
> @@ -539,4 +539,18 @@
>   	rtc {
>   		status = "okay";
>   	};
> +
> +	usb_hub_bus {
> +		compatible = "simple-bus";
> +		#address-cells =<1>;
> +		#size-cells =<0>;
> +
> +		// SMSC USB3503 connected in hardware only mode as a PHY
> +		usb_hub: usb_hub {
> +			compatible = "smsc,usb3503a";
> +
> +			reset-gpios =<&gpx3 5 1>;
> +			connect-gpios =<&gpd1 7 1>;
> +		};
> +	};
>   };

Applied, thanks.

Kukjin