drm/gma500: remove set but not used variable 'is_hdmi','is_crt'
authorzhengbin <zhengbin13@huawei.com>
Fri, 15 Nov 2019 14:27:07 +0000 (22:27 +0800)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Fri, 15 Nov 2019 15:50:24 +0000 (16:50 +0100)
Fixes gcc '-Wunused-but-set-variable' warning:

drivers/gpu/drm/gma500/cdv_intel_display.c: In function cdv_intel_crtc_mode_set:
drivers/gpu/drm/gma500/cdv_intel_display.c:594:7: warning: variable is_hdmi set but not used [-Wunused-but-set-variable]
drivers/gpu/drm/gma500/cdv_intel_display.c: In function cdv_intel_crtc_mode_set:
drivers/gpu/drm/gma500/cdv_intel_display.c:593:7: warning: variable is_crt set but not used [-Wunused-but-set-variable]

They are not used since commit acd7ef927e06 ("gma500:
Update the Cedarview clock handling")

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: zhengbin <zhengbin13@huawei.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: https://patchwork.freedesktop.org/patch/msgid/1573828027-122323-4-git-send-email-zhengbin13@huawei.com
drivers/gpu/drm/gma500/cdv_intel_display.c

index 7109d3d19be00c15457809a1f2405c7de1ccd982..1ed854f498b7a2eba99f5e99c1f600fb17497d05 100644 (file)
@@ -582,8 +582,8 @@ static int cdv_intel_crtc_mode_set(struct drm_crtc *crtc,
        struct gma_clock_t clock;
        u32 dpll = 0, dspcntr, pipeconf;
        bool ok;
-       bool is_crt = false, is_lvds = false, is_tv = false;
-       bool is_hdmi = false, is_dp = false;
+       bool is_lvds = false, is_tv = false;
+       bool is_dp = false;
        struct drm_mode_config *mode_config = &dev->mode_config;
        struct drm_connector *connector;
        const struct gma_limit_t *limit;
@@ -607,10 +607,7 @@ static int cdv_intel_crtc_mode_set(struct drm_crtc *crtc,
                        is_tv = true;
                        break;
                case INTEL_OUTPUT_ANALOG:
-                       is_crt = true;
-                       break;
                case INTEL_OUTPUT_HDMI:
-                       is_hdmi = true;
                        break;
                case INTEL_OUTPUT_DISPLAYPORT:
                        is_dp = true;