Skip to content

Commit 3bece98

Browse files
author
Andi Shyti
committed
drm/i915/gem: Return NULL instead of '0'
Commit 05da7d9 ("drm/i915/gem: Downgrade stolen lmem setup warning") returns '0' from i915_gem_stolen_lmem_setup(), but it's supposed to return a pointer to the intel_memory_region structure. Sparse complains with the following message: >> drivers/gpu/drm/i915/gem/i915_gem_stolen.c:943:32: sparse: sparse: Using plain integer as NULL pointer Return NULL. Signed-off-by: Andi Shyti <[email protected]> Cc: Jonathan Cavitt <[email protected]> Reviewed-by: Jonathan Cavitt <[email protected]> Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
1 parent 24bb052 commit 3bece98

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

drivers/gpu/drm/i915/gem/i915_gem_stolen.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -940,7 +940,7 @@ i915_gem_stolen_lmem_setup(struct drm_i915_private *i915, u16 type,
940940
drm_dbg(&i915->drm,
941941
"Disabling stolen memory support due to OOB placement: lmem_size = %lli vs dsm_base = %lli\n",
942942
lmem_size, dsm_base);
943-
return 0;
943+
return NULL;
944944
}
945945
dsm_size = ALIGN_DOWN(lmem_size - dsm_base, SZ_1M);
946946
}

0 commit comments

Comments
 (0)