Hi Biju-san, Thank you for the patch! > From: Biju Das, Sent: Tuesday, August 11, 2020 5:02 PM > > Document HS and SS data bus for the "usb-role-switch" enabled case. > > Signed-off-by: Biju Das <biju.das.jz@xxxxxxxxxxxxxx> > Reviewed-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx> > --- > v1->v2: No change > Ref:https://patchwork.kernel.org/patch/11669423/ > --- > .../bindings/usb/renesas,usb3-peri.yaml | 36 +++++++++++++++---- > 1 file changed, 30 insertions(+), 6 deletions(-) > > diff --git a/Documentation/devicetree/bindings/usb/renesas,usb3-peri.yaml > b/Documentation/devicetree/bindings/usb/renesas,usb3-peri.yaml > index e3cdeab1199f..3eb964af37b8 100644 > --- a/Documentation/devicetree/bindings/usb/renesas,usb3-peri.yaml > +++ b/Documentation/devicetree/bindings/usb/renesas,usb3-peri.yaml <snip> > @@ -79,9 +92,20 @@ examples: > companion = <&xhci0>; > usb-role-switch; > > - port { > - usb3_role_switch: endpoint { > - remote-endpoint = <&hd3ss3220_ep>; > - }; > - }; > + ports { I think we should use spaces instead of tab here and below. > + #address-cells = <1>; > + #size-cells = <0>; > + port@0 { > + reg = <0>; > + usb3_hs_ep: endpoint { > + remote-endpoint = <&hs_ep>; > + }; > + }; > + port@1 { > + reg = <1>; > + usb3_role_switch: endpoint { > + remote-endpoint = <&hd3ss3220_out_ep>; > + }; > + }; > + }; Best regards, Yoshihiro Shimoda > }; > -- > 2.17.1
