Re: [PATCH 1/7] mfd: AXP20x: Add mfd driver for AXP20x PMIC

Hi Carlo,

On Sat, Mar 08, 2014 at 12:31:41PM +0100, Carlo Caione wrote:

quoted

quoted

+	axp20x->pm_off = of_property_read_bool(node, "axp,system-power-controller");
+
+	if (axp20x->pm_off && !pm_power_off) {
+		axp20x_pm_power_off = axp20x;
+		pm_power_off = axp20x_power_off;
+	}

I don't think we have any other way to actually power down the board.

Is there any reason why we would not need this property?

In case you have multiple PMIC on the board in this way you con define
which is the one apt to power off.

Do we even have such a board?

Let's leave this out, we can always add it later if needs be.

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