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: 4.4, 4.6: camera and unlock buttons produce tons of interrupts (was Re: N900 sleep mode)
Date: Thu, 7 Apr 2016 21:48:29 +0200	[thread overview]
Message-ID: <20160407194828.GA13338@amd> (raw)
In-Reply-To: <20160407174019.GM16484@atomide.com>

[-- Attachment #1: Type: text/plain, Size: 3787 bytes --]

Hi!

> > > > Ok, I realized that _something_ set up my keyboard on console, too, so
> > > > I'm able to do "init 1" and still interact with the console.
> > > > 
> > > > cm_idlest1_core blocking bits are now 0x42 pretty consistently. But
> > > > they stay 0x42, even when screen is on (and screen on prevents low
> > > > power, right?) so I guess they are not the whole story.
> > > 
> > > Yes they stay at 0x42. And it seems you've configured the UART
> > > idle timeouts too.
> > 
> > Yes, thanks for the script ;-).
> > 
> > > > I'm able to get down to 50mA power consumption with screen off. I was
> > > > getting 90mA with X, wifi in powersave and screen off.
> > > > 
> > > > Head proximity sensor is still on in this configuration. But I guess
> > > > that should not keep the system busy...?
> > > > 
> > > > Ok, wait a moment, I'm getting "Camera Focus", "Camera Capture" and
> > > > "Lock button" interrupts ... at something like 20/second. Without
> > > > touching anything. Hmm. Also "pm_wkup", but that might be
> > > > expected. Plus, 49052000.gpio and 49054000.gpio are rather
> > > > active. Might be related to the above. Also "gp_timer" and
> > > > 48070000.i2c are active, perhaps also related. I don't think I have
> > > > the camera working...
> > > 
> > > Hmm that does not sound right at all for the GPIOs. The pm_wkup
> > > interrupt triggers every time you hit wfi pretty much, so that
> > > should be OK.
> > 
> > Wifi was down at that point.
> > 
> > Some more testing: after boot, interrupt counts stay low, as
> > expected. But when I attempt to enable power management, they start
> > rising. Sometimes 60/second, sometimes less.
> > 
> > Things are fine as long as I don't enable the off mode; when I enable
> > the off mode, "echo 1 > /sys/kernel/debug/pm_debug/enable_off_mode"
> > interrupt counts start rising immediately. "echo 0 >
> > /sys/kernel/debug/pm_debug/enable_off_mode" stops that.
> > 
> > But no matter what configuration, activity LEDs still indicate it is
> > busy, and idle power consumption is cca 53mA.
> 
> Not seeing that here at all. My n900 with v4.6-rc2 and omap2plus_defconfig
> booted with u-boot keeps hitting off mode with both LEDs going off just
> fine. It's waking up about once a second or a litte bit more often.

Ok, that was 4.4 (with patches that should not be related).

I now checked with 4.6-rc2+mini_changes, and I'm getting similar results:

Battery 4.05V 4.07V 4.11V 85% 91% 98% 1546/1569 mAh Full -131/550/100
mA
00001fff 48005020 (fa005020) cm_idlest_per blocking bits: 0007e000
f7fffebd 48004a20 (fa004a20) cm_idlest1_core blocking bits: 00000042
0000000d 48004a28 (fa004a28) cm_idlest3_core

I'm booting from NOLO, and I'm _not_ loading/removing gadget modules,
as blocking bits are 0x42, so gadget is not a problem. (Let me know if
I'm wrong).

Let me retry with 4.6-rc2 (9735a22799b9214d17d3c231fe377fc852f042e9),
no modifications. Aha, but vanilla 4.6-rc2 does not have
cm_idlest1_core display in debugfs :-(. 

And yes, I see the same effect in /proc/interrupts:

180:        280  49052000.gpio   4 Edge      Camera Focus
181:        280  49052000.gpio   5 Edge      Camera Capture
(few seconds after that)
180:        738  49052000.gpio   4 Edge      Camera Focus
181:        738  49052000.gpio   5 Edge      Camera Capture

after I do echo 1 > /sys/kernel/debug/pm_debug/enable_off_mode .

> Care to post your .config somewhere, I could give that a try?

gzipped config is attached.

Note that I'm still using NOLO. I enabled the sleep, then went to
runlevel 1. LEDs still stay on, 55mA power consumption. That was with
1 in off_mode.

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

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 24857 bytes --]

WARNING: multiple messages have this Message-ID (diff)
From: pavel@ucw.cz (Pavel Machek)
To: linux-arm-kernel@lists.infradead.org
Subject: 4.4, 4.6: camera and unlock buttons produce tons of interrupts (was Re: N900 sleep mode)
Date: Thu, 7 Apr 2016 21:48:29 +0200	[thread overview]
Message-ID: <20160407194828.GA13338@amd> (raw)
In-Reply-To: <20160407174019.GM16484@atomide.com>

Hi!

> > > > Ok, I realized that _something_ set up my keyboard on console, too, so
> > > > I'm able to do "init 1" and still interact with the console.
> > > > 
> > > > cm_idlest1_core blocking bits are now 0x42 pretty consistently. But
> > > > they stay 0x42, even when screen is on (and screen on prevents low
> > > > power, right?) so I guess they are not the whole story.
> > > 
> > > Yes they stay at 0x42. And it seems you've configured the UART
> > > idle timeouts too.
> > 
> > Yes, thanks for the script ;-).
> > 
> > > > I'm able to get down to 50mA power consumption with screen off. I was
> > > > getting 90mA with X, wifi in powersave and screen off.
> > > > 
> > > > Head proximity sensor is still on in this configuration. But I guess
> > > > that should not keep the system busy...?
> > > > 
> > > > Ok, wait a moment, I'm getting "Camera Focus", "Camera Capture" and
> > > > "Lock button" interrupts ... at something like 20/second. Without
> > > > touching anything. Hmm. Also "pm_wkup", but that might be
> > > > expected. Plus, 49052000.gpio and 49054000.gpio are rather
> > > > active. Might be related to the above. Also "gp_timer" and
> > > > 48070000.i2c are active, perhaps also related. I don't think I have
> > > > the camera working...
> > > 
> > > Hmm that does not sound right at all for the GPIOs. The pm_wkup
> > > interrupt triggers every time you hit wfi pretty much, so that
> > > should be OK.
> > 
> > Wifi was down at that point.
> > 
> > Some more testing: after boot, interrupt counts stay low, as
> > expected. But when I attempt to enable power management, they start
> > rising. Sometimes 60/second, sometimes less.
> > 
> > Things are fine as long as I don't enable the off mode; when I enable
> > the off mode, "echo 1 > /sys/kernel/debug/pm_debug/enable_off_mode"
> > interrupt counts start rising immediately. "echo 0 >
> > /sys/kernel/debug/pm_debug/enable_off_mode" stops that.
> > 
> > But no matter what configuration, activity LEDs still indicate it is
> > busy, and idle power consumption is cca 53mA.
> 
> Not seeing that here at all. My n900 with v4.6-rc2 and omap2plus_defconfig
> booted with u-boot keeps hitting off mode with both LEDs going off just
> fine. It's waking up about once a second or a litte bit more often.

Ok, that was 4.4 (with patches that should not be related).

I now checked with 4.6-rc2+mini_changes, and I'm getting similar results:

Battery 4.05V 4.07V 4.11V 85% 91% 98% 1546/1569 mAh Full -131/550/100
mA
00001fff 48005020 (fa005020) cm_idlest_per blocking bits: 0007e000
f7fffebd 48004a20 (fa004a20) cm_idlest1_core blocking bits: 00000042
0000000d 48004a28 (fa004a28) cm_idlest3_core

I'm booting from NOLO, and I'm _not_ loading/removing gadget modules,
as blocking bits are 0x42, so gadget is not a problem. (Let me know if
I'm wrong).

Let me retry with 4.6-rc2 (9735a22799b9214d17d3c231fe377fc852f042e9),
no modifications. Aha, but vanilla 4.6-rc2 does not have
cm_idlest1_core display in debugfs :-(. 

And yes, I see the same effect in /proc/interrupts:

180:        280  49052000.gpio   4 Edge      Camera Focus
181:        280  49052000.gpio   5 Edge      Camera Capture
(few seconds after that)
180:        738  49052000.gpio   4 Edge      Camera Focus
181:        738  49052000.gpio   5 Edge      Camera Capture

after I do echo 1 > /sys/kernel/debug/pm_debug/enable_off_mode .

> Care to post your .config somewhere, I could give that a try?

gzipped config is attached.

Note that I'm still using NOLO. I enabled the sleep, then went to
runlevel 1. LEDs still stay on, 55mA power consumption. That was with
1 in off_mode.

Best regards,
									Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
-------------- next part --------------
A non-text attachment was scrubbed...
Name: config.gz
Type: application/gzip
Size: 24857 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20160407/6f758f45/attachment-0001.gz>

  reply	other threads:[~2016-04-07 19:48 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
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                                       ` Pavel Machek [this message]
2016-04-07 19:48                                         ` 4.4, 4.6: " 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=20160407194828.GA13338@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.