Message ID | 20220224133125.6674-3-srinivas.kandagatla@linaro.org |
---|---|
State | Changes Requested, archived |
Headers | show |
Series | soundwire: qcom: add pm runtime support | expand |
On Thu, Feb 24, 2022 at 01:31:24PM +0000, Srinivas Kandagatla wrote: > Wake IRQ is optional interrupt that can be wired up on SoundWire controller > instances like RX path along with MBHC(Multi Button Headset connection). > Document this in bindings. > > Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org> > --- > Documentation/devicetree/bindings/soundwire/qcom,sdw.txt | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/soundwire/qcom,sdw.txt b/Documentation/devicetree/bindings/soundwire/qcom,sdw.txt > index b93a2b3e029d..bade68f429b0 100644 > --- a/Documentation/devicetree/bindings/soundwire/qcom,sdw.txt > +++ b/Documentation/devicetree/bindings/soundwire/qcom,sdw.txt > @@ -22,7 +22,7 @@ board specific bus parameters. > - interrupts: > Usage: required > Value type: <prop-encoded-array> > - Definition: should specify the SoundWire Controller IRQ > + Definition: should specify the SoundWire Controller and optional wake IRQ What about 'wakeup-source' property? > > - clock-names: > Usage: required > -- > 2.21.0 > >
On 25/02/2022 20:21, Rob Herring wrote: > On Thu, Feb 24, 2022 at 01:31:24PM +0000, Srinivas Kandagatla wrote: >> Wake IRQ is optional interrupt that can be wired up on SoundWire controller >> instances like RX path along with MBHC(Multi Button Headset connection). >> Document this in bindings. >> >> Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org> >> --- >> Documentation/devicetree/bindings/soundwire/qcom,sdw.txt | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/Documentation/devicetree/bindings/soundwire/qcom,sdw.txt b/Documentation/devicetree/bindings/soundwire/qcom,sdw.txt >> index b93a2b3e029d..bade68f429b0 100644 >> --- a/Documentation/devicetree/bindings/soundwire/qcom,sdw.txt >> +++ b/Documentation/devicetree/bindings/soundwire/qcom,sdw.txt >> @@ -22,7 +22,7 @@ board specific bus parameters. >> - interrupts: >> Usage: required >> Value type: <prop-encoded-array> >> - Definition: should specify the SoundWire Controller IRQ >> + Definition: should specify the SoundWire Controller and optional wake IRQ > > What about 'wakeup-source' property? Thanks for the hint, I was not aware of this flag. Will add this in next version. --srini > >> >> - clock-names: >> Usage: required >> -- >> 2.21.0 >> >>
diff --git a/Documentation/devicetree/bindings/soundwire/qcom,sdw.txt b/Documentation/devicetree/bindings/soundwire/qcom,sdw.txt index b93a2b3e029d..bade68f429b0 100644 --- a/Documentation/devicetree/bindings/soundwire/qcom,sdw.txt +++ b/Documentation/devicetree/bindings/soundwire/qcom,sdw.txt @@ -22,7 +22,7 @@ board specific bus parameters. - interrupts: Usage: required Value type: <prop-encoded-array> - Definition: should specify the SoundWire Controller IRQ + Definition: should specify the SoundWire Controller and optional wake IRQ - clock-names: Usage: required
Wake IRQ is optional interrupt that can be wired up on SoundWire controller instances like RX path along with MBHC(Multi Button Headset connection). Document this in bindings. Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org> --- Documentation/devicetree/bindings/soundwire/qcom,sdw.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)