[PATCH 3/4] drm/i915/gen9: Fix clearing of the BIOS power well request register

Imre Deak imre.deak at intel.com
Tue Feb 14 17:31:52 UTC 2017

Atm, in the power well sync_hw hook we are clearing all BIOS request
bits, not just the one corresponding to the given power well. This could
turn off an unrelated power well inadvertently if it didn't have a
request bit set in the driver request register.

This didn't cause a problem so far, since we enabled all power wells
explicitly before clearing the BIOS request register. A follow-up
patchset will add power wells that won't get enabled this way, so fix up
the inconsistency.

Cc: Ander Conselvan de Oliveira <conselvan2 at gmail.com>
Cc: David Weinehall <david.weinehall at linux.intel.com>
Cc: Ville Syrjälä <ville.syrjala at linux.intel.com>
Signed-off-by: Imre Deak <imre.deak at intel.com>
 drivers/gpu/drm/i915/intel_runtime_pm.c | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/i915/intel_runtime_pm.c b/drivers/gpu/drm/i915/intel_runtime_pm.c
index f9aff26..c5b7fba 100644
--- a/drivers/gpu/drm/i915/intel_runtime_pm.c
+++ b/drivers/gpu/drm/i915/intel_runtime_pm.c
@@ -887,8 +887,13 @@ static bool skl_power_well_enabled(struct drm_i915_private *dev_priv,
 static void skl_power_well_sync_hw(struct drm_i915_private *dev_priv,
 				struct i915_power_well *power_well)
+	uint32_t mask = SKL_POWER_WELL_REQ(power_well->id);
+	uint32_t val = I915_READ(HSW_PWR_WELL_BIOS);
 	/* Clear any request made by BIOS as driver is taking over */
+	if (val & mask) {
+		I915_WRITE(HSW_PWR_WELL_BIOS, val & ~mask);
+	}
 static void skl_power_well_enable(struct drm_i915_private *dev_priv,

More information about the Intel-gfx-trybot mailing list