All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ARM: dts: at91: sama5d2_xplained: Add charger node
@ 2017-08-15  6:24 ` Wenyou Yang
  0 siblings, 0 replies; 9+ messages in thread
From: Wenyou Yang @ 2017-08-15  6:24 UTC (permalink / raw)
  To: Alexandre Belloni, Nicolas Ferre, Rob Herring, Mark Rutland,
	Russell King
  Cc: linux-kernel, devicetree, linux-arm-kernel, Wenyou Yang, Wenyou Yang

From: Wenyou Yang <wenyou.yang@atmel.com>

Add the charger device node as a sub-device of act8945a mfd, move
the charger's properties in the node, and replace the
"active-semi,irq_gpios" with the "interrupts" property to denote
the act8945a charger's irq.

Signed-off-by: Wenyou Yang <wenyou.yang@microchip.com>
---

 arch/arm/boot/dts/at91-sama5d2_xplained.dts | 23 +++++++++++++++--------
 1 file changed, 15 insertions(+), 8 deletions(-)

diff --git a/arch/arm/boot/dts/at91-sama5d2_xplained.dts b/arch/arm/boot/dts/at91-sama5d2_xplained.dts
index 2e2c3d1a1fa2..2a0f5f41260f 100644
--- a/arch/arm/boot/dts/at91-sama5d2_xplained.dts
+++ b/arch/arm/boot/dts/at91-sama5d2_xplained.dts
@@ -160,14 +160,6 @@
 					compatible = "active-semi,act8945a";
 					reg = <0x5b>;
 					active-semi,vsel-high;
-					active-semi,chglev-gpios = <&pioA 12 GPIO_ACTIVE_HIGH>;
-					active-semi,lbo-gpios = <&pioA 72 GPIO_ACTIVE_LOW>;
-					active-semi,irq_gpios = <&pioA 45 GPIO_ACTIVE_LOW>;
-					active-semi,input-voltage-threshold-microvolt = <6600>;
-					active-semi,precondition-timeout = <40>;
-					active-semi,total-timeout = <3>;
-					pinctrl-names = "default";
-					pinctrl-0 = <&pinctrl_charger_chglev &pinctrl_charger_lbo &pinctrl_charger_irq>;
 					status = "okay";
 
 					regulators {
@@ -220,6 +212,21 @@
 							regulator-always-on;
 						};
 					};
+
+					charger {
+						compatible = "active-semi,act8945a-charger";
+						pinctrl-names = "default";
+						pinctrl-0 = <&pinctrl_charger_chglev &pinctrl_charger_lbo &pinctrl_charger_irq>;
+						interrupt-parent = <&pioA>;
+						interrupts = <45 GPIO_ACTIVE_LOW>;
+
+						active-semi,chglev-gpios = <&pioA 12 GPIO_ACTIVE_HIGH>;
+						active-semi,lbo-gpios = <&pioA 72 GPIO_ACTIVE_LOW>;
+						active-semi,input-voltage-threshold-microvolt = <6600>;
+						active-semi,precondition-timeout = <40>;
+						active-semi,total-timeout = <3>;
+						status = "okay";
+					};
 				};
 			};
 
-- 
2.13.0

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

* [PATCH] ARM: dts: at91: sama5d2_xplained: Add charger node
@ 2017-08-15  6:24 ` Wenyou Yang
  0 siblings, 0 replies; 9+ messages in thread
From: Wenyou Yang @ 2017-08-15  6:24 UTC (permalink / raw)
  To: Alexandre Belloni, Nicolas Ferre, Rob Herring, Mark Rutland,
	Russell King
  Cc: linux-kernel, devicetree, linux-arm-kernel, Wenyou Yang, Wenyou Yang

From: Wenyou Yang <wenyou.yang@atmel.com>

Add the charger device node as a sub-device of act8945a mfd, move
the charger's properties in the node, and replace the
"active-semi,irq_gpios" with the "interrupts" property to denote
the act8945a charger's irq.

Signed-off-by: Wenyou Yang <wenyou.yang@microchip.com>
---

 arch/arm/boot/dts/at91-sama5d2_xplained.dts | 23 +++++++++++++++--------
 1 file changed, 15 insertions(+), 8 deletions(-)

diff --git a/arch/arm/boot/dts/at91-sama5d2_xplained.dts b/arch/arm/boot/dts/at91-sama5d2_xplained.dts
index 2e2c3d1a1fa2..2a0f5f41260f 100644
--- a/arch/arm/boot/dts/at91-sama5d2_xplained.dts
+++ b/arch/arm/boot/dts/at91-sama5d2_xplained.dts
@@ -160,14 +160,6 @@
 					compatible = "active-semi,act8945a";
 					reg = <0x5b>;
 					active-semi,vsel-high;
-					active-semi,chglev-gpios = <&pioA 12 GPIO_ACTIVE_HIGH>;
-					active-semi,lbo-gpios = <&pioA 72 GPIO_ACTIVE_LOW>;
-					active-semi,irq_gpios = <&pioA 45 GPIO_ACTIVE_LOW>;
-					active-semi,input-voltage-threshold-microvolt = <6600>;
-					active-semi,precondition-timeout = <40>;
-					active-semi,total-timeout = <3>;
-					pinctrl-names = "default";
-					pinctrl-0 = <&pinctrl_charger_chglev &pinctrl_charger_lbo &pinctrl_charger_irq>;
 					status = "okay";
 
 					regulators {
@@ -220,6 +212,21 @@
 							regulator-always-on;
 						};
 					};
+
+					charger {
+						compatible = "active-semi,act8945a-charger";
+						pinctrl-names = "default";
+						pinctrl-0 = <&pinctrl_charger_chglev &pinctrl_charger_lbo &pinctrl_charger_irq>;
+						interrupt-parent = <&pioA>;
+						interrupts = <45 GPIO_ACTIVE_LOW>;
+
+						active-semi,chglev-gpios = <&pioA 12 GPIO_ACTIVE_HIGH>;
+						active-semi,lbo-gpios = <&pioA 72 GPIO_ACTIVE_LOW>;
+						active-semi,input-voltage-threshold-microvolt = <6600>;
+						active-semi,precondition-timeout = <40>;
+						active-semi,total-timeout = <3>;
+						status = "okay";
+					};
 				};
 			};
 
-- 
2.13.0

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

* [PATCH] ARM: dts: at91: sama5d2_xplained: Add charger node
@ 2017-08-15  6:24 ` Wenyou Yang
  0 siblings, 0 replies; 9+ messages in thread
From: Wenyou Yang @ 2017-08-15  6:24 UTC (permalink / raw)
  To: linux-arm-kernel

From: Wenyou Yang <wenyou.yang@atmel.com>

Add the charger device node as a sub-device of act8945a mfd, move
the charger's properties in the node, and replace the
"active-semi,irq_gpios" with the "interrupts" property to denote
the act8945a charger's irq.

Signed-off-by: Wenyou Yang <wenyou.yang@microchip.com>
---

 arch/arm/boot/dts/at91-sama5d2_xplained.dts | 23 +++++++++++++++--------
 1 file changed, 15 insertions(+), 8 deletions(-)

diff --git a/arch/arm/boot/dts/at91-sama5d2_xplained.dts b/arch/arm/boot/dts/at91-sama5d2_xplained.dts
index 2e2c3d1a1fa2..2a0f5f41260f 100644
--- a/arch/arm/boot/dts/at91-sama5d2_xplained.dts
+++ b/arch/arm/boot/dts/at91-sama5d2_xplained.dts
@@ -160,14 +160,6 @@
 					compatible = "active-semi,act8945a";
 					reg = <0x5b>;
 					active-semi,vsel-high;
-					active-semi,chglev-gpios = <&pioA 12 GPIO_ACTIVE_HIGH>;
-					active-semi,lbo-gpios = <&pioA 72 GPIO_ACTIVE_LOW>;
-					active-semi,irq_gpios = <&pioA 45 GPIO_ACTIVE_LOW>;
-					active-semi,input-voltage-threshold-microvolt = <6600>;
-					active-semi,precondition-timeout = <40>;
-					active-semi,total-timeout = <3>;
-					pinctrl-names = "default";
-					pinctrl-0 = <&pinctrl_charger_chglev &pinctrl_charger_lbo &pinctrl_charger_irq>;
 					status = "okay";
 
 					regulators {
@@ -220,6 +212,21 @@
 							regulator-always-on;
 						};
 					};
+
+					charger {
+						compatible = "active-semi,act8945a-charger";
+						pinctrl-names = "default";
+						pinctrl-0 = <&pinctrl_charger_chglev &pinctrl_charger_lbo &pinctrl_charger_irq>;
+						interrupt-parent = <&pioA>;
+						interrupts = <45 GPIO_ACTIVE_LOW>;
+
+						active-semi,chglev-gpios = <&pioA 12 GPIO_ACTIVE_HIGH>;
+						active-semi,lbo-gpios = <&pioA 72 GPIO_ACTIVE_LOW>;
+						active-semi,input-voltage-threshold-microvolt = <6600>;
+						active-semi,precondition-timeout = <40>;
+						active-semi,total-timeout = <3>;
+						status = "okay";
+					};
 				};
 			};
 
-- 
2.13.0

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

* Re: [PATCH] ARM: dts: at91: sama5d2_xplained: Add charger node
@ 2017-09-25 20:13   ` Alexandre Belloni
  0 siblings, 0 replies; 9+ messages in thread
From: Alexandre Belloni @ 2017-09-25 20:13 UTC (permalink / raw)
  To: Wenyou Yang
  Cc: Nicolas Ferre, Rob Herring, Mark Rutland, Russell King,
	linux-kernel, devicetree, linux-arm-kernel, Wenyou Yang

On 15/08/2017 at 14:24:20 +0800, Wenyou Yang wrote:
> From: Wenyou Yang <wenyou.yang@atmel.com>
> 
> Add the charger device node as a sub-device of act8945a mfd, move
> the charger's properties in the node, and replace the
> "active-semi,irq_gpios" with the "interrupts" property to denote
> the act8945a charger's irq.
> 
> Signed-off-by: Wenyou Yang <wenyou.yang@microchip.com>
> ---
> 
>  arch/arm/boot/dts/at91-sama5d2_xplained.dts | 23 +++++++++++++++--------
>  1 file changed, 15 insertions(+), 8 deletions(-)

Applied, now but I had to rework the patch a bit. Can you check this is
working for you?

https://git.kernel.org/pub/scm/linux/kernel/git/abelloni/linux.git/commit/?h=at91-dt&id=4b6ad41f267b8ddabf5dd1b02a318bc74f0b95e9

-- 
Alexandre Belloni, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

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

* Re: [PATCH] ARM: dts: at91: sama5d2_xplained: Add charger node
@ 2017-09-25 20:13   ` Alexandre Belloni
  0 siblings, 0 replies; 9+ messages in thread
From: Alexandre Belloni @ 2017-09-25 20:13 UTC (permalink / raw)
  To: Wenyou Yang
  Cc: Nicolas Ferre, Rob Herring, Mark Rutland, Russell King,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Wenyou Yang

On 15/08/2017 at 14:24:20 +0800, Wenyou Yang wrote:
> From: Wenyou Yang <wenyou.yang-AIFe0yeh4nAAvxtiuMwx3w@public.gmane.org>
> 
> Add the charger device node as a sub-device of act8945a mfd, move
> the charger's properties in the node, and replace the
> "active-semi,irq_gpios" with the "interrupts" property to denote
> the act8945a charger's irq.
> 
> Signed-off-by: Wenyou Yang <wenyou.yang-UWL1GkI3JZL3oGB3hsPCZA@public.gmane.org>
> ---
> 
>  arch/arm/boot/dts/at91-sama5d2_xplained.dts | 23 +++++++++++++++--------
>  1 file changed, 15 insertions(+), 8 deletions(-)

Applied, now but I had to rework the patch a bit. Can you check this is
working for you?

https://git.kernel.org/pub/scm/linux/kernel/git/abelloni/linux.git/commit/?h=at91-dt&id=4b6ad41f267b8ddabf5dd1b02a318bc74f0b95e9

-- 
Alexandre Belloni, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

* [PATCH] ARM: dts: at91: sama5d2_xplained: Add charger node
@ 2017-09-25 20:13   ` Alexandre Belloni
  0 siblings, 0 replies; 9+ messages in thread
From: Alexandre Belloni @ 2017-09-25 20:13 UTC (permalink / raw)
  To: linux-arm-kernel

On 15/08/2017 at 14:24:20 +0800, Wenyou Yang wrote:
> From: Wenyou Yang <wenyou.yang@atmel.com>
> 
> Add the charger device node as a sub-device of act8945a mfd, move
> the charger's properties in the node, and replace the
> "active-semi,irq_gpios" with the "interrupts" property to denote
> the act8945a charger's irq.
> 
> Signed-off-by: Wenyou Yang <wenyou.yang@microchip.com>
> ---
> 
>  arch/arm/boot/dts/at91-sama5d2_xplained.dts | 23 +++++++++++++++--------
>  1 file changed, 15 insertions(+), 8 deletions(-)

Applied, now but I had to rework the patch a bit. Can you check this is
working for you?

https://git.kernel.org/pub/scm/linux/kernel/git/abelloni/linux.git/commit/?h=at91-dt&id=4b6ad41f267b8ddabf5dd1b02a318bc74f0b95e9

-- 
Alexandre Belloni, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

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

* Re: [PATCH] ARM: dts: at91: sama5d2_xplained: Add charger node
@ 2017-09-26  1:02     ` Yang, Wenyou
  0 siblings, 0 replies; 9+ messages in thread
From: Yang, Wenyou @ 2017-09-26  1:02 UTC (permalink / raw)
  To: Alexandre Belloni
  Cc: Nicolas Ferre, Rob Herring, Mark Rutland, Russell King,
	linux-kernel, devicetree, linux-arm-kernel, Wenyou Yang

Hi Alexandre,


On 2017/9/26 4:13, Alexandre Belloni wrote:
> On 15/08/2017 at 14:24:20 +0800, Wenyou Yang wrote:
>> From: Wenyou Yang <wenyou.yang@atmel.com>
>>
>> Add the charger device node as a sub-device of act8945a mfd, move
>> the charger's properties in the node, and replace the
>> "active-semi,irq_gpios" with the "interrupts" property to denote
>> the act8945a charger's irq.
>>
>> Signed-off-by: Wenyou Yang <wenyou.yang@microchip.com>
>> ---
>>
>>   arch/arm/boot/dts/at91-sama5d2_xplained.dts | 23 +++++++++++++++--------
>>   1 file changed, 15 insertions(+), 8 deletions(-)
> Applied, now but I had to rework the patch a bit. Can you check this is
> working for you?
>
> https://git.kernel.org/pub/scm/linux/kernel/git/abelloni/linux.git/commit/?h=at91-dt&id=4b6ad41f267b8ddabf5dd1b02a318bc74f0b95e9
It is okay for me.

Thanks a lot.

Best Regards,
Wenyou Yang

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

* Re: [PATCH] ARM: dts: at91: sama5d2_xplained: Add charger node
@ 2017-09-26  1:02     ` Yang, Wenyou
  0 siblings, 0 replies; 9+ messages in thread
From: Yang, Wenyou @ 2017-09-26  1:02 UTC (permalink / raw)
  To: Alexandre Belloni
  Cc: Nicolas Ferre, Rob Herring, Mark Rutland, Russell King,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Wenyou Yang

Hi Alexandre,


On 2017/9/26 4:13, Alexandre Belloni wrote:
> On 15/08/2017 at 14:24:20 +0800, Wenyou Yang wrote:
>> From: Wenyou Yang <wenyou.yang-AIFe0yeh4nAAvxtiuMwx3w@public.gmane.org>
>>
>> Add the charger device node as a sub-device of act8945a mfd, move
>> the charger's properties in the node, and replace the
>> "active-semi,irq_gpios" with the "interrupts" property to denote
>> the act8945a charger's irq.
>>
>> Signed-off-by: Wenyou Yang <wenyou.yang-UWL1GkI3JZL3oGB3hsPCZA@public.gmane.org>
>> ---
>>
>>   arch/arm/boot/dts/at91-sama5d2_xplained.dts | 23 +++++++++++++++--------
>>   1 file changed, 15 insertions(+), 8 deletions(-)
> Applied, now but I had to rework the patch a bit. Can you check this is
> working for you?
>
> https://git.kernel.org/pub/scm/linux/kernel/git/abelloni/linux.git/commit/?h=at91-dt&id=4b6ad41f267b8ddabf5dd1b02a318bc74f0b95e9
It is okay for me.

Thanks a lot.

Best Regards,
Wenyou Yang

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

* [PATCH] ARM: dts: at91: sama5d2_xplained: Add charger node
@ 2017-09-26  1:02     ` Yang, Wenyou
  0 siblings, 0 replies; 9+ messages in thread
From: Yang, Wenyou @ 2017-09-26  1:02 UTC (permalink / raw)
  To: linux-arm-kernel

Hi Alexandre,


On 2017/9/26 4:13, Alexandre Belloni wrote:
> On 15/08/2017 at 14:24:20 +0800, Wenyou Yang wrote:
>> From: Wenyou Yang <wenyou.yang@atmel.com>
>>
>> Add the charger device node as a sub-device of act8945a mfd, move
>> the charger's properties in the node, and replace the
>> "active-semi,irq_gpios" with the "interrupts" property to denote
>> the act8945a charger's irq.
>>
>> Signed-off-by: Wenyou Yang <wenyou.yang@microchip.com>
>> ---
>>
>>   arch/arm/boot/dts/at91-sama5d2_xplained.dts | 23 +++++++++++++++--------
>>   1 file changed, 15 insertions(+), 8 deletions(-)
> Applied, now but I had to rework the patch a bit. Can you check this is
> working for you?
>
> https://git.kernel.org/pub/scm/linux/kernel/git/abelloni/linux.git/commit/?h=at91-dt&id=4b6ad41f267b8ddabf5dd1b02a318bc74f0b95e9
It is okay for me.

Thanks a lot.

Best Regards,
Wenyou Yang

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

end of thread, other threads:[~2017-09-26  1:02 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-08-15  6:24 [PATCH] ARM: dts: at91: sama5d2_xplained: Add charger node Wenyou Yang
2017-08-15  6:24 ` Wenyou Yang
2017-08-15  6:24 ` Wenyou Yang
2017-09-25 20:13 ` Alexandre Belloni
2017-09-25 20:13   ` Alexandre Belloni
2017-09-25 20:13   ` Alexandre Belloni
2017-09-26  1:02   ` Yang, Wenyou
2017-09-26  1:02     ` Yang, Wenyou
2017-09-26  1:02     ` Yang, Wenyou

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.