From: Tony Lindgren <tony@atomide.com>
To: Andreas Kemnade <andreas@kemnade.info>
Cc: Merlijn Wajer <merlijn@wizzup.org>,
linux-omap <linux-omap@vger.kernel.org>,
Adam Ford <aford173@gmail.com>,
Ivaylo Dimitrov <ivo.g.dimitrov.75@gmail.com>,
Pavel Machek <pavel@ucw.cz>, Dev Null <devnull@uvos.xyz>,
"H. Nikolaus Schaller" <hns@goldelico.com>,
Aaro Koskinen <aaro.koskinen@iki.fi>
Subject: Re: Nokia N900 OFF mode regression between v5.10 and v5.11
Date: Mon, 13 Dec 2021 07:54:46 +0200 [thread overview]
Message-ID: <YbbgJnmslVRngjuh@atomide.com> (raw)
In-Reply-To: <20211211114306.16116dde@aktux>
Hi,
* Andreas Kemnade <andreas@kemnade.info> [211211 10:43]:
> The patch fixing thermal power management is:
>
> commit 5093402e5b449b64f7bbaa09057ce40a8f3c1484
> Author: Adam Ford <aford173@gmail.com>
> Date: Fri Sep 11 07:31:56 2020 -0500
>
> thermal: ti-soc-thermal: Enable addition power management
>
> The bandgap sensor can be idled when the processor is too, but it
> isn't currently being done, so the power consumption of OMAP3
> boards can elevated if the bangap sensor is enabled.
>
> This patch attempts to use some additional power management
> to idle the clock to the bandgap when not needed.
>
> Maybe there is something specific to the N900 which causes these
> issues? Well, I'll recheck on the boards I have.
My guess the issue is that omap3-thermal needs to constantly poll
for the registers to get the status and that blocks any deeper
idle states. For PM, probably omap3-thermal needs to be tagged with
status = "disabled" assuming there is some other external thermal
sensor. I'm also guessing that the old Nokia kernel never used the
integrated thermal sensor, might be worth checking though.
Regards,
Tony
next prev parent reply other threads:[~2021-12-13 5:54 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-12-10 23:20 Nokia N900 OFF mode regression between v5.10 and v5.11 Merlijn Wajer
2021-12-11 7:54 ` Andreas Kemnade
2021-12-11 10:23 ` Merlijn Wajer
2021-12-11 10:43 ` Andreas Kemnade
2021-12-13 5:54 ` Tony Lindgren [this message]
2021-12-11 10:43 ` Merlijn Wajer
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=YbbgJnmslVRngjuh@atomide.com \
--to=tony@atomide.com \
--cc=aaro.koskinen@iki.fi \
--cc=aford173@gmail.com \
--cc=andreas@kemnade.info \
--cc=devnull@uvos.xyz \
--cc=hns@goldelico.com \
--cc=ivo.g.dimitrov.75@gmail.com \
--cc=linux-omap@vger.kernel.org \
--cc=merlijn@wizzup.org \
--cc=pavel@ucw.cz \
/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).