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] [thread-next>] [day] [month] [year] [list]

Message-ID: <ec0f5da5a4da6b2649373530d0103d65ea990c0e.camel@impulsing.ch>
Date: Thu, 24 Apr 2025 08:20:32 +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 09:21 +0200, Francesco Dolcini wrote:
> On Thu, Apr 24, 2025 at 08:16:11AM +0200, Wojciech Dubowik wrote:
> > On Wed, Apr 23, 2025 at 11:23:09AM +0000, Philippe Schenker wrote:
> > > On Wed, 2025-04-23 at 12:21 +0200, Francesco Dolcini wrote:
> > > > > > 
> > > > > > I would backport this to also older kernel, so to me
> > > > > > 
> > > > > > Fixes: 6a57f224f734 ("arm64: dts: freescale: add initial
> > > > > > support
> > > > > > for
> > > > > > verdin imx8m mini")
> > > > > 
> > > > > NACK for the proposed Fixes, this introduces a new Kconfig
> > > > > which
> > > > > could
> > > > > have side-effects in users of current stable kernels.
> > > > 
> > > > The driver for "regulator-gpio" compatible? I do not agree with
> > > > your
> > > > argument,
> > > > sorry. 
> > > > 
> > > > The previous description was not correct. There was an unused
> > > > regulator in the DT that was not switched off just by chance.
> > > > 
> > > > Francesco
> > > > 
> > > My previous reasoning about the driver is one point. The other is
> > > that
> > > the initial implementation in 6a57f224f734 ("arm64: dts:
> > > freescale: add
> > > initial support for verdin imx8m mini") was not wrong at all it
> > > was
> > > just different.
> > > 
> > > My concern is for existing users of stable kernels that you
> > > change the
> > > underlying implementation of how the SD voltage gets switched.
> > > And
> > > adding the tag
> > > 
> > > 
> > > Fixes: 6a57f224f734 ("arm64: dts: freescale: add initial support
> > > for
> > > verdin imx8m mini")
> > > 
> > > to this patch would get this new implementation also to stable
> > > kernels
> > > not affected by the issue introduced in f5aab0438ef1 ("regulator:
> > > pca9450: Fix enable register for LDO5")
> > 
> > I will wait a day and send V4 with what I beleive was result of
> > this
> > discussion.
> 
> 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.

My wish in general is that we are very careful of what is backported to
stable kernels. I do not want to edit my device-trees that are running
just fine. I agree and think it's good this is going into mainline but
I still do not agree on backporting it to every stable kernel the
Verdin iMX8M Mini is in.

Philippe

> 
> FWIW, I tested this change with both v6.1 and v6.6 and it works
> correctly, as
> expected.
> 
> Francesco

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