* [PATCH V2] dt-bindings: usb: dwc3: Add interrupt-names property support for wakeup interrupt @ 2023-05-11 7:21 Piyush Mehta 2023-05-11 9:34 ` Krzysztof Kozlowski 0 siblings, 1 reply; 4+ messages in thread From: Piyush Mehta @ 2023-05-11 7:21 UTC (permalink / raw) To: gregkh, robh+dt, krzysztof.kozlowski+dt, conor+dt, balbi, michal.simek Cc: linux-usb, devicetree, linux-kernel, michal.simek, siva.durga.prasad.paladugu, git, Piyush Mehta 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] clocks: description: -- 2.25.1 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH V2] dt-bindings: usb: dwc3: Add interrupt-names property support for wakeup interrupt 2023-05-11 7:21 [PATCH V2] dt-bindings: usb: dwc3: Add interrupt-names property support for wakeup interrupt Piyush Mehta @ 2023-05-11 9:34 ` Krzysztof Kozlowski 2023-05-12 6:57 ` Mehta, Piyush 0 siblings, 1 reply; 4+ messages in thread From: Krzysztof Kozlowski @ 2023-05-11 9:34 UTC (permalink / raw) To: Piyush Mehta, gregkh, robh+dt, krzysztof.kozlowski+dt, conor+dt, balbi, michal.simek Cc: linux-usb, devicetree, linux-kernel, michal.simek, siva.durga.prasad.paladugu, git 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 ^ permalink raw reply [flat|nested] 4+ messages in thread
* RE: [PATCH V2] dt-bindings: usb: dwc3: Add interrupt-names property support for wakeup interrupt 2023-05-11 9:34 ` Krzysztof Kozlowski @ 2023-05-12 6:57 ` Mehta, Piyush 2023-05-12 7:03 ` Krzysztof Kozlowski 0 siblings, 1 reply; 4+ messages in thread From: Mehta, Piyush @ 2023-05-12 6:57 UTC (permalink / raw) To: Krzysztof Kozlowski, 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, Paladugu, Siva Durga Prasad, git (AMD-Xilinx) 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 ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH V2] dt-bindings: usb: dwc3: Add interrupt-names property support for wakeup interrupt 2023-05-12 6:57 ` Mehta, Piyush @ 2023-05-12 7:03 ` Krzysztof Kozlowski 0 siblings, 0 replies; 4+ messages in thread From: Krzysztof Kozlowski @ 2023-05-12 7:03 UTC (permalink / raw) To: Mehta, Piyush, 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, Paladugu, Siva Durga Prasad, git (AMD-Xilinx) 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 ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2023-05-12 7:03 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2023-05-11 7:21 [PATCH V2] dt-bindings: usb: dwc3: Add interrupt-names property support for wakeup interrupt Piyush Mehta 2023-05-11 9:34 ` Krzysztof Kozlowski 2023-05-12 6:57 ` Mehta, Piyush 2023-05-12 7:03 ` Krzysztof Kozlowski
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).