* [PATCH 1/3] ARM: dts: imx6: add pm_power_off support for i.mx6 chips
2014-11-12 8:20 [PATCH v4 0/3] add power off driver for i.mx6 Robin Gong
@ 2014-11-12 8:20 ` Robin Gong
2014-11-19 3:18 ` Shawn Guo
2014-11-12 8:20 ` [PATCH 2/3] power: reset: imx-snvs-poweroff: add power off driver for i.mx6 Robin Gong
` (2 subsequent siblings)
3 siblings, 1 reply; 13+ messages in thread
From: Robin Gong @ 2014-11-12 8:20 UTC (permalink / raw)
To: shawn.guo, kernel, robh+dt, pawel.moll, mark.rutland,
ijc+devicetree, galak, linux, sre, dbaryshkov, dwmw2,
grant.likely
Cc: devicetree, linux-kernel, linux-pm, linux-arm-kernel, smoch
All chips of i.mx6 can be powered off by programming SNVS.
For example :
On i.mx6q-sabresd board, PMIC_ON_REQ connect with external
pmic ON/OFF pin, that will cause the whole PMIC powered off
except VSNVS. And system can restart once PMIC_ON_REQ goes
high by push POWRER key.
Signed-off-by: Robin Gong <b38343@freescale.com>
---
arch/arm/boot/dts/imx6qdl-sabresd.dtsi | 4 ++++
arch/arm/boot/dts/imx6qdl.dtsi | 6 ++++++
arch/arm/boot/dts/imx6sl-evk.dts | 4 ++++
arch/arm/boot/dts/imx6sl.dtsi | 6 ++++++
arch/arm/boot/dts/imx6sx-sdb.dts | 4 ++++
arch/arm/boot/dts/imx6sx.dtsi | 6 ++++++
6 files changed, 30 insertions(+)
diff --git a/arch/arm/boot/dts/imx6qdl-sabresd.dtsi b/arch/arm/boot/dts/imx6qdl-sabresd.dtsi
index 07fb302..ba12cc6 100644
--- a/arch/arm/boot/dts/imx6qdl-sabresd.dtsi
+++ b/arch/arm/boot/dts/imx6qdl-sabresd.dtsi
@@ -531,6 +531,10 @@
status = "okay";
};
+&snvs-poweroff {
+ status = "okay";
+};
+
&ssi2 {
status = "okay";
};
diff --git a/arch/arm/boot/dts/imx6qdl.dtsi b/arch/arm/boot/dts/imx6qdl.dtsi
index c701af9..3a032d1 100644
--- a/arch/arm/boot/dts/imx6qdl.dtsi
+++ b/arch/arm/boot/dts/imx6qdl.dtsi
@@ -645,6 +645,12 @@
interrupts = <0 19 IRQ_TYPE_LEVEL_HIGH>,
<0 20 IRQ_TYPE_LEVEL_HIGH>;
};
+
+ snvs-poweroff: snvs-poweroff@38 {
+ compatible = "fsl,sec-v4.0-poweroff";
+ reg = <0x38 0x4>;
+ status = "disabled";
+ };
};
epit1: epit@020d0000 { /* EPIT1 */
diff --git a/arch/arm/boot/dts/imx6sl-evk.dts b/arch/arm/boot/dts/imx6sl-evk.dts
index 3f9e041..dab588f 100644
--- a/arch/arm/boot/dts/imx6sl-evk.dts
+++ b/arch/arm/boot/dts/imx6sl-evk.dts
@@ -488,6 +488,10 @@
status = "okay";
};
+&snvs-poweroff {
+ status = "okay";
+};
+
&ssi2 {
status = "okay";
};
diff --git a/arch/arm/boot/dts/imx6sl.dtsi b/arch/arm/boot/dts/imx6sl.dtsi
index 505da9e..5e5ed07 100644
--- a/arch/arm/boot/dts/imx6sl.dtsi
+++ b/arch/arm/boot/dts/imx6sl.dtsi
@@ -565,6 +565,12 @@
interrupts = <0 19 IRQ_TYPE_LEVEL_HIGH>,
<0 20 IRQ_TYPE_LEVEL_HIGH>;
};
+
+ snvs-poweroff: snvs-poweroff@38 {
+ compatible = "fsl,sec-v4.0-poweroff";
+ reg = <0x38 0x4>;
+ status = "disabled";
+ };
};
epit1: epit@020d0000 {
diff --git a/arch/arm/boot/dts/imx6sx-sdb.dts b/arch/arm/boot/dts/imx6sx-sdb.dts
index a3980d9..78dc031 100644
--- a/arch/arm/boot/dts/imx6sx-sdb.dts
+++ b/arch/arm/boot/dts/imx6sx-sdb.dts
@@ -251,6 +251,10 @@
};
};
+&snvs-poweroff {
+ status = "okay";
+};
+
&ssi2 {
status = "okay";
};
diff --git a/arch/arm/boot/dts/imx6sx.dtsi b/arch/arm/boot/dts/imx6sx.dtsi
index f4b9da6..652be9f 100644
--- a/arch/arm/boot/dts/imx6sx.dtsi
+++ b/arch/arm/boot/dts/imx6sx.dtsi
@@ -668,6 +668,12 @@
reg = <0x34 0x58>;
interrupts = <GIC_SPI 19 IRQ_TYPE_LEVEL_HIGH>, <GIC_SPI 20 IRQ_TYPE_LEVEL_HIGH>;
};
+
+ snvs-poweroff: snvs-poweroff@38 {
+ compatible = "fsl,sec-v4.0-poweroff";
+ reg = <0x38 0x4>;
+ status = "disabled";
+ };
};
epit1: epit@020d0000 {
--
1.7.9.5
^ permalink raw reply related [flat|nested] 13+ messages in thread* Re: [PATCH 1/3] ARM: dts: imx6: add pm_power_off support for i.mx6 chips
2014-11-12 8:20 ` [PATCH 1/3] ARM: dts: imx6: add pm_power_off support for i.mx6 chips Robin Gong
@ 2014-11-19 3:18 ` Shawn Guo
2014-11-19 5:26 ` Soeren Moch
0 siblings, 1 reply; 13+ messages in thread
From: Shawn Guo @ 2014-11-19 3:18 UTC (permalink / raw)
To: Robin Gong
Cc: kernel, robh+dt, pawel.moll, mark.rutland, ijc+devicetree, galak,
linux, sre, dbaryshkov, dwmw2, grant.likely, devicetree,
linux-kernel, linux-pm, linux-arm-kernel, smoch
On Wed, Nov 12, 2014 at 04:20:37PM +0800, Robin Gong wrote:
> All chips of i.mx6 can be powered off by programming SNVS.
> For example :
> On i.mx6q-sabresd board, PMIC_ON_REQ connect with external
> pmic ON/OFF pin, that will cause the whole PMIC powered off
> except VSNVS. And system can restart once PMIC_ON_REQ goes
> high by push POWRER key.
>
> Signed-off-by: Robin Gong <b38343@freescale.com>
> ---
> arch/arm/boot/dts/imx6qdl-sabresd.dtsi | 4 ++++
> arch/arm/boot/dts/imx6qdl.dtsi | 6 ++++++
> arch/arm/boot/dts/imx6sl-evk.dts | 4 ++++
> arch/arm/boot/dts/imx6sl.dtsi | 6 ++++++
> arch/arm/boot/dts/imx6sx-sdb.dts | 4 ++++
> arch/arm/boot/dts/imx6sx.dtsi | 6 ++++++
> 6 files changed, 30 insertions(+)
>
> diff --git a/arch/arm/boot/dts/imx6qdl-sabresd.dtsi b/arch/arm/boot/dts/imx6qdl-sabresd.dtsi
> index 07fb302..ba12cc6 100644
> --- a/arch/arm/boot/dts/imx6qdl-sabresd.dtsi
> +++ b/arch/arm/boot/dts/imx6qdl-sabresd.dtsi
> @@ -531,6 +531,10 @@
> status = "okay";
> };
>
> +&snvs-poweroff {
> + status = "okay";
> +};
> +
> &ssi2 {
> status = "okay";
> };
> diff --git a/arch/arm/boot/dts/imx6qdl.dtsi b/arch/arm/boot/dts/imx6qdl.dtsi
> index c701af9..3a032d1 100644
> --- a/arch/arm/boot/dts/imx6qdl.dtsi
> +++ b/arch/arm/boot/dts/imx6qdl.dtsi
> @@ -645,6 +645,12 @@
> interrupts = <0 19 IRQ_TYPE_LEVEL_HIGH>,
> <0 20 IRQ_TYPE_LEVEL_HIGH>;
> };
> +
> + snvs-poweroff: snvs-poweroff@38 {
I'm not sure if it's because my DTC version is too low or you didn't
test, the DTC reports error in label name 'snvs-poweroff'. IIRC, hyphen
'-' cannot be used in label name, but underscore '_' can.
Shawn
> + compatible = "fsl,sec-v4.0-poweroff";
> + reg = <0x38 0x4>;
> + status = "disabled";
> + };
> };
>
> epit1: epit@020d0000 { /* EPIT1 */
> diff --git a/arch/arm/boot/dts/imx6sl-evk.dts b/arch/arm/boot/dts/imx6sl-evk.dts
> index 3f9e041..dab588f 100644
> --- a/arch/arm/boot/dts/imx6sl-evk.dts
> +++ b/arch/arm/boot/dts/imx6sl-evk.dts
> @@ -488,6 +488,10 @@
> status = "okay";
> };
>
> +&snvs-poweroff {
> + status = "okay";
> +};
> +
> &ssi2 {
> status = "okay";
> };
> diff --git a/arch/arm/boot/dts/imx6sl.dtsi b/arch/arm/boot/dts/imx6sl.dtsi
> index 505da9e..5e5ed07 100644
> --- a/arch/arm/boot/dts/imx6sl.dtsi
> +++ b/arch/arm/boot/dts/imx6sl.dtsi
> @@ -565,6 +565,12 @@
> interrupts = <0 19 IRQ_TYPE_LEVEL_HIGH>,
> <0 20 IRQ_TYPE_LEVEL_HIGH>;
> };
> +
> + snvs-poweroff: snvs-poweroff@38 {
> + compatible = "fsl,sec-v4.0-poweroff";
> + reg = <0x38 0x4>;
> + status = "disabled";
> + };
> };
>
> epit1: epit@020d0000 {
> diff --git a/arch/arm/boot/dts/imx6sx-sdb.dts b/arch/arm/boot/dts/imx6sx-sdb.dts
> index a3980d9..78dc031 100644
> --- a/arch/arm/boot/dts/imx6sx-sdb.dts
> +++ b/arch/arm/boot/dts/imx6sx-sdb.dts
> @@ -251,6 +251,10 @@
> };
> };
>
> +&snvs-poweroff {
> + status = "okay";
> +};
> +
> &ssi2 {
> status = "okay";
> };
> diff --git a/arch/arm/boot/dts/imx6sx.dtsi b/arch/arm/boot/dts/imx6sx.dtsi
> index f4b9da6..652be9f 100644
> --- a/arch/arm/boot/dts/imx6sx.dtsi
> +++ b/arch/arm/boot/dts/imx6sx.dtsi
> @@ -668,6 +668,12 @@
> reg = <0x34 0x58>;
> interrupts = <GIC_SPI 19 IRQ_TYPE_LEVEL_HIGH>, <GIC_SPI 20 IRQ_TYPE_LEVEL_HIGH>;
> };
> +
> + snvs-poweroff: snvs-poweroff@38 {
> + compatible = "fsl,sec-v4.0-poweroff";
> + reg = <0x38 0x4>;
> + status = "disabled";
> + };
> };
>
> epit1: epit@020d0000 {
> --
> 1.7.9.5
>
^ permalink raw reply [flat|nested] 13+ messages in thread* Re: [PATCH 1/3] ARM: dts: imx6: add pm_power_off support for i.mx6 chips
2014-11-19 3:18 ` Shawn Guo
@ 2014-11-19 5:26 ` Soeren Moch
2014-11-19 5:50 ` Robin Gong
0 siblings, 1 reply; 13+ messages in thread
From: Soeren Moch @ 2014-11-19 5:26 UTC (permalink / raw)
To: Shawn Guo, Robin Gong
Cc: kernel, robh+dt, pawel.moll, mark.rutland, ijc+devicetree, galak,
linux, sre, dbaryshkov, grant.likely, devicetree, linux-kernel,
linux-pm, linux-arm-kernel
On 19.11.2014 04:18, Shawn Guo wrote:
> On Wed, Nov 12, 2014 at 04:20:37PM +0800, Robin Gong wrote:
>> All chips of i.mx6 can be powered off by programming SNVS.
>> For example :
>> On i.mx6q-sabresd board, PMIC_ON_REQ connect with external
>> pmic ON/OFF pin, that will cause the whole PMIC powered off
>> except VSNVS. And system can restart once PMIC_ON_REQ goes
>> high by push POWRER key.
>>
>> Signed-off-by: Robin Gong <b38343@freescale.com>
>> ---
>> arch/arm/boot/dts/imx6qdl-sabresd.dtsi | 4 ++++
>> arch/arm/boot/dts/imx6qdl.dtsi | 6 ++++++
>> arch/arm/boot/dts/imx6sl-evk.dts | 4 ++++
>> arch/arm/boot/dts/imx6sl.dtsi | 6 ++++++
>> arch/arm/boot/dts/imx6sx-sdb.dts | 4 ++++
>> arch/arm/boot/dts/imx6sx.dtsi | 6 ++++++
>> 6 files changed, 30 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/imx6qdl-sabresd.dtsi b/arch/arm/boot/dts/imx6qdl-sabresd.dtsi
>> index 07fb302..ba12cc6 100644
>> --- a/arch/arm/boot/dts/imx6qdl-sabresd.dtsi
>> +++ b/arch/arm/boot/dts/imx6qdl-sabresd.dtsi
>> @@ -531,6 +531,10 @@
>> status = "okay";
>> };
>>
>> +&snvs-poweroff {
>> + status = "okay";
>> +};"
>> +
>> &ssi2 {
>> status = "okay";
>> };
>> diff --git a/arch/arm/boot/dts/imx6qdl.dtsi b/arch/arm/boot/dts/imx6qdl.dtsi
>> index c701af9..3a032d1 100644
>> --- a/arch/arm/boot/dts/imx6qdl.dtsi
>> +++ b/arch/arm/boot/dts/imx6qdl.dtsi
>> @@ -645,6 +645,12 @@
>> interrupts = <0 19 IRQ_TYPE_LEVEL_HIGH>,
>> <0 20 IRQ_TYPE_LEVEL_HIGH>;
>> };
>> +
>> + snvs-poweroff: snvs-poweroff@38 {
>
> I'm not sure if it's because my DTC version is too low or you didn't
> test, the DTC reports error in label name 'snvs-poweroff'. IIRC, hyphen
> '-' cannot be used in label name, but underscore '_' can.
>
> Shawn
In the patch series "[PATCH v4 0/3] add power off driver for i.mx6" from
Sep 26 (what I have tested) snvs_poweroff was used. This worked as expected.
So this series "[PATCH v4 0/3] add power off driver for i.mx6" (sent on
Nov 12) apparently contains old patches and is not simply resent as
stated in the cover letter.
Soeren
^ permalink raw reply [flat|nested] 13+ messages in thread* Re: [PATCH 1/3] ARM: dts: imx6: add pm_power_off support for i.mx6 chips
2014-11-19 5:26 ` Soeren Moch
@ 2014-11-19 5:50 ` Robin Gong
2014-11-19 6:04 ` Shawn Guo
0 siblings, 1 reply; 13+ messages in thread
From: Robin Gong @ 2014-11-19 5:50 UTC (permalink / raw)
To: Soeren Moch
Cc: Shawn Guo, kernel, robh+dt, pawel.moll, mark.rutland,
ijc+devicetree, galak, linux, sre, dbaryshkov, grant.likely,
devicetree, linux-kernel, linux-pm, linux-arm-kernel
Sorry, the patch resent wrongly...
Hi Shawn, Can you apply the first v4 patches on Sep 26, or I resend it again?
On Wed, Nov 19, 2014 at 06:26:43AM +0100, Soeren Moch wrote:
>
> On 19.11.2014 04:18, Shawn Guo wrote:
> > On Wed, Nov 12, 2014 at 04:20:37PM +0800, Robin Gong wrote:
> >> All chips of i.mx6 can be powered off by programming SNVS.
> >> For example :
> >> On i.mx6q-sabresd board, PMIC_ON_REQ connect with external
> >> pmic ON/OFF pin, that will cause the whole PMIC powered off
> >> except VSNVS. And system can restart once PMIC_ON_REQ goes
> >> high by push POWRER key.
> >>
> >> Signed-off-by: Robin Gong <b38343@freescale.com>
> >> ---
> >> arch/arm/boot/dts/imx6qdl-sabresd.dtsi | 4 ++++
> >> arch/arm/boot/dts/imx6qdl.dtsi | 6 ++++++
> >> arch/arm/boot/dts/imx6sl-evk.dts | 4 ++++
> >> arch/arm/boot/dts/imx6sl.dtsi | 6 ++++++
> >> arch/arm/boot/dts/imx6sx-sdb.dts | 4 ++++
> >> arch/arm/boot/dts/imx6sx.dtsi | 6 ++++++
> >> 6 files changed, 30 insertions(+)
> >>
> >> diff --git a/arch/arm/boot/dts/imx6qdl-sabresd.dtsi b/arch/arm/boot/dts/imx6qdl-sabresd.dtsi
> >> index 07fb302..ba12cc6 100644
> >> --- a/arch/arm/boot/dts/imx6qdl-sabresd.dtsi
> >> +++ b/arch/arm/boot/dts/imx6qdl-sabresd.dtsi
> >> @@ -531,6 +531,10 @@
> >> status = "okay";
> >> };
> >>
> >> +&snvs-poweroff {
> >> + status = "okay";
> >> +};"
> >> +
> >> &ssi2 {
> >> status = "okay";
> >> };
> >> diff --git a/arch/arm/boot/dts/imx6qdl.dtsi b/arch/arm/boot/dts/imx6qdl.dtsi
> >> index c701af9..3a032d1 100644
> >> --- a/arch/arm/boot/dts/imx6qdl.dtsi
> >> +++ b/arch/arm/boot/dts/imx6qdl.dtsi
> >> @@ -645,6 +645,12 @@
> >> interrupts = <0 19 IRQ_TYPE_LEVEL_HIGH>,
> >> <0 20 IRQ_TYPE_LEVEL_HIGH>;
> >> };
> >> +
> >> + snvs-poweroff: snvs-poweroff@38 {
> >
> > I'm not sure if it's because my DTC version is too low or you didn't
> > test, the DTC reports error in label name 'snvs-poweroff'. IIRC, hyphen
> > '-' cannot be used in label name, but underscore '_' can.
> >
> > Shawn
>
> In the patch series "[PATCH v4 0/3] add power off driver for i.mx6" from
> Sep 26 (what I have tested) snvs_poweroff was used. This worked as expected.
>
> So this series "[PATCH v4 0/3] add power off driver for i.mx6" (sent on
> Nov 12) apparently contains old patches and is not simply resent as
> stated in the cover letter.
>
> Soeren
^ permalink raw reply [flat|nested] 13+ messages in thread* Re: [PATCH 1/3] ARM: dts: imx6: add pm_power_off support for i.mx6 chips
2014-11-19 5:50 ` Robin Gong
@ 2014-11-19 6:04 ` Shawn Guo
2014-11-21 9:00 ` Robin Gong
0 siblings, 1 reply; 13+ messages in thread
From: Shawn Guo @ 2014-11-19 6:04 UTC (permalink / raw)
To: Robin Gong
Cc: Soeren Moch, kernel-bIcnvbaLZ9MEGnE8C9+IrQ,
robh+dt-DgEjT+Ai2ygdnm+yROfE0A, pawel.moll-5wv7dgnIgG8,
mark.rutland-5wv7dgnIgG8, ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg,
galak-sgV2jX0FEOL9JmXXK+q4OQ, linux-lFZ/pmaqli7XmaaqVzeoHQ,
sre-DgEjT+Ai2ygdnm+yROfE0A, dbaryshkov-Re5JQEeQqe8AvxtiuMwx3w,
grant.likely-QSEj5FYQhm4dnm+yROfE0A,
devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-pm-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
On Wed, Nov 19, 2014 at 01:50:07PM +0800, Robin Gong wrote:
> Sorry, the patch resent wrongly...
> Hi Shawn, Can you apply the first v4 patches on Sep 26, or I resend it again?
I fixed it up and tested the series on my imx6q-sabresd board.
Applied the series.
Shawn
--
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] 13+ messages in thread
* Re: [PATCH 1/3] ARM: dts: imx6: add pm_power_off support for i.mx6 chips
2014-11-19 6:04 ` Shawn Guo
@ 2014-11-21 9:00 ` Robin Gong
0 siblings, 0 replies; 13+ messages in thread
From: Robin Gong @ 2014-11-21 9:00 UTC (permalink / raw)
To: Shawn Guo
Cc: Soeren Moch, kernel, robh+dt, pawel.moll, mark.rutland,
ijc+devicetree, galak, linux, sre, dbaryshkov, grant.likely,
devicetree, linux-kernel, linux-pm, linux-arm-kernel
Thanks Shawn and Soeren. :)
On Wed, Nov 19, 2014 at 02:04:43PM +0800, Shawn Guo wrote:
> On Wed, Nov 19, 2014 at 01:50:07PM +0800, Robin Gong wrote:
> > Sorry, the patch resent wrongly...
> > Hi Shawn, Can you apply the first v4 patches on Sep 26, or I resend it again?
>
> I fixed it up and tested the series on my imx6q-sabresd board.
>
> Applied the series.
>
> Shawn
^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH 2/3] power: reset: imx-snvs-poweroff: add power off driver for i.mx6
2014-11-12 8:20 [PATCH v4 0/3] add power off driver for i.mx6 Robin Gong
2014-11-12 8:20 ` [PATCH 1/3] ARM: dts: imx6: add pm_power_off support for i.mx6 chips Robin Gong
@ 2014-11-12 8:20 ` Robin Gong
2014-11-12 16:59 ` Sebastian Reichel
2014-11-12 8:20 ` [PATCH 3/3] ARM: configs: imx_v6_v7_defconfig: add power off driver Robin Gong
2014-11-12 10:56 ` [PATCH v4 0/3] add power off driver for i.mx6 Shawn Guo
3 siblings, 1 reply; 13+ messages in thread
From: Robin Gong @ 2014-11-12 8:20 UTC (permalink / raw)
To: shawn.guo, kernel, robh+dt, pawel.moll, mark.rutland,
ijc+devicetree, galak, linux, sre, dbaryshkov, dwmw2,
grant.likely
Cc: devicetree, linux-kernel, linux-pm, linux-arm-kernel, smoch
This driver register pm_power_off with snvs power off function. If
your boards NOT use PMIC_ON_REQ to turn on/off external pmic, or use
other pin to do, please disable the driver in dts, otherwise, your
pm_power_off maybe overwrote by this driver.
Signed-off-by: Robin Gong <b38343@freescale.com>
---
.../bindings/power_supply/imx-snvs-poweroff.txt | 23 +++++++
drivers/power/reset/Kconfig | 9 +++
drivers/power/reset/Makefile | 1 +
drivers/power/reset/imx-snvs-poweroff.c | 66 ++++++++++++++++++++
4 files changed, 99 insertions(+)
create mode 100644 Documentation/devicetree/bindings/power_supply/imx-snvs-poweroff.txt
create mode 100644 drivers/power/reset/imx-snvs-poweroff.c
diff --git a/Documentation/devicetree/bindings/power_supply/imx-snvs-poweroff.txt b/Documentation/devicetree/bindings/power_supply/imx-snvs-poweroff.txt
new file mode 100644
index 0000000..3d824b6
--- /dev/null
+++ b/Documentation/devicetree/bindings/power_supply/imx-snvs-poweroff.txt
@@ -0,0 +1,23 @@
+i.mx6 Poweroff Driver
+
+SNVS_LPCR in SNVS module can power off the whole system by pull
+PMIC_ON_REQ low if PMIC_ON_REQ is connected with external PMIC.
+If you don't want to use PMIC_ON_REQ as power on/off control,
+please set status='disabled' to disable this driver.
+
+Required Properties:
+-compatible: "fsl,sec-v4.0-poweroff"
+-reg: Specifies the physical address of the SNVS_LPCR register
+
+Example:
+ snvs@020cc000 {
+ compatible = "fsl,sec-v4.0-mon", "simple-bus";
+ #address-cells = <1>;
+ #size-cells = <1>;
+ ranges = <0 0x020cc000 0x4000>;
+ .....
+ snvs-poweroff: snvs-poweroff@38 {
+ compatible = "fsl,sec-v4.0-poweroff";
+ reg = <0x38 0x4>;
+ };
+ }
diff --git a/drivers/power/reset/Kconfig b/drivers/power/reset/Kconfig
index ca41523..b26c547 100644
--- a/drivers/power/reset/Kconfig
+++ b/drivers/power/reset/Kconfig
@@ -45,6 +45,15 @@ config POWER_RESET_HISI
help
Reboot support for Hisilicon boards.
+config POWER_RESET_IMX
+ bool "IMX6 power-off driver"
+ depends on POWER_RESET && SOC_IMX6
+ help
+ This driver support power off external PMIC by PMIC_ON_REQ on i.mx6
+ boards.If you want to use other pin to control external power,please
+ say N here or disable in dts to make sure pm_power_off never be
+ overwrote wrongly by this driver.
+
config POWER_RESET_MSM
bool "Qualcomm MSM power-off driver"
depends on POWER_RESET && ARCH_QCOM
diff --git a/drivers/power/reset/Makefile b/drivers/power/reset/Makefile
index a42e70e..2524daf 100644
--- a/drivers/power/reset/Makefile
+++ b/drivers/power/reset/Makefile
@@ -3,6 +3,7 @@ obj-$(CONFIG_POWER_RESET_AXXIA) += axxia-reset.o
obj-$(CONFIG_POWER_RESET_BRCMSTB) += brcmstb-reboot.o
obj-$(CONFIG_POWER_RESET_GPIO) += gpio-poweroff.o
obj-$(CONFIG_POWER_RESET_HISI) += hisi-reboot.o
+obj-$(CONFIG_POWER_RESET_IMX) += imx-snvs-poweroff.o
obj-$(CONFIG_POWER_RESET_MSM) += msm-poweroff.o
obj-$(CONFIG_POWER_RESET_QNAP) += qnap-poweroff.o
obj-$(CONFIG_POWER_RESET_RESTART) += restart-poweroff.o
diff --git a/drivers/power/reset/imx-snvs-poweroff.c b/drivers/power/reset/imx-snvs-poweroff.c
new file mode 100644
index 0000000..ad6ce50
--- /dev/null
+++ b/drivers/power/reset/imx-snvs-poweroff.c
@@ -0,0 +1,66 @@
+/* Power off driver for i.mx6
+ * Copyright (c) 2014, FREESCALE CORPORATION. All rights reserved.
+ *
+ * based on msm-poweroff.c
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 and
+ * only version 2 as published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ */
+
+#include <linux/err.h>
+#include <linux/init.h>
+#include <linux/io.h>
+#include <linux/kernel.h>
+#include <linux/module.h>
+#include <linux/of.h>
+#include <linux/of_address.h>
+#include <linux/platform_device.h>
+
+static void __iomem *snvs_base;
+
+static void do_imx_poweroff(void)
+{
+ u32 value = readl(snvs_base);
+
+ /* set TOP and DP_EN bit */
+ writel(value | 0x60, snvs_base);
+}
+
+static int imx_poweroff_probe(struct platform_device *pdev)
+{
+ snvs_base = of_iomap(pdev->dev.of_node, 0);
+ if (!snvs_base) {
+ dev_err(&pdev->dev, "failed to get memory\n");
+ return -ENODEV;
+ }
+
+ pm_power_off = do_imx_poweroff;
+ return 0;
+}
+
+static const struct of_device_id of_imx_poweroff_match[] = {
+ { .compatible = "fsl,sec-v4.0-poweroff", },
+ {},
+};
+MODULE_DEVICE_TABLE(of, of_imx_poweroff_match);
+
+static struct platform_driver imx_poweroff_driver = {
+ .probe = imx_poweroff_probe,
+ .driver = {
+ .name = "imx-snvs-poweroff",
+ .of_match_table = of_match_ptr(of_imx_poweroff_match),
+ },
+};
+
+static int __init imx_poweroff_init(void)
+{
+ return platform_driver_register(&imx_poweroff_driver);
+}
+device_initcall(imx_poweroff_init);
--
1.7.9.5
^ permalink raw reply related [flat|nested] 13+ messages in thread* Re: [PATCH 2/3] power: reset: imx-snvs-poweroff: add power off driver for i.mx6
2014-11-12 8:20 ` [PATCH 2/3] power: reset: imx-snvs-poweroff: add power off driver for i.mx6 Robin Gong
@ 2014-11-12 16:59 ` Sebastian Reichel
0 siblings, 0 replies; 13+ messages in thread
From: Sebastian Reichel @ 2014-11-12 16:59 UTC (permalink / raw)
To: Robin Gong
Cc: shawn.guo, kernel, robh+dt, pawel.moll, mark.rutland,
ijc+devicetree, galak, linux, dbaryshkov, dwmw2, grant.likely,
devicetree, linux-kernel, linux-pm, linux-arm-kernel, smoch
[-- Attachment #1: Type: text/plain, Size: 448 bytes --]
Hi,
On Wed, Nov 12, 2014 at 04:20:38PM +0800, Robin Gong wrote:
> This driver register pm_power_off with snvs power off function. If
> your boards NOT use PMIC_ON_REQ to turn on/off external pmic, or use
> other pin to do, please disable the driver in dts, otherwise, your
> pm_power_off maybe overwrote by this driver.
>
> Signed-off-by: Robin Gong <b38343@freescale.com>
Acked-By: Sebastian Reichel <sre@kernel.org>
-- Sebastian
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH 3/3] ARM: configs: imx_v6_v7_defconfig: add power off driver
2014-11-12 8:20 [PATCH v4 0/3] add power off driver for i.mx6 Robin Gong
2014-11-12 8:20 ` [PATCH 1/3] ARM: dts: imx6: add pm_power_off support for i.mx6 chips Robin Gong
2014-11-12 8:20 ` [PATCH 2/3] power: reset: imx-snvs-poweroff: add power off driver for i.mx6 Robin Gong
@ 2014-11-12 8:20 ` Robin Gong
2014-11-12 10:56 ` [PATCH v4 0/3] add power off driver for i.mx6 Shawn Guo
3 siblings, 0 replies; 13+ messages in thread
From: Robin Gong @ 2014-11-12 8:20 UTC (permalink / raw)
To: shawn.guo, kernel, robh+dt, pawel.moll, mark.rutland,
ijc+devicetree, galak, linux, sre, dbaryshkov, dwmw2,
grant.likely
Cc: devicetree, linux-kernel, linux-pm, linux-arm-kernel, smoch
Add power off driver in config file.
Signed-off-by: Robin Gong <b38343@freescale.com>
---
arch/arm/configs/imx_v6_v7_defconfig | 3 +++
1 file changed, 3 insertions(+)
diff --git a/arch/arm/configs/imx_v6_v7_defconfig b/arch/arm/configs/imx_v6_v7_defconfig
index 16cfec4..a310e61 100644
--- a/arch/arm/configs/imx_v6_v7_defconfig
+++ b/arch/arm/configs/imx_v6_v7_defconfig
@@ -165,6 +165,9 @@ CONFIG_SPI_IMX=y
CONFIG_GPIO_SYSFS=y
CONFIG_GPIO_MC9S08DZ60=y
CONFIG_GPIO_STMPE=y
+CONFIG_POWER_SUPPLY=y
+CONFIG_POWER_RESET=y
+CONFIG_POWER_RESET_IMX=y
# CONFIG_HWMON is not set
CONFIG_WATCHDOG=y
CONFIG_IMX2_WDT=y
--
1.7.9.5
^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [PATCH v4 0/3] add power off driver for i.mx6
2014-11-12 8:20 [PATCH v4 0/3] add power off driver for i.mx6 Robin Gong
` (2 preceding siblings ...)
2014-11-12 8:20 ` [PATCH 3/3] ARM: configs: imx_v6_v7_defconfig: add power off driver Robin Gong
@ 2014-11-12 10:56 ` Shawn Guo
2014-11-12 11:26 ` Arnd Bergmann
2014-11-13 2:10 ` Robin Gong
3 siblings, 2 replies; 13+ messages in thread
From: Shawn Guo @ 2014-11-12 10:56 UTC (permalink / raw)
To: Robin Gong
Cc: kernel, robh+dt, pawel.moll, mark.rutland, ijc+devicetree, galak,
linux, sre, dbaryshkov, dwmw2, grant.likely, devicetree,
linux-kernel, linux-pm, linux-arm-kernel, smoch
On Wed, Nov 12, 2014 at 04:20:36PM +0800, Robin Gong wrote:
> Resend the patchset with the latest MAINTAINERS
> Add simple power off driver for i.mx6, including:
> - add basic imx-snvs-poweroff driver in drivers/power/reset
> - add device node in all dts files of i.mx6.
> - enable in config file
>
> Change from v3:
> - disable poweroff driver in soc level dts and enable it in board level dts
> - replace 'EIO' with 'ENODEV' while of_iomap fail.
> Change from v2:
> - remove useless included head files
> - split device binding doc from dts patch
> - add description for the case that PMIC_ON_REQ not used as external power
> control
>
> Robin Gong (3):
> ARM: dts: imx6: add pm_power_off support for i.mx6 chips
> power: reset: imx-snvs-poweroff: add power off driver for i.mx6
> ARM: configs: imx_v6_v7_defconfig: add power off driver
I'm fine with the series. But before I take the patches, I need an ACK
from DT folks on the bindings.
Shawn
^ permalink raw reply [flat|nested] 13+ messages in thread* Re: [PATCH v4 0/3] add power off driver for i.mx6
2014-11-12 10:56 ` [PATCH v4 0/3] add power off driver for i.mx6 Shawn Guo
@ 2014-11-12 11:26 ` Arnd Bergmann
2014-11-13 2:10 ` Robin Gong
1 sibling, 0 replies; 13+ messages in thread
From: Arnd Bergmann @ 2014-11-12 11:26 UTC (permalink / raw)
To: Shawn Guo
Cc: Robin Gong, kernel, robh+dt, pawel.moll, mark.rutland,
ijc+devicetree, galak, linux, sre, dbaryshkov, dwmw2,
grant.likely, devicetree, linux-kernel, linux-pm,
linux-arm-kernel, smoch
On Wednesday 12 November 2014 18:56:22 Shawn Guo wrote:
> On Wed, Nov 12, 2014 at 04:20:36PM +0800, Robin Gong wrote:
> > Resend the patchset with the latest MAINTAINERS
> > Add simple power off driver for i.mx6, including:
> > - add basic imx-snvs-poweroff driver in drivers/power/reset
> > - add device node in all dts files of i.mx6.
> > - enable in config file
> >
> > Change from v3:
> > - disable poweroff driver in soc level dts and enable it in board level dts
> > - replace 'EIO' with 'ENODEV' while of_iomap fail.
> > Change from v2:
> > - remove useless included head files
> > - split device binding doc from dts patch
> > - add description for the case that PMIC_ON_REQ not used as external power
> > control
> >
> > Robin Gong (3):
> > ARM: dts: imx6: add pm_power_off support for i.mx6 chips
> > power: reset: imx-snvs-poweroff: add power off driver for i.mx6
> > ARM: configs: imx_v6_v7_defconfig: add power off driver
>
> I'm fine with the series. But before I take the patches, I need an ACK
> from DT folks on the bindings.
Looks ok to me. Using a syscon for fsl,sec-v4.0-mon might have been nicer
than making it a simple-bus, but that decision has been made already,
so this version is a logical extension of what we have already.
Arnd
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v4 0/3] add power off driver for i.mx6
2014-11-12 10:56 ` [PATCH v4 0/3] add power off driver for i.mx6 Shawn Guo
2014-11-12 11:26 ` Arnd Bergmann
@ 2014-11-13 2:10 ` Robin Gong
1 sibling, 0 replies; 13+ messages in thread
From: Robin Gong @ 2014-11-13 2:10 UTC (permalink / raw)
To: Shawn Guo
Cc: kernel, robh+dt, pawel.moll, mark.rutland, ijc+devicetree, galak,
linux, sre, dbaryshkov, dwmw2, grant.likely, devicetree,
linux-kernel, linux-pm, linux-arm-kernel, smoch
Thanks, Shawn.
On Wed, Nov 12, 2014 at 06:56:22PM +0800, Shawn Guo wrote:
> On Wed, Nov 12, 2014 at 04:20:36PM +0800, Robin Gong wrote:
> > Resend the patchset with the latest MAINTAINERS
> > Add simple power off driver for i.mx6, including:
> > - add basic imx-snvs-poweroff driver in drivers/power/reset
> > - add device node in all dts files of i.mx6.
> > - enable in config file
> >
> > Change from v3:
> > - disable poweroff driver in soc level dts and enable it in board level dts
> > - replace 'EIO' with 'ENODEV' while of_iomap fail.
> > Change from v2:
> > - remove useless included head files
> > - split device binding doc from dts patch
> > - add description for the case that PMIC_ON_REQ not used as external power
> > control
> >
> > Robin Gong (3):
> > ARM: dts: imx6: add pm_power_off support for i.mx6 chips
> > power: reset: imx-snvs-poweroff: add power off driver for i.mx6
> > ARM: configs: imx_v6_v7_defconfig: add power off driver
>
> I'm fine with the series. But before I take the patches, I need an ACK
> from DT folks on the bindings.
>
> Shawn
^ permalink raw reply [flat|nested] 13+ messages in thread