On 10/24/21 2:36 AM, Marek Vasut wrote: > Copy the otg-rev property documentation from usb-drd.yaml into the DWC2 > binding document, since some users of the DWC2 IP like STM32MP1 use the > otg-rev property in DT bindings for this controller. > > Signed-off-by: Marek Vasut <marex@xxxxxxx> > Cc: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> > Cc: Rob Herring <robh+dt@xxxxxxxxxx> > Cc: devicetree@xxxxxxxxxxxxxxx > --- > NOTE: Shouldn't this somehow be part of the USB core bindings instead? Hi Marek, Yes, this is similar to patch series I sent recently, see [1]. [1] https://lore.kernel.org/linux-usb/1634133425-25670-1-git-send-email-fabrice.gasnier@xxxxxxxxxxx/ BR, Fabrice > --- > Documentation/devicetree/bindings/usb/dwc2.yaml | 10 ++++++++++ > 1 file changed, 10 insertions(+) > > diff --git a/Documentation/devicetree/bindings/usb/dwc2.yaml b/Documentation/devicetree/bindings/usb/dwc2.yaml > index 10c7d9b6cc53f..254436c19870b 100644 > --- a/Documentation/devicetree/bindings/usb/dwc2.yaml > +++ b/Documentation/devicetree/bindings/usb/dwc2.yaml > @@ -120,6 +120,16 @@ properties: > $ref: /schemas/types.yaml#/definitions/uint32-array > description: size of periodic tx fifo per endpoint (except ep0) in gadget mode. > > + otg-rev: > + description: > + Tells usb driver the release number of the OTG and EH supplement with > + which the device and its descriptors are compliant, in binary-coded > + decimal (i.e. 2.0 is 0200H). This property is used if any real OTG > + features (HNP/SRP/ADP) is enabled. If ADP is required, otg-rev should be > + 0x0200 or above. > + $ref: /schemas/types.yaml#/definitions/uint32 > + enum: [0x0100, 0x0120, 0x0130, 0x0200] > + > snps,need-phy-for-wake: > $ref: /schemas/types.yaml#/definitions/flag > description: If present indicates that the phy needs to be left on for >
