net/mac8390: discard unnecessary breaks
authorZhang Qilong <zhangqilong3@huawei.com>
Tue, 27 Oct 2020 13:51:59 +0000 (21:51 +0800)
committerJakub Kicinski <kuba@kernel.org>
Fri, 30 Oct 2020 02:03:46 +0000 (19:03 -0700)
The 'break' is unnecessary because of previous 'return',
and we could discard it.

Signed-off-by: Zhang Qilong <zhangqilong3@huawei.com>
Link: https://lore.kernel.org/r/20201027135159.71444-1-zhangqilong3@huawei.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/8390/mac8390.c

index d60a86a..9aac711 100644 (file)
@@ -175,7 +175,6 @@ static enum mac8390_type mac8390_ident(struct nubus_rsrc *fres)
                default:
                        return MAC8390_APPLE;
                }
-               break;
 
        case NUBUS_DRSW_APPLE:
                switch (fres->dr_hw) {
@@ -186,11 +185,9 @@ static enum mac8390_type mac8390_ident(struct nubus_rsrc *fres)
                default:
                        return MAC8390_APPLE;
                }
-               break;
 
        case NUBUS_DRSW_ASANTE:
                return MAC8390_ASANTE;
-               break;
 
        case NUBUS_DRSW_TECHWORKS:
        case NUBUS_DRSW_DAYNA2:
@@ -199,11 +196,9 @@ static enum mac8390_type mac8390_ident(struct nubus_rsrc *fres)
                        return MAC8390_CABLETRON;
                else
                        return MAC8390_APPLE;
-               break;
 
        case NUBUS_DRSW_FARALLON:
                return MAC8390_FARALLON;
-               break;
 
        case NUBUS_DRSW_KINETICS:
                switch (fres->dr_hw) {
@@ -212,7 +207,6 @@ static enum mac8390_type mac8390_ident(struct nubus_rsrc *fres)
                default:
                        return MAC8390_KINETICS;
                }
-               break;
 
        case NUBUS_DRSW_DAYNA:
                /*
@@ -224,7 +218,6 @@ static enum mac8390_type mac8390_ident(struct nubus_rsrc *fres)
                        return MAC8390_NONE;
                else
                        return MAC8390_DAYNA;
-               break;
        }
        return MAC8390_NONE;
 }