[PATCH 27/31] arm64: dts: renesas: r8a7795: Add usb companion property in EHCI

Simon Horman horms at verge.net.au
Tue Aug 8 00:40:41 PDT 2017


On Wed, Aug 02, 2017 at 02:20:59PM +0200, Simon Horman wrote:
> On Mon, Jul 31, 2017 at 07:40:23PM +0300, Sergei Shtylyov wrote:
> > On 07/31/2017 06:03 PM, Simon Horman wrote:
> > 
> > >From: Kazuya Mizuguchi <kazuya.mizuguchi.ks at renesas.com>
> > >
> > >This patch adds the "companion" property in the EHCI ch0, ch1 and
> > >ch2 nodes to wait for the usb companion controller startup at resume.
> > >
> > >Signed-off-by: Kazuya Mizuguchi <kazuya.mizuguchi.ks at renesas.com>
> > >Signed-off-by: Takeshi Kihara <takeshi.kihara.df at renesas.com>
> > >[remove ch3 node and revise the commit log]
> > >Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh at renesas.com>
> > >
> > >Signed-off-by: Simon Horman <horms+renesas at verge.net.au>
> > >---
> > >  arch/arm64/boot/dts/renesas/r8a7795.dtsi | 3 +++
> > >  1 file changed, 3 insertions(+)
> > >
> > >diff --git a/arch/arm64/boot/dts/renesas/r8a7795.dtsi b/arch/arm64/boot/dts/renesas/r8a7795.dtsi
> > >index d73d986039c8..7246e689b619 100644
> > >--- a/arch/arm64/boot/dts/renesas/r8a7795.dtsi
> > >+++ b/arch/arm64/boot/dts/renesas/r8a7795.dtsi
> > >@@ -1583,6 +1583,7 @@
> > >  			clocks = <&cpg CPG_MOD 703>;
> > >  			phys = <&usb2_phy0>;
> > >  			phy-names = "usb";
> > >+			companion= <&ohci0>;
> > >  			power-domains = <&sysc R8A7795_PD_ALWAYS_ON>;
> > >  			resets = <&cpg 703>;
> > >  			status = "disabled";
> > >@@ -1595,6 +1596,7 @@
> > >  			clocks = <&cpg CPG_MOD 702>;
> > >  			phys = <&usb2_phy1>;
> > >  			phy-names = "usb";
> > >+			companion= <&ohci1>;
> > >  			power-domains = <&sysc R8A7795_PD_ALWAYS_ON>;
> > >  			resets = <&cpg 702>;
> > >  			status = "disabled";
> > >@@ -1607,6 +1609,7 @@
> > >  			clocks = <&cpg CPG_MOD 701>;
> > >  			phys = <&usb2_phy2>;
> > >  			phy-names = "usb";
> > >+			companion= <&ohci2>;
> > >  			power-domains = <&sysc R8A7795_PD_ALWAYS_ON>;
> > >  			resets = <&cpg 701>;
> > >  			status = "disabled";
> > 
> >    Looks like a space before = is missing in all 3 cases.
> 
> Thanks.
> 
> I'll post a follow-up patch for that. I don't think it warrants respinning
> this pull request at this point.

Follow-up sent as
[PATCH] arm64: dts: renesas: r8a7795: correct whitespace of companion property



More information about the linux-arm-kernel mailing list