Commit 5a83381f authored by Gustavo Sousa's avatar Gustavo Sousa Committed by Matt Roper
Browse files

drm/i915/dmc_wl: Couple enable/disable with dynamic DC states 



Enabling and disabling the DMC wakelock should be done as part of
enabling and disabling of dynamic DC states, respectively. We should not
enable or disable DMC wakelock independently of DC states, otherwise we
would risk ending up with an inconsistent state where dynamic DC states
are enabled and the DMC wakelock is disabled, going against current
recommendations and making MMIO transactions potentially slower. In
future display IPs that could have a worse outcome if DMC trap
implementation is completely removed.

So, let's make things safer by tying stuff together, removing the
independent calls, and also put warnings in place to detect inconsistent
calls.

Reviewed-by: default avatarLuca Coelho <luciano.coelho@intel.com>
Signed-off-by: default avatarGustavo Sousa <gustavo.sousa@intel.com>
Signed-off-by: default avatarMatt Roper <matthew.d.roper@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20241108130218.24125-13-gustavo.sousa@intel.com
parent 9fe9cd95

drivers/gpu/drm/i915/display/intel_display_power_well.c

+4 −1
Original line number Diff line number Diff line
@@ -988,6 +988,7 @@ void gen9_disable_dc_states(struct intel_display *display) 
	struct drm_i915_private *dev_priv = to_i915(display->drm);

	struct i915_power_domains *power_domains = &display->power.domains;

	struct intel_cdclk_config cdclk_config = {};

	u32 old_state = power_domains->dc_state;



	if (power_domains->target_dc_state == DC_STATE_EN_DC3CO) {

		tgl_disable_dc3co(display);
@@ -1003,6 +1004,8 @@ void gen9_disable_dc_states(struct intel_display *display) 
		return;

	}



	if (old_state == DC_STATE_EN_UPTO_DC5 ||

	    old_state == DC_STATE_EN_UPTO_DC6)

		intel_dmc_wl_disable(display);



	intel_cdclk_get_cdclk(display, &cdclk_config);

drivers/gpu/drm/i915/display/intel_dmc.c

+0 −4
Original line number Diff line number Diff line
@@ -638,8 +638,6 @@ void intel_dmc_disable_program(struct intel_display *display) 
	pipedmc_clock_gating_wa(display, true);

	disable_all_event_handlers(display);

	pipedmc_clock_gating_wa(display, false);



	intel_dmc_wl_disable(display);

}



void assert_dmc_loaded(struct intel_display *display)
@@ -1146,8 +1144,6 @@ void intel_dmc_suspend(struct intel_display *display) 
	if (dmc)

		flush_work(&dmc->work);



	intel_dmc_wl_disable(display);



	/* Drop the reference held in case DMC isn't loaded. */

	if (!intel_dmc_has_payload(display))

		intel_dmc_runtime_pm_put(display);

drivers/gpu/drm/i915/display/intel_dmc_wl.c

+4 −2
Original line number Diff line number Diff line
@@ -283,6 +283,7 @@ void intel_dmc_wl_init(struct intel_display *display) 
	refcount_set(&wl->refcount, 0);

}



/* Must only be called as part of enabling dynamic DC states. */

void intel_dmc_wl_enable(struct intel_display *display, u32 dc_state)

{

	struct intel_dmc_wl *wl = &display->wl;
@@ -295,7 +296,7 @@ void intel_dmc_wl_enable(struct intel_display *display, u32 dc_state) 


	wl->dc_state = dc_state;



	if (wl->enabled)

	if (drm_WARN_ON(display->drm, wl->enabled))

		goto out_unlock;



	/*
@@ -328,6 +329,7 @@ void intel_dmc_wl_enable(struct intel_display *display, u32 dc_state) 
	spin_unlock_irqrestore(&wl->lock, flags);

}



/* Must only be called as part of disabling dynamic DC states. */

void intel_dmc_wl_disable(struct intel_display *display)

{

	struct intel_dmc_wl *wl = &display->wl;
@@ -340,7 +342,7 @@ void intel_dmc_wl_disable(struct intel_display *display) 


	spin_lock_irqsave(&wl->lock, flags);



	if (!wl->enabled)

	if (drm_WARN_ON(display->drm, !wl->enabled))

		goto out_unlock;



	/* Disable wakelock in DMC */