linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: pavel@ucw.cz (Pavel Machek)
To: linux-arm-kernel@lists.infradead.org
Subject: N900 sleep mode (in 4.5-rc0, if that matters)
Date: Wed, 23 Mar 2016 13:38:01 +0100	[thread overview]
Message-ID: <20160323123801.GC32031@amd> (raw)
In-Reply-To: <20160209172403.GK19432@atomide.com>

Hi!

> > > 055555fc459 ("usb: musb: core: Fix handling of the phy notifications")
> > > 03e43528ab68 ("usb: musb: Fix unbalanced pm_runtime_enable")
> > 
> > Ok, with that, I can insmod and rmmod. But I still get:
> > 
> > 00001fff 48005020 (fa005020) cm_idlest_per blocking bits: 0007e000
> 
> I think the cm_idlest_per is fine.
> 
> > ffdffebd 48004a20 (fa004a20) cm_idlest1_core blocking bits: 00200042
> > 0000000d 48004a28 (fa004a28) cm_idlest3_core
> 
> Bit 21 in cm_idlest1_core is for MCSPI4 so WLAN. Does that go
> down if do sleep 5; cat /sys/kernel/debug/pm_debug/count ?
> 
> If not, the're PM runtime missing or broken somewhere.
> 
> FYI, below is my omap3 usb test script that I use to start and
> stop USB, it also works on n900. And after stopping it n900
> continues hitting deeper idle states just fine.

You rmmod musb_hdrc, so I checked, and found:

550a7375f (Felipe Balbi              2008-07-24 12:27:36 +0300 1093)
static void musb_shutdown(struct platform_de
vice *pdev)
550a7375f (Felipe Balbi              2008-07-24 12:27:36 +0300 1094) {
550a7375f (Felipe Balbi              2008-07-24 12:27:36 +0300 1095)
struct musb     *musb = dev_to_musb(&pdev
->dev);
550a7375f (Felipe Balbi              2008-07-24 12:27:36 +0300 1096)
unsigned long   flags;
550a7375f (Felipe Balbi              2008-07-24 12:27:36 +0300 1097)
4f9edd2d7 (Hema HK                   2011-03-22 16:02:12 +0530 1098)
pm_runtime_get_sync(musb->controller);
24307caef (Grazvydas Ignotas         2012-01-12 15:22:45 +0200 1099)
2cc65feab (Daniel Mack               2013-04-10 21:55:47 +0200 1100)
musb_host_cleanup(musb);
24307caef (Grazvydas Ignotas         2012-01-12 15:22:45 +0200 1101)
musb_gadget_cleanup(musb);
24307caef (Grazvydas Ignotas         2012-01-12 15:22:45 +0200 1102)
550a7375f (Felipe Balbi              2008-07-24 12:27:36 +0300 1103)
spin_lock_irqsave(&musb->lock, flags);
550a7375f (Felipe Balbi              2008-07-24 12:27:36 +0300 1104)
musb_platform_disable(musb);
550a7375f (Felipe Balbi              2008-07-24 12:27:36 +0300 1105)
musb_generic_disable(musb);
550a7375f (Felipe Balbi              2008-07-24 12:27:36 +0300 1106)
spin_unlock_irqrestore(&musb->lock, flags
);
550a7375f (Felipe Balbi              2008-07-24 12:27:36 +0300 1107)
120d074c5 (Grazvydas Ignotas         2010-10-10 13:52:22 -0500 1108)
musb_writeb(musb->mregs, MUSB_DEVCTL, 0);
120d074c5 (Grazvydas Ignotas         2010-10-10 13:52:22 -0500 1109)
musb_platform_exit(musb);
120d074c5 (Grazvydas Ignotas         2010-10-10 13:52:22 -0500 1110)
4f9edd2d7 (Hema HK                   2011-03-22 16:02:12 +0530 1111)
pm_runtime_put(musb->controller);
550a7375f (Felipe Balbi              2008-07-24 12:27:36 +0300 1112)
/* FIXME power down */
550a7375f (Felipe Balbi              2008-07-24 12:27:36 +0300 1113) }

I thought i was responsible for the FIXME, but apparently not... If
you happen to have some changes there, that would be useful to
know....

Ok, another attempt at shutting USB down:

00001fff 48005020 (fa005020) cm_idlest_per blocking bits: 0007e000
f7dffe9d 48004a20 (fa004a20) cm_idlest1_core blocking bits: 00200062
0000000d 48004a28 (fa004a28) cm_idlest3_core

Tried again today:

00001fff 48005020 (fa005020) cm_idlest_per blocking bits: 0007e000
f7dffe9d 48004a20 (fa004a20) cm_idlest1_core blocking bits: 00200062
0000000d 48004a28 (fa004a28) cm_idlest3_core
pavel at n900:/my/tui/ofone$ sleep test

00001fff 48005020 (fa005020) cm_idlest_per blocking bits: 0007e000
ffde7e9d 48004a20 (fa004a20) cm_idlest1_core blocking bits: 00218062
0000000d 48004a28 (fa004a28) cm_idlest3_core

00001fff 48005020 (fa005020) cm_idlest_per blocking bits: 0007e000
fedffe9d 48004a20 (fa004a20) cm_idlest1_core blocking bits: 01200062
0000000d 48004a28 (fa004a28) cm_idlest3_core

00001fff 48005020 (fa005020) cm_idlest_per blocking bits: 0007e000
ffde7e9d 48004a20 (fa004a20) cm_idlest1_core blocking bits: 00218062
0000000d 48004a28 (fa004a28) cm_idlest3_core

Is there documentation for the cm_idlest1_ bits?

How idle system do I need to have? Screen is blanked and machine
should be mostly idle, but there's X running on another vt with Mate
desktop, and some python scripts... GSM modem should be online.

Are you booting over USB from NOLO?

Thanks and best regards,

								Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

  parent reply	other threads:[~2016-03-23 12:38 UTC|newest]

Thread overview: 58+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-01-23 12:10 N900 sleep mode (in 4.5-rc0, if that matters) Pavel Machek
2016-01-25 16:33 ` Tony Lindgren
2016-01-25 22:23   ` Pavel Machek
2016-01-25 22:50     ` Tony Lindgren
2016-01-26 14:00   ` Pavel Machek
2016-01-26 17:25     ` Tony Lindgren
2016-01-26 22:51       ` Tony Lindgren
2016-01-30 20:02       ` Pavel Machek
2016-01-30 20:14         ` Pavel Machek
2016-01-30 22:15           ` Pavel Machek
2016-02-01 18:13             ` Tony Lindgren
2016-02-01 21:17               ` Pavel Machek
2016-02-01 22:11                 ` Tony Lindgren
2016-02-04  5:35                   ` Tony Lindgren
2016-02-07 21:37                     ` Pavel Machek
2016-02-08  8:51                       ` Pali Rohár
2016-02-07 21:23                   ` Pavel Machek
2016-02-09 17:24                     ` Tony Lindgren
2016-02-09 17:38                       ` Tony Lindgren
2016-03-20  8:38                         ` Pavel Machek
2016-02-11  1:08                       ` Tony Lindgren
2016-03-23 14:37                         ` Pavel Machek
2016-03-20  8:33                       ` Pavel Machek
2016-03-23 12:38                       ` Pavel Machek [this message]
2016-03-30 19:12                         ` Tony Lindgren
2016-04-04 11:09                           ` Pali Rohár
2016-04-04 22:31                             ` Tony Lindgren
2016-04-04 21:30                           ` Pavel Machek
2016-04-04 22:07                             ` Tony Lindgren
2016-04-05 10:09                               ` Pavel Machek
2016-04-05 13:17                               ` Pavel Machek
2016-04-05 14:22                               ` 4.4: camera and unlock buttons produce tons of interrupts (was Re: N900 sleep mode) Pavel Machek
2016-04-05 15:29                                 ` Tony Lindgren
2016-04-05 20:51                                   ` Pavel Machek
2016-04-07 17:40                                     ` Tony Lindgren
2016-04-07 19:48                                       ` 4.4, 4.6: " Pavel Machek
2016-04-07 21:32                                         ` Tony Lindgren
2016-04-07 23:01                                           ` Pavel Machek
2016-04-07 23:41                                             ` Tony Lindgren
2016-04-08  9:19                                               ` Pavel Machek
2016-04-11  8:27                                               ` 4.6-rc2: regression with omap video and lockdep (was Re: 4.4, 4.6: camera and unlock buttons produce tons of interrupts (was Re: N900 sleep mode)) Pavel Machek
2016-04-11  9:30                                               ` 4.4, 4.6: camera and unlock buttons produce tons of interrupts (was Re: N900 sleep mode) Pavel Machek
2016-04-11  9:41                                                 ` Pavel Machek
2016-04-11 21:10                                                   ` Tony Lindgren
     [not found]                                                     ` <20160412121658.GA32486@amd>
2016-04-12 12:30                                                       ` Nokia N900 retention mode in v4.6, camera buttons fun Pavel Machek
2016-04-12 16:30                                                         ` Tony Lindgren
2016-04-13  5:52                                                           ` Tomi Valkeinen
2016-04-13 15:15                                                             ` Tony Lindgren
2016-04-17 17:55                                                           ` Pavel Machek
2016-04-18 23:47                                                             ` Tony Lindgren
2016-04-21 13:04                                                             ` Pali Rohár
2016-04-21 21:28                                                               ` Pavel Machek
2016-04-22  7:05                                                                 ` Pali Rohár
2016-04-29 20:06                                                               ` Sebastian Reichel
2016-05-01  8:52                                                                 ` Pavel Machek
2016-05-01 15:48                                                                   ` Sebastian Reichel
2016-05-02 20:12                                                                     ` Sebastian Reichel
2016-05-28 11:09                                                                       ` Pavel Machek

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=20160323123801.GC32031@amd \
    --to=pavel@ucw.cz \
    --cc=linux-arm-kernel@lists.infradead.org \
    /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).