Unverified Commit 22581e7c authored Jul 29, 2019 by Gustavo A. R. Silva Committed by Mark Brown Jul 31, 2019

ASoC: ti: Mark expected switch fall-throughs



Mark switch cases where we are expecting to fall through.

This patch fixes the following warning (Building: arm):

sound/soc/ti/n810.c: In function ‘n810_ext_control’:
sound/soc/ti/n810.c:48:10: warning: this statement may fall through [-Wimplicit-fallthrough=]
   line1l = 1;
   ~~~~~~~^~~
sound/soc/ti/n810.c:49:2: note: here
  case N810_JACK_HP:
  ^~~~

sound/soc/ti/rx51.c: In function ‘rx51_ext_control’:
sound/soc/ti/rx51.c:57:6: warning: this statement may fall through [-Wimplicit-fallthrough=]
   hs = 1;
   ~~~^~~
sound/soc/ti/rx51.c:58:2: note: here
  case RX51_JACK_HP:
  ^~~~

Signed-off-by: Gustavo A. R. Silva <gustavo@embeddedor.com>
Acked-by: Jarkko Nikula <jarkko.nikula@bitmer.com>
Link: https://lore.kernel.org/r/20190729221534.GA18696@embeddedor


Signed-off-by: Mark Brown <broonie@kernel.org>

parent ebdd7be5

sound/soc/ti/n810.c

+1 −0

Original line number	Original line	Diff line number	Diff line
	@@ -46,6 +46,7 @@ static void n810_ext_control(struct snd_soc_dapm_context *dapm)
	switch (n810_jack_func) {		switch (n810_jack_func) {
	case N810_JACK_HS:		case N810_JACK_HS:
	line1l = 1;		line1l = 1;
			/* fall through */
	case N810_JACK_HP:		case N810_JACK_HP:
	hp = 1;		hp = 1;
	break;		break;

sound/soc/ti/rx51.c

+1 −0

Original line number	Original line	Diff line number	Diff line
	@@ -55,6 +55,7 @@ static void rx51_ext_control(struct snd_soc_dapm_context *dapm)
	break;		break;
	case RX51_JACK_HS:		case RX51_JACK_HS:
	hs = 1;		hs = 1;
			/* fall through */
	case RX51_JACK_HP:		case RX51_JACK_HP:
	hp = 1;		hp = 1;
	break;		break;

Admin message