diff mbox

[02/13] ARM: STi: DT: STiH416: Add DT node for the stih415/6 usb2 phy

Message ID 1415361475-6218-3-git-send-email-peter.griffin@linaro.org
State New
Headers show

Commit Message

Peter Griffin Nov. 7, 2014, 11:57 a.m. UTC
This usb picophy is found on stih415/6 SoC.

Signed-off-by: Peter Griffin <peter.griffin@linaro.org>
---
 arch/arm/boot/dts/stih416.dtsi | 9 +++++++++
 1 file changed, 9 insertions(+)

Comments

Lee Jones Nov. 10, 2014, 10:49 a.m. UTC | #1
On Fri, 07 Nov 2014, Peter Griffin wrote:

> This usb picophy is found on stih415/6 SoC.

Then shouldn't it live in stih41x.dtsi?

> Signed-off-by: Peter Griffin <peter.griffin@linaro.org>
> ---
>  arch/arm/boot/dts/stih416.dtsi | 9 +++++++++
>  1 file changed, 9 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/stih416.dtsi b/arch/arm/boot/dts/stih416.dtsi
> index 1137bdf..0f3019a 100644
> --- a/arch/arm/boot/dts/stih416.dtsi
> +++ b/arch/arm/boot/dts/stih416.dtsi
> @@ -316,5 +316,14 @@
>  
>  			status	        = "disabled";
>  		};
> +
> +		usb2_phy: usb2phy@0 {
> +			compatible = "st,stih416-usb-phy";
> +			#phy-cells = <0>;
> +			st,syscfg = <&syscfg_rear>;
> +			clocks = <&clk_sysin>;
> +			clock-names = "osc_phy";
> +

Remove this '\n'.

> +		};
>  	};
>  };
Peter Griffin Nov. 12, 2014, 1:55 p.m. UTC | #2
Hi Lee,

Thanks for reviewing.

On Mon, 10 Nov 2014, Lee Jones wrote:

> On Fri, 07 Nov 2014, Peter Griffin wrote:
> 
> > This usb picophy is found on stih415/6 SoC.
> 
> Then shouldn't it live in stih41x.dtsi?

No, it is the same ip block, and underlying driver, but needs a different
compatible string as it's wired up to different sysconfig registers on the SoC.

> 
> > Signed-off-by: Peter Griffin <peter.griffin@linaro.org>
> > ---
> >  arch/arm/boot/dts/stih416.dtsi | 9 +++++++++
> >  1 file changed, 9 insertions(+)
> > 
> > diff --git a/arch/arm/boot/dts/stih416.dtsi b/arch/arm/boot/dts/stih416.dtsi
> > index 1137bdf..0f3019a 100644
> > --- a/arch/arm/boot/dts/stih416.dtsi
> > +++ b/arch/arm/boot/dts/stih416.dtsi
> > @@ -316,5 +316,14 @@
> >  
> >  			status	        = "disabled";
> >  		};
> > +
> > +		usb2_phy: usb2phy@0 {
> > +			compatible = "st,stih416-usb-phy";
> > +			#phy-cells = <0>;
> > +			st,syscfg = <&syscfg_rear>;
> > +			clocks = <&clk_sysin>;
> > +			clock-names = "osc_phy";
> > +
> 
> Remove this '\n'.

Fixed in v2.

regards,

Peter.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/
diff mbox

Patch

diff --git a/arch/arm/boot/dts/stih416.dtsi b/arch/arm/boot/dts/stih416.dtsi
index 1137bdf..0f3019a 100644
--- a/arch/arm/boot/dts/stih416.dtsi
+++ b/arch/arm/boot/dts/stih416.dtsi
@@ -316,5 +316,14 @@ 
 
 			status	        = "disabled";
 		};
+
+		usb2_phy: usb2phy@0 {
+			compatible = "st,stih416-usb-phy";
+			#phy-cells = <0>;
+			st,syscfg = <&syscfg_rear>;
+			clocks = <&clk_sysin>;
+			clock-names = "osc_phy";
+
+		};
 	};
 };