| Message ID | 20230511072154.2030703-1-piyush.mehta@amd.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | [V2] dt-bindings: usb: dwc3: Add interrupt-names property support for wakeup interrupt | expand |
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
Hi, > -----Original Message----- > From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > Sent: Thursday, May 11, 2023 3:05 PM > To: Mehta, Piyush <piyush.mehta@amd.com>; gregkh@linuxfoundation.org; > robh+dt@kernel.org; krzysztof.kozlowski+dt@linaro.org; > conor+dt@kernel.org; balbi@kernel.org; michal.simek@xilinx.com > Cc: linux-usb@vger.kernel.org; devicetree@vger.kernel.org; linux- > kernel@vger.kernel.org; Simek, Michal <michal.simek@amd.com>; Paladugu, > Siva Durga Prasad <siva.durga.prasad.paladugu@amd.com>; git (AMD-Xilinx) > <git@amd.com> > Subject: Re: [PATCH V2] dt-bindings: usb: dwc3: Add interrupt-names property > support for wakeup interrupt > > 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+EQvPPmbmSOpfo3JDb0xDN4St > u > > HUm1kgchw@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. > We have dedicated irq line for hibernation feature, "wakeup" irq line triggers hibernation interrupt and the system wakes up. DWC3 core supports the hibernation feature, we have a dedicated code which is yet to be upstreamed. As the hibernation feature provided by dwc3-core, so this will be supported by other SOC/vendors. For this, we need the interrupt-name/ interrupt property. To enable the hibernation feature for Xilinx Versal NET SoC, the interrupt-name enum would be: - enum [host, peripheral, otg, wakeup] Please provide valuable input for the same. > Best regards, > Krzysztof Regards, Piyush Mehta
On 12/05/2023 08:57, Mehta, Piyush wrote: > Hi, > >> -----Original Message----- >> From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> >> Sent: Thursday, May 11, 2023 3:05 PM >> To: Mehta, Piyush <piyush.mehta@amd.com>; gregkh@linuxfoundation.org; >> robh+dt@kernel.org; krzysztof.kozlowski+dt@linaro.org; >> conor+dt@kernel.org; balbi@kernel.org; michal.simek@xilinx.com >> Cc: linux-usb@vger.kernel.org; devicetree@vger.kernel.org; linux- >> kernel@vger.kernel.org; Simek, Michal <michal.simek@amd.com>; Paladugu, >> Siva Durga Prasad <siva.durga.prasad.paladugu@amd.com>; git (AMD-Xilinx) >> <git@amd.com> >> Subject: Re: [PATCH V2] dt-bindings: usb: dwc3: Add interrupt-names property >> support for wakeup interrupt >> >> 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+EQvPPmbmSOpfo3JDb0xDN4St >> u >>> HUm1kgchw@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. >> > > We have dedicated irq line for hibernation feature, "wakeup" irq line triggers hibernation interrupt and the system wakes up. > DWC3 core supports the hibernation feature, we have a dedicated code which is yet to be upstreamed. > As the hibernation feature provided by dwc3-core, so this will be supported by other SOC/vendors. > > For this, we need the interrupt-name/ interrupt property. To enable the hibernation feature for > Xilinx Versal NET SoC, the interrupt-name enum would be: > - enum [host, peripheral, otg, wakeup] Yes, this makes sense. Other combination could be dwc_usb3 and wakeup but only if it is real... adding random entries sounds like not really. Best regards, Krzysztof
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] clocks: description:
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(+)