[PATCH v8 2/2] ARM: imx: support suspend states on imx7D

-----Original Message-----
From: Shawn Guo [mailto:shawnguo at kernel.org]

quoted

This patch adds a new suspend driver to manage the power states on IMX7D.
It currently supports "SUSPEND_STANDBY" and "SUSPEND_MEM" states.

Signed-off-by: Shenwei Wang <redacted>
Signed-off-by: Anson Huang <redacted>

Please stop sending patches to my Linaro mailbox, and use
shawnguo at kernel.org instead.  You should already get that if you ever
run ./scripts/get_maintainer.pl on the patch.  Also please always copy
kernel at pengutronix.de for i.MX platform patches like this.

Noted.

quoted

---
 arch/arm/mach-imx/Kconfig        |   1 +
 arch/arm/mach-imx/Makefile       |   2 +
 arch/arm/mach-imx/common.h       |   4 +
 arch/arm/mach-imx/pm-imx7.c      | 917

+++++++++++++++++++++++++++++++++++++++

quoted

 arch/arm/mach-imx/suspend-imx7.S | 529 ++++++++++++++++++++++
 5 files changed, 1453 insertions(+)

1453 lines addition to kernel only for i.MX7D suspend support.  Yes, this is the
way we support suspend on i.MX6, but that's enough, and we have to stop this
somewhere.  I would ask you to take Sudeep's comment and adopt PSCI for
i.MX7D power management.

According to PSCI spec, a supervisory software must be implemented under either
HVC(EL2) or SMC(EL3) privilege level. I agree that, with this idea, we could finally move 
these 1453 lines of codes from kernel into a platform specific firmware. The problem
is we don't have the firmware so far, and we haven't even had a plan to enable the
hypervisor or secure software on this platform. Because low power consumption is
the key feature of this i.MX7D platform, please continue to consider the current 
implementation and get this feature supported first. 

Thanks,
Shenwei

 

Shawn

[1] https://lkml.org/lkml/2015/8/26/554