Commit 64909aa6 authored by Ville Syrjälä's avatar Ville Syrjälä
Browse files

drm/i915/fbc: Remove pointless "stride is multiple of 64 bytes" check 



Plane stride is always a multiple of 64 bytes. Remove the
pointless check that really doesn't have anything to do
with FBC.

Signed-off-by: default avatarVille Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20231003194256.28569-6-ville.syrjala@linux.intel.com


Reviewed-by: default avatarVinod Govindapillai <vinod.govindapillai@intel.com>
parent 1fe5c43e

drivers/gpu/drm/i915/display/intel_fbc.c

+0 −7
Original line number Diff line number Diff line
@@ -897,13 +897,6 @@ static bool icl_fbc_stride_is_valid(const struct intel_plane_state *plane_state) 
static bool stride_is_valid(const struct intel_plane_state *plane_state)

{

	struct drm_i915_private *i915 = to_i915(plane_state->uapi.plane->dev);

	const struct drm_framebuffer *fb = plane_state->hw.fb;

	unsigned int stride = intel_fbc_plane_stride(plane_state) *

		fb->format->cpp[0];



	/* This should have been caught earlier. */

	if (drm_WARN_ON_ONCE(&i915->drm, (stride & (64 - 1)) != 0))

		return false;



	if (DISPLAY_VER(i915) >= 11)

		return icl_fbc_stride_is_valid(plane_state);