On 27.07.2018 11:33, Robin Gong wrote:

quoted

-----Original Message-----
From: Oleksij Rempel [mailto:o.rempel@pengutronix.de]
Sent: 2018年7月26日 17:22
To: Shawn Guo <shawnguo@kernel.org>; Mark Brown <broonie@kernel.org>;
Rafael J. Wysocki [off-list ref]
Cc: Oleksij Rempel <o.rempel@pengutronix.de>; kernel@pengutronix.de;
devicetree@vger.kernel.org; linux-arm-kernel@lists.infradead.org;
linux-clk@vger.kernel.org; linux-kernel@vger.kernel.org; Andrew Morton
[off-list ref]; Liam Girdwood [off-list ref];
Leonard Crestez [off-list ref]; Rob Herring
[off-list ref]; Mark Rutland [off-list ref]; Michael
Turquette [off-list ref]; Stephen Boyd
[off-list ref]; Fabio Estevam [off-list ref]; Russell
King [off-list ref]; dl-linux-imx [off-list ref]; Robin Gong
[off-list ref]; A.s. Dong [off-list ref]
Subject: [PATCH v8 6/6] ARM: dts: imx6: RIoTboard provide standby on power
off option

This board, as well as some other boards with i.MX6 and a PMIC, uses a
"PMIC_STBY_REQ" line to notify the PMIC about a state change.
The PMIC is programmed for a specific state change before triggering the line.
In this case, PMIC_STBY_REQ can be used for stand by, sleep and power off
modes.

Signed-off-by: Oleksij Rempel <o.rempel@pengutronix.de>
---
 arch/arm/boot/dts/imx6dl-riotboard.dts | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/arch/arm/boot/dts/imx6dl-riotboard.dts

b/arch/arm/boot/dts/imx6dl-riotboard.dts
index 2e98c92adff7..a0e9753ee767 100644

--- a/arch/arm/boot/dts/imx6dl-riotboard.dts
+++ b/arch/arm/boot/dts/imx6dl-riotboard.dts

@@ -90,6 +90,10 @@
 	status = "okay";
 };

+&clks {
+	fsl,pmic-stby-poweroff;
+};

It's better remove the default "syscon-poweroff" power off node.

"syscon-poweroff" is by default disabled and not enabled in
arch/arm/boot/dts/imx6dl-riotboard.dts