Commit 4651fb23 authored by Paulo Zanoni's avatar Paulo Zanoni Committed by Daniel Vetter
Browse files

drm/i915: remove useless runtime PM get calls 



We already call intel_display_power_get, which will get a power
domain, and every power domain should get a runtime PM reference,
which will wake up the machine.

v2: - Also touch intel_crt_detect() (Ville).

Signed-off-by: default avatarPaulo Zanoni <paulo.r.zanoni@intel.com>
Reviewed-by: default avatarVille Syrjälä <ville.syrjala@linux.intel.com>
[danvet: Fixup commit message as spotted by Ville.]
Signed-off-by: default avatarDaniel Vetter <daniel.vetter@ffwll.ch>
parent b47adc17

drivers/gpu/drm/i915/intel_crt.c

+0 −4
Original line number Diff line number Diff line
@@ -660,8 +660,6 @@ intel_crt_detect(struct drm_connector *connector, bool force) 
	struct intel_load_detect_pipe tmp;

	struct drm_modeset_acquire_ctx ctx;



	intel_runtime_pm_get(dev_priv);



	DRM_DEBUG_KMS("[CONNECTOR:%d:%s] force=%d\n",

		      connector->base.id, connector->name,

		      force);
@@ -713,8 +711,6 @@ intel_crt_detect(struct drm_connector *connector, bool force) 


out:

	intel_display_power_put(dev_priv, power_domain);

	intel_runtime_pm_put(dev_priv);



	return status;

}

drivers/gpu/drm/i915/intel_dp.c

+0 −5
Original line number Diff line number Diff line
@@ -3621,8 +3621,6 @@ intel_dp_detect(struct drm_connector *connector, bool force) 
	enum intel_display_power_domain power_domain;

	struct edid *edid = NULL;



	intel_runtime_pm_get(dev_priv);



	power_domain = intel_display_port_power_domain(intel_encoder);

	intel_display_power_get(dev_priv, power_domain);
@@ -3657,9 +3655,6 @@ intel_dp_detect(struct drm_connector *connector, bool force) 


out:

	intel_display_power_put(dev_priv, power_domain);



	intel_runtime_pm_put(dev_priv);



	return status;

}

CRA Git | Maintained and supported by SUSTech CRA and CCSE