linux-kernel - Re: [PATCH v1] power: reset: tdx-ec-poweroff: fix restart

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20260130080943.GA157514@francesco-nb>
Date: Fri, 30 Jan 2026 09:09:59 +0100
From: Francesco Dolcini <francesco@...cini.it>
To: Emanuele Ghidoli <ghidoliemanuele@...il.com>
Cc: Francesco Dolcini <francesco@...cini.it>,
	Sebastian Reichel <sre@...nel.org>, Rob Herring <robh@...nel.org>,
	Krzysztof Kozlowski <krzk+dt@...nel.org>,
	Conor Dooley <conor+dt@...nel.org>,
	Emanuele Ghidoli <emanuele.ghidoli@...adex.com>,
	linux-pm@...r.kernel.org, devicetree@...r.kernel.org,
	linux-kernel@...r.kernel.org, stable@...r.kernel.org
Subject: Re: [PATCH v1] power: reset: tdx-ec-poweroff: fix restart

On Fri, Jan 30, 2026 at 08:11:35AM +0100, Emanuele Ghidoli wrote:
> From: Emanuele Ghidoli <emanuele.ghidoli@...adex.com>
> 
> During testing, restart occasionally failed on Toradex modules.
> 
> The issue was traced to an interaction between the EC-based reset/poweroff
> handler and the PSCI restart handler. While the embedded controller is
> resetting or powering off the module, the PSCI code may still be invoked,
> triggering an I2C transaction to the PMIC. This can leave the PMIC I2C
> in a frozen state.
> 
> Add a delay after issuing the EC reset or power-off command to give the
> controller time to complete the operation and avoid falling back to another
> restart/poweroff provider.
> 
> Also print an error message if sending the command to the embedded controller
> fails.
> 
> Fixes: 18672fe12367 ("power: reset: add Toradex Embedded Controller")
> Cc: stable@...r.kernel.org
> Signed-off-by: Emanuele Ghidoli <emanuele.ghidoli@...adex.com>

Reviewed-by: Francesco Dolcini <francesco.dolcini@...adex.com>