Re: [PATCH 00/10] ASoC: qcom: remove cppcheck warnings

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Thanks Pierre for taking time to fix these warning!

On 18/02/2021 22:29, Pierre-Louis Bossart wrote:
Second batch of cleanups for Qualcomm SOCs and codecs. The only
functional change is the addition of a missing error check in the last
patch.

Pierre-Louis Bossart (10):
   ASoC: qcom: lpass-hdmi: remove useless return
   ASoC: qcom: lpass-platform: remove useless assignment
   ASoC: qcom: q6dsp-dai: clarify expression
   ASoC: qcom: q6afe: remove useless assignments
   ASoC: qcom: q6afe: align function prototype
   ASoC: qcom: q6asm: align function prototypes
   ASoC: wcd-clsh-v2: align function prototypes
   ASoC: wcd9335: clarify return value
   ASoC: wcd934x: remove useless return
   ASoC: lpass-wsa-macro: add missing test

  sound/soc/codecs/lpass-wsa-macro.c | 2 ++
  sound/soc/codecs/wcd-clsh-v2.h     | 6 +++---
  sound/soc/codecs/wcd9335.c         | 2 +-
  sound/soc/codecs/wcd934x.c         | 2 --
  sound/soc/qcom/lpass-hdmi.c        | 4 ----
  sound/soc/qcom/lpass-platform.c    | 2 +-
  sound/soc/qcom/qdsp6/q6afe-dai.c   | 2 +-
  sound/soc/qcom/qdsp6/q6afe.c       | 5 ++---
  sound/soc/qcom/qdsp6/q6afe.h       | 2 +-
  sound/soc/qcom/qdsp6/q6asm.h       | 6 +++---
  10 files changed, 14 insertions(+), 19 deletions(-)


All for these changes LGTM,

Reviewed-by: Srinivas Kandagatla <srinivas.kandagatla@xxxxxxxxxx>


--srini




[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux