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.Cui at 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];