public inbox for linux-sunxi@lists.linux.dev
 help / color / mirror / Atom feed
* Re: (subset) [PATCH v3] mfd: axp20x: AXP717: Fix missing IRQ status registers range
  2024-06-13 23:31 Andre Przywara
@ 2024-06-14  7:43 ` Lee Jones
  0 siblings, 0 replies; 3+ messages in thread
From: Lee Jones @ 2024-06-14  7:43 UTC (permalink / raw)
  To: Chen-Yu Tsai, Andre Przywara
  Cc: Jernej Skrabec, linux-kernel, linux-sunxi, Chris Morgan,
	Ryan Walklin, Philippe Simons

On Fri, 14 Jun 2024 00:31:04 +0100, Andre Przywara wrote:
> While we list the "IRQ status *and acknowledge*" registers as volatile
> in the MFD description, they are missing from the writable range array,
> so acknowledging any interrupts was met with an -EIO error.
> This error propagates up, leading to the whole AXP717 driver failing to
> probe, which is fatal to most systems using this PMIC, since most
> peripherals refer one of the PMIC voltage rails.
> This wasn't noticed on the initial submission, since the interrupt was
> completely missing at this point, but the DTs now merged describe the
> interrupt, creating the problem.
> 
> [...]

Applied, thanks!

[1/1] mfd: axp20x: AXP717: Fix missing IRQ status registers range
      commit: 569c5136baf12c3b8dd137ac356f2a7ceb38bdb5

--
Lee Jones [李琼斯]


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: (subset) [PATCH v3] mfd: axp20x: AXP717: Fix missing IRQ status registers range
@ 2024-06-14 16:43 Philippe Simons
  2024-06-14 17:11 ` Lee Jones
  0 siblings, 1 reply; 3+ messages in thread
From: Philippe Simons @ 2024-06-14 16:43 UTC (permalink / raw)
  To: lee
  Cc: andre.przywara, jernej.skrabec, linux-kernel, linux-sunxi,
	macroalpha82, ryan, Philippe Simons, wens

Tested on my RG35XX-H, v6.10rc3 boots with the patch applied

Tested-by: Philippe Simons <simons.philippe@gmail.com>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: (subset) [PATCH v3] mfd: axp20x: AXP717: Fix missing IRQ status registers range
  2024-06-14 16:43 (subset) [PATCH v3] mfd: axp20x: AXP717: Fix missing IRQ status registers range Philippe Simons
@ 2024-06-14 17:11 ` Lee Jones
  0 siblings, 0 replies; 3+ messages in thread
From: Lee Jones @ 2024-06-14 17:11 UTC (permalink / raw)
  To: Philippe Simons
  Cc: andre.przywara, jernej.skrabec, linux-kernel, linux-sunxi,
	macroalpha82, ryan, wens

On Fri, 14 Jun 2024, Philippe Simons wrote:

> Tested on my RG35XX-H, v6.10rc3 boots with the patch applied
> 
> Tested-by: Philippe Simons <simons.philippe@gmail.com>

It looks like the way you replied to this mail failed to attach itself
to the original thread, so I have no idea what this tag applies to.

However this point may be moot since, by the look of the subject line,
you're attempting to apply tags to patches which have already been
committed.

-- 
Lee Jones [李琼斯]

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2024-06-14 17:11 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-06-14 16:43 (subset) [PATCH v3] mfd: axp20x: AXP717: Fix missing IRQ status registers range Philippe Simons
2024-06-14 17:11 ` Lee Jones
  -- strict thread matches above, loose matches on Subject: below --
2024-06-13 23:31 Andre Przywara
2024-06-14  7:43 ` (subset) " Lee Jones

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox