linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/4] dt-bindings: mfd: Update TPS65217 interrupts
@ 2016-12-09  6:28 Milo Kim
  2016-12-09  6:28 ` [PATCH 1/4] ARM: dts: am335x: Fix the interrupt name of TPS65217 Milo Kim
                   ` (4 more replies)
  0 siblings, 5 replies; 13+ messages in thread
From: Milo Kim @ 2016-12-09  6:28 UTC (permalink / raw)
  To: bcousson, Tony Lindgren
  Cc: Arnd Bergmann, Lee Jones, Sebastian Reichel, Dmitry Torokhov,
	linux-omap, devicetree, linux-arm-kernel, linux-kernel, Milo Kim

This patch-set fixes wrong property name and uses TPS65217 HW interrupt 
number from the datasheet instead of the DT ABI. DT bindings are also 
updated.

Milo Kim (4):
  ARM: dts: am335x: Fix the interrupt name of TPS65217
  dt-bindings: mfd: Remove TPS65217 interrupts
  dt-bindings: power/supply: Update TPS65217 properties
  dt-bindings: input: Add interrupt number for TPS65217

 .../bindings/input/tps65218-pwrbutton.txt          |  4 +++-
 .../bindings/power/supply/tps65217_charger.txt     |  7 +++++-
 arch/arm/boot/dts/am335x-bone-common.dtsi          |  8 +++----
 include/dt-bindings/mfd/tps65217.h                 | 26 ----------------------
 4 files changed, 12 insertions(+), 33 deletions(-)
 delete mode 100644 include/dt-bindings/mfd/tps65217.h

-- 
2.9.3

^ permalink raw reply	[flat|nested] 13+ messages in thread

* [PATCH 1/4] ARM: dts: am335x: Fix the interrupt name of TPS65217
  2016-12-09  6:28 [PATCH 0/4] dt-bindings: mfd: Update TPS65217 interrupts Milo Kim
@ 2016-12-09  6:28 ` Milo Kim
  2016-12-09  6:28 ` [PATCH 2/4] dt-bindings: mfd: Remove TPS65217 interrupts Milo Kim
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 13+ messages in thread
From: Milo Kim @ 2016-12-09  6:28 UTC (permalink / raw)
  To: bcousson, Tony Lindgren
  Cc: Arnd Bergmann, Lee Jones, Sebastian Reichel, Dmitry Torokhov,
	linux-omap, devicetree, linux-arm-kernel, linux-kernel, Milo Kim

Use 'interrupt-names' for getting the charger interrupt number.

Fixes: 1934e89a769b ("ARM: dts: am335x: Add the charger interrupt")
Signed-off-by: Milo Kim <woogyom.kim@gmail.com>
---
 arch/arm/boot/dts/am335x-bone-common.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/am335x-bone-common.dtsi b/arch/arm/boot/dts/am335x-bone-common.dtsi
index dc561d5..14b6269 100644
--- a/arch/arm/boot/dts/am335x-bone-common.dtsi
+++ b/arch/arm/boot/dts/am335x-bone-common.dtsi
@@ -320,7 +320,7 @@
 
 	charger {
 		interrupts = <TPS65217_IRQ_AC>, <TPS65217_IRQ_USB>;
-		interrupts-names = "AC", "USB";
+		interrupt-names = "AC", "USB";
 		status = "okay";
 	};
 
-- 
2.9.3

^ permalink raw reply	[flat|nested] 13+ messages in thread

* [PATCH 2/4] dt-bindings: mfd: Remove TPS65217 interrupts
  2016-12-09  6:28 [PATCH 0/4] dt-bindings: mfd: Update TPS65217 interrupts Milo Kim
  2016-12-09  6:28 ` [PATCH 1/4] ARM: dts: am335x: Fix the interrupt name of TPS65217 Milo Kim
@ 2016-12-09  6:28 ` Milo Kim
  2016-12-12 17:25   ` Rob Herring
  2017-01-04 10:21   ` Lee Jones
  2016-12-09  6:28 ` [PATCH 3/4] dt-bindings: power/supply: Update TPS65217 properties Milo Kim
                   ` (2 subsequent siblings)
  4 siblings, 2 replies; 13+ messages in thread
From: Milo Kim @ 2016-12-09  6:28 UTC (permalink / raw)
  To: bcousson, Tony Lindgren
  Cc: Arnd Bergmann, Lee Jones, Sebastian Reichel, Dmitry Torokhov,
	linux-omap, devicetree, linux-arm-kernel, linux-kernel, Milo Kim

Interrupt numbers are from the datasheet, so no need to keep them in
the ABI. Use the number in the DT file.

Signed-off-by: Milo Kim <woogyom.kim@gmail.com>
---
 arch/arm/boot/dts/am335x-bone-common.dtsi |  8 +++-----
 include/dt-bindings/mfd/tps65217.h        | 26 --------------------------
 2 files changed, 3 insertions(+), 31 deletions(-)
 delete mode 100644 include/dt-bindings/mfd/tps65217.h

diff --git a/arch/arm/boot/dts/am335x-bone-common.dtsi b/arch/arm/boot/dts/am335x-bone-common.dtsi
index 14b6269..3e32dd1 100644
--- a/arch/arm/boot/dts/am335x-bone-common.dtsi
+++ b/arch/arm/boot/dts/am335x-bone-common.dtsi
@@ -6,8 +6,6 @@
  * published by the Free Software Foundation.
  */
 
-#include <dt-bindings/mfd/tps65217.h>
-
 / {
 	cpus {
 		cpu@0 {
@@ -319,13 +317,13 @@
 	ti,pmic-shutdown-controller;
 
 	charger {
-		interrupts = <TPS65217_IRQ_AC>, <TPS65217_IRQ_USB>;
-		interrupt-names = "AC", "USB";
+		interrupts = <0>, <1>;
+		interrupt-names = "USB", "AC";
 		status = "okay";
 	};
 
 	pwrbutton {
-		interrupts = <TPS65217_IRQ_PB>;
+		interrupts = <2>;
 		status = "okay";
 	};
 
diff --git a/include/dt-bindings/mfd/tps65217.h b/include/dt-bindings/mfd/tps65217.h
deleted file mode 100644
index cafb9e6..0000000
--- a/include/dt-bindings/mfd/tps65217.h
+++ /dev/null
@@ -1,26 +0,0 @@
-/*
- * This header provides macros for TI TPS65217 DT bindings.
- *
- * Copyright (C) 2016 Texas Instruments
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License version 2 as
- * published by the Free Software Foundation.
- *
- * This program is distributed in the hope that it will be useful, but
- * WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
- * General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License along with
- * this program.  If not, see <http://www.gnu.org/licenses/>.
- */
-
-#ifndef __DT_BINDINGS_TPS65217_H__
-#define __DT_BINDINGS_TPS65217_H__
-
-#define TPS65217_IRQ_USB	0
-#define TPS65217_IRQ_AC		1
-#define TPS65217_IRQ_PB		2
-
-#endif
-- 
2.9.3

^ permalink raw reply	[flat|nested] 13+ messages in thread

* [PATCH 3/4] dt-bindings: power/supply: Update TPS65217 properties
  2016-12-09  6:28 [PATCH 0/4] dt-bindings: mfd: Update TPS65217 interrupts Milo Kim
  2016-12-09  6:28 ` [PATCH 1/4] ARM: dts: am335x: Fix the interrupt name of TPS65217 Milo Kim
  2016-12-09  6:28 ` [PATCH 2/4] dt-bindings: mfd: Remove TPS65217 interrupts Milo Kim
@ 2016-12-09  6:28 ` Milo Kim
  2016-12-12 17:26   ` Rob Herring
  2017-01-12  3:47   ` Sebastian Reichel
  2016-12-09  6:28 ` [PATCH 4/4] dt-bindings: input: Specify the interrupt number of TPS65217 power button Milo Kim
  2016-12-27 17:52 ` [PATCH 0/4] dt-bindings: mfd: Update TPS65217 interrupts Tony Lindgren
  4 siblings, 2 replies; 13+ messages in thread
From: Milo Kim @ 2016-12-09  6:28 UTC (permalink / raw)
  To: bcousson, Tony Lindgren
  Cc: Arnd Bergmann, Lee Jones, Sebastian Reichel, Dmitry Torokhov,
	linux-omap, devicetree, linux-arm-kernel, linux-kernel, Milo Kim

Add interrupt specifiers for USB and AC charger input. Interrupt numbers
are from the datasheet.
Fix wrong property for compatible string.

Signed-off-by: Milo Kim <woogyom.kim@gmail.com>
---
 .../devicetree/bindings/power/supply/tps65217_charger.txt          | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/power/supply/tps65217_charger.txt b/Documentation/devicetree/bindings/power/supply/tps65217_charger.txt
index 98d131a..a11072c 100644
--- a/Documentation/devicetree/bindings/power/supply/tps65217_charger.txt
+++ b/Documentation/devicetree/bindings/power/supply/tps65217_charger.txt
@@ -2,11 +2,16 @@ TPS65217 Charger
 
 Required Properties:
 -compatible: "ti,tps65217-charger"
+-interrupts: TPS65217 interrupt numbers for the AC and USB charger input change.
+             Should be <0> for the USB charger and <1> for the AC adapter.
+-interrupt-names: Should be "USB" and "AC"
 
 This node is a subnode of the tps65217 PMIC.
 
 Example:
 
 	tps65217-charger {
-		compatible = "ti,tps65090-charger";
+		compatible = "ti,tps65217-charger";
+		interrupts = <0>, <1>;
+		interrupt-names = "USB", "AC";
 	};
-- 
2.9.3

^ permalink raw reply	[flat|nested] 13+ messages in thread

* [PATCH 4/4] dt-bindings: input: Specify the interrupt number of TPS65217 power button
  2016-12-09  6:28 [PATCH 0/4] dt-bindings: mfd: Update TPS65217 interrupts Milo Kim
                   ` (2 preceding siblings ...)
  2016-12-09  6:28 ` [PATCH 3/4] dt-bindings: power/supply: Update TPS65217 properties Milo Kim
@ 2016-12-09  6:28 ` Milo Kim
  2016-12-12 17:27   ` Rob Herring
  2016-12-27 17:52 ` [PATCH 0/4] dt-bindings: mfd: Update TPS65217 interrupts Tony Lindgren
  4 siblings, 1 reply; 13+ messages in thread
From: Milo Kim @ 2016-12-09  6:28 UTC (permalink / raw)
  To: bcousson, Tony Lindgren
  Cc: Arnd Bergmann, Lee Jones, Sebastian Reichel, Dmitry Torokhov,
	linux-omap, devicetree, linux-arm-kernel, linux-kernel, Milo Kim

Specify the power button interrupt number which is from the datasheet.

Signed-off-by: Milo Kim <woogyom.kim@gmail.com>
---
 Documentation/devicetree/bindings/input/tps65218-pwrbutton.txt | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/input/tps65218-pwrbutton.txt b/Documentation/devicetree/bindings/input/tps65218-pwrbutton.txt
index 3e5b979..8682ab6 100644
--- a/Documentation/devicetree/bindings/input/tps65218-pwrbutton.txt
+++ b/Documentation/devicetree/bindings/input/tps65218-pwrbutton.txt
@@ -8,8 +8,9 @@ This driver provides a simple power button event via an Interrupt.
 Required properties:
 - compatible: should be "ti,tps65217-pwrbutton" or "ti,tps65218-pwrbutton"
 
-Required properties for TPS65218:
+Required properties:
 - interrupts: should be one of the following
+   - <2>: For controllers compatible with tps65217
    - <3 IRQ_TYPE_EDGE_BOTH>: For controllers compatible with tps65218
 
 Examples:
@@ -17,6 +18,7 @@ Examples:
 &tps {
 	tps65217-pwrbutton {
 		compatible = "ti,tps65217-pwrbutton";
+		interrupts = <2>;
 	};
 };
 
-- 
2.9.3

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 2/4] dt-bindings: mfd: Remove TPS65217 interrupts
  2016-12-09  6:28 ` [PATCH 2/4] dt-bindings: mfd: Remove TPS65217 interrupts Milo Kim
@ 2016-12-12 17:25   ` Rob Herring
  2016-12-12 23:24     ` Milo Kim
  2017-01-04 10:21   ` Lee Jones
  1 sibling, 1 reply; 13+ messages in thread
From: Rob Herring @ 2016-12-12 17:25 UTC (permalink / raw)
  To: Milo Kim
  Cc: bcousson, Tony Lindgren, Arnd Bergmann, Lee Jones,
	Sebastian Reichel, Dmitry Torokhov, linux-omap, devicetree,
	linux-arm-kernel, linux-kernel

On Fri, Dec 09, 2016 at 03:28:31PM +0900, Milo Kim wrote:
> Interrupt numbers are from the datasheet, so no need to keep them in
> the ABI. Use the number in the DT file.

I don't see the purpose of ripping this out. The headers have always 
been for convienence, not whether the values come from the datasheet or 
not.

> Signed-off-by: Milo Kim <woogyom.kim@gmail.com>
> ---
>  arch/arm/boot/dts/am335x-bone-common.dtsi |  8 +++-----
>  include/dt-bindings/mfd/tps65217.h        | 26 --------------------------
>  2 files changed, 3 insertions(+), 31 deletions(-)
>  delete mode 100644 include/dt-bindings/mfd/tps65217.h

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 3/4] dt-bindings: power/supply: Update TPS65217 properties
  2016-12-09  6:28 ` [PATCH 3/4] dt-bindings: power/supply: Update TPS65217 properties Milo Kim
@ 2016-12-12 17:26   ` Rob Herring
  2017-01-12  3:47   ` Sebastian Reichel
  1 sibling, 0 replies; 13+ messages in thread
From: Rob Herring @ 2016-12-12 17:26 UTC (permalink / raw)
  To: Milo Kim
  Cc: bcousson, Tony Lindgren, Arnd Bergmann, Lee Jones,
	Sebastian Reichel, Dmitry Torokhov, linux-omap, devicetree,
	linux-arm-kernel, linux-kernel

On Fri, Dec 09, 2016 at 03:28:32PM +0900, Milo Kim wrote:
> Add interrupt specifiers for USB and AC charger input. Interrupt numbers
> are from the datasheet.
> Fix wrong property for compatible string.
> 
> Signed-off-by: Milo Kim <woogyom.kim@gmail.com>
> ---
>  .../devicetree/bindings/power/supply/tps65217_charger.txt          | 7 ++++++-
>  1 file changed, 6 insertions(+), 1 deletion(-)

Acked-by: Rob Herring <robh@kernel.org>

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 4/4] dt-bindings: input: Specify the interrupt number of TPS65217 power button
  2016-12-09  6:28 ` [PATCH 4/4] dt-bindings: input: Specify the interrupt number of TPS65217 power button Milo Kim
@ 2016-12-12 17:27   ` Rob Herring
  0 siblings, 0 replies; 13+ messages in thread
From: Rob Herring @ 2016-12-12 17:27 UTC (permalink / raw)
  To: Milo Kim
  Cc: bcousson, Tony Lindgren, Arnd Bergmann, Lee Jones,
	Sebastian Reichel, Dmitry Torokhov, linux-omap, devicetree,
	linux-arm-kernel, linux-kernel

On Fri, Dec 09, 2016 at 03:28:33PM +0900, Milo Kim wrote:
> Specify the power button interrupt number which is from the datasheet.
> 
> Signed-off-by: Milo Kim <woogyom.kim@gmail.com>
> ---
>  Documentation/devicetree/bindings/input/tps65218-pwrbutton.txt | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)

Acked-by: Rob Herring <robh@kernel.org>

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 2/4] dt-bindings: mfd: Remove TPS65217 interrupts
  2016-12-12 17:25   ` Rob Herring
@ 2016-12-12 23:24     ` Milo Kim
  2016-12-13 21:09       ` Rob Herring
  0 siblings, 1 reply; 13+ messages in thread
From: Milo Kim @ 2016-12-12 23:24 UTC (permalink / raw)
  To: Rob Herring, Arnd Bergmann
  Cc: bcousson, Tony Lindgren, Lee Jones, Sebastian Reichel,
	Dmitry Torokhov, linux-omap, devicetree, linux-arm-kernel,
	linux-kernel

On 12/13/2016 02:25 AM, Rob Herring wrote:
> On Fri, Dec 09, 2016 at 03:28:31PM +0900, Milo Kim wrote:
>> Interrupt numbers are from the datasheet, so no need to keep them in
>> the ABI. Use the number in the DT file.
> I don't see the purpose of ripping this out. The headers have always
> been for convienence, not whether the values come from the datasheet or
> not.

My understanding is it's a same rule as other interrupt controllers.
I'd like to have Arnd's opinion for this.

Best regards,
Milo

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 2/4] dt-bindings: mfd: Remove TPS65217 interrupts
  2016-12-12 23:24     ` Milo Kim
@ 2016-12-13 21:09       ` Rob Herring
  0 siblings, 0 replies; 13+ messages in thread
From: Rob Herring @ 2016-12-13 21:09 UTC (permalink / raw)
  To: Milo Kim
  Cc: Arnd Bergmann, Benoit Cousson, Tony Lindgren, Lee Jones,
	Sebastian Reichel, Dmitry Torokhov, linux-omap, devicetree,
	linux-arm-kernel, linux-kernel

On Mon, Dec 12, 2016 at 5:24 PM, Milo Kim <woogyom.kim@gmail.com> wrote:
> On 12/13/2016 02:25 AM, Rob Herring wrote:
>>
>> On Fri, Dec 09, 2016 at 03:28:31PM +0900, Milo Kim wrote:
>>>
>>> Interrupt numbers are from the datasheet, so no need to keep them in
>>> the ABI. Use the number in the DT file.
>>
>> I don't see the purpose of ripping this out. The headers have always
>> been for convienence, not whether the values come from the datasheet or
>> not.
>
>
> My understanding is it's a same rule as other interrupt controllers.

Oh yes, that's true. We never use defines for interrupts. In that case:

Acked-by: Rob Herring <robh@kernel.org>

Rob

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 0/4] dt-bindings: mfd: Update TPS65217 interrupts
  2016-12-09  6:28 [PATCH 0/4] dt-bindings: mfd: Update TPS65217 interrupts Milo Kim
                   ` (3 preceding siblings ...)
  2016-12-09  6:28 ` [PATCH 4/4] dt-bindings: input: Specify the interrupt number of TPS65217 power button Milo Kim
@ 2016-12-27 17:52 ` Tony Lindgren
  4 siblings, 0 replies; 13+ messages in thread
From: Tony Lindgren @ 2016-12-27 17:52 UTC (permalink / raw)
  To: Milo Kim
  Cc: bcousson, Arnd Bergmann, Lee Jones, Sebastian Reichel,
	Dmitry Torokhov, linux-omap, devicetree, linux-arm-kernel,
	linux-kernel

* Milo Kim <woogyom.kim@gmail.com> [161208 22:29]:
> This patch-set fixes wrong property name and uses TPS65217 HW interrupt 
> number from the datasheet instead of the DT ABI. DT bindings are also 
> updated.

Applying into omap-for-v4.10/fixes before this usage pops up on other
am335x boards.

Regards,

Tony

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 2/4] dt-bindings: mfd: Remove TPS65217 interrupts
  2016-12-09  6:28 ` [PATCH 2/4] dt-bindings: mfd: Remove TPS65217 interrupts Milo Kim
  2016-12-12 17:25   ` Rob Herring
@ 2017-01-04 10:21   ` Lee Jones
  1 sibling, 0 replies; 13+ messages in thread
From: Lee Jones @ 2017-01-04 10:21 UTC (permalink / raw)
  To: Milo Kim
  Cc: bcousson, Tony Lindgren, Arnd Bergmann, Sebastian Reichel,
	Dmitry Torokhov, linux-omap, devicetree, linux-arm-kernel,
	linux-kernel

On Fri, 09 Dec 2016, Milo Kim wrote:

> Interrupt numbers are from the datasheet, so no need to keep them in
> the ABI. Use the number in the DT file.
> 
> Signed-off-by: Milo Kim <woogyom.kim@gmail.com>
> ---
>  arch/arm/boot/dts/am335x-bone-common.dtsi |  8 +++-----
>  include/dt-bindings/mfd/tps65217.h        | 26 --------------------------
>  2 files changed, 3 insertions(+), 31 deletions(-)
>  delete mode 100644 include/dt-bindings/mfd/tps65217.h

Acked-by: Lee Jones <lee.jones@linaro.org>

I'm happy for this to go in via your respective ARM tree.

> diff --git a/arch/arm/boot/dts/am335x-bone-common.dtsi b/arch/arm/boot/dts/am335x-bone-common.dtsi
> index 14b6269..3e32dd1 100644
> --- a/arch/arm/boot/dts/am335x-bone-common.dtsi
> +++ b/arch/arm/boot/dts/am335x-bone-common.dtsi
> @@ -6,8 +6,6 @@
>   * published by the Free Software Foundation.
>   */
>  
> -#include <dt-bindings/mfd/tps65217.h>
> -
>  / {
>  	cpus {
>  		cpu@0 {
> @@ -319,13 +317,13 @@
>  	ti,pmic-shutdown-controller;
>  
>  	charger {
> -		interrupts = <TPS65217_IRQ_AC>, <TPS65217_IRQ_USB>;
> -		interrupt-names = "AC", "USB";
> +		interrupts = <0>, <1>;
> +		interrupt-names = "USB", "AC";
>  		status = "okay";
>  	};
>  
>  	pwrbutton {
> -		interrupts = <TPS65217_IRQ_PB>;
> +		interrupts = <2>;
>  		status = "okay";
>  	};
>  
> diff --git a/include/dt-bindings/mfd/tps65217.h b/include/dt-bindings/mfd/tps65217.h
> deleted file mode 100644
> index cafb9e6..0000000
> --- a/include/dt-bindings/mfd/tps65217.h
> +++ /dev/null
> @@ -1,26 +0,0 @@
> -/*
> - * This header provides macros for TI TPS65217 DT bindings.
> - *
> - * Copyright (C) 2016 Texas Instruments
> - *
> - * This program is free software; you can redistribute it and/or modify
> - * it under the terms of the GNU General Public License version 2 as
> - * published by the Free Software Foundation.
> - *
> - * This program is distributed in the hope that it will be useful, but
> - * WITHOUT ANY WARRANTY; without even the implied warranty of
> - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
> - * General Public License for more details.
> - *
> - * You should have received a copy of the GNU General Public License along with
> - * this program.  If not, see <http://www.gnu.org/licenses/>.
> - */
> -
> -#ifndef __DT_BINDINGS_TPS65217_H__
> -#define __DT_BINDINGS_TPS65217_H__
> -
> -#define TPS65217_IRQ_USB	0
> -#define TPS65217_IRQ_AC		1
> -#define TPS65217_IRQ_PB		2
> -
> -#endif

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 3/4] dt-bindings: power/supply: Update TPS65217 properties
  2016-12-09  6:28 ` [PATCH 3/4] dt-bindings: power/supply: Update TPS65217 properties Milo Kim
  2016-12-12 17:26   ` Rob Herring
@ 2017-01-12  3:47   ` Sebastian Reichel
  1 sibling, 0 replies; 13+ messages in thread
From: Sebastian Reichel @ 2017-01-12  3:47 UTC (permalink / raw)
  To: Milo Kim
  Cc: bcousson, Tony Lindgren, Arnd Bergmann, Lee Jones,
	Dmitry Torokhov, linux-omap, devicetree, linux-arm-kernel,
	linux-kernel

[-- Attachment #1: Type: text/plain, Size: 1415 bytes --]

Hi Milo,

On Fri, Dec 09, 2016 at 03:28:32PM +0900, Milo Kim wrote:
> Add interrupt specifiers for USB and AC charger input. Interrupt numbers
> are from the datasheet.
> Fix wrong property for compatible string.
> 
> Signed-off-by: Milo Kim <woogyom.kim@gmail.com>
> ---
>  .../devicetree/bindings/power/supply/tps65217_charger.txt          | 7 ++++++-
>  1 file changed, 6 insertions(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/power/supply/tps65217_charger.txt b/Documentation/devicetree/bindings/power/supply/tps65217_charger.txt
> index 98d131a..a11072c 100644
> --- a/Documentation/devicetree/bindings/power/supply/tps65217_charger.txt
> +++ b/Documentation/devicetree/bindings/power/supply/tps65217_charger.txt
> @@ -2,11 +2,16 @@ TPS65217 Charger
>  
>  Required Properties:
>  -compatible: "ti,tps65217-charger"
> +-interrupts: TPS65217 interrupt numbers for the AC and USB charger input change.
> +             Should be <0> for the USB charger and <1> for the AC adapter.
> +-interrupt-names: Should be "USB" and "AC"
>  
>  This node is a subnode of the tps65217 PMIC.
>  
>  Example:
>  
>  	tps65217-charger {
> -		compatible = "ti,tps65090-charger";
> +		compatible = "ti,tps65217-charger";
> +		interrupts = <0>, <1>;
> +		interrupt-names = "USB", "AC";
>  	};

Thanks, I queued this into power-supply's for-next branch.

-- Sebastian

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 13+ messages in thread

end of thread, other threads:[~2017-01-12  3:47 UTC | newest]

Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-12-09  6:28 [PATCH 0/4] dt-bindings: mfd: Update TPS65217 interrupts Milo Kim
2016-12-09  6:28 ` [PATCH 1/4] ARM: dts: am335x: Fix the interrupt name of TPS65217 Milo Kim
2016-12-09  6:28 ` [PATCH 2/4] dt-bindings: mfd: Remove TPS65217 interrupts Milo Kim
2016-12-12 17:25   ` Rob Herring
2016-12-12 23:24     ` Milo Kim
2016-12-13 21:09       ` Rob Herring
2017-01-04 10:21   ` Lee Jones
2016-12-09  6:28 ` [PATCH 3/4] dt-bindings: power/supply: Update TPS65217 properties Milo Kim
2016-12-12 17:26   ` Rob Herring
2017-01-12  3:47   ` Sebastian Reichel
2016-12-09  6:28 ` [PATCH 4/4] dt-bindings: input: Specify the interrupt number of TPS65217 power button Milo Kim
2016-12-12 17:27   ` Rob Herring
2016-12-27 17:52 ` [PATCH 0/4] dt-bindings: mfd: Update TPS65217 interrupts Tony Lindgren

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).