[Intel-gfx] [PATCH v2 00/26] drm/i915: Refactor the display power domain mappings

[Imre Deak]

On Thu, Feb 17, 2022 at 01:59:26PM +0200, Hogander, Jouni wrote:
> Reviewed-by: Jouni Högander <jouni.hogander at intel.com>

Thanks.

> for all the patches except. I'm expecting you send patch 09/26 again?

Yes, I decided to resend patches 1-8 in a separate patchset and
resend the remaining ones after the first half is merged.

> On Tue, 2022-02-08 at 13:36 +0200, Imre Deak wrote:
> > This is v2 of [1] addressing the review comments from Jani, moving
> > the
> > power well ops/desc struct accessor functions and implementation of
> > the
> > platform specific power well hooks to a separate file.
> >
> > [1] https://patchwork.freedesktop.org/series/99476/
> >
> > Cc: Jani Nikula <jani.nikula at intel.com>
> >
> > Imre Deak (26):
> >   drm/i915: Fix the VDSC_PW2 power domain enum value
> >   drm/i915: Sanitize open-coded power well enable()/disable() calls
> >   drm/i915: Remove redundant state verification during TypeC AUX
> > power
> >     well disabling
> >   drm/i915: Move i915_power_well_regs struct into i915_power_well_ops
> >   drm/i915: Move power well get/put/enable/disable functions to a new
> >     file
> >   drm/i915: Add function to call a power well's sync_hw() hook
> >   drm/i915: Add functions to get a power well's
> >     state/name/domains/mask/refcount
> >   drm/i915: Move intel_display_power_well_is_enabled() to
> >     intel_display_power_well.c
> >   drm/i915: Move per-platform power well hooks to
> >     intel_display_power_well.c
> >   drm/i915: Unexport the for_each_power_well() macros
> >   drm/i915: Move the power domain->well mappings to
> >     intel_display_power_map.c
> >   drm/i915: Move the dg2 fixed_enable_delay power well param to a
> > common
> >     bitfield
> >   drm/i915: Move the HSW power well flags to a common bitfield
> >   drm/i915: Rename the power domain names to end with pipes/ports
> >   drm/i915: Sanitize the power well names
> >   drm/i915: Convert the power well descriptor domain mask to an array
> > of
> >     domains
> >   drm/i915: Convert the u64 power well domains mask to a bitmap
> >   drm/i915: Simplify power well definitions by adding power well
> >     instances
> >   drm/i915: Allow platforms to share power well descriptors
> >   drm/i915: Simplify the DG1 power well descriptors
> >   drm/i915: Sanitize the ADL-S power well definition
> >   drm/i915: Sanitize the port -> DDI/AUX power domain mapping for
> > each
> >     platform
> >   drm/i915: Remove the aliasing of power domain enum values
> >   drm/i915: Remove the ICL specific TBT power domains
> >   drm/i915: Remove duplicate DDI/AUX power domain mappings
> >   drm/i915: Remove the XELPD specific AUX and DDI power domains
> >
> >  drivers/gpu/drm/i915/Makefile                 |    2 +
> >  drivers/gpu/drm/i915/display/g4x_dp.c         |    3 +-
> >  drivers/gpu/drm/i915/display/g4x_hdmi.c       |    3 +-
> >  drivers/gpu/drm/i915/display/icl_dsi.c        |    8 +-
> >  drivers/gpu/drm/i915/display/intel_ddi.c      |    6 +-
> >  drivers/gpu/drm/i915/display/intel_display.c  |  150 +-
> >  drivers/gpu/drm/i915/display/intel_display.h  |    4 +-
> >  .../drm/i915/display/intel_display_debugfs.c  |    1 +
> >  .../drm/i915/display/intel_display_power.c    | 4842 ++-------------
> > --
> >  .../drm/i915/display/intel_display_power.h    |  154 +-
> >  .../i915/display/intel_display_power_map.c    | 1487 +++++
> >  .../i915/display/intel_display_power_map.h    |   14 +
> >  .../i915/display/intel_display_power_well.c   | 1937 +++++++
> >  .../i915/display/intel_display_power_well.h   |  168 +
> >  drivers/gpu/drm/i915/display/intel_hdcp.c     |    1 +
> >  drivers/gpu/drm/i915/display/intel_tc.c       |    5 +-
> >  16 files changed, 4105 insertions(+), 4680 deletions(-)
> >  create mode 100644
> > drivers/gpu/drm/i915/display/intel_display_power_map.c
> >  create mode 100644
> > drivers/gpu/drm/i915/display/intel_display_power_map.h
> >  create mode 100644
> > drivers/gpu/drm/i915/display/intel_display_power_well.c
> >  create mode 100644
> > drivers/gpu/drm/i915/display/intel_display_power_well.h
> >
> 
> BR,
> 
> Jouni Högander