Re: EXTERNAL - [PATCH v3] arm64: dts: imx8mm-verdin: Link reg_usdhc2_vqmmc to usdhc2

[Philippe Schenker <philippe.schenker@impulsing.ch>]

[-- Attachment #1: Type: text/plain, Size: 1237 bytes --]

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

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 717 bytes --]