drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c:1148:26: error: variable 'priv' set but not used

From: kernel test robot
Date: Sun Feb 13 2022 - 15:21:12 EST


tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: c5d714aa6dce262482992cccc0ab01383294f7ac
commit: 167dac97eb46c7b8a15b2195080e191bb0c9ce35 drm/msm/dp: Modify prototype of encoder based API
date: 4 months ago
config: arm-randconfig-r006-20220213 (https://download.01.org/0day-ci/archive/20220214/202202140432.v7pvcbKc-lkp@xxxxxxxxx/config)
compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project 478c237e21b2c3a83e46f26fcbeb3876682f9b14)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install arm cross compiling tool for clang build
# apt-get install binutils-arm-linux-gnueabi
# https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=167dac97eb46c7b8a15b2195080e191bb0c9ce35
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout 167dac97eb46c7b8a15b2195080e191bb0c9ce35
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=arm SHELL=/bin/bash drivers/gpu/drm/msm/

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@xxxxxxxxx>

All errors (new ones prefixed by >>):

>> drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c:1148:26: error: variable 'priv' set but not used [-Werror,-Wunused-but-set-variable]
struct msm_drm_private *priv;
^
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c:1197:26: error: variable 'priv' set but not used [-Werror,-Wunused-but-set-variable]
struct msm_drm_private *priv;
^
2 errors generated.


vim +/priv +1148 drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c

25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 1143
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 1144 static void dpu_encoder_virt_enable(struct drm_encoder *drm_enc)
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 1145 {
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 1146 struct dpu_encoder_virt *dpu_enc = NULL;
86b89080368b46 Jeykumar Sankaran 2018-09-05 1147 int ret = 0;
c943b4948b5848 Chandan Uddaraju 2020-08-27 @1148 struct msm_drm_private *priv;
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 1149 struct drm_display_mode *cur_mode = NULL;
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 1150
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 1151 if (!drm_enc) {
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 1152 DPU_ERROR("invalid encoder\n");
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 1153 return;
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 1154 }
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 1155 dpu_enc = to_dpu_encoder_virt(drm_enc);
fba7427eb59496 Sean Paul 2018-11-16 1156
fba7427eb59496 Sean Paul 2018-11-16 1157 mutex_lock(&dpu_enc->enc_lock);
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 1158 cur_mode = &dpu_enc->base.crtc->state->adjusted_mode;
c943b4948b5848 Chandan Uddaraju 2020-08-27 1159 priv = drm_enc->dev->dev_private;
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 1160
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 1161 trace_dpu_enc_enable(DRMID(drm_enc), cur_mode->hdisplay,
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 1162 cur_mode->vdisplay);
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 1163
86b89080368b46 Jeykumar Sankaran 2018-09-05 1164 /* always enable slave encoder before master */
86b89080368b46 Jeykumar Sankaran 2018-09-05 1165 if (dpu_enc->cur_slave && dpu_enc->cur_slave->ops.enable)
86b89080368b46 Jeykumar Sankaran 2018-09-05 1166 dpu_enc->cur_slave->ops.enable(dpu_enc->cur_slave);
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 1167
86b89080368b46 Jeykumar Sankaran 2018-09-05 1168 if (dpu_enc->cur_master && dpu_enc->cur_master->ops.enable)
86b89080368b46 Jeykumar Sankaran 2018-09-05 1169 dpu_enc->cur_master->ops.enable(dpu_enc->cur_master);
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 1170
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 1171 ret = dpu_encoder_resource_control(drm_enc, DPU_ENC_RC_EVENT_KICKOFF);
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 1172 if (ret) {
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 1173 DPU_ERROR_ENC(dpu_enc, "dpu resource control failed: %d\n",
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 1174 ret);
fba7427eb59496 Sean Paul 2018-11-16 1175 goto out;
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 1176 }
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 1177
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 1178 _dpu_encoder_virt_enable_helper(drm_enc);
fba7427eb59496 Sean Paul 2018-11-16 1179
167dac97eb46c7 Bjorn Andersson 2021-10-16 1180 if (drm_enc->encoder_type == DRM_MODE_ENCODER_TMDS) {
167dac97eb46c7 Bjorn Andersson 2021-10-16 1181 ret = msm_dp_display_enable(dpu_enc->dp, drm_enc);
c943b4948b5848 Chandan Uddaraju 2020-08-27 1182 if (ret) {
c943b4948b5848 Chandan Uddaraju 2020-08-27 1183 DPU_ERROR_ENC(dpu_enc, "dp display enable failed: %d\n",
c943b4948b5848 Chandan Uddaraju 2020-08-27 1184 ret);
c943b4948b5848 Chandan Uddaraju 2020-08-27 1185 goto out;
c943b4948b5848 Chandan Uddaraju 2020-08-27 1186 }
c943b4948b5848 Chandan Uddaraju 2020-08-27 1187 }
fba7427eb59496 Sean Paul 2018-11-16 1188 dpu_enc->enabled = true;
fba7427eb59496 Sean Paul 2018-11-16 1189
fba7427eb59496 Sean Paul 2018-11-16 1190 out:
fba7427eb59496 Sean Paul 2018-11-16 1191 mutex_unlock(&dpu_enc->enc_lock);
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 1192 }
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 1193

:::::: The code at line 1148 was first introduced by commit
:::::: c943b4948b5848fc0e07f875edbd35a973879e22 drm/msm/dp: add displayPort driver support

:::::: TO: Chandan Uddaraju <chandanu@xxxxxxxxxxxxxx>
:::::: CC: Rob Clark <robdclark@xxxxxxxxxxxx>

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@xxxxxxxxxxxx