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@xxxxxxxxxxx> > > > >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@xxxxxxxxxxx> > >Signed-off-by: Takeshi Kihara <takeshi.kihara.df@xxxxxxxxxxx> > >[remove ch3 node and revise the commit log] > >Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@xxxxxxxxxxx> > > > >Signed-off-by: Simon Horman <horms+renesas@xxxxxxxxxxxx> > >--- > > 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.
