Re: [PATCH v3 4/7] drm/i915/huc: differentiate the 2 steps of the MTL HuC auth flow

[kernel test robot <lkp@intel.com>]

Hi Daniele,

kernel test robot noticed the following build errors:

[auto build test ERROR on drm-tip/drm-tip]

url:    https://github.com/intel-lab-lkp/linux/commits/Daniele-Ceraolo-Spurio/drm-i915-uc-perma-pin-firmwares/20230527-085429
base:   git://anongit.freedesktop.org/drm/drm-tip drm-tip
patch link:    https://lore.kernel.org/r/20230527005242.1346093-5-daniele.ceraolospurio%40intel.com
patch subject: [PATCH v3 4/7] drm/i915/huc: differentiate the 2 steps of the MTL HuC auth flow
config: x86_64-randconfig-a001-20230526 (https://download.01.org/0day-ci/archive/20230527/202305271233.h1MUODDM-lkp@intel.com/config)
compiler: gcc-11 (Debian 11.3.0-12) 11.3.0
reproduce (this is a W=1 build):
        # https://github.com/intel-lab-lkp/linux/commit/5e7ad1c45a6cbb80d8b9f71212d410d19bbe96d9
        git remote add linux-review https://github.com/intel-lab-lkp/linux
        git fetch --no-tags linux-review Daniele-Ceraolo-Spurio/drm-i915-uc-perma-pin-firmwares/20230527-085429
        git checkout 5e7ad1c45a6cbb80d8b9f71212d410d19bbe96d9
        # save the config file
        mkdir build_dir && cp config build_dir/.config
        make W=1 O=build_dir ARCH=x86_64 olddefconfig
        make W=1 O=build_dir ARCH=x86_64 SHELL=/bin/bash drivers/gpu/drm/i915/

If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202305271233.h1MUODDM-lkp@intel.com/

All errors (new ones prefixed by >>):

   drivers/gpu/drm/i915/pxp/intel_pxp_gsccs.c: In function 'intel_pxp_gsccs_is_ready_for_sessions':
>> drivers/gpu/drm/i915/pxp/intel_pxp_gsccs.c:199:13: error: too few arguments to function 'intel_huc_is_authenticated'
     199 |         if (intel_huc_is_authenticated(&pxp->ctrl_gt->uc.huc) &&
         |             ^~~~~~~~~~~~~~~~~~~~~~~~~~
   In file included from drivers/gpu/drm/i915/gt/uc/intel_uc.h:14,
                    from drivers/gpu/drm/i915/gt/intel_gt_types.h:19,
                    from drivers/gpu/drm/i915/gt/intel_engine.h:18,
                    from drivers/gpu/drm/i915/i915_drv.h:46,
                    from drivers/gpu/drm/i915/gt/intel_context.h:14,
                    from drivers/gpu/drm/i915/pxp/intel_pxp_gsccs.c:8:
   drivers/gpu/drm/i915/gt/uc/intel_huc.h:60:6: note: declared here
      60 | bool intel_huc_is_authenticated(struct intel_huc *huc,
         |      ^~~~~~~~~~~~~~~~~~~~~~~~~~


vim +/intel_huc_is_authenticated +199 drivers/gpu/drm/i915/pxp/intel_pxp_gsccs.c

dc9ac125d81faf Alan Previn 2023-05-11  188  
99afb7cc8c4457 Alan Previn 2023-05-11  189  bool intel_pxp_gsccs_is_ready_for_sessions(struct intel_pxp *pxp)
99afb7cc8c4457 Alan Previn 2023-05-11  190  {
99afb7cc8c4457 Alan Previn 2023-05-11  191  	/*
99afb7cc8c4457 Alan Previn 2023-05-11  192  	 * GSC-fw loading, HuC-fw loading, HuC-fw authentication and
99afb7cc8c4457 Alan Previn 2023-05-11  193  	 * GSC-proxy init flow (requiring an mei component driver)
99afb7cc8c4457 Alan Previn 2023-05-11  194  	 * must all occur first before we can start requesting for PXP
99afb7cc8c4457 Alan Previn 2023-05-11  195  	 * sessions. Checking for completion on HuC authentication and
99afb7cc8c4457 Alan Previn 2023-05-11  196  	 * gsc-proxy init flow (the last set of dependencies that
99afb7cc8c4457 Alan Previn 2023-05-11  197  	 * are out of order) will suffice.
99afb7cc8c4457 Alan Previn 2023-05-11  198  	 */
99afb7cc8c4457 Alan Previn 2023-05-11 @199  	if (intel_huc_is_authenticated(&pxp->ctrl_gt->uc.huc) &&
99afb7cc8c4457 Alan Previn 2023-05-11  200  	    intel_gsc_uc_fw_proxy_init_done(&pxp->ctrl_gt->uc.gsc))
99afb7cc8c4457 Alan Previn 2023-05-11  201  		return true;
99afb7cc8c4457 Alan Previn 2023-05-11  202  
99afb7cc8c4457 Alan Previn 2023-05-11  203  	return false;
99afb7cc8c4457 Alan Previn 2023-05-11  204  }
99afb7cc8c4457 Alan Previn 2023-05-11  205  

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