On Mon, Feb 03, 2025 at 11:23:23AM +0200, Jani Nikula wrote:
> On Mon, 03 Feb 2025, Mohammed Thasleem <mohammed.thasleem@xxxxxxxxx> wrote:
> > Starting from MTl we don't have a platform agnostic way to validate DC6 state
> > due to dc6 counter has been removed to validate DC state.
> > Adding dc6_entry_counter at display dirver to validate dc6 state.
> >
> > Signed-off-by: Mohammed Thasleem <mohammed.thasleem@xxxxxxxxx>
> > ---
> > drivers/gpu/drm/i915/display/intel_display_core.h | 1 +
> > drivers/gpu/drm/i915/display/intel_display_power_well.c | 2 ++
> > drivers/gpu/drm/i915/display/intel_dmc.c | 1 +
> > 3 files changed, 4 insertions(+)
> >
> > diff --git a/drivers/gpu/drm/i915/display/intel_display_core.h b/drivers/gpu/drm/i915/display/intel_display_core.h
> > index 554870d2494b..cc244617011f 100644
> > --- a/drivers/gpu/drm/i915/display/intel_display_core.h
> > +++ b/drivers/gpu/drm/i915/display/intel_display_core.h
> > @@ -376,6 +376,7 @@ struct intel_display {
> > struct {
> > struct intel_dmc *dmc;
> > intel_wakeref_t wakeref;
> > + u32 dc6_entry_counter;
> > } dmc;
> >
> > struct {
> > diff --git a/drivers/gpu/drm/i915/display/intel_display_power_well.c b/drivers/gpu/drm/i915/display/intel_display_power_well.c
> > index f45a4f9ba23c..0eb178aa618d 100644
> > --- a/drivers/gpu/drm/i915/display/intel_display_power_well.c
> > +++ b/drivers/gpu/drm/i915/display/intel_display_power_well.c
> > @@ -869,6 +869,8 @@ void skl_enable_dc6(struct intel_display *display)
> > intel_dmc_wl_enable(display, DC_STATE_EN_UPTO_DC6);
> >
> > gen9_set_dc_state(display, DC_STATE_EN_UPTO_DC6);
> > +
> > + display->dmc.dc6_entry_counter++;
>
> This file has no business touching the guts of display->dmc.
my bad... I might have suggested that because here we are covering
something up for DMC :)
But yet, maybe better another place of i915_display.
>
> BR,
> Jani.
>
>
> > }
> >
> > void bxt_enable_dc9(struct intel_display *display)
> > diff --git a/drivers/gpu/drm/i915/display/intel_dmc.c b/drivers/gpu/drm/i915/display/intel_dmc.c
> > index 221d3abda791..f51bd8e6011d 100644
> > --- a/drivers/gpu/drm/i915/display/intel_dmc.c
> > +++ b/drivers/gpu/drm/i915/display/intel_dmc.c
> > @@ -1293,6 +1293,7 @@ static int intel_dmc_debugfs_status_show(struct seq_file *m, void *unused)
> > if (i915_mmio_reg_valid(dc6_reg))
> > seq_printf(m, "DC5 -> DC6 count: %d\n",
> > intel_de_read(display, dc6_reg));
> > + seq_printf(m, "DC6 entry count: %d\n", display->dmc.dc6_entry_counter);
> >
> > seq_printf(m, "program base: 0x%08x\n",
> > intel_de_read(display, DMC_PROGRAM(dmc->dmc_info[DMC_FW_MAIN].start_mmioaddr, 0)));
>
> --
> Jani Nikula, Intel
