RE: [PATCH] drm/i915/display: Don't program DBUF_CTL tracker state service

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




> -----Original Message-----
> From: Vodapalli, Ravi Kumar <ravi.kumar.vodapalli@xxxxxxxxx>
> Sent: Tuesday, December 17, 2024 11:00 PM
> To: intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> Cc: Vivekanandan, Balasubramani
> <balasubramani.vivekanandan@xxxxxxxxx>; Roper, Matthew D
> <matthew.d.roper@xxxxxxxxx>; De Marchi, Lucas
> <lucas.demarchi@xxxxxxxxx>; Sousa, Gustavo <gustavo.sousa@xxxxxxxxx>;
> Taylor, Clinton A <clinton.a.taylor@xxxxxxxxx>; Atwood, Matthew S
> <matthew.s.atwood@xxxxxxxxx>; Bhadane, Dnyaneshwar
> <dnyaneshwar.bhadane@xxxxxxxxx>; Kalvala, Haridhar
> <haridhar.kalvala@xxxxxxxxx>; Chauhan, Shekhar
> <shekhar.chauhan@xxxxxxxxx>
> Subject: [PATCH] drm/i915/display: Don't program DBUF_CTL tracker state
> service
> 
> While display initialization along with MBUS credits programming DBUF_CTL
> register is also programmed, as a part of it the tracker state service field is
> also set to 0x8 value when default value is other than 0x8 which are for
> platforms past display version 13.
> For remaining platforms the default value is already 0x8 so don't program
> them.
> 
> Bspec: 49213
> Signed-off-by: Ravi Kumar Vodapalli <ravi.kumar.vodapalli@xxxxxxxxx>
> ---
>  drivers/gpu/drm/i915/display/intel_display_power.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/i915/display/intel_display_power.c
> b/drivers/gpu/drm/i915/display/intel_display_power.c
> index 34465d56def0..cbcc4bddc01f 100644
> --- a/drivers/gpu/drm/i915/display/intel_display_power.c
> +++ b/drivers/gpu/drm/i915/display/intel_display_power.c
> @@ -1126,7 +1126,7 @@ static void gen12_dbuf_slices_config(struct
> intel_display *display)  {
>  	enum dbuf_slice slice;
> 
> -	if (display->platform.alderlake_p)
> +	if (DISPLAY_VER(display) >= 13)
Hi Ravi, 
This new condition is not needed here since there's already a similar check in the caller function icl_display_core_init().
Please update the condition at the caller function and remove this code from here.

Dnyaneshwar,
>  		return;
> 
>  	for_each_dbuf_slice(display, slice)
> --
> 2.25.1





[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux