The 'break' is not necessary and reachable
because of previous 'return', and we could
discard it for better view.

Signed-off-by: Zhang Qilong <zhangqilo...@huawei.com>
---
 drivers/gpu/drm/nouveau/nvkm/subdev/bios/pll.c  | 1 -
 drivers/gpu/drm/nouveau/nvkm/subdev/clk/mcp77.c | 2 --
 2 files changed, 3 deletions(-)

diff --git a/drivers/gpu/drm/nouveau/nvkm/subdev/bios/pll.c 
b/drivers/gpu/drm/nouveau/nvkm/subdev/bios/pll.c
index 350f10a3de37..2ec84b8a3b3a 100644
--- a/drivers/gpu/drm/nouveau/nvkm/subdev/bios/pll.c
+++ b/drivers/gpu/drm/nouveau/nvkm/subdev/bios/pll.c
@@ -123,7 +123,6 @@ pll_map(struct nvkm_bios *bios)
        case NV_20:
        case NV_30:
                return nv04_pll_mapping;
-               break;
        case NV_40:
                return nv40_pll_mapping;
        case NV_50:
diff --git a/drivers/gpu/drm/nouveau/nvkm/subdev/clk/mcp77.c 
b/drivers/gpu/drm/nouveau/nvkm/subdev/clk/mcp77.c
index efa50274df97..295ee352b5c6 100644
--- a/drivers/gpu/drm/nouveau/nvkm/subdev/clk/mcp77.c
+++ b/drivers/gpu/drm/nouveau/nvkm/subdev/clk/mcp77.c
@@ -147,10 +147,8 @@ mcp77_clk_read(struct nvkm_clk *base, enum nv_clk_src src)
                switch (mast & 0x00400000) {
                case 0x00400000:
                        return nvkm_clk_read(&clk->base, nv_clk_src_core) >> P;
-                       break;
                default:
                        return 500000 >> P;
-                       break;
                }
                break;
        default:
-- 
2.17.1

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to