linux-kernel - Re: EXTERNAL - [PATCH v3] arm64: dts: imx8mm-verdin: Link reg_usdhc2

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: <84dc6fdf295902044d49cafc4479eb75477bba5c.camel@impulsing.ch>
Date: Thu, 24 Apr 2025 09:24:16 +0000
From: Philippe Schenker <philippe.schenker@...ulsing.ch>
To: Francesco Dolcini <francesco@...cini.it>, Wojciech Dubowik
	<Wojciech.Dubowik@...com>
CC: "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, Rob Herring
	<robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley
	<conor+dt@...nel.org>, Shawn Guo <shawnguo@...nel.org>, Sascha Hauer
	<s.hauer@...gutronix.de>, Pengutronix Kernel Team <kernel@...gutronix.de>,
	Fabio Estevam <festevam@...il.com>, "devicetree@...r.kernel.org"
	<devicetree@...r.kernel.org>, "imx@...ts.linux.dev" <imx@...ts.linux.dev>,
	"linux-arm-kernel@...ts.infradead.org"
	<linux-arm-kernel@...ts.infradead.org>, "stable@...r.kernel.org"
	<stable@...r.kernel.org>
Subject: Re: EXTERNAL - [PATCH v3] arm64: dts: imx8mm-verdin: Link
 reg_usdhc2_vqmmc to usdhc2

On Thu, 2025-04-24 at 10:34 +0200, Francesco Dolcini wrote:
> > > 
> > > My opinion is that we should backport this fix. The DT
> > > description
> > > was
> > > wrong, that change on the PMIC driver just made the issue
> > > visible,
> > > the DT is about the HW description.
> > 
> > From what I understand it was not wrong but it became wrong with
> > the
> > PMIC driver changes.
> 
> It was wrong. You had a regulator described as not used, and
> therefore the OS
> was free to switch it off. For a bug in the driver this regulator was
> not
> switched off by the old kernels.
> 
> The DT description was never correct.
> 
> With that said, hopefully it makes no difference in pratice,
> f5aab0438ef17f01c5ecd25e61ae6a03f82a4586 will be backported, and
> therefore also
> any fix to it should.

Aah I overlooked the Fixes: tag there. Yes in this case we need your
Fixes tag, you are of course correct as always.

@Wojciech, please send a v4 with:

Fixes: 6a57f224f734 ("arm64: dts: freescale: add initial support for
verdin imx8m mini")

Philippe

> 
> If for some unfortunate reason we do not backport also this one,
> you'll get a
> non-working SD card on your LTS kernel.
> 
> Francesco

Download attachment "signature.asc" of type "application/pgp-signature" (704 bytes)