| Message ID | 20241217172955.3027271-1-ravi.kumar.vodapalli@intel.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | drm/i915/display: Don't program DBUF_CTL tracker state service | expand |
> -----Original Message----- > From: Vodapalli, Ravi Kumar <ravi.kumar.vodapalli@intel.com> > Sent: Tuesday, December 17, 2024 11:00 PM > To: intel-gfx@lists.freedesktop.org > Cc: Vivekanandan, Balasubramani > <balasubramani.vivekanandan@intel.com>; Roper, Matthew D > <matthew.d.roper@intel.com>; De Marchi, Lucas > <lucas.demarchi@intel.com>; Sousa, Gustavo <gustavo.sousa@intel.com>; > Taylor, Clinton A <clinton.a.taylor@intel.com>; Atwood, Matthew S > <matthew.s.atwood@intel.com>; Bhadane, Dnyaneshwar > <dnyaneshwar.bhadane@intel.com>; Kalvala, Haridhar > <haridhar.kalvala@intel.com>; Chauhan, Shekhar > <shekhar.chauhan@intel.com> > 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@intel.com> > --- > 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
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) return; for_each_dbuf_slice(display, slice)
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@intel.com> --- drivers/gpu/drm/i915/display/intel_display_power.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)