Skip to content

Merge PC8 with runtime PM, v3

Paulo Zanoni intel-gfx 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest16 patches 6 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/16drm/i915: extract __hsw_do_{en, dis}able_package_c8drivers/gpu/drm/i915/intel_display.c+38 -260 replies
2/16drm/i915: make PC8 be part of runtime PM suspend/resumedrivers/gpu/drm/i915/i915_dma.c drivers/gpu/drm/i915/i915_drv.c drivers/gpu/drm/i915/i915_drv.h +4 more+43 -532 replies
3/16drm/i915: get/put runtime PM when we get/put a power domaindrivers/gpu/drm/i915/intel_pm.c+4 -00 replies
4/16drm/i915: remove dev_priv->pc8.requirements_metdrivers/gpu/drm/i915/i915_debugfs.c drivers/gpu/drm/i915/i915_drv.h drivers/gpu/drm/i915/intel_display.c +1 more+3 -591 reply
5/16drm/i915: get runtime PM references when the GPU is idle/busydrivers/gpu/drm/i915/intel_display.c+2 -20 replies
6/16drm/i915: kill pc8.disable_countdrivers/gpu/drm/i915/i915_debugfs.c drivers/gpu/drm/i915/i915_drv.h drivers/gpu/drm/i915/intel_display.c +1 more+0 -170 replies
7/16drm/i915: remove an indirection level on PC8 functionsdrivers/gpu/drm/i915/intel_display.c+2 -142 replies
8/16drm/i915: don't get/put PC8 reference on freeze/thawdrivers/gpu/drm/i915/i915_drv.c+0 -50 replies
9/16drm/i915: make intel_aux_display_runtime_get get runtime PM, not PC8drivers/gpu/drm/i915/intel_pm.c+2 -31 reply
10/16drm/i915: don't get/put PC8 when getting/putting power wellsdrivers/gpu/drm/i915/intel_display.c drivers/gpu/drm/i915/intel_drv.h drivers/gpu/drm/i915/intel_pm.c+0 -240 replies
11/16drm/i915: remove dev_priv->pc8.enableddrivers/gpu/drm/i915/i915_debugfs.c drivers/gpu/drm/i915/i915_drv.h drivers/gpu/drm/i915/intel_display.c +1 more+0 -90 replies
12/16drm/i915: move pc8.irqs_disabled to pm.irqs_disableddrivers/gpu/drm/i915/i915_debugfs.c drivers/gpu/drm/i915/i915_drv.h drivers/gpu/drm/i915/i915_gem.c +4 more+42 -410 replies
13/16drm/i915: kill struct i915_package_c8drivers/gpu/drm/i915/i915_debugfs.c drivers/gpu/drm/i915/i915_drv.h drivers/gpu/drm/i915/intel_pm.c+0 -90 replies
14/16drm/i915: rename __hsw_do_{en, dis}able_pc8drivers/gpu/drm/i915/i915_drv.c drivers/gpu/drm/i915/intel_display.c drivers/gpu/drm/i915/intel_drv.h+6 -60 replies
15/16drm/i915: update the PC8 and runtime PM documentationdrivers/gpu/drm/i915/i915_drv.h drivers/gpu/drm/i915/intel_display.c+35 -400 replies
16/16drm/i915: init pm.suspended earlierdrivers/gpu/drm/i915/intel_pm.c+1 -20 replies

Reviews trailers per patch

All 16 patches have a Reviewed-by or Acked-by.

PatchImre DeakJesse Barnes
1/16R-b
2/16R-b
3/16R-b
4/16R-b
5/16R-b
6/16R-b
7/16R-b
8/16R-b
9/16R-b
10/16R-b
11/16R-b
12/16R-b
13/16R-b
14/16R-b
15/16R-b
16/16R-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [PATCH 00/16] Merge PC8 with runtime PM, v3

From: Paulo Zanoni <[email protected]>

Hi

So this series is obviously the follow up of "Merge PC8 with runtime PM, v2". It contains the changes requested by our reviewers, a rebase, and the latest Reviewed-by tags. Daniel asked me to send it on a new thread to avoid confusion, and I agree this is the best way to proceed.

I based this series on top of "[PATCH 0/6] More runtime PM fixes". So please grab that before applying these patches.

This series is mostly reviewed. Patches that still don't have Reviewed-by tags are just: 2, 4 and 9.

Notes to Daniel and other possible people applying these patches:
- Please merge patches in the exact order I sent! The order I used here is strategically designed to avoid regressions in the middle.
- After applying patch 3, please read the notice on its commit message and check if the result is correct.

Thanks,
Paulo

Paulo Zanoni (16):
drm/i915: extract __hsw_do_{en,dis}able_package_c8 drm/i915: make PC8 be part of runtime PM suspend/resume drm/i915: get/put runtime PM when we get/put a power domain drm/i915: remove dev_priv->pc8.requirements_met
drm/i915: get runtime PM references when the GPU is idle/busy drm/i915: kill pc8.disable_count
drm/i915: remove an indirection level on PC8 functions drm/i915: don't get/put PC8 reference on freeze/thaw drm/i915: make intel_aux_display_runtime_get get runtime PM, not PC8 drm/i915: don't get/put PC8 when getting/putting power wells drm/i915: remove dev_priv->pc8.enabled
drm/i915: move pc8.irqs_disabled to pm.irqs_disabled drm/i915: kill struct i915_package_c8
drm/i915: rename __hsw_do_{en,dis}able_pc8
drm/i915: update the PC8 and runtime PM documentation drm/i915: init pm.suspended earlier

 drivers/gpu/drm/i915/i915_debugfs.c  |   8 +-
 drivers/gpu/drm/i915/i915_dma.c      |   2 -
 drivers/gpu/drm/i915/i915_drv.c      |  13 ++-
 drivers/gpu/drm/i915/i915_drv.h      |  65 +++----------
 drivers/gpu/drm/i915/i915_gem.c      |   2 +-
 drivers/gpu/drm/i915/i915_irq.c      |  58 ++++++------
 drivers/gpu/drm/i915/i915_params.c   |  10 --
 drivers/gpu/drm/i915/intel_display.c | 177 ++++++++++-------------------------
 drivers/gpu/drm/i915/intel_drv.h     |   9 +-
 drivers/gpu/drm/i915/intel_pm.c      |  26 +++--
 10 files changed, 119 insertions(+), 251 deletions(-)
Signature · 1 line
1.8.5.3