Hi Mark
Thank you for appling my patch.
It seems this is on "topic" branch.
But as I mention on this patch
> This is bugfix for v4.10.
> Can you please apply this patch to it?
is it possible to go to "fix" branch for v4.10-rcX ?
> The patch
>
> ASoC: rsnd: fixup for_each_rsnd_mod_array{s} iterator increment
>
> has been applied to the asoc tree at
>
> git://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git
>
> 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
>
> From 826e83de58e3bb243b154380837c6506a0575395 Mon Sep 17 00:00:00 2001
> From: Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx>
> Date: Fri, 20 Jan 2017 04:23:29 +0000
> Subject: [PATCH] ASoC: rsnd: fixup for_each_rsnd_mod_array{s} iterator
> increment
>
> commit 5f222a292 ("ASoC: rsnd: use for_each_rsnd_mod_xxx() ...")
> modifies rsnd_dai_call() to use for_each_rsnd_mod_arrays().
>
> Current rsnd is incrementing iterator in rsnd_mod_next(),
> but the iterator will indicate +1 position in for_each loop in
> this case. Incremental position should be inside for()
>
> Reported-by: Hoan Nguyen An <na-hoan@xxxxxxxxxxx>
> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx>
> Signed-off-by: Mark Brown <broonie@xxxxxxxxxx>
> ---
> sound/soc/sh/rcar/core.c | 2 --
> sound/soc/sh/rcar/rsnd.h | 4 ++--
> 2 files changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/sound/soc/sh/rcar/core.c b/sound/soc/sh/rcar/core.c
> index 4bd68de76130..948c5ec87980 100644
> --- a/sound/soc/sh/rcar/core.c
> +++ b/sound/soc/sh/rcar/core.c
> @@ -363,8 +363,6 @@ struct rsnd_mod *rsnd_mod_next(int *iterator,
> if (!mod)
> continue;
>
> - (*iterator)++;
> -
> return mod;
> }
>
> diff --git a/sound/soc/sh/rcar/rsnd.h b/sound/soc/sh/rcar/rsnd.h
> index b90df77662df..7410ec0174db 100644
> --- a/sound/soc/sh/rcar/rsnd.h
> +++ b/sound/soc/sh/rcar/rsnd.h
> @@ -374,10 +374,10 @@ struct rsnd_mod *rsnd_mod_next(int *iterator,
> int array_size);
> #define for_each_rsnd_mod(iterator, pos, io) \
> for (iterator = 0; \
> - (pos = rsnd_mod_next(&iterator, io, NULL, 0));)
> + (pos = rsnd_mod_next(&iterator, io, NULL, 0)); iterator++)
> #define for_each_rsnd_mod_arrays(iterator, pos, io, array, size) \
> for (iterator = 0; \
> - (pos = rsnd_mod_next(&iterator, io, array, size));)
> + (pos = rsnd_mod_next(&iterator, io, array, size)); iterator++)
> #define for_each_rsnd_mod_array(iterator, pos, io, array) \
> for_each_rsnd_mod_arrays(iterator, pos, io, array, ARRAY_SIZE(array))
>
> --
> 2.11.0
>
Best regards
---
Kuninori Morimoto
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
