Thread (1 message) 1 message, 1 author, 2016-05-01

Re: [PATCH v2 5/6] ARM: dts: axp209: Add ac_power_supply child node to the ax209 node

From: Chen-Yu Tsai <hidden>
Date: 2016-05-01 09:50:53
Also in: linux-pm

Hi,

On Sun, May 1, 2016 at 4:57 PM, Michael Haas [off-list ref] wrote:
quoted hunk
Add a node representing the ac power supply part of the axp209 pmic.

Signed-off-by: Michael Haas <redacted>
---
 arch/arm/boot/dts/axp209.dtsi | 6 ++++++
 1 file changed, 6 insertions(+)
diff --git a/arch/arm/boot/dts/axp209.dtsi b/arch/arm/boot/dts/axp209.dtsi
index 051ab3b..9046f0a 100644
--- a/arch/arm/boot/dts/axp209.dtsi
+++ b/arch/arm/boot/dts/axp209.dtsi
@@ -94,4 +94,10 @@
                compatible = "x-powers,axp202-usb-power-supply";
                status = "disabled";
        };
+
+       ac_power_supply: ac_power_supply {
Please keep them alphabetically sorted.
+               compatible = "x-powers,axp202-ac-power-supply";
+               status = "disabled";
I see no reason why we shouldn't just enable it by default.
It is almost always going to be used. Same for the VBUS power supply.
+       };
+
Extra line here.


ChenYu
 };
--
2.8.0
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help