[lvc-project] [PATCH 0/3] ASoC: make some DAPM functions *void* instead of *int*

Sergey Shtylyov s.shtylyov at omp.ru
Tue Apr 15 22:06:42 MSK 2025


First, we had 2 complaints from Svace (static analysis tool) about the
*if* statement conditions inside the loops in dapm_update_dai_unlocked()
always being false -- because dapm_update_dai_chan() (called from there)
only ever returned 0. After removing those *if* statements, in its turn,
dapm_update_dai_unlocked() also started to only return 0, so there was
no sense anymore to propagate its result from snd_soc_dapm_update_dai();
after removing now useless local variable from that function, there was
no sense anymore to return an *int* result (and the function's callers
do ignore its result anyway)...

The patches are against the for-next branch of Mark Brown's sound.git repo.

Sergey Shtylyov (3):
  ASoC: dapm: make dapm_update_dai_chan() *void*
  ASoC: dapm: make dapm_update_dai_unlocked() *void*
  ASoC: dapm: make snd_soc_dapm_update_dai() *void*

 include/sound/soc-dapm.h |  4 ++--
 sound/soc/soc-dapm.c     | 46 ++++++++++++++--------------------------
 2 files changed, 18 insertions(+), 32 deletions(-)

-- 
2.49.0



More information about the lvc-project mailing list