Subject: [PATCH 3/4] ARM: dts: exynos4: add hsotg device node

From: [email-masked] (Tomasz Figa)
Date: Thu, 24 Apr 2014 18:16:37 +0200

Hi Chanho,

On 14.04.2014 14:48, Chanho Park wrote:
> This patch adds a hsotg node for exynos4 USB2.0 device controller.
>
> Cc: Tomasz Figa <[email-masked]>
> Cc: Kamil Debski <[email-masked]>
> Cc: Marek Szyprowski <[email-masked]>
> Signed-off-by: Chanho Park <[email-masked]>
> ---
>   arch/arm/boot/dts/exynos4.dtsi | 11 +++++++++++
>   1 file changed, 11 insertions(+)
>
> diff --git a/arch/arm/boot/dts/exynos4.dtsi b/arch/arm/boot/dts/exynos4.dtsi
> index 0e32d7f..e541ecb 100644
> --- a/arch/arm/boot/dts/exynos4.dtsi
> +++ b/arch/arm/boot/dts/exynos4.dtsi
> @@ -288,6 +288,17 @@
>   		#phy-cells = <1>;
>   	};
>
> +	hsotg at 12480000 {
> +		compatible = "samsung,s3c6400-hsotg";
> +		reg = <0x12480000 0x20000>;
> +		interrupts = <0 71 0>;
> +		clocks = <&clock 305>;

Please use clock macros.

> +		clock-names = "otg";
> +		phys = <&exynos_usbphy 0>;
> +		phy-names = "device";

This is not the correct phy name for this binding. According to what the 
driver uses and the example in ...bindings/usb/dwc2.txt "usb2-phy" 
should be used.

Best regards,
Tomasz