<span class="Apple-style-span" style="font-family: 'Times New Roman'; font-size: 14px; border-collapse: collapse; "><pre style="white-space: pre-wrap; word-wrap: break-word; ">"fsl,mpc8349-pmc" has .has_deep_sleep = 0, deep_sleeping=0 so the mem should not do anything and just do a standby.</pre>
<pre style="white-space: pre-wrap; word-wrap: break-word; ">I am not sure if mem is a valid state in that case under /sys/power/state. </pre><pre style="white-space: pre-wrap; word-wrap: break-word; ">Scott, u can fix it!</pre>
<pre style="white-space: pre-wrap; word-wrap: break-word; ">-mj</pre></span><br><div class="gmail_quote">On Mon, Mar 23, 2009 at 11:15 AM, Li Yang-R58472 <span dir="ltr"><<a href="mailto:LeoLi@freescale.com">LeoLi@freescale.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div class="im">> -----Original Message-----<br>
> From: Wood Scott-B07421<br>
</div><div class="im">> Sent: Friday, March 20, 2009 10:42 PM<br>
> To: Li Yang-R58472<br>
> Cc: Soohyung Cho; <a href="mailto:linuxppc-dev@ozlabs.org">linuxppc-dev@ozlabs.org</a><br>
> Subject: Re: suspend-to-mem on the mpc8349e-mitx-gp?<br>
><br>
</div><div class="im">> Li Yang-R58472 wrote:<br>
> >> However, the code should treat "mem" as "standby" on chips<br>
> that don't<br>
> >> support deep sleep. What does the device tree<br>
> ><br>
> > Well, shouldn't the valid() callback reject unsupported<br>
> states instead<br>
> > of covering up?<br>
><br>
> I don't think so, in this case. The user is not asking for<br>
> "sleep" or deep sleep"; they are asking for a power state<br>
> that meets the definition of "standby" (which sleep does) or<br>
> which meets the definition of "mem"<br>
> (which both sleep and deep sleep do). When the user asks for<br>
> "mem", we provide the lowest power mode that qualifies.<br>
<br>
</div>In my understanding, "mem" which is suspend-to-ram means all CPU states and registers are kept in memory and the CPU is completely off during suspension. I don't think the sleep mode of 8349 qualifies, does it?<br>
<br>
- Leo<br>
<div><div></div><div class="h5">_______________________________________________<br>
Linuxppc-dev mailing list<br>
<a href="mailto:Linuxppc-dev@ozlabs.org">Linuxppc-dev@ozlabs.org</a><br>
<a href="https://ozlabs.org/mailman/listinfo/linuxppc-dev" target="_blank">https://ozlabs.org/mailman/listinfo/linuxppc-dev</a><br>
</div></div></blockquote></div><br>