diff mbox

[v5,5/6] dt-bindings: power: reset: qcom: Add resin binding

Message ID 20180628150850.15358-6-vkoul@kernel.org (mailing list archive)
State Not Applicable, archived
Headers show

Commit Message

Vinod Koul June 28, 2018, 3:08 p.m. UTC
Resin is similar to pwrkey, add the description and example
for resin bindings

Signed-off-by: Vinod Koul <vkoul@kernel.org>
---
 .../devicetree/bindings/input/qcom,pm8941-pwrkey.txt       |  1 +
 Documentation/devicetree/bindings/power/reset/qcom,pon.txt | 14 ++++++++++++--
 2 files changed, 13 insertions(+), 2 deletions(-)

Comments

Rob Herring (Arm) July 3, 2018, 5:19 p.m. UTC | #1
On Thu, Jun 28, 2018 at 08:38:49PM +0530, Vinod Koul wrote:
> Resin is similar to pwrkey, add the description and example
> for resin bindings
> 
> Signed-off-by: Vinod Koul <vkoul@kernel.org>
> ---
>  .../devicetree/bindings/input/qcom,pm8941-pwrkey.txt       |  1 +
>  Documentation/devicetree/bindings/power/reset/qcom,pon.txt | 14 ++++++++++++--
>  2 files changed, 13 insertions(+), 2 deletions(-)

Reviewed-by: Rob Herring <robh@kernel.org>
Vinod Koul July 7, 2018, 5:14 a.m. UTC | #2
Hello Sebastian,

On 28-06-18, 20:38, Vinod Koul wrote:
> Resin is similar to pwrkey, add the description and example
> for resin bindings

I think this should go thru your tree as well

> 
> Signed-off-by: Vinod Koul <vkoul@kernel.org>
> ---
>  .../devicetree/bindings/input/qcom,pm8941-pwrkey.txt       |  1 +
>  Documentation/devicetree/bindings/power/reset/qcom,pon.txt | 14 ++++++++++++--
>  2 files changed, 13 insertions(+), 2 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/input/qcom,pm8941-pwrkey.txt b/Documentation/devicetree/bindings/input/qcom,pm8941-pwrkey.txt
> index bf719396eadd..34ab5763f494 100644
> --- a/Documentation/devicetree/bindings/input/qcom,pm8941-pwrkey.txt
> +++ b/Documentation/devicetree/bindings/input/qcom,pm8941-pwrkey.txt
> @@ -7,6 +7,7 @@ PROPERTIES
>  	Value type: <string>
>  	Definition: must be one of:
>  		    "qcom,pm8941-pwrkey"
> +		    "qcom,pm8941-resin"
>  
>  - reg:
>  	Usage: required
> diff --git a/Documentation/devicetree/bindings/power/reset/qcom,pon.txt b/Documentation/devicetree/bindings/power/reset/qcom,pon.txt
> index f32cdd358267..651491bb63b7 100644
> --- a/Documentation/devicetree/bindings/power/reset/qcom,pon.txt
> +++ b/Documentation/devicetree/bindings/power/reset/qcom,pon.txt
> @@ -1,9 +1,9 @@
>  Qualcomm PON Device
>  
>  The Power On device for Qualcomm PM8xxx is MFD supporting pwrkey
> -along with the Android reboot-mode.
> +and resin along with the Android reboot-mode.
>  
> -This DT node has pwrkey as sub node.
> +This DT node has pwrkey and resin as sub nodes.
>  
>  Required Properties:
>  -compatible: "qcom,pm8916-pon"
> @@ -12,6 +12,8 @@ Required Properties:
>  Optional subnode:
>  -pwrkey: Specifies the subnode pwrkey and should follow the
>   qcom,pm8941-pwrkey.txt description.
> +-resin: Specifies the subnode resin and should follow the
> + qcom,pm8xxx-pwrkey.txt description.
>  
>  The rest of the properties should follow the generic reboot-mode description
>  found in reboot-mode.txt
> @@ -32,4 +34,12 @@ Example:
>  			bias-pull-up;
>  			linux,code = <KEY_POWER>;
>  		};
> +
> +		resin {
> +			compatible = "qcom,pm8941-resin";
> +			interrupts = <0x0 0x8 1 IRQ_TYPE_EDGE_BOTH>;
> +			debounce = <15625>;
> +			bias-pull-up;
> +			linux,code = <KEY_VOLUMEDOWN>;
> +		};
>  	};
> -- 
> 2.14.4
Sebastian Reichel July 7, 2018, 8:24 p.m. UTC | #3
Hi Vinod,

On Sat, Jul 07, 2018 at 10:44:14AM +0530, Vinod wrote:
> On 28-06-18, 20:38, Vinod Koul wrote:
> > Resin is similar to pwrkey, add the description and example
> > for resin bindings
> 
> I think this should go thru your tree as well

Right. I just queued this one with Rob's Reviewed-by.

-- Sebastian

> > Signed-off-by: Vinod Koul <vkoul@kernel.org>
> > ---
> >  .../devicetree/bindings/input/qcom,pm8941-pwrkey.txt       |  1 +
> >  Documentation/devicetree/bindings/power/reset/qcom,pon.txt | 14 ++++++++++++--
> >  2 files changed, 13 insertions(+), 2 deletions(-)
> > 
> > diff --git a/Documentation/devicetree/bindings/input/qcom,pm8941-pwrkey.txt b/Documentation/devicetree/bindings/input/qcom,pm8941-pwrkey.txt
> > index bf719396eadd..34ab5763f494 100644
> > --- a/Documentation/devicetree/bindings/input/qcom,pm8941-pwrkey.txt
> > +++ b/Documentation/devicetree/bindings/input/qcom,pm8941-pwrkey.txt
> > @@ -7,6 +7,7 @@ PROPERTIES
> >  	Value type: <string>
> >  	Definition: must be one of:
> >  		    "qcom,pm8941-pwrkey"
> > +		    "qcom,pm8941-resin"
> >  
> >  - reg:
> >  	Usage: required
> > diff --git a/Documentation/devicetree/bindings/power/reset/qcom,pon.txt b/Documentation/devicetree/bindings/power/reset/qcom,pon.txt
> > index f32cdd358267..651491bb63b7 100644
> > --- a/Documentation/devicetree/bindings/power/reset/qcom,pon.txt
> > +++ b/Documentation/devicetree/bindings/power/reset/qcom,pon.txt
> > @@ -1,9 +1,9 @@
> >  Qualcomm PON Device
> >  
> >  The Power On device for Qualcomm PM8xxx is MFD supporting pwrkey
> > -along with the Android reboot-mode.
> > +and resin along with the Android reboot-mode.
> >  
> > -This DT node has pwrkey as sub node.
> > +This DT node has pwrkey and resin as sub nodes.
> >  
> >  Required Properties:
> >  -compatible: "qcom,pm8916-pon"
> > @@ -12,6 +12,8 @@ Required Properties:
> >  Optional subnode:
> >  -pwrkey: Specifies the subnode pwrkey and should follow the
> >   qcom,pm8941-pwrkey.txt description.
> > +-resin: Specifies the subnode resin and should follow the
> > + qcom,pm8xxx-pwrkey.txt description.
> >  
> >  The rest of the properties should follow the generic reboot-mode description
> >  found in reboot-mode.txt
> > @@ -32,4 +34,12 @@ Example:
> >  			bias-pull-up;
> >  			linux,code = <KEY_POWER>;
> >  		};
> > +
> > +		resin {
> > +			compatible = "qcom,pm8941-resin";
> > +			interrupts = <0x0 0x8 1 IRQ_TYPE_EDGE_BOTH>;
> > +			debounce = <15625>;
> > +			bias-pull-up;
> > +			linux,code = <KEY_VOLUMEDOWN>;
> > +		};
> >  	};
> > -- 
> > 2.14.4
> 
> -- 
> ~Vinod
diff mbox

Patch

diff --git a/Documentation/devicetree/bindings/input/qcom,pm8941-pwrkey.txt b/Documentation/devicetree/bindings/input/qcom,pm8941-pwrkey.txt
index bf719396eadd..34ab5763f494 100644
--- a/Documentation/devicetree/bindings/input/qcom,pm8941-pwrkey.txt
+++ b/Documentation/devicetree/bindings/input/qcom,pm8941-pwrkey.txt
@@ -7,6 +7,7 @@  PROPERTIES
 	Value type: <string>
 	Definition: must be one of:
 		    "qcom,pm8941-pwrkey"
+		    "qcom,pm8941-resin"
 
 - reg:
 	Usage: required
diff --git a/Documentation/devicetree/bindings/power/reset/qcom,pon.txt b/Documentation/devicetree/bindings/power/reset/qcom,pon.txt
index f32cdd358267..651491bb63b7 100644
--- a/Documentation/devicetree/bindings/power/reset/qcom,pon.txt
+++ b/Documentation/devicetree/bindings/power/reset/qcom,pon.txt
@@ -1,9 +1,9 @@ 
 Qualcomm PON Device
 
 The Power On device for Qualcomm PM8xxx is MFD supporting pwrkey
-along with the Android reboot-mode.
+and resin along with the Android reboot-mode.
 
-This DT node has pwrkey as sub node.
+This DT node has pwrkey and resin as sub nodes.
 
 Required Properties:
 -compatible: "qcom,pm8916-pon"
@@ -12,6 +12,8 @@  Required Properties:
 Optional subnode:
 -pwrkey: Specifies the subnode pwrkey and should follow the
  qcom,pm8941-pwrkey.txt description.
+-resin: Specifies the subnode resin and should follow the
+ qcom,pm8xxx-pwrkey.txt description.
 
 The rest of the properties should follow the generic reboot-mode description
 found in reboot-mode.txt
@@ -32,4 +34,12 @@  Example:
 			bias-pull-up;
 			linux,code = <KEY_POWER>;
 		};
+
+		resin {
+			compatible = "qcom,pm8941-resin";
+			interrupts = <0x0 0x8 1 IRQ_TYPE_EDGE_BOTH>;
+			debounce = <15625>;
+			bias-pull-up;
+			linux,code = <KEY_VOLUMEDOWN>;
+		};
 	};