isdn: mark expected switch fall-throughs
authorGustavo A. R. Silva <gustavo@embeddedor.com>
Tue, 3 Jul 2018 21:17:31 +0000 (16:17 -0500)
committerDavid S. Miller <davem@davemloft.net>
Wed, 4 Jul 2018 13:17:32 +0000 (22:17 +0900)
In preparation to enabling -Wimplicit-fallthrough, mark switch cases
where we are expecting to fall through.

Warning level 2 was used: -Wimplicit-fallthrough=2

Signed-off-by: Gustavo A. R. Silva <gustavo@embeddedor.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
15 files changed:
drivers/isdn/gigaset/bas-gigaset.c
drivers/isdn/hardware/mISDN/avmfritz.c
drivers/isdn/hardware/mISDN/hfcpci.c
drivers/isdn/hardware/mISDN/mISDNinfineon.c
drivers/isdn/hardware/mISDN/mISDNisar.c
drivers/isdn/hisax/avm_pci.c
drivers/isdn/hisax/callc.c
drivers/isdn/hisax/config.c
drivers/isdn/hisax/gazel.c
drivers/isdn/hisax/isar.c
drivers/isdn/hisax/l3_1tr6.c
drivers/isdn/hisax/l3dss1.c
drivers/isdn/hysdn/hysdn_boot.c
drivers/isdn/i4l/isdn_v110.c
drivers/isdn/mISDN/stack.c

index 40c141163f0f98e39478ffddf8a1371b4c41dd16..ecdeb89645d00454381ef4020986fdf5e38f2eec 100644 (file)
@@ -739,6 +739,7 @@ static void read_int_callback(struct urb *urb)
 
        case HD_OPEN_B2CHANNEL_ACK:
                ++channel;
+               /* fall through */
        case HD_OPEN_B1CHANNEL_ACK:
                bcs = cs->bcs + channel;
                update_basstate(ucs, BS_B1OPEN << channel, 0);
@@ -752,6 +753,7 @@ static void read_int_callback(struct urb *urb)
 
        case HD_CLOSE_B2CHANNEL_ACK:
                ++channel;
+               /* fall through */
        case HD_CLOSE_B1CHANNEL_ACK:
                bcs = cs->bcs + channel;
                update_basstate(ucs, 0, BS_B1OPEN << channel);
@@ -765,6 +767,7 @@ static void read_int_callback(struct urb *urb)
 
        case HD_B2_FLOW_CONTROL:
                ++channel;
+               /* fall through */
        case HD_B1_FLOW_CONTROL:
                bcs = cs->bcs + channel;
                atomic_add((l - BAS_NORMFRAME) * BAS_CORRFRAMES,
index ae2b2669af1bc44667e49e60148c9591b7fa4e85..8eb28a83832eddc11f5fab235ad29f0252a89268 100644 (file)
@@ -361,6 +361,7 @@ modehdlc(struct bchannel *bch, int protocol)
        switch (protocol) {
        case -1: /* used for init */
                bch->state = -1;
+               /* fall through */
        case ISDN_P_NONE:
                if (bch->state == ISDN_P_NONE)
                        break;
index 34c93874af23bc43565119eab4b8b9b5d84a0815..72a271b988730d071f4a4c28bb0b0f2418391954 100644 (file)
@@ -1296,6 +1296,7 @@ mode_hfcpci(struct bchannel *bch, int bc, int protocol)
        case (-1): /* used for init */
                bch->state = -1;
                bch->nr = bc;
+               /* fall through */
        case (ISDN_P_NONE):
                if (bch->state == ISDN_P_NONE)
                        return 0;
index 1fc290659e945a5ffdf2efcda8cf687c4bedd42d..3e01012be4abc3a8abe51d727ad1fe70b83badd7 100644 (file)
@@ -887,6 +887,7 @@ release_card(struct inf_hw *card) {
                                release_card(card->sc[i]);
                        card->sc[i] = NULL;
                }
+               /* fall through */
        default:
                pci_disable_device(card->pdev);
                pci_set_drvdata(card->pdev, NULL);
index b791688d0228ccb8921b57775bf6330adb8a96ea..386731ec248912f327afe1e327306274add1876e 100644 (file)
@@ -972,6 +972,7 @@ isar_pump_statev_fax(struct isar_ch *ch, u8 devt) {
                                break;
                        case PCTRL_CMD_FTM:
                                p1 = 2;
+                               /* fall through */
                        case PCTRL_CMD_FTH:
                                send_mbox(ch->is, dps | ISAR_HIS_PUMPCTRL,
                                          PCTRL_CMD_SILON, 1, &p1);
@@ -1177,6 +1178,7 @@ setup_pump(struct isar_ch *ch) {
                        send_mbox(ch->is, dps | ISAR_HIS_PUMPCFG,
                                  PMOD_DTMF, 1, param);
                }
+               /* fall through */
        case ISDN_P_B_MODEM_ASYNC:
                ctrl = PMOD_DATAMODEM;
                if (test_bit(FLG_ORIGIN, &ch->bch.Flags)) {
@@ -1268,6 +1270,7 @@ setup_iom2(struct isar_ch *ch) {
        case ISDN_P_B_MODEM_ASYNC:
        case ISDN_P_B_T30_FAX:
                cmsb |= IOM_CTRL_RCV;
+               /* fall through */
        case ISDN_P_B_L2DTMF:
                if (test_bit(FLG_DTMFSEND, &ch->bch.Flags))
                        cmsb |= IOM_CTRL_RCV;
@@ -1560,6 +1563,7 @@ isar_l2l1(struct mISDNchannel *ch, struct sk_buff *skb)
                                ich->is->name, hh->id);
                        ret = -EINVAL;
                }
+               /* fall through */
        default:
                pr_info("%s: %s unknown prim(%x,%x)\n",
                        ich->is->name, __func__, hh->prim, hh->id);
index a18b605fb4f23e97d863b6bc64db924ad58053eb..b161456c942e2ea94796bfceb4df71f8400a1bf1 100644 (file)
@@ -207,6 +207,7 @@ modehdlc(struct BCState *bcs, int mode, int bc)
                bcs->mode = 1;
                bcs->channel = bc;
                bc = 0;
+               /* fall through */
        case (L1_MODE_NULL):
                if (bcs->mode == L1_MODE_NULL)
                        return;
index ddec47a911a0df6ea74c1a9f5e926fbbacc54a2a..5f43783039d4617dcf5e328435192a3beed235fa 100644 (file)
@@ -1369,6 +1369,7 @@ leased_l1l2(struct PStack *st, int pr, void *arg)
        case (PH_ACTIVATE | INDICATION):
        case (PH_ACTIVATE | CONFIRM):
                event = EV_LEASED;
+               /* fall through */
        case (PH_DEACTIVATE | INDICATION):
        case (PH_DEACTIVATE | CONFIRM):
                if (test_bit(FLG_TWO_DCHAN, &chanp->cs->HW_Flags))
index 7108bdb8742e7a41b5aac61ada26ad212211fe67..fcc9c46127b40da1bf55ff346a69190bbc4a9fea 100644 (file)
@@ -1843,6 +1843,7 @@ static void hisax_b_l2l1(struct PStack *st, int pr, void *arg)
        case PH_DEACTIVATE | REQUEST:
                test_and_clear_bit(BC_FLG_BUSY, &bcs->Flag);
                skb_queue_purge(&bcs->squeue);
+               /* fall through */
        default:
                B_L2L1(b_if, pr, arg);
                break;
index 35c6df6534ecde8507c417a61a1c93f41fa4a293..a6d8af02354a4eb7240a0393f65226f424f0c223 100644 (file)
@@ -108,6 +108,7 @@ ReadISAC(struct IsdnCardState *cs, u_char offset)
        switch (cs->subtyp) {
        case R647:
                off2 = ((off2 << 8 & 0xf000) | (off2 & 0xf));
+               /* fall through */
        case R685:
                return (readreg(cs->hw.gazel.isac, off2));
        case R753:
@@ -125,6 +126,7 @@ WriteISAC(struct IsdnCardState *cs, u_char offset, u_char value)
        switch (cs->subtyp) {
        case R647:
                off2 = ((off2 << 8 & 0xf000) | (off2 & 0xf));
+               /* fall through */
        case R685:
                writereg(cs->hw.gazel.isac, off2, value);
                break;
@@ -203,6 +205,7 @@ ReadHSCX(struct IsdnCardState *cs, int hscx, u_char offset)
        switch (cs->subtyp) {
        case R647:
                off2 = ((off2 << 8 & 0xf000) | (off2 & 0xf));
+               /* fall through */
        case R685:
                return (readreg(cs->hw.gazel.hscx[hscx], off2));
        case R753:
@@ -220,6 +223,7 @@ WriteHSCX(struct IsdnCardState *cs, int hscx, u_char offset, u_char value)
        switch (cs->subtyp) {
        case R647:
                off2 = ((off2 << 8 & 0xf000) | (off2 & 0xf));
+               /* fall through */
        case R685:
                writereg(cs->hw.gazel.hscx[hscx], off2, value);
                break;
index d01ff116797b937d02f04ce17abc8fe8802cd528..82c1879f56647e7bc39b789587c78dafa6367cfa 100644 (file)
@@ -1089,6 +1089,7 @@ isar_pump_statev_fax(struct BCState *bcs, u_char devt) {
                                break;
                        case PCTRL_CMD_FTM:
                                p1 = 2;
+                               /* fall through */
                        case PCTRL_CMD_FTH:
                                sendmsg(cs, dps | ISAR_HIS_PUMPCTRL,
                                        PCTRL_CMD_SILON, 1, &p1);
@@ -1097,6 +1098,7 @@ isar_pump_statev_fax(struct BCState *bcs, u_char devt) {
                        case PCTRL_CMD_FRM:
                                if (frm_extra_delay)
                                        mdelay(frm_extra_delay);
+                               /* fall through */
                        case PCTRL_CMD_FRH:
                                p1 = bcs->hw.isar.mod = bcs->hw.isar.newmod;
                                bcs->hw.isar.newmod = 0;
index da0a1c6aa32996e6cab700b19654b819cfa0afa4..98f60d1523f4ec06fb761014c3ebdcd592608cfe 100644 (file)
@@ -88,6 +88,7 @@ l3_1tr6_setup_req(struct l3_process *pc, u_char pr, void *arg)
                        break;
                case 'C':
                        channel = 0x08;
+                       /* fall through */
                case 'P':
                        channel |= 0x80;
                        teln++;
index 18a3484b1f7e96f33439875ede43704f91feaab8..368d152a8f1d7d00b6414f9963a7cad727beb77e 100644 (file)
@@ -1282,6 +1282,7 @@ l3dss1_setup_req(struct l3_process *pc, u_char pr,
                        switch (0x5f & *teln) {
                        case 'C':
                                channel = 0x08;
+                               /* fall through */
                        case 'P':
                                channel |= 0x80;
                                teln++;
index 4a0425378f37864779135048c097ec1e73cc165a..ba177c3a621b1371e0c50ded7a3d0967c4309039 100644 (file)
@@ -99,6 +99,7 @@ pof_handle_data(hysdn_card *card, int datlen)
 
        case TAG_CBOOTDTA:
                DecryptBuf(boot, datlen);       /* we need to encrypt the buffer */
+               /* fall through */
        case TAG_BOOTDTA:
                if (card->debug_flags & LOG_POF_RECORD)
                        hysdn_addlog(card, "POF got %s len=%d offs=0x%lx",
@@ -137,6 +138,7 @@ pof_handle_data(hysdn_card *card, int datlen)
 
        case TAG_CABSDATA:
                DecryptBuf(boot, datlen);       /* we need to encrypt the buffer */
+               /* fall through */
        case TAG_ABSDATA:
                if (card->debug_flags & LOG_POF_RECORD)
                        hysdn_addlog(card, "POF got %s len=%d offs=0x%lx",
index 8b74ce412524827c14333e4444e275ac4fdcc723..2a5f6668756cd6ea79cc8e98e18e7b606fecd2e7 100644 (file)
@@ -354,6 +354,7 @@ EncodeMatrix(unsigned char *buf, int len, unsigned char *m, int mlen)
                                printk(KERN_WARNING "isdn_v110 (EncodeMatrix): buffer full!\n");
                                return line;
                        }
+                       /* else: fall through */
                case 128:
                        m[line] = 128;  /* leftmost -> set byte to 1000000 */
                        mbit = 64;      /* current bit in the matrix line */
@@ -386,20 +387,28 @@ EncodeMatrix(unsigned char *buf, int len, unsigned char *m, int mlen)
                switch (++line % 10) {
                case 1:
                        m[line++] = 0xfe;
+                       /* fall through */
                case 2:
                        m[line++] = 0xfe;
+                       /* fall through */
                case 3:
                        m[line++] = 0xfe;
+                       /* fall through */
                case 4:
                        m[line++] = 0xfe;
+                       /* fall through */
                case 5:
                        m[line++] = 0xbf;
+                       /* fall through */
                case 6:
                        m[line++] = 0xfe;
+                       /* fall through */
                case 7:
                        m[line++] = 0xfe;
+                       /* fall through */
                case 8:
                        m[line++] = 0xfe;
+                       /* fall through */
                case 9:
                        m[line++] = 0xfe;
                }
index 422dced7c90ac26dcf0d366fedb32ab9edf44207..d97c6dd52223c9519e2af28e150cbc9f2a1faa0c 100644 (file)
@@ -539,6 +539,7 @@ create_l2entity(struct mISDNdevice *dev, struct mISDNchannel *ch,
                rq.protocol = ISDN_P_NT_S0;
                if (dev->Dprotocols & (1 << ISDN_P_NT_E1))
                        rq.protocol = ISDN_P_NT_E1;
+               /* fall through */
        case ISDN_P_LAPD_TE:
                ch->recv = mISDN_queue_message;
                ch->peer = &dev->D.st->own;