diff mbox series

[23/42] drm/i915/xe2lpd: FBC is now supported on all pipes

Message ID 20230823170740.1180212-24-lucas.demarchi@intel.com (mailing list archive)
State New, archived
Headers show
Series Enable Lunar Lake display | expand

Commit Message

Lucas De Marchi Aug. 23, 2023, 5:07 p.m. UTC 
From: Matt Roper <matthew.d.roper@intel.com>

FBC is no longer limited by pipe.

Bspec: 68881, 68904
Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
---
 drivers/gpu/drm/i915/display/intel_fbc.h | 2 ++
 1 file changed, 2 insertions(+)

Comments

Matt Roper Aug. 23, 2023, 7:49 p.m. UTC | #1 
On Wed, Aug 23, 2023 at 10:07:21AM -0700, Lucas De Marchi wrote:
> From: Matt Roper <matthew.d.roper@intel.com>
> 
> FBC is no longer limited by pipe.

It looks like we lost the part of this patch that adds this to the
xe2_lpd_display device info structure.


Matt

> 
> Bspec: 68881, 68904
> Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
> Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
> ---
>  drivers/gpu/drm/i915/display/intel_fbc.h | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/gpu/drm/i915/display/intel_fbc.h b/drivers/gpu/drm/i915/display/intel_fbc.h
> index 4adb98afe6ff..6720ec8ee8a2 100644
> --- a/drivers/gpu/drm/i915/display/intel_fbc.h
> +++ b/drivers/gpu/drm/i915/display/intel_fbc.h
> @@ -20,6 +20,8 @@ struct intel_plane_state;
>  enum intel_fbc_id {
>  	INTEL_FBC_A,
>  	INTEL_FBC_B,
> +	INTEL_FBC_C,
> +	INTEL_FBC_D,
>  
>  	I915_MAX_FBCS,
>  };
> -- 
> 2.40.1
>
Lucas De Marchi Aug. 24, 2023, 3:53 p.m. UTC | #2 
On Wed, Aug 23, 2023 at 12:49:36PM -0700, Matt Roper wrote:
>On Wed, Aug 23, 2023 at 10:07:21AM -0700, Lucas De Marchi wrote:
>> From: Matt Roper <matthew.d.roper@intel.com>
>>
>> FBC is no longer limited by pipe.
>
>It looks like we lost the part of this patch that adds this to the
>xe2_lpd_display device info structure.

ack, but I will just move this patch first in the series so when we
introduce xe2_lpd_display we can already add it.

Lucas De Marchi

>
>
>Matt
>
>>
>> Bspec: 68881, 68904
>> Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
>> Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
>> ---
>>  drivers/gpu/drm/i915/display/intel_fbc.h | 2 ++
>>  1 file changed, 2 insertions(+)
>>
>> diff --git a/drivers/gpu/drm/i915/display/intel_fbc.h b/drivers/gpu/drm/i915/display/intel_fbc.h
>> index 4adb98afe6ff..6720ec8ee8a2 100644
>> --- a/drivers/gpu/drm/i915/display/intel_fbc.h
>> +++ b/drivers/gpu/drm/i915/display/intel_fbc.h
>> @@ -20,6 +20,8 @@ struct intel_plane_state;
>>  enum intel_fbc_id {
>>  	INTEL_FBC_A,
>>  	INTEL_FBC_B,
>> +	INTEL_FBC_C,
>> +	INTEL_FBC_D,
>>
>>  	I915_MAX_FBCS,
>>  };
>> --
>> 2.40.1
>>
>
>-- 
>Matt Roper
>Graphics Software Engineer
>Linux GPU Platform Enablement
>Intel Corporation
diff mbox series

Patch

diff --git a/drivers/gpu/drm/i915/display/intel_fbc.h b/drivers/gpu/drm/i915/display/intel_fbc.h
index 4adb98afe6ff..6720ec8ee8a2 100644
--- a/drivers/gpu/drm/i915/display/intel_fbc.h
+++ b/drivers/gpu/drm/i915/display/intel_fbc.h
@@ -20,6 +20,8 @@  struct intel_plane_state;
 enum intel_fbc_id {
 	INTEL_FBC_A,
 	INTEL_FBC_B,
+	INTEL_FBC_C,
+	INTEL_FBC_D,
 
 	I915_MAX_FBCS,
 };