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 40c1411..ecdeb89 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 ae2b266..8eb28a8 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 34c9387..72a271b 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 1fc2906..3e01012 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 b791688..386731e 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 a18b605..b161456 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 ddec47a..5f43783 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 7108bdb..fcc9c46 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 35c6df6..a6d8af0 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 d01ff11..82c1879 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 da0a1c6..98f60d1 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 18a3484..368d152 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 4a04253..ba177c3 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 8b74ce4..2a5f666 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 422dced..d97c6dd 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;