[PATCH/RFC 3/6] drivers: firmware: psci: Implement shallow suspend mode

Sudeep Holla
2017-02-22T14:35:37Z

On 22/02/17 13:47, Geert Uytterhoeven wrote:
> Hi Sudeep,
>
> On Tue, Feb 21, 2017 at 6:22 PM, Sudeep Holla <sudeep.holla@arm.com> wrote:

[...]

>>
>> IIUC, it's not implemented today. I can't talk about future ;), but your
>
> Good, so there's no need for the DT property, and drivers/firmware/psci.c
> should aways call do_cpu_idle() instead of PSCI SYSTEM_SUSPEND if any
> other wake-up sources are configured?
>

No.

> That follows the principle of least surprise: it doesn't leave the user with
> a system that won't wake up the way he configured it to wake up.
>

But he can still wake up with the "switch" so there's no surprise. He
just need to better understand his system before playing with it ;)

--
Regards,
Sudeep