Messages in this thread | | | Date | Thu, 11 May 2023 11:34:42 +0200 | Subject | Re: [PATCH V2] dt-bindings: usb: dwc3: Add interrupt-names property support for wakeup interrupt | From | Krzysztof Kozlowski <> |
| |
On 11/05/2023 09:21, Piyush Mehta wrote: > The hibernation feature enabled for Xilinx Versal NET SoC in DWC3 IP. > As the DWC3 IP supports the hibernation feature, to handle the wakeup > or hibernation interrupt, add host mode "wakeup" interrupt-names > optional property in the binding schema to capture remote-wakeup and > connect/ disconnect event in the hibernation state. > > Signed-off-by: Piyush Mehta <piyush.mehta@amd.com> > --- > Change in V2: > - Addressed ROB review comments > - Updated name of interrupt-names property with "wakeup" > - Move interrupt-names property from dwc3-xilinx core to dwc3 core. > > Link: https://lore.kernel.org/all/CAL_JsqK6_7XD7+w+EQvPPmbmSOpfo3JDb0xDN4StuHUm1kgchw@mail.gmail.com/ > --- > Documentation/devicetree/bindings/usb/snps,dwc3.yaml | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/Documentation/devicetree/bindings/usb/snps,dwc3.yaml b/Documentation/devicetree/bindings/usb/snps,dwc3.yaml > index 50edc4da780e..db512769bd80 100644 > --- a/Documentation/devicetree/bindings/usb/snps,dwc3.yaml > +++ b/Documentation/devicetree/bindings/usb/snps,dwc3.yaml > @@ -53,6 +53,8 @@ properties: > - const: dwc_usb3 > - items: > enum: [host, peripheral, otg] > + - items: > + enum: [dwc_usb3, otg, wakeup]
You didn't add only "wakeup" but also dwc_usb3 which does not look correct here as it is common interrupt. The change does not match commit msg and I don't know what you actually want to achieve.
Best regards, Krzysztof
| |