All of lore.kernel.org
 help / color / mirror / Atom feed
From: kbuild test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: Re: [PATCH 2/8] drm/msm/dpu: update UBWC config for sm8150 and sm8250
Date: Fri, 29 May 2020 14:50:15 +0800	[thread overview]
Message-ID: <202005291424.eDvbVKWH%lkp@intel.com> (raw)
In-Reply-To: <20200526032235.21930-3-jonathan@marek.ca>

[-- Attachment #1: Type: text/plain, Size: 4046 bytes --]

Hi Jonathan,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on linus/master]
[also build test WARNING on v5.7-rc7 next-20200528]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]

url:    https://github.com/0day-ci/linux/commits/Jonathan-Marek/Initial-SM8150-and-SM8250-DPU-bringup/20200526-112725
base:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 9cb1fd0efd195590b828b9b865421ad345a4a145
config: arm-defconfig (attached as .config)
compiler: arm-linux-gnueabi-gcc (GCC) 9.3.0
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
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=arm 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kbuild test robot <lkp@intel.com>

All warnings (new ones prefixed by >>, old ones prefixed by <<):

drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c: In function 'dpu_encoder_virt_mode_set':
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c:958:30: warning: variable 'topology' set but not used [-Wunused-but-set-variable]
958 |  struct msm_display_topology topology;
|                              ^~~~~~~~
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c: In function '_dpu_encoder_virt_enable_helper':
>> drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c:1074:18: warning: variable 'dpu_kms' set but not used [-Wunused-but-set-variable]
1074 |  struct dpu_kms *dpu_kms;
|                  ^~~~~~~

vim +/dpu_kms +1074 drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c

25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  1069  
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  1070  static void _dpu_encoder_virt_enable_helper(struct drm_encoder *drm_enc)
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  1071  {
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  1072  	struct dpu_encoder_virt *dpu_enc = NULL;
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  1073  	struct msm_drm_private *priv;
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 @1074  	struct dpu_kms *dpu_kms;
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  1075  
422ed75581176c Drew Davenport    2019-09-06  1076  	if (!drm_enc || !drm_enc->dev) {
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  1077  		DPU_ERROR("invalid parameters\n");
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  1078  		return;
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  1079  	}
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  1080  
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  1081  	priv = drm_enc->dev->dev_private;
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  1082  	dpu_kms = to_dpu_kms(priv->kms);
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  1083  
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  1084  	dpu_enc = to_dpu_encoder_virt(drm_enc);
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  1085  	if (!dpu_enc || !dpu_enc->cur_master) {
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  1086  		DPU_ERROR("invalid dpu encoder/master\n");
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  1087  		return;
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  1088  	}
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  1089  
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  1090  	_dpu_encoder_update_vsync_source(dpu_enc, &dpu_enc->disp_info);
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  1091  }
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  1092  

:::::: The code at line 1074 was first introduced by commit
:::::: 25fdd5933e4c0f5fe2ea5cd59994f8ac5fbe90ef drm/msm: Add SDM845 DPU support

:::::: TO: Jeykumar Sankaran <jsanka@codeaurora.org>
:::::: CC: Sean Paul <seanpaul@chromium.org>

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

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 51919 bytes --]

  reply	other threads:[~2020-05-29  6:50 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-05-26  3:22 [PATCH 0/8] Initial SM8150 and SM8250 DPU bringup Jonathan Marek
2020-05-26  3:22 ` Jonathan Marek
2020-05-26  3:22 ` [PATCH 1/8] drm/msm/dpu: use right setup_blend_config for sm8150 and sm8250 Jonathan Marek
2020-05-26  3:22   ` Jonathan Marek
2020-05-26  3:22 ` [PATCH 2/8] drm/msm/dpu: update UBWC config " Jonathan Marek
2020-05-26  3:22   ` Jonathan Marek
2020-05-29  6:50   ` kbuild test robot [this message]
2020-05-26  3:22 ` [PATCH 3/8] drm/msm/dpu: move some sspp caps to dpu_caps Jonathan Marek
2020-05-26  3:22   ` Jonathan Marek
2020-05-26  3:22 ` [PATCH 4/8] drm/msm/dpu: don't use INTF_INPUT_CTRL feature on sdm845 Jonathan Marek
2020-05-26  3:22   ` Jonathan Marek
2020-05-26  3:22 ` [PATCH 5/8] drm/msm/dpu: set missing flush bits for INTF_2 and INTF_3 Jonathan Marek
2020-05-26  3:22   ` Jonathan Marek
2020-05-26  3:22 ` [PATCH 6/8] drm/msm/dpu: intf timing path for displayport Jonathan Marek
2020-05-26  3:22   ` Jonathan Marek
2020-05-26  3:22 ` [PATCH 7/8] drm/msm/dpu: add SM8150 to hw catalog Jonathan Marek
2020-05-26  3:22   ` Jonathan Marek
2020-05-27 11:09   ` Dan Carpenter
2020-05-27 11:09     ` [kbuild] " Dan Carpenter
2020-05-30 12:28   ` kbuild test robot
2020-06-11 14:37   ` Dmitry Baryshkov
2020-06-11 14:37     ` Dmitry Baryshkov
2020-06-11 14:45     ` Jonathan Marek
2020-06-11 14:45       ` Jonathan Marek
2020-05-26  3:22 ` [PATCH 8/8] drm/msm/dpu: add SM8250 " Jonathan Marek
2020-05-26  3:22   ` Jonathan Marek

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=202005291424.eDvbVKWH%lkp@intel.com \
    --to=lkp@intel.com \
    --cc=kbuild-all@lists.01.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.