Commit dfd1d2f7 authored by Ville Syrjälä's avatar Ville Syrjälä Committed by Vinod Govindapillai
Browse files

drm/xe/fbdev: Fix BIOS FB vs. stolen size check 



Looks like stolen->size is in bytes, not pages. Remove the
bogus PAGE_SHIFT stuff.

Also for some random reason xe rejects the FB if it takes up
exactly half of stolen, whereas i915 allows it to be used
in that case. Adjust xe to follow the i915 rule for consistency.

v2: rebase related updates

Signed-off-by: default avatarVille Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: default avatarVinod Govindapillai <vinod.govindapillai@intel.com>
Reviewed-by: default avatarUma Shankar <uma.shankar@intel.com>
Acked-by: default avatarMika Kahola <mika.kahola@intel.com>
Link: https://patch.msgid.link/20260220170908.201422-2-vinod.govindapillai@intel.com
parent f09812b8

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

+1 −1
Original line number Diff line number Diff line
@@ -99,7 +99,7 @@ initial_plane_bo(struct xe_device *xe, 
		 * features.

		 */

		if (IS_ENABLED(CONFIG_FRAMEBUFFER_CONSOLE) &&

		    plane_config->size * 2 >> PAGE_SHIFT >= stolen->size)

		    plane_config->size * 2 > stolen->size)

			return NULL;

	}