drm/msm/dpu: Remove set but not used variables 'dpu_cstate', 'priv' (f09662c1) · Commits · 戴 / test

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 ("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

+0 −4

Original line number	Diff line number	Diff line
		@@ -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);

Admin message