Re: [Intel-gfx] [trivial PATCH] treewide: Convert switch/case fallthrough; to break;

[Matthias Brugger]

On 09/09/2020 22:06, Joe Perches wrote:

diff --git a/drivers/net/wireless/mediatek/mt7601u/dma.c 
b/drivers/net/wireless/mediatek/mt7601u/dma.c
index 09f931d4598c..778be26d329f 100644
--- a/drivers/net/wireless/mediatek/mt7601u/dma.c
+++ b/drivers/net/wireless/mediatek/mt7601u/dma.c
@@ -193,11 +193,11 @@ static void mt7601u_complete_rx(struct urb *urb)
        case -ESHUTDOWN:
        case -ENOENT:
                return;
+       case 0:
+               break;
        default:
                dev_err_ratelimited(dev->dev, "rx urb failed: %d\n",
                                    urb->status);
-               fallthrough;
-       case 0:
                break;
        }

@@ -238,11 +238,11 @@ static void mt7601u_complete_tx(struct urb *urb)

        case -ESHUTDOWN:
        case -ENOENT:
                return;
+       case 0:
+               break;
        default:
                dev_err_ratelimited(dev->dev, "tx urb failed: %d\n",
                                    urb->status);
-               fallthrough;
-       case 0:
                break;
        }

Reviewed-by: Matthias Brugger <matthias....@gmail.com>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx