All of lore.kernel.org
 help / color / mirror / Atom feed
From: Pavel Machek <pavel@ucw.cz>
To: Tony Lindgren <tony@atomide.com>
Cc: pali.rohar@gmail.com, sre@kernel.org,
	kernel list <linux-kernel@vger.kernel.org>,
	linux-arm-kernel <linux-arm-kernel@lists.infradead.org>,
	linux-omap@vger.kernel.org, khilman@kernel.org,
	aaro.koskinen@iki.fi, ivo.g.dimitrov.75@gmail.com,
	patrikbachan@gmail.com, serge@hallyn.com
Subject: Re: 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@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

WARNING: multiple messages have this Message-ID (diff)
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: 120+ 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-23 12:10 ` Pavel Machek
2016-01-25 16:33 ` Tony Lindgren
2016-01-25 16:33   ` Tony Lindgren
2016-01-25 22:23   ` Pavel Machek
2016-01-25 22:23     ` Pavel Machek
2016-01-25 22:50     ` Tony Lindgren
2016-01-25 22:50       ` Tony Lindgren
2016-01-26 14:00   ` Pavel Machek
2016-01-26 14:00     ` Pavel Machek
2016-01-26 17:25     ` Tony Lindgren
2016-01-26 17:25       ` Tony Lindgren
2016-01-26 22:51       ` Tony Lindgren
2016-01-26 22:51         ` Tony Lindgren
2016-01-30 20:02       ` Pavel Machek
2016-01-30 20:02         ` Pavel Machek
2016-01-30 20:14         ` Pavel Machek
2016-01-30 20:14           ` Pavel Machek
2016-01-30 22:15           ` Pavel Machek
2016-01-30 22:15             ` Pavel Machek
2016-02-01 18:13             ` Tony Lindgren
2016-02-01 18:13               ` Tony Lindgren
2016-02-01 21:17               ` Pavel Machek
2016-02-01 21:17                 ` Pavel Machek
2016-02-01 22:11                 ` Tony Lindgren
2016-02-01 22:11                   ` Tony Lindgren
2016-02-04  5:35                   ` Tony Lindgren
2016-02-04  5:35                     ` Tony Lindgren
2016-02-07 21:37                     ` Pavel Machek
2016-02-07 21:37                       ` Pavel Machek
2016-02-07 21:37                       ` Pavel Machek
2016-02-08  8:51                       ` Pali Rohár
2016-02-08  8:51                         ` Pali Rohár
2016-02-07 21:23                   ` Pavel Machek
2016-02-07 21:23                     ` Pavel Machek
2016-02-09 17:24                     ` Tony Lindgren
2016-02-09 17:24                       ` Tony Lindgren
2016-02-09 17:38                       ` Tony Lindgren
2016-02-09 17:38                         ` Tony Lindgren
2016-03-20  8:38                         ` Pavel Machek
2016-03-20  8:38                           ` Pavel Machek
2016-02-11  1:08                       ` Tony Lindgren
2016-02-11  1:08                         ` Tony Lindgren
2016-02-11  1:08                         ` Tony Lindgren
2016-03-23 14:37                         ` Pavel Machek
2016-03-23 14:37                           ` Pavel Machek
2016-03-20  8:33                       ` Pavel Machek
2016-03-20  8:33                         ` Pavel Machek
2016-03-23 12:38                       ` Pavel Machek [this message]
2016-03-23 12:38                         ` Pavel Machek
2016-03-30 19:12                         ` Tony Lindgren
2016-03-30 19:12                           ` Tony Lindgren
2016-04-04 11:09                           ` Pali Rohár
2016-04-04 11:09                             ` Pali Rohár
2016-04-04 22:31                             ` Tony Lindgren
2016-04-04 22:31                               ` Tony Lindgren
2016-04-04 21:30                           ` Pavel Machek
2016-04-04 21:30                             ` Pavel Machek
2016-04-04 22:07                             ` Tony Lindgren
2016-04-04 22:07                               ` Tony Lindgren
2016-04-05 10:09                               ` Pavel Machek
2016-04-05 10:09                                 ` Pavel Machek
2016-04-05 13:17                               ` 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 14:22                                 ` Pavel Machek
2016-04-05 15:29                                 ` Tony Lindgren
2016-04-05 15:29                                   ` Tony Lindgren
2016-04-05 20:51                                   ` Pavel Machek
2016-04-05 20:51                                     ` Pavel Machek
2016-04-07 17:40                                     ` Tony Lindgren
2016-04-07 17:40                                       ` Tony Lindgren
2016-04-07 19:48                                       ` 4.4, 4.6: " Pavel Machek
2016-04-07 19:48                                         ` Pavel Machek
2016-04-07 21:32                                         ` Tony Lindgren
2016-04-07 21:32                                           ` Tony Lindgren
2016-04-07 23:01                                           ` Pavel Machek
2016-04-07 23:01                                             ` Pavel Machek
2016-04-07 23:41                                             ` Tony Lindgren
2016-04-07 23:41                                               ` Tony Lindgren
2016-04-08  9:19                                               ` Pavel Machek
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  8:27                                                 ` 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:30                                                 ` Pavel Machek
2016-04-11  9:41                                                 ` Pavel Machek
2016-04-11  9:41                                                   ` Pavel Machek
2016-04-11 21:10                                                   ` Tony Lindgren
2016-04-11 21:10                                                     ` Tony Lindgren
2016-04-12 12:16                                                     ` Nokia N900 retention mode in v4.6, camera buttons fun Pavel Machek
2016-04-12 12:30                                                       ` Pavel Machek
2016-04-12 12:30                                                         ` Pavel Machek
2016-04-12 16:30                                                         ` Tony Lindgren
2016-04-12 16:30                                                           ` Tony Lindgren
2016-04-13  5:52                                                           ` Tomi Valkeinen
2016-04-13  5:52                                                             ` Tomi Valkeinen
2016-04-13  5:52                                                             ` Tomi Valkeinen
2016-04-13 15:15                                                             ` Tony Lindgren
2016-04-13 15:15                                                               ` Tony Lindgren
2016-04-17 17:55                                                           ` Pavel Machek
2016-04-17 17:55                                                             ` Pavel Machek
2016-04-18 23:47                                                             ` Tony Lindgren
2016-04-18 23:47                                                               ` Tony Lindgren
2016-04-21 13:04                                                             ` Pali Rohár
2016-04-21 13:04                                                               ` Pali Rohár
2016-04-21 21:28                                                               ` Pavel Machek
2016-04-21 21:28                                                                 ` Pavel Machek
2016-04-22  7:05                                                                 ` Pali Rohár
2016-04-22  7:05                                                                   ` Pali Rohár
2016-04-29 20:06                                                               ` Sebastian Reichel
2016-04-29 20:06                                                                 ` Sebastian Reichel
2016-05-01  8:52                                                                 ` Pavel Machek
2016-05-01  8:52                                                                   ` Pavel Machek
2016-05-01 15:48                                                                   ` Sebastian Reichel
2016-05-01 15:48                                                                     ` Sebastian Reichel
2016-05-02 20:12                                                                     ` Sebastian Reichel
2016-05-02 20:12                                                                       ` Sebastian Reichel
2016-05-28 11:09                                                                       ` Pavel Machek
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=aaro.koskinen@iki.fi \
    --cc=ivo.g.dimitrov.75@gmail.com \
    --cc=khilman@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=pali.rohar@gmail.com \
    --cc=patrikbachan@gmail.com \
    --cc=serge@hallyn.com \
    --cc=sre@kernel.org \
    --cc=tony@atomide.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.