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

drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c: In function _dpu_danger_signal_status:
drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c:80:26: warning: variable priv set but not used [-Wunused-but-set-variable]
drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c: In function dpu_kms_prepare_commit:
drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c:271:26: warning: variable priv set but not used [-Wunused-but-set-variable]

It is 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_kms.c

index 144bbbe..b1645ad 100644 (file)
@@ -64,7 +64,6 @@ static int _dpu_danger_signal_status(struct seq_file *s,
                bool danger_status)
 {
        struct dpu_kms *kms = (struct dpu_kms *)s->private;
-       struct msm_drm_private *priv;
        struct dpu_danger_safe_status status;
        int i;
 
@@ -73,7 +72,6 @@ static int _dpu_danger_signal_status(struct seq_file *s,
                return 0;
        }
 
-       priv = kms->dev->dev_private;
        memset(&status, 0, sizeof(struct dpu_danger_safe_status));
 
        pm_runtime_get_sync(&kms->pdev->dev);
@@ -270,7 +268,6 @@ static void dpu_kms_prepare_commit(struct msm_kms *kms,
                struct drm_atomic_state *state)
 {
        struct dpu_kms *dpu_kms;
-       struct msm_drm_private *priv;
        struct drm_device *dev;
        struct drm_crtc *crtc;
        struct drm_crtc_state *crtc_state;
@@ -281,7 +278,6 @@ static void dpu_kms_prepare_commit(struct msm_kms *kms,
                return;
        dpu_kms = to_dpu_kms(kms);
        dev = dpu_kms->dev;
-       priv = dev->dev_private;
 
        /* Call prepare_commit for all affected encoders */
        for_each_new_crtc_in_state(state, crtc, crtc_state, i) {