Re: [PATCH i-g-t v2] lib/i915/fbc: xe3_lpd+ update the plane size checks for FBC support

"Reddy Guddati, Santhosh" <[email protected]> Tue, 28 Jul 2026 20:52:40 +0530
Newsgroups org.freedesktop.lists.igt-dev
Message-ID <[email protected]>

On 20-07-2026 20:43, Vinod Govindapillai wrote:
>  From xe3_lpd+ there is no specific restriction on the plane
> width to enable FBC. Width can be the maximum supported width
> per platform.  The restriction on the number of lines still
> holds. Update the plane size check function accordingly so that
> FBC tests dont get skipped for valid plane sizes.
> 
> v2: copy the values from the driver.
> 
> Bspec: 69560
> Signed-off-by: Vinod Govindapillai <[email protected]>

LGTM
Reviewed-by: Santhosh Reddy Guddati <[email protected]>
> ---
>   lib/i915/intel_fbc.c | 5 ++++-
>   1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/lib/i915/intel_fbc.c b/lib/i915/intel_fbc.c
> index b8c714d43..9aa7b3408 100644
> --- a/lib/i915/intel_fbc.c
> +++ b/lib/i915/intel_fbc.c
> @@ -127,7 +127,10 @@ static void intel_fbc_max_plane_size(igt_display_t *display, uint32_t *width, ui
>   	const struct intel_device_info *info = intel_get_device_info(dev_id);
>   	int ver = info->graphics_ver;
>   
> -	if (ver >= 10) {
> +	if (ver >= 30) {
> +		*width = 6144;
> +		*height = 4096;
> +	} else if (ver >= 10) {
>   		*width = 5120;
>   		*height = 4096;
>   	} else if (ver >= 8 || IS_HASWELL(fd)) {