Skip to content

ASoC: codecs: Use guard() for mutex & spin locks - part 2

[email protected] linux-arm-kernel linux-arm-msm linux-mediatek linux-sound lkml 7 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v4 Latest posting 2/229 patches 1 replyIn review
  2. v3 posting 2/227 patches 8 repliesSuperseded
  3. v2 posting 2/227 patches 2 repliesSuperseded
Earlier versions · 4
VersionPostedPatchesRepliesStatus
v4 posting 1/224 patches2 repliesSuperseded
v3 posting 1/224 patches5 repliesSuperseded
v2 posting 1/224 patches13 repliesSuperseded
v127 patches6 repliesSuperseded

Changes in v4 from the cover letter

- Included 2 additional codec drivers for cleanup.
- Simplify regcache error handling in resume for both rt712 and rt721.
- Remove goto labels to streamline error paths and prepare for guard cleanup.

Patches in v4 posting 2/2 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/29ASoC: codecs: da7213: Use guard() for mutex lockssound/soc/codecs/da7213.c+12 -250 replies
2/29ASoC: codecs: da7219: Use guard() for mutex lockssound/soc/codecs/da7219.c+19 -430 replies
3/29ASoC: codecs: es8316: Use guard() for mutex lockssound/soc/codecs/es8316.c+13 -180 replies
4/29ASoC: codecs: es8326: Use guard() for mutex lockssound/soc/codecs/es8326.c+13 -170 replies
5/29ASoC: codecs: es9356: Use guard() for mutex lockssound/soc/codecs/es9356.c+16 -130 replies
6/29ASoC: codecs: fs210x: Use guard() for mutex lockssound/soc/codecs/fs210x.c+31 -580 replies
7/29ASoC: codecs: hdac_hdmi: Use guard() for mutex lockssound/soc/codecs/hdac_hdmi.c+57 -600 replies
8/29ASoC: codecs: hdmi-codec: Use guard() for mutex lockssound/soc/codecs/hdmi-codec.c+6 -90 replies
9/29ASoC: codecs: idt821034: Use guard() for mutex lockssound/soc/codecs/idt821034.c+57 -870 replies
10/29ASoC: codecs: lpass-macro: Use guard() for mutex lockssound/soc/codecs/lpass-macro-common.c+4 -80 replies
11/29ASoC: codecs: madera: Use guard() for mutex lockssound/soc/codecs/madera.c+13 -260 replies
12/29ASoC: codecs: max98095: Use guard() for mutex lockssound/soc/codecs/max98095.c+20 -150 replies
13/29ASoC: codecs: mt6359-accdet: Use guard() for mutex lockssound/soc/codecs/mt6359-accdet.c+4 -90 replies
14/29ASoC: codecs: pcm512x: Use guard() for mutex lockssound/soc/codecs/pcm512x.c+6 -130 replies
15/29ASoC: codecs: pcm6240: Use guard() for mutex lockssound/soc/codecs/pcm6240.c+12 -130 replies
16/29ASoC: codecs: peb2466: Use guard() for mutex lockssound/soc/codecs/peb2466.c+6 -100 replies
17/29ASoC: codecs: rt5514-spi: Use guard() for mutex lockssound/soc/codecs/rt5514-spi.c+8 -130 replies
18/29ASoC: codecs: rt5645: Use guard() for mutex lockssound/soc/codecs/rt5645.c+79 -810 replies
19/29ASoC: codecs: rt5665: Use guard() for mutex lockssound/soc/codecs/rt5665.c+7 -80 replies
20/29ASoC: codecs: rt5668: Use guard() for mutex lockssound/soc/codecs/rt5668.c+3 -70 replies
21/29ASoC: codecs: rt5677: Use guard() for mutex lockssound/soc/codecs/rt5677-spi.c sound/soc/codecs/rt5677.c+49 -620 replies
22/29ASoC: codecs: rt5682: Use guard() for mutex lockssound/soc/codecs/rt5682-sdw.c sound/soc/codecs/rt5682.c sound/soc/codecs/rt5682s.c+19 -270 replies
23/29ASoC: codecs: rt700: Use guard() for mutex lockssound/soc/codecs/rt700-sdw.c+7 -70 replies
24/29ASoC: codecs: rt711: Use guard() for mutex lockssound/soc/codecs/rt711-sdca-sdw.c sound/soc/codecs/rt711-sdca.c sound/soc/codecs/rt711-sdw.c +1 more+54 -540 replies
25/29ASoC: codecs: rt712-sdca-sdw: Simplify regcache error handling in resumesound/soc/codecs/rt712-sdca-sdw.c+9 -110 replies
26/29ASoC: codecs: rt712: Use guard() for mutex lockssound/soc/codecs/rt712-sdca-sdw.c sound/soc/codecs/rt712-sdca.c+18 -170 replies
27/29ASoC: codecs: rt721-sdca-sdw: Simplify regcache error handling in resumesound/soc/codecs/rt721-sdca-sdw.c+9 -110 replies
28/29ASoC: codecs: rt721: Use guard() for mutex lockssound/soc/codecs/rt721-sdca-sdw.c sound/soc/codecs/rt721-sdca.c+18 -160 replies
29/29ASoC: codecs: rt722: Use guard() for mutex lockssound/soc/codecs/rt722-sdca-sdw.c sound/soc/codecs/rt722-sdca.c+18 -150 replies

Reviews trailers per patch

2 of 29 patches have a Reviewed-by or Acked-by.

PatchCharles KeepaxHerve Codina
1/29
2/29
3/29
4/29
5/29
6/29
7/29
8/29
9/29
10/29
11/29R-b
12/29
13/29
14/29
15/29
16/29R-b
17/29
18/29
19/29
20/29
21/29
22/29
23/29
24/29
25/29
26/29
27/29
28/29
29/29

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v4 thread

Cover letter [PATCH v4 00/29] ASoC: codecs: Use guard() for mutex & spin locks - part 2

From: bui duc phuc <[email protected]>

Hi all,

This series converts mutex and spinlock handling in ASoC codec drivers to use the guard() and scoped_guard() helpers.

Compile-tested only.

Changes in v4:
- Included 2 additional codec drivers for cleanup.
- Simplify regcache error handling in resume for both rt712 and rt721.
- Remove goto labels to streamline error paths and prepare for guard cleanup.

Changes in v3:
- Rebased onto the latest broonie/for-next.

Changes in v2:
- Add Reviewed-by tags from Charles and Herve.
- Address Herve's review comments.

Link v3:
https://lore.kernel.org/all/[email protected]/
Link v2:
https://lore.kernel.org/all/[email protected]/
Link v1 :
https://lore.kernel.org/all/[email protected]/

Best regards,
Phuc

bui duc phuc (29):
ASoC: codecs: da7213: Use guard() for mutex locks
ASoC: codecs: da7219: Use guard() for mutex locks
ASoC: codecs: es8316: Use guard() for mutex locks
ASoC: codecs: es8326: Use guard() for mutex locks
ASoC: codecs: es9356: Use guard() for mutex locks
ASoC: codecs: fs210x: Use guard() for mutex locks
ASoC: codecs: hdac_hdmi: Use guard() for mutex locks
ASoC: codecs: hdmi-codec: Use guard() for mutex locks
ASoC: codecs: idt821034: Use guard() for mutex locks
ASoC: codecs: lpass-macro: Use guard() for mutex locks
ASoC: codecs: madera: Use guard() for mutex locks
ASoC: codecs: max98095: Use guard() for mutex locks
ASoC: codecs: mt6359-accdet: Use guard() for mutex locks
ASoC: codecs: pcm512x: Use guard() for mutex locks
ASoC: codecs: pcm6240: Use guard() for mutex locks
ASoC: codecs: peb2466: Use guard() for mutex locks
ASoC: codecs: rt5514-spi: Use guard() for mutex locks
ASoC: codecs: rt5645: Use guard() for mutex locks
ASoC: codecs: rt5665: Use guard() for mutex locks
ASoC: codecs: rt5668: Use guard() for mutex locks
ASoC: codecs: rt5677: Use guard() for mutex locks
ASoC: codecs: rt5682: Use guard() for mutex locks
ASoC: codecs: rt700: Use guard() for mutex locks
ASoC: codecs: rt711: Use guard() for mutex locks
ASoC: codecs: rt712-sdca-sdw: Simplify regcache error handling in
resume
ASoC: codecs: rt712: Use guard() for mutex locks
ASoC: codecs: rt721-sdca-sdw: Simplify regcache error handling in
resume
ASoC: codecs: rt721: Use guard() for mutex locks
ASoC: codecs: rt722: Use guard() for mutex locks

 sound/soc/codecs/da7213.c             |  37 ++----
 sound/soc/codecs/da7219.c             |  62 +++-------
 sound/soc/codecs/es8316.c             |  31 +++--
 sound/soc/codecs/es8326.c             |  30 +++--
 sound/soc/codecs/es9356.c             |  29 ++---
 sound/soc/codecs/fs210x.c             |  89 +++++---------
 sound/soc/codecs/hdac_hdmi.c          | 117 +++++++++----------
 sound/soc/codecs/hdmi-codec.c         |  15 +--
 sound/soc/codecs/idt821034.c          | 144 +++++++++--------------
 sound/soc/codecs/lpass-macro-common.c |  12 +-
 sound/soc/codecs/madera.c             |  39 +++----
 sound/soc/codecs/max98095.c           |  35 +++---
 sound/soc/codecs/mt6359-accdet.c      |  13 +--
 sound/soc/codecs/pcm512x.c            |  19 +--
 sound/soc/codecs/pcm6240.c            |  25 ++--
 sound/soc/codecs/peb2466.c            |  16 +--
 sound/soc/codecs/rt5514-spi.c         |  21 ++--
 sound/soc/codecs/rt5645.c             | 160 +++++++++++++-------------
 sound/soc/codecs/rt5665.c             |  15 ++-
 sound/soc/codecs/rt5668.c             |  10 +-
 sound/soc/codecs/rt5677-spi.c         |  36 +++---
 sound/soc/codecs/rt5677.c             |  75 ++++++------
 sound/soc/codecs/rt5682-sdw.c         |  24 ++--
 sound/soc/codecs/rt5682.c             |   5 +-
 sound/soc/codecs/rt5682s.c            |  17 +--
 sound/soc/codecs/rt700-sdw.c          |  14 +--
 sound/soc/codecs/rt711-sdca-sdw.c     |  29 ++---
 sound/soc/codecs/rt711-sdca.c         |   5 +-
 sound/soc/codecs/rt711-sdw.c          |  24 ++--
 sound/soc/codecs/rt711.c              |  50 ++++----
 sound/soc/codecs/rt712-sdca-sdw.c     |  50 ++++----
 sound/soc/codecs/rt712-sdca.c         |   5 +-
 sound/soc/codecs/rt721-sdca-sdw.c     |  49 ++++----
 sound/soc/codecs/rt721-sdca.c         |   5 +-
 sound/soc/codecs/rt722-sdca-sdw.c     |  29 ++---
 sound/soc/codecs/rt722-sdca.c         |   4 +-
 36 files changed, 587 insertions(+), 753 deletions(-)
Signature · 1 line
2.43.0