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 [李琼斯]