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: <aAnXK0sAXqfTNaXg@mt.com>
Date: Thu, 24 Apr 2025 08:16:11 +0200
From: Wojciech Dubowik <Wojciech.Dubowik@...com>
To: Philippe Schenker <philippe.schenker@...ulsing.ch>
Cc: Francesco Dolcini <francesco@...cini.it>,
	"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 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.

Wojtek

> 
> Philippe