diff options
author | Gustavo A. R. Silva <gustavo@embeddedor.com> | 2019-02-13 13:54:03 -0500 |
---|---|---|
committer | Gustavo A. R. Silva <gustavo@embeddedor.com> | 2019-04-09 12:48:38 -0400 |
commit | 9fe0a75908903428ef33d05a4f4842a057b36a62 (patch) | |
tree | f63955ffdbfa02db4d778fa0194923370b82c0a4 /drivers/nfc | |
parent | e16fb3a8d27189a76037e143fec8b89f551b96bf (diff) |
NFC: pn533: mark expected switch fall-throughs
In preparation to enabling -Wimplicit-fallthrough, mark switch
cases where we are expecting to fall through.
This patch fixes the following warnings:
drivers/nfc/pn533/pn533.c: In function ‘pn533_transceive’:
drivers/nfc/pn533/pn533.c:2142:6: warning: this statement may fall through [-Wimplicit-fallthrough=]
if (dev->tgt_active_prot == NFC_PROTO_FELICA) {
^
drivers/nfc/pn533/pn533.c:2150:2: note: here
default:
^~~~~~~
drivers/nfc/pn533/pn533.c: In function ‘pn533_wq_mi_recv’:
drivers/nfc/pn533/pn533.c:2267:6: warning: this statement may fall through [-Wimplicit-fallthrough=]
if (dev->tgt_active_prot == NFC_PROTO_FELICA) {
^
drivers/nfc/pn533/pn533.c:2276:2: note: here
default:
^~~~~~~
Warning level 3 was used: -Wimplicit-fallthrough=3
This patch is part of the ongoing efforts to enable
-Wimplicit-fallthrough.
Addresses-Coverity-ID: 1230487 ("Missing break in switch")
Addresses-Coverity-ID: 1230488 ("Missing break in switch")
Reviewed-by: Kees Cook <keescook@chromium.org>
Signed-off-by: Gustavo A. R. Silva <gustavo@embeddedor.com>
Diffstat (limited to 'drivers/nfc')
-rw-r--r-- | drivers/nfc/pn533/pn533.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/nfc/pn533/pn533.c b/drivers/nfc/pn533/pn533.c index a0cc1cc45292..5961f14259e5 100644 --- a/drivers/nfc/pn533/pn533.c +++ b/drivers/nfc/pn533/pn533.c | |||
@@ -2147,6 +2147,7 @@ static int pn533_transceive(struct nfc_dev *nfc_dev, | |||
2147 | 2147 | ||
2148 | break; | 2148 | break; |
2149 | } | 2149 | } |
2150 | /* fall through */ | ||
2150 | default: | 2151 | default: |
2151 | /* jumbo frame ? */ | 2152 | /* jumbo frame ? */ |
2152 | if (skb->len > PN533_CMD_DATAEXCH_DATA_MAXLEN) { | 2153 | if (skb->len > PN533_CMD_DATAEXCH_DATA_MAXLEN) { |
@@ -2273,6 +2274,7 @@ static void pn533_wq_mi_recv(struct work_struct *work) | |||
2273 | 2274 | ||
2274 | break; | 2275 | break; |
2275 | } | 2276 | } |
2277 | /* fall through */ | ||
2276 | default: | 2278 | default: |
2277 | skb_put_u8(skb, 1); /*TG*/ | 2279 | skb_put_u8(skb, 1); /*TG*/ |
2278 | 2280 | ||