mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Charles Keepax <ckeepax@opensource.cirrus.com>
To: broonie@kernel.org, lee@kernel.org
Cc: lgirdwood@gmail.com, linux-sound@vger.kernel.org,
	linux-kernel@vger.kernel.org, patches@opensource.cirrus.com
Subject: [PATCH 0/6] Improve cs42l43 suspend/IRQ interactions
Date: Wed,  3 Sep 2025 10:45:43 +0100	[thread overview]
Message-ID: <20250903094549.271068-1-ckeepax@opensource.cirrus.com> (raw)

cs42l43 uses pm_runtime_force_suspend() during system suspend, however
this means care must be taken that IRQ handler code isn't running when
entering system suspend as force suspend will ignore that the handler
is holding a pm reference. Typically the result of this is just a few
error messages, but better to improve the handling and ensure that all
IRQ processing is synchronised in before system suspend.

Note the ASoC and MFD bits of this series have no build dependencies,
so could go through their seperate trees, however, it would be nice if
all the patches could land in the same kernel version to avoid the IRQ
handling being in a transitional state.

Thanks,
Charles

Charles Keepax (6):
  ASoC: cs42l43: Rename system suspend callback and fix debug print
  ASoC: cs42l43: Store IRQ domain in codec private data
  ASoC: cs42l43: Disable IRQs in system suspend
  ASoC: cs42l43: Shutdown jack detection on suspend
  mfd: cs42l43: Move IRQ enable/disable to encompass force suspend
  mfd: cs42l43: Remove IRQ masking in suspend

 drivers/mfd/cs42l43.c           |  32 +---------
 sound/soc/codecs/cs42l43-jack.c |   8 ++-
 sound/soc/codecs/cs42l43.c      | 109 +++++++++++++++++++++++---------
 sound/soc/codecs/cs42l43.h      |   3 +
 4 files changed, 89 insertions(+), 63 deletions(-)

-- 
2.47.2


             reply	other threads:[~2025-09-03  9:46 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-09-03  9:45 Charles Keepax [this message]
2025-09-03  9:45 ` [PATCH 1/6] ASoC: cs42l43: Rename system suspend callback and fix debug print Charles Keepax
2025-09-03  9:45 ` [PATCH 2/6] ASoC: cs42l43: Store IRQ domain in codec private data Charles Keepax
2025-09-03  9:45 ` [PATCH 3/6] ASoC: cs42l43: Disable IRQs in system suspend Charles Keepax
2025-09-03  9:45 ` [PATCH 4/6] ASoC: cs42l43: Shutdown jack detection on suspend Charles Keepax
2025-09-03  9:45 ` [PATCH 5/6] mfd: cs42l43: Move IRQ enable/disable to encompass force suspend Charles Keepax
2025-09-03  9:45 ` [PATCH 6/6] mfd: cs42l43: Remove IRQ masking in suspend Charles Keepax
2025-09-04 18:09 ` (subset) [PATCH 0/6] Improve cs42l43 suspend/IRQ interactions Mark Brown
2025-09-11 15:02 ` Lee Jones

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20250903094549.271068-1-ckeepax@opensource.cirrus.com \
    --to=ckeepax@opensource.cirrus.com \
    --cc=broonie@kernel.org \
    --cc=lee@kernel.org \
    --cc=lgirdwood@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-sound@vger.kernel.org \
    --cc=patches@opensource.cirrus.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®