This function is never called without a pagelist at the moment.

So there is no justification for this change.

Christian.

Am 18.08.2016 um 04:58 schrieb Flora Cui:
Signed-off-by: Flora Cui <flora....@amd.com>
---
  drivers/gpu/drm/amd/amdgpu/amdgpu_gart.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_gart.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_gart.c
index 0feea34..03b9fd6 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_gart.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_gart.c
@@ -287,7 +287,7 @@ int amdgpu_gart_bind(struct amdgpu_device *adev, uint64_t 
offset,
for (i = 0; i < pages; i++, p++) {
  #ifdef CONFIG_AMDGPU_GART_DEBUGFS
-               adev->gart.pages[p] = pagelist[i];
+               adev->gart.pages[p] = pagelist ? pagelist[i] : NULL;
  #endif
                if (adev->gart.ptr) {
                        page_base = dma_addr[i];


_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to