linux-omap.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Kevin Hilman <khilman@ti.com>
To: Joe Woodward <jw@terrafix.co.uk>
Cc: Archit Taneja <a0393947@ti.com>,
	tomi.valkeinen@ti.com, linux-omap@vger.kernel.org,
	jaswinder.singh@linaro.org
Subject: Re: 3.5-rc3: PM/DSS broken (was vdd_mpu_iva warnings)
Date: Wed, 04 Jul 2012 07:01:39 -0700	[thread overview]
Message-ID: <87vci339rw.fsf@ti.com> (raw)
In-Reply-To: <WC20120703135255.990389@terrafix.co.uk> (Joe Woodward's message of "Tue, 03 Jul 2012 14:52:55 +0100")

"Joe Woodward" <jw@terrafix.co.uk> writes:

> ...snip...
>
>> > #
>> 
>> Thanks for testing this out.
>> 
>> I was going through Tomi's queue for the 3.6 merge window:
>> 
>> git://gitorious.org/linux-omap-dss2/linux.git master
>> 
>> There is a commit called:
>> 
>> 2b8501d777346ce1d4fe99167e9b3c0e42aae7a8
>> 
>> OMAPDSS: Use PM notifiers for system suspend
>> 
>> The commit message mentions the issue you see, and seems to resolve it.
>> Could you give this a try?
>> 
>> Tomi is out on vacation, and I don't know why this wasn't intended for 
>> the 3.5-rcs, maybe there are still some discussion going on about this?
>> 
>
> Thanks, this did remove the DSS warnings when suspending.
>
> Is there any chance this patch will make 3.5, or is it far too late now?
>
> However, core still fails to power down:
> # echo mem > /sys/power/state
> [   64.905975] PM: Syncing filesystems ... done.
> [   64.926879] Freezing user space processes ... (elapsed 0.01 seconds) done.
> [   64.952697] Freezing remaining freezable tasks ... (elapsed 0.02 seconds) done.
> [   64.983947] Suspending console(s) (use no_console_suspend to debug)
> [   65.115112] PM: suspend of devices complete after 121.215 msecs
> [   65.117797] PM: late suspend of devices complete after 2.655 msecs
> [   65.122314] PM: noirq suspend of devices complete after 4.486 msecs
> [   65.122344] Disabling non-boot CPUs ...
> [   66.095733] Powerdomain (core_pwrdm) didn't enter target state 1
> [   66.095733] Could not enter target state in pm_suspend
> [   66.098236] PM: noirq resume of devices complete after 2.319 msecs
> [   66.101409] PM: early resume of devices complete after 1.770 msecs
> [   66.471008] mmc1: error -110 during resume (card was removed?)
> [   66.516784] PM: resume of devices complete after 415.252 msecs
> [   66.580841] Restarting tasks ... done.
> sh: write error: Operation not permitted
>
> I'm guessing this is a question for Kevin, any idea how to debug this?

Already debugged and fixed. The exact solution is still under
discussion, but while waiting for the right fix, you can use this patch
and you should see CORE hitting retention:

   http://marc.info/?l=linux-omap&m=134000053229888&w=2

>
> Also, why is it now saing "sh: write error: Operation not permitted" after waking, this never happened with 3.4?

I have seen this here and there and have never looked into why that is
happening.  Clearly the write actually worked because the suspend
happened, so I'm not sure what's going on there.

Kevin


  reply	other threads:[~2012-07-04 14:01 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-06-18 10:39 3.5-rc3: vdd_mpu_iva warnings Joe Woodward
2012-06-27 15:22 ` Joe Woodward
2012-07-02 21:18 ` Kevin Hilman
2012-07-02 21:19 ` Kevin Hilman
2012-07-03  8:37   ` 3.5-rc3: PM/DSS broken (was vdd_mpu_iva warnings) Joe Woodward
2012-07-03  8:59     ` Archit Taneja
2012-07-03  9:12       ` Joe Woodward
2012-07-03 10:59         ` Archit Taneja
2012-07-03 12:31           ` Jassi Brar
2012-07-03 13:52           ` Joe Woodward
2012-07-04 14:01             ` Kevin Hilman [this message]
2012-07-05  7:58               ` Joe Woodward
2012-07-04  6:28           ` Tomi Valkeinen
2012-07-04 10:38             ` Archit Taneja

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87vci339rw.fsf@ti.com \
    --to=khilman@ti.com \
    --cc=a0393947@ti.com \
    --cc=jaswinder.singh@linaro.org \
    --cc=jw@terrafix.co.uk \
    --cc=linux-omap@vger.kernel.org \
    --cc=tomi.valkeinen@ti.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).