Hello Wenyou,
On Mon, Jan 26, 2015 at 05:42:11PM +0800, Wenyou Yang wrote:
quoted hunk ↗ jump to hunk
+static int at91_pm_verify_clocks(suspend_state_t state)
{
unsigned long scsr;
int i;
+ /* For PM_SUSPEND_STANDBY, skip verifying the clock */
+ if (state == PM_SUSPEND_STANDBY)
+ return 1;
+
In my opinion we should use the select() already in place in
at91_pm_enter() to do that:
static int at91_pm_enter(suspend_state_t state)
{
at91_pinctrl_gpio_suspend();
switch (state) {
(...)
+ case PM_SUSPEND_MEM:
/*
* Ensure that clocks are in a valid state.
*/
if (!at91_pm_verify_clocks())
goto error;
/* FALLTHROUGH */
quoted hunk ↗ jump to hunk
+ case PM_SUSPEND_STANDBY:
/*
- * Suspend-to-RAM is like STANDBY plus slow clock mode, so
Sylvain