On Wed, Apr 24, 2019 at 10:22:18AM +0100, Biju Das wrote: > Add an optional property renesas,usb-role-switch to support > dual role switch for USB Type-C DRP port controller devices > using USB role switch class framework. > > Signed-off-by: Biju Das <biju.das@xxxxxxxxxxxxxx> > --- > V4-->V5 > * No Change > V3-->V4 > * No Change > V2-->V3 > * Added optional renesas,usb-role-switch property. > V1-->V2 > * Added usb-role-switch-property > * Updated the example with usb-role-switch property. > --- > .../devicetree/bindings/usb/renesas_usb3.txt | 22 ++++++++++++++++++++++ > 1 file changed, 22 insertions(+) > > diff --git a/Documentation/devicetree/bindings/usb/renesas_usb3.txt b/Documentation/devicetree/bindings/usb/renesas_usb3.txt > index 35039e7..f1cb06a 100644 > --- a/Documentation/devicetree/bindings/usb/renesas_usb3.txt > +++ b/Documentation/devicetree/bindings/usb/renesas_usb3.txt > @@ -22,6 +22,7 @@ Required properties: > Optional properties: > - phys: phandle + phy specifier pair > - phy-names: must be "usb" > + - renesas,usb-role-switch: use USB role switch to handle role switch events Mediatek and HiSilicon both have same or similar properties in patches under review. Please coordinate and document a common property. Really, I'm wondering why this is needed. Can't you walk the graph to the connector and determine if dual role is supported by the connector type? Rob
