Commit 692a4802 authored by Thomas Hellström's avatar Thomas Hellström
Browse files

drm/xe: Fix uninitialized return values 



clang warned about two uninitialized variables used as return
values in the exhaustive eviction series.

Fix those.

Fixes: 1f154172 ("drm/xe: Rework instances of variants of xe_bo_create_locked()")
Fixes: 7bcb6e38 ("drm/xe/display: Convert __xe_pin_fb_vma()")
Cc: Matthew Brost <matthew.brost@intel.com>
Cc: Jani Nikula <jani.nikula@linux.intel.com>
Signed-off-by: default avatarThomas Hellström <thomas.hellstrom@linux.intel.com>
Reviewed-by: default avatarJonathan Cavitt <jonathan.cavitt@intel.com>
Reviewed-by: default avatarTejas Upadhyay <tejas.upadhyay@intel.com>
Link: https://lore.kernel.org/r/20250910151128.49693-1-thomas.hellstrom@linux.intel.com
parent b98775bc

drivers/gpu/drm/xe/compat-i915-headers/gem/i915_gem_stolen.h

+1 −1
Original line number Diff line number Diff line
@@ -22,7 +22,7 @@ static inline int i915_gem_stolen_insert_node_in_range(struct xe_device *xe, 
						       u32 start, u32 end)

{

	struct xe_bo *bo;

	int err;

	int err = 0;

	u32 flags = XE_BO_FLAG_PINNED | XE_BO_FLAG_STOLEN;



	if (start < SZ_4K)

drivers/gpu/drm/xe/display/xe_fb_pin.c

+1 −1
Original line number Diff line number Diff line
@@ -283,7 +283,7 @@ static struct i915_vma *__xe_pin_fb_vma(const struct intel_framebuffer *fb, 
	struct xe_bo *bo = gem_to_xe_bo(obj);

	struct xe_validation_ctx ctx;

	struct drm_exec exec;

	int ret;

	int ret = 0;



	if (!vma)

		return ERR_PTR(-ENODEV);