From: Colin Ian King colin.king@canonical.com

The current null check is checking the wrong pointer atif. Fix this to check the correct pointer atcs.

Addresses-Coverity: ("Uninitialized pointer read") Fixes: c1c4d8efddde ("drm/amdgpu/acpi: unify ATCS handling (v2)") Signed-off-by: Colin Ian King colin.king@canonical.com --- drivers/gpu/drm/amd/amdgpu/amdgpu_acpi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_acpi.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_acpi.c index c39f447df21d..b3ee7fb72b81 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_acpi.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_acpi.c @@ -901,7 +901,7 @@ int amdgpu_acpi_init(struct amdgpu_device *adev) goto out;

atcs = kzalloc(sizeof(*atcs), GFP_KERNEL); - if (!atif) { + if (!atcs) { DRM_WARN("Not enough memory to initialize ATCS\n"); goto out; }