> -----Original Message-----
> From: Vodapalli, Ravi Kumar <[email protected]>
> Sent: Tuesday, December 17, 2024 11:00 PM
> To: [email protected]
> Cc: Vivekanandan, Balasubramani
> <[email protected]>; Roper, Matthew D
> <[email protected]>; De Marchi, Lucas
> <[email protected]>; Sousa, Gustavo <[email protected]>;
> Taylor, Clinton A <[email protected]>; Atwood, Matthew S
> <[email protected]>; Bhadane, Dnyaneshwar
> <[email protected]>; Kalvala, Haridhar
> <[email protected]>; Chauhan, Shekhar
> <[email protected]>
> 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 <[email protected]>
> ---
> 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
