[0/8] ASoC: sh: remove cppcheck warnings

Message

Pierre-Louis Bossart Feb. 19, 2021, 11:16 p.m. UTC

No functional changes except for patch 4 where a missing error check
was added for consistency.

Pierre-Louis Bossart (8):
  ASoC: sh: dma-sh7760: remove unused variable
  ASoC: sh: rcar: align function prototypes
  ASoC: sh: rcar: simplify return
  ASoC: sh: rcar: core: rename shadowing variables
  ASoC: sh: rcar: ctu: add  missing error check
  ASoC: sh: rcar: ssi: remove redundant assignment
  ASoC: sh: siu_pcm: remove useless assignment
  ASoC: sh: siu_pcm: remove unused variable

 sound/soc/sh/dma-sh7760.c |  1 -
 sound/soc/sh/rcar/core.c  | 14 +++++++-------
 sound/soc/sh/rcar/ctu.c   |  2 ++
 sound/soc/sh/rcar/rsnd.h  |  6 +++---
 sound/soc/sh/rcar/ssi.c   |  1 -
 sound/soc/sh/siu_pcm.c    |  6 +-----
 6 files changed, 13 insertions(+), 17 deletions(-)

Comments

Kuninori Morimoto Feb. 21, 2021, 11:32 p.m. UTC | #1

Hi Pierre-Louis

Thank you for your patches.

> No functional changes except for patch 4 where a missing error check
> was added for consistency.
> 
> Pierre-Louis Bossart (8):
>   ASoC: sh: dma-sh7760: remove unused variable
>   ASoC: sh: rcar: align function prototypes
>   ASoC: sh: rcar: simplify return
>   ASoC: sh: rcar: core: rename shadowing variables
>   ASoC: sh: rcar: ctu: add  missing error check
>   ASoC: sh: rcar: ssi: remove redundant assignment
>   ASoC: sh: siu_pcm: remove useless assignment
>   ASoC: sh: siu_pcm: remove unused variable

Except [4/8] patch,

	Acked-by: Kuninori Morimoto <[e-mail]>

Thank you for your help !!

Best regards
---
Kuninori Morimoto

Mark Brown March 1, 2021, 11:34 p.m. UTC | #2

On Fri, 19 Feb 2021 17:16:27 -0600, Pierre-Louis Bossart wrote:
> No functional changes except for patch 4 where a missing error check
> was added for consistency.
> 
> Pierre-Louis Bossart (8):
>   ASoC: sh: dma-sh7760: remove unused variable
>   ASoC: sh: rcar: align function prototypes
>   ASoC: sh: rcar: simplify return
>   ASoC: sh: rcar: core: rename shadowing variables
>   ASoC: sh: rcar: ctu: add  missing error check
>   ASoC: sh: rcar: ssi: remove redundant assignment
>   ASoC: sh: siu_pcm: remove useless assignment
>   ASoC: sh: siu_pcm: remove unused variable
> 
> [...]

Applied to

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

Thanks!

[1/8] ASoC: sh: dma-sh7760: remove unused variable
      commit: 2497d156c297992ef1254fda2277ce055f48db64
[2/8] ASoC: sh: rcar: align function prototypes
      commit: 8644cbf296de7440f8d53d333da89f79d3258d25
[3/8] ASoC: sh: rcar: simplify return
      commit: d15737e24a962941bae0373b8a89173e2451e7c9
[5/8] ASoC: sh: rcar: ctu: add missing error check
      commit: a029ad994d24c92228b4460cc374846170720a05
[6/8] ASoC: sh: rcar: ssi: remove redundant assignment
      commit: 0711cb41ce02ae8994e4f75551545b114e8922d6
[7/8] ASoC: sh: siu_pcm: remove useless assignment
      commit: 075bfe61c88a267755bf4cc54af19cab3141f23f
[8/8] ASoC: sh: siu_pcm: remove unused variable
      commit: 69011a3193be54b04b3cfd652c484e6b77cf59b0

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