In function mgag200_set_pci_regs, there are some switch cases
returned, then break. These break will never run.
This patch is to make the code a bit readable.

Signed-off-by: Bernard Zhao <bern...@vivo.com>
---
 drivers/gpu/drm/mgag200/mgag200_mode.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/mgag200/mgag200_mode.c 
b/drivers/gpu/drm/mgag200/mgag200_mode.c
index 38672f9e5c4f..de873a5d276e 100644
--- a/drivers/gpu/drm/mgag200/mgag200_mode.c
+++ b/drivers/gpu/drm/mgag200/mgag200_mode.c
@@ -794,20 +794,17 @@ static int mgag200_crtc_set_plls(struct mga_device *mdev, 
long clock)
        case G200_SE_A:
        case G200_SE_B:
                return mga_g200se_set_plls(mdev, clock);
-               break;
        case G200_WB:
        case G200_EW3:
                return mga_g200wb_set_plls(mdev, clock);
-               break;
        case G200_EV:
                return mga_g200ev_set_plls(mdev, clock);
-               break;
        case G200_EH:
        case G200_EH3:
                return mga_g200eh_set_plls(mdev, clock);
-               break;
        case G200_ER:
                return mga_g200er_set_plls(mdev, clock);
+       default:
                break;
        }
 
-- 
2.28.0

Reply via email to