]> git.baikalelectronics.ru Git - kernel.git/commit
drm/amdgpu: fix error handling in df_v3_6_pmc_start
authorArnd Bergmann <arnd@arndb.de>
Mon, 17 Jun 2019 12:51:45 +0000 (14:51 +0200)
committerAlex Deucher <alexander.deucher@amd.com>
Mon, 17 Jun 2019 16:02:03 +0000 (11:02 -0500)
commitcbc68b2646af9d551c29c48f96eaafc1b0ed0096
tree4027612589817c2279c6aa69eb96a0cd531a94a3
parent20af7c7f8c0ef85220ea3e7365c9a449a824d914
drm/amdgpu: fix error handling in df_v3_6_pmc_start

When df_v3_6_pmc_get_ctrl_settings() fails for some reason, we
store uninitialized data in a register, as gcc points out:

drivers/gpu/drm/amd/amdgpu/df_v3_6.c: In function 'df_v3_6_pmc_start':
drivers/gpu/drm/amd/amdgpu/amdgpu.h:1012:29: error: 'lo_val' may be used uninitialized in this function [-Werror=maybe-uninitialized]
 #define WREG32_PCIE(reg, v) adev->pcie_wreg(adev, (reg), (v))
                             ^~~~
drivers/gpu/drm/amd/amdgpu/df_v3_6.c:334:39: note: 'lo_val' was declared here
  uint32_t lo_base_addr, hi_base_addr, lo_val, hi_val;
                                       ^~~~~~

Make it return a proper error code that we can catch in the caller.

Fixes: b381697a4369 ("drm/amdgpu: add df perfmon regs and funcs for xgmi")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/amdgpu/df_v3_6.c