All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: kaixuxia@tencent.com, frankjpliu@tencent.com, kasong@tencent.com,
	sagazchen@tencent.com, kernelxing@tencent.com,
	aurelianliu@tencent.com, deshengwu@tencent.com,
	flyingpeng@tencent.com, jason.zeng@intel.com, wu.zheng@intel.com,
	yingbao.jia@intel.com, pei.p.jia@intel.com
Cc: oe-kbuild-all@lists.linux.dev
Subject: [opencloudos:linux-5.4/lts/5.4.119-20.0009.spr 232/2443] drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c:728:31: error: 'struct device' has no member named 'iommu_fwspec'
Date: Fri, 27 Sep 2024 00:05:42 +0800	[thread overview]
Message-ID: <202409262306.PD5PWMZv-lkp@intel.com> (raw)

tree:   https://gitee.com/OpenCloudOS/OpenCloudOS-Kernel.git linux-5.4/lts/5.4.119-20.0009.spr
head:   9d665359e14c559b74a94a55057e9c0fbd31a81a
commit: af26f4f021e66233136ec1ec03363a08cbef4da0 [232/2443] iommu: Move iommu_fwspec to struct dev_iommu
config: arm64-allmodconfig (https://download.01.org/0day-ci/archive/20240926/202409262306.PD5PWMZv-lkp@intel.com/config)
compiler: aarch64-linux-gcc (GCC) 14.1.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240926/202409262306.PD5PWMZv-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202409262306.PD5PWMZv-lkp@intel.com/

Note: the opencloudos/linux-5.4/lts/5.4.119-20.0009.spr HEAD 9d665359e14c559b74a94a55057e9c0fbd31a81a builds fine.
      It only hurts bisectability.

All errors (new ones prefixed by >>):

   drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c:310:5: warning: no previous prototype for 'mdp5_disable' [-Wmissing-prototypes]
     310 | int mdp5_disable(struct mdp5_kms *mdp5_kms)
         |     ^~~~~~~~~~~~
   drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c:326:5: warning: no previous prototype for 'mdp5_enable' [-Wmissing-prototypes]
     326 | int mdp5_enable(struct mdp5_kms *mdp5_kms)
         |     ^~~~~~~~~~~
   drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c: In function 'modeset_init':
   drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c:469:35: warning: variable 'hw_cfg' set but not used [-Wunused-but-set-variable]
     469 |         const struct mdp5_cfg_hw *hw_cfg;
         |                                   ^~~~~~
   drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c: In function 'mdp5_kms_init':
>> drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c:728:31: error: 'struct device' has no member named 'iommu_fwspec'
     728 |                 if (!iommu_dev->iommu_fwspec)
         |                               ^~


vim +728 drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c

e2dd9f9ffa7fff drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2015-10-26  677  
06c0dd96bfbba8 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Rob Clark           2013-11-30  678  struct msm_kms *mdp5_kms_init(struct drm_device *dev)
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  679  {
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  680  	struct msm_drm_private *priv = dev->dev_private;
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  681  	struct platform_device *pdev;
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  682  	struct mdp5_kms *mdp5_kms;
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  683  	struct mdp5_cfg *config;
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  684  	struct msm_kms *kms;
667ce33e57d0de drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Rob Clark           2016-09-28  685  	struct msm_gem_address_space *aspace;
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  686  	int irq, i, ret;
518304cbf3b245 drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c Jeffrey Hugo        2019-07-08  687  	struct device *iommu_dev;
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  688  
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  689  	/* priv->kms would have been populated by the MDP5 driver */
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  690  	kms = priv->kms;
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  691  	if (!kms)
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  692  		return NULL;
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  693  
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  694  	mdp5_kms = to_mdp5_kms(to_mdp_kms(kms));
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  695  
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  696  	mdp_kms_init(&mdp5_kms->base, &kms_funcs);
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  697  
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  698  	pdev = mdp5_kms->pdev;
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  699  
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  700  	irq = irq_of_parse_and_map(pdev->dev.of_node, 0);
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  701  	if (irq < 0) {
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  702  		ret = irq;
6a41da17e87dee drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c Mamta Shukla        2018-10-20  703  		DRM_DEV_ERROR(&pdev->dev, "failed to get irq: %d\n", ret);
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  704  		goto fail;
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  705  	}
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  706  
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  707  	kms->irq = irq;
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  708  
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  709  	config = mdp5_cfg_get_config(mdp5_kms->cfg);
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  710  
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  711  	/* make sure things are off before attaching iommu (bootloader could
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  712  	 * have left things on, in which case we'll start getting faults if
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  713  	 * we don't disable):
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  714  	 */
d68fe15b1878ac drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2017-07-28  715  	pm_runtime_get_sync(&pdev->dev);
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  716  	for (i = 0; i < MDP5_INTF_NUM_MAX; i++) {
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  717  		if (mdp5_cfg_intf_is_virtual(config->hw->intf.connect[i]) ||
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  718  		    !config->hw->intf.base[i])
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  719  			continue;
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  720  		mdp5_write(mdp5_kms, REG_MDP5_INTF_TIMING_ENGINE_EN(i), 0);
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  721  
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  722  		mdp5_write(mdp5_kms, REG_MDP5_INTF_FRAME_LINE_COUNT_EN(i), 0x3);
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  723  	}
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  724  	mdelay(16);
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  725  
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  726  	if (config->platform.iommu) {
518304cbf3b245 drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c Jeffrey Hugo        2019-07-08  727  		iommu_dev = &pdev->dev;
518304cbf3b245 drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c Jeffrey Hugo        2019-07-08 @728  		if (!iommu_dev->iommu_fwspec)
518304cbf3b245 drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c Jeffrey Hugo        2019-07-08  729  			iommu_dev = iommu_dev->parent;
518304cbf3b245 drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c Jeffrey Hugo        2019-07-08  730  
518304cbf3b245 drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c Jeffrey Hugo        2019-07-08  731  		aspace = msm_gem_address_space_create(iommu_dev,
667ce33e57d0de drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Rob Clark           2016-09-28  732  				config->platform.iommu, "mdp5");
667ce33e57d0de drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Rob Clark           2016-09-28  733  		if (IS_ERR(aspace)) {
667ce33e57d0de drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Rob Clark           2016-09-28  734  			ret = PTR_ERR(aspace);
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  735  			goto fail;
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  736  		}
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  737  
f59f62d592a055 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Rob Clark           2017-06-13  738  		kms->aspace = aspace;
667ce33e57d0de drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Rob Clark           2016-09-28  739  
667ce33e57d0de drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Rob Clark           2016-09-28  740  		ret = aspace->mmu->funcs->attach(aspace->mmu, iommu_ports,
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  741  				ARRAY_SIZE(iommu_ports));
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  742  		if (ret) {
6a41da17e87dee drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c Mamta Shukla        2018-10-20  743  			DRM_DEV_ERROR(&pdev->dev, "failed to attach iommu: %d\n",
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  744  				ret);
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  745  			goto fail;
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  746  		}
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  747  	} else {
6a41da17e87dee drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c Mamta Shukla        2018-10-20  748  		DRM_DEV_INFO(&pdev->dev,
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  749  			 "no iommu, fallback to phys contig buffers for scanout\n");
52a8988de97f5e drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Luis de Bethencourt 2018-01-17  750  		aspace = NULL;
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  751  	}
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  752  
3c352b66905b29 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Rob Clark           2017-10-20  753  	pm_runtime_put_sync(&pdev->dev);
d68fe15b1878ac drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2017-07-28  754  
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  755  	ret = modeset_init(mdp5_kms);
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  756  	if (ret) {
6a41da17e87dee drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c Mamta Shukla        2018-10-20  757  		DRM_DEV_ERROR(&pdev->dev, "modeset_init failed: %d\n", ret);
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  758  		goto fail;
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  759  	}
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  760  
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  761  	dev->mode_config.min_width = 0;
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  762  	dev->mode_config.min_height = 0;
9708ebbe1728e5 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Rob Clark           2016-11-05  763  	dev->mode_config.max_width = 0xffff;
9708ebbe1728e5 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Rob Clark           2016-11-05  764  	dev->mode_config.max_height = 0xffff;
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  765  
1bf6ad622b9be5 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Daniel Vetter       2017-05-09  766  	dev->driver->get_vblank_timestamp = drm_calc_vbltimestamp_from_scanoutpos;
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  767  	dev->driver->get_scanout_position = mdp5_get_scanoutpos;
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  768  	dev->driver->get_vblank_counter = mdp5_get_vblank_counter;
2bab52af6fe68c drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c Brian Masney        2019-05-31  769  	dev->max_vblank_count = 0; /* max_vblank_count is set on each CRTC */
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  770  	dev->vblank_disable_immediate = true;
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  771  
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  772  	return kms;
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  773  fail:
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  774  	if (kms)
392ae6e0efa5e2 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  775  		mdp5_kms_destroy(kms);
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  776  	return ERR_PTR(ret);
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  777  }
aec095ecbcc706 drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c  Archit Taneja       2016-06-14  778  

:::::: The code at line 728 was first introduced by commit
:::::: 518304cbf3b2452e1e592e06bed65baced0a530d drm/msm/mdp5: Find correct node for creating gem address space

:::::: TO: Jeffrey Hugo <jeffrey.l.hugo@gmail.com>
:::::: CC: Rob Clark <robdclark@chromium.org>

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

                 reply	other threads:[~2024-09-26 16:06 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

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=202409262306.PD5PWMZv-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=aurelianliu@tencent.com \
    --cc=deshengwu@tencent.com \
    --cc=flyingpeng@tencent.com \
    --cc=frankjpliu@tencent.com \
    --cc=jason.zeng@intel.com \
    --cc=kaixuxia@tencent.com \
    --cc=kasong@tencent.com \
    --cc=kernelxing@tencent.com \
    --cc=oe-kbuild-all@lists.linux.dev \
    --cc=pei.p.jia@intel.com \
    --cc=sagazchen@tencent.com \
    --cc=wu.zheng@intel.com \
    --cc=yingbao.jia@intel.com \
    /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.