From: Sasha Levin <sashal@kernel.org>
To: patches@lists.linux.dev, stable@vger.kernel.org
Cc: Janne Grunau <j@jannau.net>, Ulf Hansson <ulf.hansson@linaro.org>,
Neal Gompa <neal@gompa.dev>,
"Rob Herring (Arm)" <robh@kernel.org>,
Sven Peter <sven@kernel.org>, Sasha Levin <sashal@kernel.org>,
asahi@lists.linux.dev, linux-arm-kernel@lists.infradead.org
Subject: [PATCH AUTOSEL 6.17-6.6] pmdomain: apple: Add "apple,t8103-pmgr-pwrstate"
Date: Thu, 9 Oct 2025 11:55:55 -0400 [thread overview]
Message-ID: <20251009155752.773732-89-sashal@kernel.org> (raw)
In-Reply-To: <20251009155752.773732-1-sashal@kernel.org>
From: Janne Grunau <j@jannau.net>
[ Upstream commit 442816f97a4f84cb321d3359177a3b9b0ce48a60 ]
After discussion with the devicetree maintainers we agreed to not extend
lists with the generic compatible "apple,pmgr-pwrstate" anymore [1]. Use
"apple,t8103-pmgr-pwrstate" as base compatible as it is the SoC the
driver and bindings were written for.
[1]: https://lore.kernel.org/asahi/12ab93b7-1fc2-4ce0-926e-c8141cfe81bf@kernel.org/
Signed-off-by: Janne Grunau <j@jannau.net>
Acked-by: Ulf Hansson <ulf.hansson@linaro.org>
Reviewed-by: Neal Gompa <neal@gompa.dev>
Acked-by: Rob Herring (Arm) <robh@kernel.org>
Signed-off-by: Sven Peter <sven@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
---
LLM Generated explanations, may be completely bogus:
YES
- What changed: The patch adds a single OF compatible to the driver
match table to explicitly support the Apple M1 (t8103) PMGR power-
state nodes. Concretely, it inserts `{ .compatible =
"apple,t8103-pmgr-pwrstate" }` in `drivers/pmdomain/apple/pmgr-
pwrstate.c:309`, keeping the existing generic fallback `{ .compatible
= "apple,pmgr-pwrstate" }` just below it (drivers/pmdomain/apple/pmgr-
pwrstate.c:310).
- Rationale from commit message: DT maintainers agreed to stop extending
nodes with the generic `"apple,pmgr-pwrstate"` and instead use the
SoC-specific string as the base compatible. Without this addition, a
DT using only `"apple,t8103-pmgr-pwrstate"` would fail to bind the
driver.
- Scope and risk: The change is a single-line device-ID addition to the
OF match table. It does not touch runtime logic, data structures,
Kconfig, or probe/remove paths. There are no architectural changes.
This kind of “add a new compatible ID” patch is routinely accepted for
stable because it’s low-risk and purely affects device binding.
- User impact fixed: On systems providing a DTB that omits the generic
fallback (aligned with the new binding guidance), the driver would not
probe, leaving PM domains and reset control unmanaged. Adding the
t8103 compatible restores functionality. This is a real, user-visible
failure mode when booting older kernels with newer DTBs or bootloader-
provided DTBs following updated conventions.
- Alignment with bindings: The bindings already enumerate SoC-specific
compatibles, including `"apple,t8103-pmgr-pwrstate"`
(Documentation/devicetree/bindings/power/apple,pmgr-pwrstate.yaml:41).
The driver now matches that SoC-specific string directly, which aligns
the implementation with binding expectations going forward.
- No regressions expected: Existing DTs that list both
`"apple,t8103-pmgr-pwrstate", "apple,pmgr-pwrstate"` continue to bind
identically, only preferring the SoC-specific match. DTs that still
only have the generic string continue to work via the unchanged `{
.compatible = "apple,pmgr-pwrstate" }` entry
(drivers/pmdomain/apple/pmgr-pwrstate.c:310).
- Stable policy fit: This is a minimal, contained, non-feature change
that enables correct binding for an existing driver on an existing SoC
and prevents regressions with evolving DTs. There’s no security
implication and no interaction with critical core subsystems beyond OF
matching.
- Backport note: On older stable series, the file may reside under a
different path (pre-“pmdomain” rename). The change remains a trivial
OF-ID addition to the same match table in that file.
Given the above, this is a textbook low-risk backport that prevents real
breakage with newer DTBs and should be backported to stable trees that
already contain the Apple PMGR power-state driver.
drivers/pmdomain/apple/pmgr-pwrstate.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/pmdomain/apple/pmgr-pwrstate.c b/drivers/pmdomain/apple/pmgr-pwrstate.c
index 9467235110f46..82c33cf727a82 100644
--- a/drivers/pmdomain/apple/pmgr-pwrstate.c
+++ b/drivers/pmdomain/apple/pmgr-pwrstate.c
@@ -306,6 +306,7 @@ static int apple_pmgr_ps_probe(struct platform_device *pdev)
}
static const struct of_device_id apple_pmgr_ps_of_match[] = {
+ { .compatible = "apple,t8103-pmgr-pwrstate" },
{ .compatible = "apple,pmgr-pwrstate" },
{}
};
--
2.51.0
next prev parent reply other threads:[~2025-10-09 16:00 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20251009155752.773732-1-sashal@kernel.org>
2025-10-09 15:54 ` [PATCH AUTOSEL 6.17-6.12] soc: apple: mailbox: Add Apple A11 and T2 mailbox support Sasha Levin
2025-10-10 2:22 ` Nick Chan
2025-11-04 0:22 ` Sasha Levin
2025-10-09 15:54 ` [PATCH AUTOSEL 6.17-6.12] soc: sunxi: sram: add entry for a523 Sasha Levin
2025-10-09 16:38 ` Andre Przywara
2025-11-04 0:22 ` Sasha Levin
2025-10-09 15:54 ` [PATCH AUTOSEL 6.17-5.10] pinctrl: single: fix bias pull up/down handling in pin_config_set Sasha Levin
2025-10-09 15:54 ` [PATCH AUTOSEL 6.17-6.16] soc: ti: k3-socinfo: Add information for AM62L SR1.1 Sasha Levin
2025-10-09 15:55 ` [PATCH AUTOSEL 6.17] mfd: macsmc: Add "apple,t8103-smc" compatible Sasha Levin
2025-10-09 15:55 ` [PATCH AUTOSEL 6.17-5.4] mfd: stmpe: Remove IRQ domain upon removal Sasha Levin
2025-10-09 15:55 ` [PATCH AUTOSEL 6.17-5.4] cpufreq/longhaul: handle NULL policy in longhaul_exit Sasha Levin
2025-10-09 15:55 ` [PATCH AUTOSEL 6.17-6.16] firmware: ti_sci: Enable abort handling of entry to LPM Sasha Levin
2025-10-09 15:55 ` [PATCH AUTOSEL 6.17-5.15] soc: aspeed: socinfo: Add AST27xx silicon IDs Sasha Levin
2025-10-09 15:55 ` [PATCH AUTOSEL 6.17-6.6] arm64: zynqmp: Revert usb node drive strength and slew rate for zcu106 Sasha Levin
2025-10-09 15:55 ` [PATCH AUTOSEL 6.17-6.12] arm64: zynqmp: Disable coresight by default Sasha Levin
2025-10-09 15:55 ` Sasha Levin [this message]
2025-10-09 15:56 ` [PATCH AUTOSEL 6.17-6.16] arm64: versal-net: Update rtc calibration value Sasha Levin
2025-10-09 15:56 ` [PATCH AUTOSEL 6.17-5.10] soc: ti: pruss: don't use %pK through printk Sasha Levin
2025-10-09 15:56 ` [PATCH AUTOSEL 6.17-5.4] mfd: stmpe-i2c: Add missing MODULE_LICENSE Sasha Levin
2025-10-09 15:56 ` [PATCH AUTOSEL 6.17-5.4] irqchip/gic-v2m: Handle Multiple MSI base IRQ Alignment Sasha Levin
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=20251009155752.773732-89-sashal@kernel.org \
--to=sashal@kernel.org \
--cc=asahi@lists.linux.dev \
--cc=j@jannau.net \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=neal@gompa.dev \
--cc=patches@lists.linux.dev \
--cc=robh@kernel.org \
--cc=stable@vger.kernel.org \
--cc=sven@kernel.org \
--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 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).