> -----Original Message----- > From: Intel-xe <[email protected]> On Behalf Of Vinod > Govindapillai > Sent: Wednesday, 16 September 2026 22.56 > To: [email protected]; [email protected] > Cc: Govindapillai, Vinod <[email protected]> > Subject: [RESEND PATCH] drm/i915/fbc: remove uint16 from supported fbc > formats in xe3plpd > > As UINT16 pixel formats are not listed as supported formats in any > of the display versions so far, there is no point is having these > formats listed as supported for FBC. So remove the UINT16 formats > from the supported formats for FBC. > > v2: removed an unused variable >
LGTM, Reviewed-by: Mika Kahola <[email protected]> > Signed-off-by: Vinod Govindapillai <[email protected]> > --- > drivers/gpu/drm/i915/display/intel_fbc.c | 12 +----------- > 1 file changed, 1 insertion(+), 11 deletions(-) > > diff --git a/drivers/gpu/drm/i915/display/intel_fbc.c > b/drivers/gpu/drm/i915/display/intel_fbc.c > index f61b4a218d6e..5694d20773f8 100644 > --- a/drivers/gpu/drm/i915/display/intel_fbc.c > +++ b/drivers/gpu/drm/i915/display/intel_fbc.c > @@ -1196,23 +1196,13 @@ xe3p_lpd_fbc_fp16_format_is_valid(const struct > intel_plane_state *plane_state) > > static bool xe3p_lpd_fbc_pixel_format_is_valid(const struct > intel_plane_state *plane_state) > { > - const struct drm_framebuffer *fb = plane_state->hw.fb; > - > if (lnl_fbc_pixel_format_is_valid(plane_state)) > return true; > > if (xe3p_lpd_fbc_fp16_format_is_valid(plane_state)) > return true; > > - switch (fb->format->format) { > - case DRM_FORMAT_XRGB16161616: > - case DRM_FORMAT_XBGR16161616: > - case DRM_FORMAT_ARGB16161616: > - case DRM_FORMAT_ABGR16161616: > - return true; > - default: > - return false; > - } > + return false; > } > > bool intel_fbc_need_pixel_normalizer(const struct intel_plane_state > *plane_state) > -- > 2.43.0
