Re: [Intel-gfx] [PATCH v14 0/7] mtl: add support for pmdemand

[Jani Nikula <jani.nikula@linux.intel.com>]

On Tue, 06 Jun 2023, Vinod Govindapillai <vinod.govindapillai@intel.com> wrote:
> SAGV configuration support for MTL
>
> v2: added one missing patch in the previous version
>
> v3: chekcpatch warning fixes
>     update index handling for the icl/tgl QGV point handling
>     program pmdemand code simplified
>
> v4: update to debufs and pipe values pmdemand regiters
>     removed the macro usage in update_pmdemand_values
>
> V5: Addressing comments from Gustavo and Jani
>     And some other fixes for issues from CI
>
> v6: Addressing some comments from Gustavo
>     Updates to pmdemand state struct, active phys calculations
>     Got rid of suppress warning patch from v5
>
> v7: Rebased and updates to max ddiclk and active phys calculations
>
> v8: updates to active phys calcuations
>
> v9: Address styling issues
>
> v10: Updates to phys calculation, pmdemand state initialization during
>      HW readout / sanitization
>
> v11: Fix CI checkpatch warnings
>
> v12: Addressing comments
>
> v13: Updates based on imre's comments to handle non serialize cases,
>      updates tp phys mask during sanitize calls after HW readout
>
> v14: check display version before accessig pmdemand functions

Okay, please stop sending new versions now.

We don't want to reach v14 on any patch series. It's a burden on the
developer, the reviewers, the CI, everyone. Please adjust towards
waiting for more review comments before sending another version, and
making the most of each version sent. There are several versions of the
series that didn't receive any feedback; why send another version?

In this case, AFAICT, some of the patches in the series have Reviewed-by
but haven't been changed in the last ten versions of the series. Why
haven't they been merged already? What's the point in resending them
over and over again?


BR,
Jani.


>
> Mika Kahola (1):
>   drm/i915/mtl: Add support for PM DEMAND
>
> Vinod Govindapillai (6):
>   drm/i915: fix the derating percentage for MTL
>   drm/i915: update the QGV point frequency calculations
>   drm/i915: store the peak bw per QGV point
>   drm/i915: extract intel_bw_check_qgv_points()
>   drm/i915: modify max_bw to return index to intel_bw_info
>   drm/i915/mtl: find the best QGV point for the SAGV configuration
>
>  drivers/gpu/drm/i915/Makefile                 |   1 +
>  drivers/gpu/drm/i915/display/intel_bw.c       | 345 ++++++---
>  drivers/gpu/drm/i915/display/intel_bw.h       |   6 +
>  drivers/gpu/drm/i915/display/intel_display.c  |  14 +
>  .../gpu/drm/i915/display/intel_display_core.h |  11 +
>  .../drm/i915/display/intel_display_driver.c   |   7 +
>  .../gpu/drm/i915/display/intel_display_irq.c  |  23 +-
>  .../drm/i915/display/intel_display_power.c    |  14 +-
>  .../drm/i915/display/intel_modeset_setup.c    |  36 +
>  drivers/gpu/drm/i915/display/intel_pmdemand.c | 714 ++++++++++++++++++
>  drivers/gpu/drm/i915/display/intel_pmdemand.h |  69 ++
>  drivers/gpu/drm/i915/i915_reg.h               |  36 +-
>  12 files changed, 1153 insertions(+), 123 deletions(-)
>  create mode 100644 drivers/gpu/drm/i915/display/intel_pmdemand.c
>  create mode 100644 drivers/gpu/drm/i915/display/intel_pmdemand.h

-- 
Jani Nikula, Intel Open Source Graphics Center