From: "Heiko Stübner" <heiko@sntech.de>
To: Geert Uytterhoeven <geert@linux-m68k.org>,
Nicolas Frattaroli <nicolas.frattaroli@collabora.com>,
Saravana Kannan <saravanak@google.com>,
linux-pm@vger.kernel.org, Ulf Hansson <ulf.hansson@linaro.org>
Cc: Stephen Boyd <sboyd@kernel.org>,
"Rafael J . Wysocki" <rafael@kernel.org>,
Sebastian Reichel <sebastian.reichel@collabora.com>,
Sebin Francis <sebin.francis@ti.com>,
Diederik de Haas <didi.debian@cknow.org>,
Bjorn Andersson <andersson@kernel.org>,
Abel Vesa <abel.vesa@linaro.org>, Peng Fan <peng.fan@oss.nxp.com>,
Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>,
Johan Hovold <johan@kernel.org>,
Maulik Shah <maulik.shah@oss.qualcomm.com>,
Michal Simek <michal.simek@amd.com>,
Konrad Dybcio <konradybcio@kernel.org>,
Thierry Reding <thierry.reding@gmail.com>,
Jonathan Hunter <jonathanh@nvidia.com>,
Ulf Hansson <ulf.hansson@linaro.org>,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/5] pmdomain: core: Restore behaviour for disabling unused PM domains
Date: Tue, 09 Sep 2025 14:50:09 +0200 [thread overview]
Message-ID: <6841173.4vTCxPXJkl@diego> (raw)
In-Reply-To: <20250909111130.132976-2-ulf.hansson@linaro.org>
Am Dienstag, 9. September 2025, 13:11:20 Mitteleuropäische Sommerzeit schrieb Ulf Hansson:
> Recent changes to genpd prevents those PM domains being powered-on during
> initialization from being powered-off during the boot sequence. Based upon
> whether CONFIG_PM_CONFIG_PM_GENERIC_DOMAINS_OF is set of not, genpd relies
> on the sync_state mechanism or the genpd_power_off_unused() (which is a
> late_initcall_sync), to understand when it's okay to allow these PM domains
> to be powered-off.
>
> This new behaviour in genpd has lead to problems on different platforms.
> Let's therefore restore the behavior of genpd_power_off_unused().
> Moreover, let's introduce GENPD_FLAG_NO_STAY_ON, to allow genpd OF
> providers to opt-out from the new behaviour.
>
> Link: https://lore.kernel.org/all/20250701114733.636510-1-ulf.hansson@linaro.org/
> Reported-by: Geert Uytterhoeven <geert@linux-m68k.org>
> Link: https://lore.kernel.org/all/20250902-rk3576-lockup-regression-v1-1-c4a0c9daeb00@collabora.com/
> Reported-by: Nicolas Frattaroli <nicolas.frattaroli@collabora.com>
> Fixes: 0e789b491ba0 ("pmdomain: core: Leave powered-on genpds on until sync_state")
> Fixes: 13a4b7fb6260 ("pmdomain: core: Leave powered-on genpds on until late_initcall_sync")
> Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
NPU on rk3588 now again correctly gets its supplying regulator and
running said npu does not end up in a kernel panic :-)
Tested-by: Heiko Stuebner <heiko@sntech.de>
next prev parent reply other threads:[~2025-09-09 17:17 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-09-09 11:11 [PATCH 0/5] pmdomain: Restore behaviour for disabling unused PM domains Ulf Hansson
2025-09-09 11:11 ` [PATCH 1/5] pmdomain: core: " Ulf Hansson
2025-09-09 12:50 ` Heiko Stübner [this message]
2025-09-11 7:56 ` Geert Uytterhoeven
2025-09-11 10:25 ` Ulf Hansson
2025-09-09 11:11 ` [PATCH 2/5] pmdomain: rockchip: Fix regulator dependency with GENPD_FLAG_NO_STAY_ON Ulf Hansson
2025-09-09 12:50 ` Heiko Stübner
2025-09-09 13:49 ` Nicolas Frattaroli
2025-09-09 11:11 ` [PATCH 3/5] pmdomain: renesas: rcar-sysc: Don't keep unused PM domains powered-on Ulf Hansson
2025-09-11 7:56 ` Geert Uytterhoeven
2025-09-11 9:01 ` Geert Uytterhoeven
2025-09-11 10:31 ` Ulf Hansson
2025-09-09 11:11 ` [PATCH 4/5] pmdomain: renesas: rcar-gen4-sysc: " Ulf Hansson
2025-09-11 7:57 ` Geert Uytterhoeven
2025-09-09 11:11 ` [PATCH 5/5] pmdomain: renesas: rmobile-sysc: " Ulf Hansson
2025-09-11 7:58 ` Geert Uytterhoeven
2025-09-10 19:33 ` [PATCH 0/5] pmdomain: Restore behaviour for disabling unused PM domains Diederik de Haas
2025-09-10 19:33 ` Diederik de Haas
2025-09-11 7:18 ` Geert Uytterhoeven
2025-09-11 7:18 ` Geert Uytterhoeven
2025-09-11 8:26 ` Diederik de Haas
2025-09-11 8:26 ` Diederik de Haas
2025-09-12 18:32 ` Saravana Kannan
2025-09-12 18:32 ` Saravana Kannan
2025-09-12 18:36 ` Saravana Kannan
2025-09-12 18:49 ` Nicolas Frattaroli
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=6841173.4vTCxPXJkl@diego \
--to=heiko@sntech.de \
--cc=abel.vesa@linaro.org \
--cc=andersson@kernel.org \
--cc=didi.debian@cknow.org \
--cc=geert@linux-m68k.org \
--cc=johan@kernel.org \
--cc=jonathanh@nvidia.com \
--cc=konradybcio@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=maulik.shah@oss.qualcomm.com \
--cc=michal.simek@amd.com \
--cc=nicolas.frattaroli@collabora.com \
--cc=peng.fan@oss.nxp.com \
--cc=rafael@kernel.org \
--cc=saravanak@google.com \
--cc=sboyd@kernel.org \
--cc=sebastian.reichel@collabora.com \
--cc=sebin.francis@ti.com \
--cc=thierry.reding@gmail.com \
--cc=tomi.valkeinen@ideasonboard.com \
--cc=ulf.hansson@linaro.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.