All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ARM: dts: sun7i: Enable power supplies on A20-OLinuxIno-Micro
@ 2017-08-29 17:08 ` Harald Geyer
  0 siblings, 0 replies; 4+ messages in thread
From: Harald Geyer @ 2017-08-29 17:08 UTC (permalink / raw)
  To: Russell King, Maxime Ripard, Chen-Yu Tsai
  Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Harald Geyer

The axp209.dtsi defines nodes for ac, usb and battery supplies for a
while now. This patch enables these nodes for the A20-Olinuxino-Micro,
which has connectors for all three of them.

The patch was run-tested against linux-next-20170825.

Signed-off-by: Harald Geyer <harald-95f8Dae0BrPYtjvyW6yDsg@public.gmane.org>
---
Note: I guess the same change is good for the olinuxino lime boards too.
However I don't have such boards to test, so I don't formally submit a
patch for them.

 arch/arm/boot/dts/sun7i-a20-olinuxino-micro.dts | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/arch/arm/boot/dts/sun7i-a20-olinuxino-micro.dts b/arch/arm/boot/dts/sun7i-a20-olinuxino-micro.dts
index 0b7403e..64c993a 100644
--- a/arch/arm/boot/dts/sun7i-a20-olinuxino-micro.dts
+++ b/arch/arm/boot/dts/sun7i-a20-olinuxino-micro.dts
@@ -256,6 +256,18 @@
 
 #include "axp209.dtsi"
 
+&ac_power_supply {
+	status = "okay";
+};
+
+&battery_power_supply {
+	status = "okay";
+};
+
+&usb_power_supply {
+	status = "okay";
+};
+
 &reg_dcdc2 {
 	regulator-always-on;
 	regulator-min-microvolt = <1000000>;
-- 
2.1.4

--
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 related	[flat|nested] 4+ messages in thread

* [PATCH] ARM: dts: sun7i: Enable power supplies on A20-OLinuxIno-Micro
@ 2017-08-29 17:08 ` Harald Geyer
  0 siblings, 0 replies; 4+ messages in thread
From: Harald Geyer @ 2017-08-29 17:08 UTC (permalink / raw)
  To: linux-arm-kernel

The axp209.dtsi defines nodes for ac, usb and battery supplies for a
while now. This patch enables these nodes for the A20-Olinuxino-Micro,
which has connectors for all three of them.

The patch was run-tested against linux-next-20170825.

Signed-off-by: Harald Geyer <harald@ccbib.org>
---
Note: I guess the same change is good for the olinuxino lime boards too.
However I don't have such boards to test, so I don't formally submit a
patch for them.

 arch/arm/boot/dts/sun7i-a20-olinuxino-micro.dts | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/arch/arm/boot/dts/sun7i-a20-olinuxino-micro.dts b/arch/arm/boot/dts/sun7i-a20-olinuxino-micro.dts
index 0b7403e..64c993a 100644
--- a/arch/arm/boot/dts/sun7i-a20-olinuxino-micro.dts
+++ b/arch/arm/boot/dts/sun7i-a20-olinuxino-micro.dts
@@ -256,6 +256,18 @@
 
 #include "axp209.dtsi"
 
+&ac_power_supply {
+	status = "okay";
+};
+
+&battery_power_supply {
+	status = "okay";
+};
+
+&usb_power_supply {
+	status = "okay";
+};
+
 &reg_dcdc2 {
 	regulator-always-on;
 	regulator-min-microvolt = <1000000>;
-- 
2.1.4

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

* Re: [PATCH] ARM: dts: sun7i: Enable power supplies on A20-OLinuxIno-Micro
  2017-08-29 17:08 ` Harald Geyer
@ 2017-08-30 14:40     ` Maxime Ripard
  -1 siblings, 0 replies; 4+ messages in thread
From: Maxime Ripard @ 2017-08-30 14:40 UTC (permalink / raw)
  To: Harald Geyer
  Cc: Russell King, Chen-Yu Tsai, devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r

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

On Tue, Aug 29, 2017 at 07:08:10PM +0200, Harald Geyer wrote:
> The axp209.dtsi defines nodes for ac, usb and battery supplies for a
> while now. This patch enables these nodes for the A20-Olinuxino-Micro,
> which has connectors for all three of them.
> 
> The patch was run-tested against linux-next-20170825.
> 
> Signed-off-by: Harald Geyer <harald-95f8Dae0BrPYtjvyW6yDsg@public.gmane.org>
> ---
> Note: I guess the same change is good for the olinuxino lime boards too.
> However I don't have such boards to test, so I don't formally submit a
> patch for them.
> 
>  arch/arm/boot/dts/sun7i-a20-olinuxino-micro.dts | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/sun7i-a20-olinuxino-micro.dts b/arch/arm/boot/dts/sun7i-a20-olinuxino-micro.dts
> index 0b7403e..64c993a 100644
> --- a/arch/arm/boot/dts/sun7i-a20-olinuxino-micro.dts
> +++ b/arch/arm/boot/dts/sun7i-a20-olinuxino-micro.dts
> @@ -256,6 +256,18 @@
>  
>  #include "axp209.dtsi"
>  
> +&ac_power_supply {
> +	status = "okay";
> +};
> +
> +&battery_power_supply {
> +	status = "okay";
> +};
> +
> +&usb_power_supply {
> +	status = "okay";
> +};
> +

We usually try to sort the nodes per alphabetical order (even though
it's a bit difficult for the battery and AC-IN power supplies).

I've moved the usb_power_supply node a bit below, where it belongs,
and it will be part of 4.15.

Thanks!
Maxime

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

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

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

* [PATCH] ARM: dts: sun7i: Enable power supplies on A20-OLinuxIno-Micro
@ 2017-08-30 14:40     ` Maxime Ripard
  0 siblings, 0 replies; 4+ messages in thread
From: Maxime Ripard @ 2017-08-30 14:40 UTC (permalink / raw)
  To: linux-arm-kernel

On Tue, Aug 29, 2017 at 07:08:10PM +0200, Harald Geyer wrote:
> The axp209.dtsi defines nodes for ac, usb and battery supplies for a
> while now. This patch enables these nodes for the A20-Olinuxino-Micro,
> which has connectors for all three of them.
> 
> The patch was run-tested against linux-next-20170825.
> 
> Signed-off-by: Harald Geyer <harald@ccbib.org>
> ---
> Note: I guess the same change is good for the olinuxino lime boards too.
> However I don't have such boards to test, so I don't formally submit a
> patch for them.
> 
>  arch/arm/boot/dts/sun7i-a20-olinuxino-micro.dts | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/sun7i-a20-olinuxino-micro.dts b/arch/arm/boot/dts/sun7i-a20-olinuxino-micro.dts
> index 0b7403e..64c993a 100644
> --- a/arch/arm/boot/dts/sun7i-a20-olinuxino-micro.dts
> +++ b/arch/arm/boot/dts/sun7i-a20-olinuxino-micro.dts
> @@ -256,6 +256,18 @@
>  
>  #include "axp209.dtsi"
>  
> +&ac_power_supply {
> +	status = "okay";
> +};
> +
> +&battery_power_supply {
> +	status = "okay";
> +};
> +
> +&usb_power_supply {
> +	status = "okay";
> +};
> +

We usually try to sort the nodes per alphabetical order (even though
it's a bit difficult for the battery and AC-IN power supplies).

I've moved the usb_power_supply node a bit below, where it belongs,
and it will be part of 4.15.

Thanks!
Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170830/c01f4c47/attachment.sig>

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

end of thread, other threads:[~2017-08-30 14:40 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-08-29 17:08 [PATCH] ARM: dts: sun7i: Enable power supplies on A20-OLinuxIno-Micro Harald Geyer
2017-08-29 17:08 ` Harald Geyer
     [not found] ` <20170829170810.1302-1-harald-95f8Dae0BrPYtjvyW6yDsg@public.gmane.org>
2017-08-30 14:40   ` Maxime Ripard
2017-08-30 14:40     ` Maxime Ripard

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.