Am 28.09.22 um 19:37 schrieb Vignesh Chander:
Change-Id: Ifd6121fb94db3fadaa1dee61d35699abe1259409
Signed-off-by: Vignesh Chander <Vignesh.Chander@xxxxxxx>
---
drivers/gpu/drm/amd/amdgpu/amdgpu_xgmi.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_xgmi.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_xgmi.c
index 47159e9a0884..80fb6ef929e5 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_xgmi.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_xgmi.c
@@ -217,7 +217,8 @@ static void amdgpu_xgmi_hive_release(struct kobject *kobj)
struct amdgpu_hive_info *hive = container_of(
kobj, struct amdgpu_hive_info, kobj);
- amdgpu_reset_put_reset_domain(hive->reset_domain);
+ if (hive->reset_domain)
+ amdgpu_reset_put_reset_domain(hive->reset_domain);
As general coding style rule _put functions are usually NULL pointer
tolerant. So this shouldn't be necessary.
If that really is necessary then please adjust the
amdgpu_reset_put_reset_domain() function instead.
Christian.
hive->reset_domain = NULL;
mutex_destroy(&hive->hive_lock);
