drm/msm/dpu: Remove set but not used variables 'dpu_cstate', 'priv'
authorzhengbin <zhengbin13@huawei.com>
Sat, 5 Oct 2019 04:33:47 +0000 (12:33 +0800)
committerRob Clark <robdclark@chromium.org>
Mon, 7 Oct 2019 16:33:30 +0000 (09:33 -0700)
Fixes gcc '-Wunused-but-set-variable' warning:

drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.c: In function dpu_core_perf_crtc_release_bw:
drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.c:248:25: warning: variable dpu_cstate set but not used [-Wunused-but-set-variable]
drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.c: In function dpu_core_perf_crtc_update:
drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.c:337:26: warning: variable priv set but not used [-Wunused-but-set-variable]

They are not used since commit 25fdd5933e4c ("drm/msm:
Add SDM845 DPU support")

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: zhengbin <zhengbin13@huawei.com>
Signed-off-by: Rob Clark <robdclark@chromium.org>
drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.c

index e968430..11f2beb 100644 (file)
@@ -204,7 +204,6 @@ static int _dpu_core_perf_crtc_update_bus(struct dpu_kms *kms,
 void dpu_core_perf_crtc_release_bw(struct drm_crtc *crtc)
 {
        struct dpu_crtc *dpu_crtc;
-       struct dpu_crtc_state *dpu_cstate;
        struct dpu_kms *kms;
 
        if (!crtc) {
@@ -219,7 +218,6 @@ void dpu_core_perf_crtc_release_bw(struct drm_crtc *crtc)
        }
 
        dpu_crtc = to_dpu_crtc(crtc);
-       dpu_cstate = to_dpu_crtc_state(crtc->state);
 
        if (atomic_dec_return(&kms->bandwidth_ref) > 0)
                return;
@@ -276,7 +274,6 @@ int dpu_core_perf_crtc_update(struct drm_crtc *crtc,
        u64 clk_rate = 0;
        struct dpu_crtc *dpu_crtc;
        struct dpu_crtc_state *dpu_cstate;
-       struct msm_drm_private *priv;
        struct dpu_kms *kms;
        int ret;
 
@@ -290,7 +287,6 @@ int dpu_core_perf_crtc_update(struct drm_crtc *crtc,
                DPU_ERROR("invalid kms\n");
                return -EINVAL;
        }
-       priv = kms->dev->dev_private;
 
        dpu_crtc = to_dpu_crtc(crtc);
        dpu_cstate = to_dpu_crtc_state(crtc->state);