Hi Andre, kernel test robot noticed the following build errors:
[auto build test ERROR on drm-misc/drm-misc-next] [also build test ERROR on linus/master v7.1-rc3 next-20260508] [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#_base_tree_information] url: https://github.com/intel-lab-lkp/linux/commits/Andre-Jun-Hirata/drm-amd-pm-Use-guard-mutex-instead-of-manual-lock-unlock/20260513-062000 base: https://gitlab.freedesktop.org/drm/misc/kernel.git drm-misc-next patch link: https://lore.kernel.org/r/20260508172345.6680-1-andrejhirata%40usp.br patch subject: [PATCH v4] drm/amd/pm: Use guard(mutex) instead of manual lock+unlock config: x86_64-randconfig-072-20260513 (https://download.01.org/0day-ci/archive/20260513/[email protected]/config) compiler: gcc-12 (Debian 12.4.0-5) 12.4.0 reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260513/[email protected]/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 <[email protected]> | Closes: https://lore.kernel.org/oe-kbuild-all/[email protected]/ All error/warnings (new ones prefixed by >>): drivers/gpu/drm/amd/amdgpu/../pm/amdgpu_dpm.c: In function 'amdgpu_dpm_is_mode1_reset_supported': drivers/gpu/drm/amd/amdgpu/../pm/amdgpu_dpm.c:247:47: error: expected ';' before '}' token 247 | return smu_mode1_reset_is_support(smu) | ^ | ; 248 | } | ~ drivers/gpu/drm/amd/amdgpu/../pm/amdgpu_dpm.c: In function 'amdgpu_dpm_smu_i2c_bus_access': >> drivers/gpu/drm/amd/amdgpu/../pm/amdgpu_dpm.c:405:16: error: expected >> expression before '=' token 405 | return = pp_funcs->smu_i2c_bus_access(pp_handle, | ^ drivers/gpu/drm/amd/amdgpu/../pm/amdgpu_dpm.c: In function 'amdgpu_dpm_get_vce_clock_state': drivers/gpu/drm/amd/amdgpu/../pm/amdgpu_dpm.c:833:31: warning: unused variable 'vstate' [-Wunused-variable] 833 | struct amd_vce_state *vstate = NULL; | ^~~~~~ drivers/gpu/drm/amd/amdgpu/../pm/amdgpu_dpm.c: In function 'amdgpu_dpm_get_pp_table': drivers/gpu/drm/amd/amdgpu/../pm/amdgpu_dpm.c:1016:16: error: expected expression before '=' token 1016 | return = pp_funcs->get_pp_table(adev->powerplay.pp_handle, | ^ drivers/gpu/drm/amd/amdgpu/../pm/amdgpu_dpm.c: In function 'amdgpu_dpm_set_fine_grain_clk_vol': drivers/gpu/drm/amd/amdgpu/../pm/amdgpu_dpm.c:1031:16: error: expected expression before '=' token 1031 | return = pp_funcs->set_fine_grain_clk_vol(adev->powerplay.pp_handle, | ^ drivers/gpu/drm/amd/amdgpu/../pm/amdgpu_dpm.c: In function 'amdgpu_dpm_get_display_mode_validation_clks': drivers/gpu/drm/amd/amdgpu/../pm/amdgpu_dpm.c:1498:16: error: expected expression before '=' token 1498 | return = pp_funcs->get_display_mode_validation_clocks(adev->powerplay.pp_handle, | ^ drivers/gpu/drm/amd/amdgpu/../pm/amdgpu_dpm.c: In function 'amdgpu_dpm_smu_i2c_bus_access': >> drivers/gpu/drm/amd/amdgpu/../pm/amdgpu_dpm.c:407:1: warning: control >> reaches end of non-void function [-Wreturn-type] 407 | } | ^ drivers/gpu/drm/amd/amdgpu/../pm/amdgpu_dpm.c: In function 'amdgpu_dpm_get_pp_table': drivers/gpu/drm/amd/amdgpu/../pm/amdgpu_dpm.c:1018:1: warning: control reaches end of non-void function [-Wreturn-type] 1018 | } | ^ drivers/gpu/drm/amd/amdgpu/../pm/amdgpu_dpm.c: In function 'amdgpu_dpm_set_fine_grain_clk_vol': drivers/gpu/drm/amd/amdgpu/../pm/amdgpu_dpm.c:1035:1: warning: control reaches end of non-void function [-Wreturn-type] 1035 | } | ^ drivers/gpu/drm/amd/amdgpu/../pm/amdgpu_dpm.c: In function 'amdgpu_dpm_get_display_mode_validation_clks': drivers/gpu/drm/amd/amdgpu/../pm/amdgpu_dpm.c:1500:1: warning: control reaches end of non-void function [-Wreturn-type] 1500 | } | ^ vim +405 drivers/gpu/drm/amd/amdgpu/../pm/amdgpu_dpm.c 393 394 int amdgpu_dpm_smu_i2c_bus_access(struct amdgpu_device *adev, 395 bool acquire) 396 { 397 void *pp_handle = adev->powerplay.pp_handle; 398 const struct amd_pm_funcs *pp_funcs = 399 adev->powerplay.pp_funcs; 400 401 if (!pp_funcs || !pp_funcs->smu_i2c_bus_access) 402 return -EOPNOTSUPP; 403 404 guard(mutex)(&adev->pm.mutex); > 405 return = pp_funcs->smu_i2c_bus_access(pp_handle, 406 acquire); > 407 } 408 -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki
