Skip to content

Commit

Permalink
drm/msm: Fix wrong pointer check in a5xx_destroy
Browse files Browse the repository at this point in the history
Instead of checking for a5xx_gpu->gpmu_iova during destroy we
accidently check a5xx_gpu->gpmu_bo.

Signed-off-by: Jordan Crouse <[email protected]>
Signed-off-by: Rob Clark <[email protected]>
  • Loading branch information
Jordan Crouse authored and robclark committed Apr 3, 2017
1 parent 280489d commit f456d34
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions drivers/gpu/drm/msm/adreno/a5xx_gpu.c
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
/* Copyright (c) 2016 The Linux Foundation. All rights reserved.
/* Copyright (c) 2016-2017 The Linux Foundation. All rights reserved.
*
* This program is free software; you can redistribute it and/or modify
* it under the terms of the GNU General Public License version 2 and
Expand Down Expand Up @@ -534,7 +534,7 @@ static void a5xx_destroy(struct msm_gpu *gpu)
}

if (a5xx_gpu->gpmu_bo) {
if (a5xx_gpu->gpmu_bo)
if (a5xx_gpu->gpmu_iova)
msm_gem_put_iova(a5xx_gpu->gpmu_bo, gpu->id);
drm_gem_object_unreference_unlocked(a5xx_gpu->gpmu_bo);
}
Expand Down

0 comments on commit f456d34

Please sign in to comment.