* [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 related [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 related [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 related [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 related [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).