From add12477e9a20a36e91622f01bdc72dbd3e73245 Mon Sep 17 00:00:00 2001
From: Hin-Tak Leung <htl10@users.sourceforge.net>
Date: Mon, 27 Apr 2020 01:04:16 +0100
Subject: [PATCH] use the kernel's way of suppressing fallthrough

See include/linux/compiler_attributes.h:

/*
 * Add the pseudo keyword 'fallthrough' so case statement blocks
 * must end with any of these keywords:
 *   break;
 *   fallthrough;
 *   goto <label>;
 *   return [expression];
 *
 *  gcc: https://gcc.gnu.org/onlinedocs/gcc/Statement-Attributes.html#Statement-Attributes
 */
-if __has_attribute(__fallthrough__)
- define fallthrough                    __attribute__((__fallthrough__))
-else
- define fallthrough                    do {} while (0)  /* fallthrough */
-endif
---
 ac108.c  | 2 +-
 wm8960.c | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/ac108.c b/ac108.c
index d202dc3..8c09143 100644
--- a/ac108.c
+++ b/ac108.c
@@ -865,7 +865,7 @@ static int ac108_set_fmt(struct snd_soc_dai *dai, unsigned int fmt) {
 			/* TODO: Both cpu_dai and codec_dai(AC108) be set as slave in DTS */
 			dev_dbg(dai->dev, "used as slave when AC101 is master\n");
 		}
-		/* fall through */
+		fallthrough;
 	case SND_SOC_DAIFMT_CBS_CFS:    /*AC108 Slave*/
 		dev_dbg(dai->dev, "AC108 set to work as Slave\n");
 		/**
diff --git a/wm8960.c b/wm8960.c
index c2975e1..34d4dad 100644
--- a/wm8960.c
+++ b/wm8960.c
@@ -752,8 +752,8 @@ static int wm8960_hw_params(struct snd_pcm_substream *substream,
 		if ((iface & 0x3) != 0) {
 			iface |= 0x000c;
 			break;
-		};
-		/* fall through */
+		}
+		fallthrough;
 	default:
 		dev_err(codec->dev, "unsupported width %d\n",
 			params_width(params));