All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tony Lindgren <tony@atomide.com>
To: Pavel Machek <pavel@ucw.cz>
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: Tue, 26 Jan 2016 09:25:28 -0800	[thread overview]
Message-ID: <20160126172527.GY3500@atomide.com> (raw)
In-Reply-To: <20160126140010.GA29723@amd>

* Pavel Machek <pavel@ucw.cz> [160126 06:01]:
> 
> It seems like I have rather lot of blocking bits:
> 
> 00001fff 48005020 (fa005020) cm_idlest_per blocking bits: 0007e000

Looks like most of these are for GPIO banks, that's OK those get saved
and restored in the idle loop.

Here bit 18 UART4 is a mystery though.. It's uart4 on 36xx but reserved
on 34xx. I do have that too on my n900, but it's hitting off mode with
v4.4.

> ffdffe8d 48004a20 (fa004a20) cm_idlest1_core blocking bits: 00200072
> 0000000d 48004a28 (fa004a28) cm_idlest3_core
> 
> cm_idlest1_core changes periodicall often, to 00218072. The rest seems
> constant.

For cm_idlest1_core 42 is the answer.. Here you have bits 4 and 5
blocking which is for OTG and it's PHY. That's a known issue with
musb and setting pm_runtime_irq_safe() on the MUSB parent.

If you do rmmod omap2430 and phy-twl4030usb chances are the LEDs will
start going off assuming the McSPI bit goes low with WLAN idling.

Looks like we have some regression with v4.5-rc1 where n900 is not
hitting deeper idle states though. I'll run git bisect between
v4.4..v4.5-rc1.

Regards,

Tony

WARNING: multiple messages have this Message-ID (diff)
From: tony@atomide.com (Tony Lindgren)
To: linux-arm-kernel@lists.infradead.org
Subject: N900 sleep mode (in 4.5-rc0, if that matters)
Date: Tue, 26 Jan 2016 09:25:28 -0800	[thread overview]
Message-ID: <20160126172527.GY3500@atomide.com> (raw)
In-Reply-To: <20160126140010.GA29723@amd>

* Pavel Machek <pavel@ucw.cz> [160126 06:01]:
> 
> It seems like I have rather lot of blocking bits:
> 
> 00001fff 48005020 (fa005020) cm_idlest_per blocking bits: 0007e000

Looks like most of these are for GPIO banks, that's OK those get saved
and restored in the idle loop.

Here bit 18 UART4 is a mystery though.. It's uart4 on 36xx but reserved
on 34xx. I do have that too on my n900, but it's hitting off mode with
v4.4.

> ffdffe8d 48004a20 (fa004a20) cm_idlest1_core blocking bits: 00200072
> 0000000d 48004a28 (fa004a28) cm_idlest3_core
> 
> cm_idlest1_core changes periodicall often, to 00218072. The rest seems
> constant.

For cm_idlest1_core 42 is the answer.. Here you have bits 4 and 5
blocking which is for OTG and it's PHY. That's a known issue with
musb and setting pm_runtime_irq_safe() on the MUSB parent.

If you do rmmod omap2430 and phy-twl4030usb chances are the LEDs will
start going off assuming the McSPI bit goes low with WLAN idling.

Looks like we have some regression with v4.5-rc1 where n900 is not
hitting deeper idle states though. I'll run git bisect between
v4.4..v4.5-rc1.

Regards,

Tony

  reply	other threads:[~2016-01-26 17:25 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 [this message]
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
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=20160126172527.GY3500@atomide.com \
    --to=tony@atomide.com \
    --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=pavel@ucw.cz \
    --cc=serge@hallyn.com \
    --cc=sre@kernel.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 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.