summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDan Carpenter <dan.carpenter@oracle.com>2019-10-17 12:12:16 +0300
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2020-01-26 10:01:05 +0100
commitbdfb7621c4c79ec0bd86494823ae36a651a9d693 (patch)
tree8a6c25ce9491af52a34f7dd301351ef3d3535317
parentb6cb98e60d0649e0d7c59da550c9860e03483656 (diff)
downloadlinux-bdfb7621c4c79ec0bd86494823ae36a651a9d693.tar.gz
linux-bdfb7621c4c79ec0bd86494823ae36a651a9d693.tar.bz2
linux-bdfb7621c4c79ec0bd86494823ae36a651a9d693.zip
drm/amdgpu/vi: silence an uninitialized variable warning
[ Upstream commit 4ff17a1df7d550257972a838220a8af4611c8f2c ] Smatch complains that we need to initialized "*cap" otherwise it can lead to an uninitialized variable bug in the caller. This seems like a reasonable warning and it doesn't hurt to silence it at least. drivers/gpu/drm/amd/amdgpu/vi.c:767 vi_asic_reset_method() error: uninitialized symbol 'baco_reset'. Fixes: 425db2553e43 ("drm/amdgpu: expose BACO interfaces to upper level from PP") Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com> Signed-off-by: Sasha Levin <sashal@kernel.org>
-rw-r--r--drivers/gpu/drm/amd/powerplay/amd_powerplay.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/gpu/drm/amd/powerplay/amd_powerplay.c b/drivers/gpu/drm/amd/powerplay/amd_powerplay.c
index fa8ad7db2b3a..d306cc711997 100644
--- a/drivers/gpu/drm/amd/powerplay/amd_powerplay.c
+++ b/drivers/gpu/drm/amd/powerplay/amd_powerplay.c
@@ -1421,6 +1421,7 @@ static int pp_get_asic_baco_capability(void *handle, bool *cap)
{
struct pp_hwmgr *hwmgr = handle;
+ *cap = false;
if (!hwmgr)
return -EINVAL;