Re: [PATCH 3/5] drm/msm/adreno: Load the firmware before bringing up the hardware

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Jordan,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on robclark/msm-next]
[also build test ERROR on v4.17 next-20180608]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Jordan-Crouse/drm-msm-Add-support-for-Adreno-a6xx/20180612-022828
base:   git://people.freedesktop.org/~robclark/linux msm-next
config: arm64-defconfig (attached as .config)
compiler: aarch64-linux-gnu-gcc (Debian 7.2.0-11) 7.2.0
reproduce:
        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
        GCC_VERSION=7.2.0 make.cross ARCH=arm64 

Note: the linux-review/Jordan-Crouse/drm-msm-Add-support-for-Adreno-a6xx/20180612-022828 HEAD 716a0bd2934f1b01e622ac7a6714789861824f8f builds fine.
      It only hurts bisectibility.

All errors (new ones prefixed by >>):

   drivers/gpu/drm/msm/adreno/adreno_device.c: In function 'adreno_load_gpu':
>> drivers/gpu/drm/msm/adreno/adreno_device.c:177:8: error: implicit declaration of function 'adreno_load_fw'; did you mean 'adreno_load_gpu'? [-Werror=implicit-function-declaration]
     ret = adreno_load_fw(adreno_gpu);
           ^~~~~~~~~~~~~~
           adreno_load_gpu
   cc1: some warnings being treated as errors

vim +177 drivers/gpu/drm/msm/adreno/adreno_device.c

   152	
   153	struct msm_gpu *adreno_load_gpu(struct drm_device *dev)
   154	{
   155		struct msm_drm_private *priv = dev->dev_private;
   156		struct platform_device *pdev = priv->gpu_pdev;
   157		struct msm_gpu *gpu = NULL;
   158		struct adreno_gpu *adreno_gpu;
   159		int ret;
   160	
   161		if (pdev)
   162			gpu = platform_get_drvdata(pdev);
   163	
   164		if (!gpu) {
   165			dev_err_once(dev->dev, "no GPU device was found\n");
   166			return NULL;
   167		}
   168	
   169		adreno_gpu = to_adreno_gpu(gpu);
   170	
   171		/*
   172		 * The number one reason for HW init to fail is if the firmware isn't
   173		 * loaded yet. Try that first and don't bother continuing on
   174		 * otherwise
   175		 */
   176	
 > 177		ret = adreno_load_fw(adreno_gpu);
   178		if (ret)
   179			return NULL;
   180	
   181		/* Make sure pm runtime is active and reset any previous errors */
   182		pm_runtime_set_active(&pdev->dev);
   183	
   184		ret = pm_runtime_get_sync(&pdev->dev);
   185		if (ret < 0) {
   186			dev_err(dev->dev, "Couldn't power up the GPU: %d\n", ret);
   187			return NULL;
   188		}
   189	
   190		mutex_lock(&dev->struct_mutex);
   191		ret = msm_gpu_hw_init(gpu);
   192		mutex_unlock(&dev->struct_mutex);
   193		pm_runtime_put_autosuspend(&pdev->dev);
   194		if (ret) {
   195			dev_err(dev->dev, "gpu hw init failed: %d\n", ret);
   196			return NULL;
   197		}
   198	

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Attachment: .config.gz
Description: application/gzip


[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [Linux for Sparc]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux