Hi Lucas, Thank you for the patch! Yet something to improve:
[auto build test ERROR on drm-tip/drm-tip] [also build test ERROR on next-20220125] [cannot apply to drm-intel/for-linux-next drm-exynos/exynos-drm-next drm/drm-next tegra-drm/drm/tegra/for-next linus/master airlied/drm-next v5.17-rc1] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to use '--base' as documented in https://git-scm.com/docs/git-format-patch] url: https://github.com/0day-ci/linux/commits/Lucas-De-Marchi/drm-i915-guc-Refactor-ADS-access-to-use-dma_buf_map/20220127-043912 base: git://anongit.freedesktop.org/drm/drm-tip drm-tip config: i386-allyesconfig (https://download.01.org/0day-ci/archive/20220127/202201271208.kelpe3mn-...@intel.com/config) compiler: gcc-9 (Debian 9.3.0-22) 9.3.0 reproduce (this is a W=1 build): # https://github.com/0day-ci/linux/commit/313757d9ed833acea4ee2bb0e3f3565d6efcf3cc git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Lucas-De-Marchi/drm-i915-guc-Refactor-ADS-access-to-use-dma_buf_map/20220127-043912 git checkout 313757d9ed833acea4ee2bb0e3f3565d6efcf3cc # save the config file to linux build tree mkdir build_dir make W=1 O=build_dir ARCH=i386 SHELL=/bin/bash If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <l...@intel.com> All errors (new ones prefixed by >>): In file included from include/drm/drm_mm.h:51, from drivers/gpu/drm/i915/i915_vma.h:31, from drivers/gpu/drm/i915/gt/uc/intel_uc_fw.h:13, from drivers/gpu/drm/i915/gt/uc/intel_guc.h:20, from drivers/gpu/drm/i915/gt/uc/intel_uc.h:9, from drivers/gpu/drm/i915/gt/intel_gt_types.h:18, from drivers/gpu/drm/i915/gt/intel_gt.h:10, from drivers/gpu/drm/i915/gt/uc/intel_guc_ads.c:9: drivers/gpu/drm/i915/gt/uc/intel_guc_ads.c: In function 'guc_mmio_reg_state_create': >> drivers/gpu/drm/i915/gt/uc/intel_guc_ads.c:369:38: error: format '%lu' >> expects argument of type 'long unsigned int', but argument 4 has type 'u32' >> {aka 'unsigned int'} [-Werror=format=] 369 | drm_dbg(&guc_to_gt(guc)->i915->drm, "Used %lu KB for temporary ADS regset\n", | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 370 | (temp_set.storage_max * sizeof(struct guc_mmio_reg)) >> 10); | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ | | | u32 {aka unsigned int} include/drm/drm_print.h:461:56: note: in definition of macro 'drm_dbg' 461 | drm_dev_dbg((drm) ? (drm)->dev : NULL, DRM_UT_DRIVER, fmt, ##__VA_ARGS__) | ^~~ drivers/gpu/drm/i915/gt/uc/intel_guc_ads.c:369:46: note: format string is defined here 369 | drm_dbg(&guc_to_gt(guc)->i915->drm, "Used %lu KB for temporary ADS regset\n", | ~~^ | | | long unsigned int | %u cc1: all warnings being treated as errors vim +369 drivers/gpu/drm/i915/gt/uc/intel_guc_ads.c 348 349 static long guc_mmio_reg_state_create(struct intel_guc *guc) 350 { 351 struct intel_gt *gt = guc_to_gt(guc); 352 struct intel_engine_cs *engine; 353 enum intel_engine_id id; 354 struct temp_regset temp_set = {}; 355 long total = 0; 356 357 for_each_engine(engine, gt, id) { 358 u32 used = temp_set.storage_used; 359 360 if (guc_mmio_regset_init(&temp_set, engine) < 0) 361 return -1; 362 363 guc->ads_regset_count[id] = temp_set.storage_used - used; 364 total += guc->ads_regset_count[id]; 365 } 366 367 guc->ads_regset = temp_set.storage; 368 > 369 drm_dbg(&guc_to_gt(guc)->i915->drm, "Used %lu KB for temporary > ADS regset\n", 370 (temp_set.storage_max * sizeof(struct guc_mmio_reg)) >> 10); 371 372 return total * sizeof(struct guc_mmio_reg); 373 } 374 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-...@lists.01.org
