drm/msm/dpu: Remove set but not used variable 'priv' in dpu_kms.c (a663a2b1) · Commits · 戴 / test

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

+0 −4

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

Admin message