[PATCH v3 0/5] drm/i915/display: reduce clock_gating interface between core and display

Luca Coelho <[email protected]>
Newsgroups org.freedesktop.lists.intel-xe,org.freedesktop.lists.intel-gfx
Message-ID <[email protected]>
Hi,

As a follow up to my previous clock-gating patches, this series
reduces the interface between core and display by combining all the
platform-specific exported functions and letting the display code
decide what to do for each platform.

In v2:

  * Reworked the previous version so we have only one entry point,
    instead of an _early() and a _late() function.  It turns out that
    this is not necessary because some platforms were using only
    _early() and the others were using _late() (not both);
    
  * Removed back-dependency from the display code into the core code
    by avoiding calls to intel_clock_gating_init() (Jani);

In v3:

   * Added intel_display_clock_gating.o to xe's Makefile to prevent
     linking breakage;

Please review.

Cheers,
Luca.


Luca Coelho (5):
  drm/i915/display: consolidate Gen9 clock gating initialization
  drm/i915/display: consolidate remaining clock gating initialization
  drm/i915/display: remove unnecessary clock_gating include from
    intel_display.c
  drm/i915/display: add a way to restore only display/pch registers in
    clock_gating
  drm/i915/display: split part of intel_display_reset_finish() to a new
    function

 drivers/gpu/drm/i915/display/intel_display.c  |  1 -
 .../i915/display/intel_display_clock_gating.c | 74 +++++++++++++++----
 .../i915/display/intel_display_clock_gating.h | 17 +----
 .../drm/i915/display/intel_display_power.c    |  4 +-
 .../drm/i915/display/intel_display_reset.c    | 19 +++--
 .../drm/i915/display/intel_display_reset.h    |  1 +
 drivers/gpu/drm/i915/gt/intel_reset.c         |  5 ++
 drivers/gpu/drm/i915/intel_clock_gating.c     | 30 +++-----
 drivers/gpu/drm/xe/Makefile                   |  1 +
 9 files changed, 95 insertions(+), 57 deletions(-)

-- 
2.53.0
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.