From: mateusz.nowicki@posteo.net
To: Arnd Bergmann <arnd@arndb.de>
Cc: Marek Vasut <marex@nabladev.com>,
Alexandre Torgue <alexandre.torgue@foss.st.com>,
Maxime Coquelin <mcoquelin.stm32@gmail.com>,
Thomas BOURGOIN <thomas.bourgoin@st.com>,
Rob Herring <robh@kernel.org>,
krzk+dt@kernel.org, Conor Dooley <conor+dt@kernel.org>,
Lionel DEBIEVE <lionel.debieve@foss.st.com>,
Amelie DELAUNAY <amelie.delaunay@foss.st.com>,
Patrice Chotard <patrice.chotard@foss.st.com>,
devicetree@vger.kernel.org,
linux-stm32@st-md-mailman.stormreply.com,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, kernel <kernel@dh-electronics.com>,
Geert Uytterhoeven <geert+renesas@glider.be>,
"laurent.pinchart" <laurent.pinchart@ideasonboard.com>,
Fabrice GASNIER <fabrice.gasnier@foss.st.com>,
Stefano Babic <sbabic@nabladev.com>,
Etienne Carriere <etienne.carriere@foss.st.com>
Subject: Re: [PATCH] arm64: dts: st: fix arm,smc-id of the SMC watchdog on stm32mp2
Date: Sat, 31 Oct 2026 12:00:27 +0000 [thread overview]
Message-ID: <f7010f07fdcdb4ff04a0dbfac1cc5421@posteo.net> (raw)
In-Reply-To: <cbcb8d78-51f3-41ab-80d6-fe3f0623da5c@app.fastmail.com>
Hi Arnd, Marek,
On Sat, Oct 31, 2026, Arnd Bergmann wrote:
> I think for the kernel it's pretty clear, we have to revert it
> to avoid the regression on old firmware, at least for boards
> that have ever shipped with that version. I count eight boards
> for stm32mp2 in upstream Linux, do we know how many of them
> are affected by the ABI break?
As the author of the patch, I looked at where both IDs come from.
0xb200005a only ever matched ST's *downstream* OP-TEE 3.19.0-stm32mp,
the base of the first OpenSTLinux releases for stm32mp25 (last update
2024-11-25). No *upstream* OP-TEE release ever used it on stm32mp2.
0xbc000000 is used by:
- ST's downstream OP-TEE since 4.0.0-stm32mp-r1 (2024-11-04), the base
of the current OpenSTLinux releases, up to 4.8.0-stm32mp-r1
(2026-10-27)
- upstream OP-TEE since 4.8.0 (2025-10-24), commit 11d68b6754
("plat-stm32mp2: enable watchdog SMC service"). Before 4.8.0
upstream OP-TEE has no watchdog SMC service on stm32mp2 at all.
On the Linux side:
- 89fca38168c9 (v6.7, 2023-09) added the node with 0xb200005a, which
matched the 3.19 based firmware of that time.
- 7a57b1bb1afb and e9b03ef21386 (v6.15, 2025-03) added stm32mp21 with
0xbc000000 and stm32mp23 with 0xb200005a on the same day.
- ae4d08bbe3f4 (v7.4 merge window) moved stm32mp23 and stm32mp25 to
0xbc000000.
So from v6.7 to v7.3 the watchdog never worked with upstream OP-TEE,
nor with ST's own firmware since November 2024: arm_smc_wdt fails to
probe with -EIO and an IWDG started by the boot loader is never
refreshed. That is how I found it on a STM32MP257F-DK with upstream
OP-TEE 4.10.
A revert would move the breakage to everybody running current
firmware. For stm32mp23 there is nothing to restore: ST's OP-TEE
supports stm32mp21 and stm32mp23 only since 4.0.0-stm32mp, so
0xb200005a never matched any stm32mp23 firmware.
About the boards: arm_wdt is enabled on all eight.
- stm32mp215f-dk is not affected, stm32mp211.dtsi always had
0xbc000000.
- stm32mp235f-dk and stm32mp23xx-dhcos-bb can only run firmware with
0xbc000000.
- That leaves the stm32mp25 boards: stm32mp257f-dk, stm32mp257f-ev1,
stm32mp257d-engicam-microgea-rmm, stm32mp255c-dhcos-dhsbc and
stm32mp25xx-dhcos-bb. Only those still shipping a 3.19 based OP-TEE
need the old ID.
So I would suggest the opposite direction: keep 0xbc000000 in the SoC
dtsi, which matches upstream OP-TEE and current ST firmware, and let
the boards that shipped with the old firmware override it in their
board dts:
&arm_wdt {
arm,smc-id = <0xb200005a>;
};
Marek, would that work for the DHCOS boards? I can send the patch if
you tell me which of them ship the old OP-TEE. Your OP-TEE change to
accept both IDs is a good mitigation on top of it for anybody who
updates the firmware.
Best regards,
Mateusz
next prev parent reply other threads:[~2026-10-31 12:00 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-07 9:14 [PATCH] arm64: dts: st: fix arm,smc-id of the SMC watchdog on stm32mp2 Mateusz Nowicki
2026-09-08 8:52 ` [Linux-stm32] [PATCH] arm64: dts: st: fix arm, smc-id " Antonio Borneo
2026-09-23 11:52 ` [PATCH] arm64: dts: st: fix arm,smc-id " Mateusz Nowicki
2026-09-28 8:45 ` Alexandre TORGUE
2026-10-31 5:59 ` Marek Vasut
2026-10-31 8:45 ` Arnd Bergmann
2026-10-31 12:00 ` mateusz.nowicki [this message]
2026-10-31 13:35 ` Marek Vasut
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=f7010f07fdcdb4ff04a0dbfac1cc5421@posteo.net \
--to=mateusz.nowicki@posteo.net \
--cc=alexandre.torgue@foss.st.com \
--cc=amelie.delaunay@foss.st.com \
--cc=arnd@arndb.de \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=etienne.carriere@foss.st.com \
--cc=fabrice.gasnier@foss.st.com \
--cc=geert+renesas@glider.be \
--cc=kernel@dh-electronics.com \
--cc=krzk+dt@kernel.org \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-stm32@st-md-mailman.stormreply.com \
--cc=lionel.debieve@foss.st.com \
--cc=marex@nabladev.com \
--cc=mcoquelin.stm32@gmail.com \
--cc=patrice.chotard@foss.st.com \
--cc=robh@kernel.org \
--cc=sbabic@nabladev.com \
--cc=thomas.bourgoin@st.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox