linux-kernel - [PATCH] fix semicolon.cocci warnings

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Tue, 19 Mar 2019 18:34:36 +0800
From:   kbuild test robot <lkp@...el.com>
To:     Shawn Landden <shawn@....icu>
Cc:     kbuild-all@...org, linux-kernel@...r.kernel.org,
        "Gustavo A . R . Silva" <gustavo@...eddedor.com>,
        Joe Perches <joe@...ches.com>,
        Andy Whitcroft <apw@...onical.com>,
        Shawn Landden <shawn@....icu>
Subject: [PATCH] fix semicolon.cocci warnings

From: kbuild test robot <lkp@...el.com>

drivers/media/dvb-frontends/drx39xyj/drxj.c:11123:48-49: Unneeded semicolon
drivers/media/dvb-frontends/drx39xyj/drxj.c:11124:48-49: Unneeded semicolon
drivers/media/dvb-frontends/drx39xyj/drxj.c:11125:47-48: Unneeded semicolon
drivers/media/dvb-frontends/drx39xyj/drxj.c:11126:47-48: Unneeded semicolon
drivers/media/dvb-frontends/drx39xyj/drxj.c:11127:48-49: Unneeded semicolon
drivers/media/dvb-frontends/drx39xyj/drxj.c:11128:40-41: Unneeded semicolon
drivers/media/dvb-frontends/drx39xyj/drxj.c:11139:40-41: Unneeded semicolon
drivers/media/dvb-frontends/drx39xyj/drxj.c:11276:39-40: Unneeded semicolon
drivers/media/dvb-frontends/drx39xyj/drxj.c:11278:40-41: Unneeded semicolon
drivers/media/dvb-frontends/drx39xyj/drxj.c:11279:40-41: Unneeded semicolon
drivers/media/dvb-frontends/drx39xyj/drxj.c:11313:40-41: Unneeded semicolon
drivers/media/dvb-frontends/drx39xyj/drxj.c:11314:40-41: Unneeded semicolon
drivers/media/dvb-frontends/drx39xyj/drxj.c:11232:40-41: Unneeded semicolon
drivers/media/dvb-frontends/drx39xyj/drxj.c:11233:40-41: Unneeded semicolon


 Remove unneeded semicolon.

Generated by: scripts/coccinelle/misc/semicolon.cocci

Fixes: 5cf7e99ecb7d ("Use fall-through attribute rather than magic comments")
CC: Shawn Landden <shawn@....icu>
Signed-off-by: kbuild test robot <lkp@...el.com>
---

url:    https://github.com/0day-ci/linux/commits/Shawn-Landden/Use-fall-through-attribute-rather-than-magic-comments/20190318-194548

 drxj.c |   28 ++++++++++++++--------------
 1 file changed, 14 insertions(+), 14 deletions(-)

--- a/drivers/media/dvb-frontends/drx39xyj/drxj.c
+++ b/drivers/media/dvb-frontends/drx39xyj/drxj.c
@@ -11120,12 +11120,12 @@ ctrl_power_mode(struct drx_demod_instanc
 				goto rw_error;
 			}
 			break;
-		case DRX_STANDARD_PAL_SECAM_BG:	__fallthrough;;
-		case DRX_STANDARD_PAL_SECAM_DK:	__fallthrough;;
-		case DRX_STANDARD_PAL_SECAM_I:	__fallthrough;;
-		case DRX_STANDARD_PAL_SECAM_L:	__fallthrough;;
-		case DRX_STANDARD_PAL_SECAM_LP:	__fallthrough;;
-		case DRX_STANDARD_NTSC:	__fallthrough;;
+		case DRX_STANDARD_PAL_SECAM_BG:	__fallthrough;
+		case DRX_STANDARD_PAL_SECAM_DK:	__fallthrough;
+		case DRX_STANDARD_PAL_SECAM_I:	__fallthrough;
+		case DRX_STANDARD_PAL_SECAM_L:	__fallthrough;
+		case DRX_STANDARD_PAL_SECAM_LP:	__fallthrough;
+		case DRX_STANDARD_NTSC:	__fallthrough;
 		case DRX_STANDARD_FM:
 			rc = power_down_atv(demod, ext_attr->standard, true);
 			if (rc != 0) {
@@ -11136,7 +11136,7 @@ ctrl_power_mode(struct drx_demod_instanc
 		case DRX_STANDARD_UNKNOWN:
 			/* Do nothing */
 			break;
-		case DRX_STANDARD_AUTO:	__fallthrough;;
+		case DRX_STANDARD_AUTO:	__fallthrough;
 		default:
 			return -EIO;
 		}
@@ -11229,8 +11229,8 @@ ctrl_set_cfg_pre_saw(struct drx_demod_in
 		ext_attr->vsb_pre_saw_cfg = *pre_saw;
 		break;
 #ifndef DRXJ_VSB_ONLY
-	case DRX_STANDARD_ITU_A:	__fallthrough;;
-	case DRX_STANDARD_ITU_B:	__fallthrough;;
+	case DRX_STANDARD_ITU_A:	__fallthrough;
+	case DRX_STANDARD_ITU_B:	__fallthrough;
 	case DRX_STANDARD_ITU_C:
 		ext_attr->qam_pre_saw_cfg = *pre_saw;
 		break;
@@ -11273,10 +11273,10 @@ ctrl_set_cfg_afe_gain(struct drx_demod_i
 	ext_attr = (struct drxj_data *) demod->my_ext_attr;
 
 	switch (afe_gain->standard) {
-	case DRX_STANDARD_8VSB:	__fallthrough;;
+	case DRX_STANDARD_8VSB:	__fallthrough;
 #ifndef DRXJ_VSB_ONLY
-	case DRX_STANDARD_ITU_A:	__fallthrough;;
-	case DRX_STANDARD_ITU_B:	__fallthrough;;
+	case DRX_STANDARD_ITU_A:	__fallthrough;
+	case DRX_STANDARD_ITU_B:	__fallthrough;
 	case DRX_STANDARD_ITU_C:
 #endif
 		/* Do nothing */
@@ -11310,8 +11310,8 @@ ctrl_set_cfg_afe_gain(struct drx_demod_i
 		ext_attr->vsb_pga_cfg = gain * 13 + 140;
 		break;
 #ifndef DRXJ_VSB_ONLY
-	case DRX_STANDARD_ITU_A:	__fallthrough;;
-	case DRX_STANDARD_ITU_B:	__fallthrough;;
+	case DRX_STANDARD_ITU_A:	__fallthrough;
+	case DRX_STANDARD_ITU_B:	__fallthrough;
 	case DRX_STANDARD_ITU_C:
 		ext_attr->qam_pga_cfg = gain * 13 + 140;
 		break;