Re: [Intel-xe] [PATCH v2 14/15] drm/xe/display: Disable PSR HW tracking by default in all display versions

[Rodrigo Vivi <rodrigo.vivi@kernel.org>]

On Tue, Apr 25, 2023 at 12:26:23PM -0700, José Roberto de Souza wrote:
> Of the platforms supported by Xe, only TGL and DG1 has PSR HW
> tracking.
> 
> Signed-off-by: José Roberto de Souza <jose.souza@intel.com>

Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>

> ---
>  drivers/gpu/drm/xe/xe_display.c | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_display.c b/drivers/gpu/drm/xe/xe_display.c
> index fbae60c179be1..376186bc02e51 100644
> --- a/drivers/gpu/drm/xe/xe_display.c
> +++ b/drivers/gpu/drm/xe/xe_display.c
> @@ -449,7 +449,7 @@ __diag_ignore_all("-Woverride-init", "Allow field overrides in table");
>  	.has_hdcp = 1,							\
>  	.has_ipc = 1,							\
>  	.has_psr = 1,							\
> -	.has_psr_hw_tracking = 1,					\
> +	.has_psr_hw_tracking = 0,					\
>  	.color = { .degamma_lut_size = 33, .gamma_lut_size = 262145 }
>  
>  #define XE_LPD								\
> @@ -472,7 +472,8 @@ __diag_ignore_all("-Woverride-init", "Allow field overrides in table");
>  	.has_fpga_dbg = 1,						\
>  	.has_hdcp = 1,							\
>  	.has_ipc = 1,							\
> -	.has_psr = 1
> +	.has_psr = 1,							\
> +	.has_psr_hw_tracking = 0
>  
>  #define XE_LPDP								\
>  	XE_LPD,								\
> @@ -492,6 +493,7 @@ void xe_display_info_init(struct xe_device *xe)
>  		xe->info.display = (struct xe_device_display_info) {
>  			GEN12_DISPLAY,
>  			.has_modular_fia = 1,
> +			.has_psr_hw_tracking = 1,
>  		};
>  		break;
>  	case XE_ROCKETLAKE:
> @@ -511,15 +513,13 @@ void xe_display_info_init(struct xe_device *xe)
>  		xe->info.display = (struct xe_device_display_info) {
>  			GEN12_DISPLAY,
>  			.has_hti = 1,
> -			.has_psr_hw_tracking = 0,
>  		};
>  		break;
>  	case XE_ALDERLAKE_P:
>  		xe->info.display = (struct xe_device_display_info) {
>  			XE_LPD,
>  			.has_cdclk_crawl = 1,
> -			.has_modular_fia = 1,
> -			.has_psr_hw_tracking = 0,
> +			.has_modular_fia = 1
>  		};
>  		break;
>  	case XE_DG2:
> -- 
> 2.40.0
>