> -----Original Message----- > From: Intel-gfx <intel-gfx-bounces@xxxxxxxxxxxxxxxxxxxxx> On Behalf Of Jani > Nikula > Sent: Thursday, November 9, 2023 9:37 PM > To: intel-gfx@xxxxxxxxxxxxxxxxxxxxx > Cc: Nikula, Jani <jani.nikula@xxxxxxxxx> > Subject: [PATCH 1/2] drm/i915: abstract plane protection check > > Centralize the conditions in a function. > > Signed-off-by: Jani Nikula <jani.nikula@xxxxxxxxx> LGTM. Reviewed-by: Suraj Kandpal <suraj.kandpal@xxxxxxxxx> > --- > .../gpu/drm/i915/display/skl_universal_plane.c | 18 ++++++++++++++---- > 1 file changed, 14 insertions(+), 4 deletions(-) > > diff --git a/drivers/gpu/drm/i915/display/skl_universal_plane.c > b/drivers/gpu/drm/i915/display/skl_universal_plane.c > index 68035675ae3c..98acf25a5ca3 100644 > --- a/drivers/gpu/drm/i915/display/skl_universal_plane.c > +++ b/drivers/gpu/drm/i915/display/skl_universal_plane.c > @@ -1866,6 +1866,19 @@ static bool pxp_is_borked(struct > drm_i915_gem_object *obj) > return i915_gem_object_is_protected(obj) && > !bo_has_valid_encryption(obj); } > > +static void check_protection(struct intel_plane_state *plane_state) { > + struct intel_plane *plane = to_intel_plane(plane_state->uapi.plane); > + struct drm_i915_private *i915 = to_i915(plane->base.dev); > + const struct drm_framebuffer *fb = plane_state->hw.fb; > + > + if (DISPLAY_VER(i915) < 11) > + return; > + > + plane_state->decrypt = bo_has_valid_encryption(intel_fb_obj(fb)); > + plane_state->force_black = pxp_is_borked(intel_fb_obj(fb)); } > + > static int skl_plane_check(struct intel_crtc_state *crtc_state, > struct intel_plane_state *plane_state) { @@ - > 1910,10 +1923,7 @@ static int skl_plane_check(struct intel_crtc_state > *crtc_state, > if (ret) > return ret; > > - if (DISPLAY_VER(dev_priv) >= 11) { > - plane_state->decrypt = > bo_has_valid_encryption(intel_fb_obj(fb)); > - plane_state->force_black = pxp_is_borked(intel_fb_obj(fb)); > - } > + check_protection(plane_state); > > /* HW only has 8 bits pixel precision, disable plane if invisible */ > if (!(plane_state->hw.alpha >> 8)) > -- > 2.39.2