[PATCH 00/14] ASoC: codecs: propagate component resume cache sync failures

Pengpeng Hou posted 14 patches 3 weeks, 3 days ago
sound/soc/codecs/adau17x1.c | 4 +---
sound/soc/codecs/ak4642.c   | 3 +--
sound/soc/codecs/es8316.c   | 4 +---
sound/soc/codecs/es8323.c   | 4 +---
sound/soc/codecs/nau8540.c  | 4 +---
sound/soc/codecs/rt1011.c   | 4 +---
sound/soc/codecs/rt1016.c   | 4 +---
sound/soc/codecs/rt1305.c   | 4 +---
sound/soc/codecs/rt1308.c   | 4 +---
sound/soc/codecs/rt1318.c   | 3 +--
sound/soc/codecs/rt5616.c   | 3 +--
sound/soc/codecs/rt5659.c   | 4 +---
sound/soc/codecs/rt5660.c   | 4 +---
sound/soc/codecs/rt5665.c   | 4 +---
14 files changed, 14 insertions(+), 39 deletions(-)
[PATCH 00/14] ASoC: codecs: propagate component resume cache sync failures
Posted by Pengpeng Hou 3 weeks, 3 days ago
Commit 951af101c730 ("ASoC: report component resume callback errors")
made negative component resume results visible through the existing ASoC
component error helper while preserving best-effort resume.

These 14 component resume callbacks still discard the final
regcache_sync() result and return success. Return that result so the common
wrapper can report a failed cache replay with component context. None of
these callbacks has dependent work after the sync, so the patches do not
add rollback or retry semantics and do not change the successful path.

The issues were found by a tool we developed.
GPT-5 assisted with source-history review, patch construction and
changelog drafting.

No dynamic reproducer was produced. The series was not built or tested on
codec hardware. The 14 patches were manually reviewed, checked with
checkpatch.pl and applied in order to sound/for-next at 27a50351cbc8.

The shared error-reporting contract was discussed at:
https://lore.kernel.org/r/20260825085239.82282-1-pengpeng@iscas.ac.cn

Pengpeng Hou (14):
  ASoC: adau17x1: return cache sync errors from resume
  ASoC: ak4642: return cache sync errors from resume
  ASoC: es8316: return cache sync errors from resume
  ASoC: es8323: return cache sync errors from resume
  ASoC: nau8540: return cache sync errors from resume
  ASoC: rt1011: return cache sync errors from resume
  ASoC: rt1016: return cache sync errors from resume
  ASoC: rt1305: return cache sync errors from resume
  ASoC: rt1308: return cache sync errors from resume
  ASoC: rt1318: return cache sync errors from resume
  ASoC: rt5616: return cache sync errors from resume
  ASoC: rt5659: return cache sync errors from resume
  ASoC: rt5660: return cache sync errors from resume
  ASoC: rt5665: return cache sync errors from resume

 sound/soc/codecs/adau17x1.c | 4 +---
 sound/soc/codecs/ak4642.c   | 3 +--
 sound/soc/codecs/es8316.c   | 4 +---
 sound/soc/codecs/es8323.c   | 4 +---
 sound/soc/codecs/nau8540.c  | 4 +---
 sound/soc/codecs/rt1011.c   | 4 +---
 sound/soc/codecs/rt1016.c   | 4 +---
 sound/soc/codecs/rt1305.c   | 4 +---
 sound/soc/codecs/rt1308.c   | 4 +---
 sound/soc/codecs/rt1318.c   | 3 +--
 sound/soc/codecs/rt5616.c   | 3 +--
 sound/soc/codecs/rt5659.c   | 4 +---
 sound/soc/codecs/rt5660.c   | 4 +---
 sound/soc/codecs/rt5665.c   | 4 +---
 14 files changed, 14 insertions(+), 39 deletions(-)


base-commit: 27a50351cbc82e9f0811df417c5e7d2a72f60ef5
-- 
2.50.1 (Apple Git-155)
Re: [PATCH 00/14] ASoC: codecs: propagate component resume cache sync failures
Posted by Mark Brown 1 day, 12 hours ago
On Wed, 02 Sep 2026 09:32:36 +0800, Pengpeng Hou wrote:
> ASoC: codecs: propagate component resume cache sync failures
> 
> Commit 951af101c730 ("ASoC: report component resume callback errors")
> made negative component resume results visible through the existing ASoC
> component error helper while preserving best-effort resume.
> 
> These 14 component resume callbacks still discard the final
> regcache_sync() result and return success. Return that result so the common
> wrapper can report a failed cache replay with component context. None of
> these callbacks has dependent work after the sync, so the patches do not
> add rollback or retry semantics and do not change the successful path.
> 
> [...]

Applied to

   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-7.4

Thanks!

[01/14] ASoC: adau17x1: return cache sync errors from resume
        https://git.kernel.org/broonie/sound/c/8ca84987c11d
[02/14] ASoC: ak4642: return cache sync errors from resume
        https://git.kernel.org/broonie/sound/c/7142c34ed92f
[03/14] ASoC: es8316: return cache sync errors from resume
        https://git.kernel.org/broonie/sound/c/243d0c47e57a
[04/14] ASoC: es8323: return cache sync errors from resume
        https://git.kernel.org/broonie/sound/c/b7c3f95915a9
[05/14] ASoC: nau8540: return cache sync errors from resume
        https://git.kernel.org/broonie/sound/c/0bf2b3411537
[06/14] ASoC: rt1011: return cache sync errors from resume
        https://git.kernel.org/broonie/sound/c/432734a00bfa
[07/14] ASoC: rt1016: return cache sync errors from resume
        https://git.kernel.org/broonie/sound/c/98ace05793c7
[08/14] ASoC: rt1305: return cache sync errors from resume
        https://git.kernel.org/broonie/sound/c/30858325aa90
[09/14] ASoC: rt1308: return cache sync errors from resume
        https://git.kernel.org/broonie/sound/c/3db3289178b9
[10/14] ASoC: rt1318: return cache sync errors from resume
        https://git.kernel.org/broonie/sound/c/a5735b341c6e
[11/14] ASoC: rt5616: return cache sync errors from resume
        https://git.kernel.org/broonie/sound/c/8fc9d353d537
[12/14] ASoC: rt5659: return cache sync errors from resume
        https://git.kernel.org/broonie/sound/c/3a9b5bce85a1
[13/14] ASoC: rt5660: return cache sync errors from resume
        https://git.kernel.org/broonie/sound/c/70fe8997b030
[14/14] ASoC: rt5665: return cache sync errors from resume
        https://git.kernel.org/broonie/sound/c/9f2933d9a437

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark