Re: [PATCH v2 1/5] mfd: axp20x: AXP717: Fix missing IRQ status registers range

From: Lee Jones
Date: Wed Jun 12 2024 - 11:25:28 EST


On Wed, 12 Jun 2024, Andre Przywara wrote:

> Hi Lee,
>
> On 02/05/2024 10:39, Lee Jones wrote:
> > On Thu, 18 Apr 2024, Andre Przywara wrote:
> >
> > > While we list the "IRQ status *and acknowledge*" registers as volatile,
> > > they are missing from the writable range array, so acknowledging any
> > > interrupts was met with an -EIO error.
> > >
> > > Add the five registers that hold those bits to the writable array.
> > >
> > > Fixes: b5bfc8ab2484 ("mfd: axp20x: Add support for AXP717 PMIC")
> > > Reported-by: Chris Morgan <macromorgan@xxxxxxxxxxx>
> > > Signed-off-by: Andre Przywara <andre.przywara@xxxxxxx>
> > > ---
> > > drivers/mfd/axp20x.c | 1 +
> > > 1 file changed, 1 insertion(+)
> >
> > Acked-by: Lee Jones <lee@xxxxxxxxxx>
>
> Can you please take just this patch as a fix for 6.10? This fixes the power
> key operation.
> This applies cleanly on top of v6.10-rc3, so there is no need for any extra
> immutable branch or coordination with regulator.
> (The same is true independently for patch 2/5, on the regulator side).

What does the Fixes: commit break?

Or is it the case that it never worked properly?

--
Lee Jones [李琼斯]