aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/gpu/drm/i915/intel_display.c
diff options
context:
space:
mode:
authorDaniel Vetter <daniel.vetter@ffwll.ch>2015-06-18 04:30:24 -0400
committerDaniel Vetter <daniel.vetter@ffwll.ch>2015-06-23 18:22:36 -0400
commit251ac8621921d3936ea2eff6790fe35b25cf28a4 (patch)
treed312c53d6780c78a123258d9730750dd92c0e479 /drivers/gpu/drm/i915/intel_display.c
parent9a851789e8a0cf38f9c04705a8e01b572e61f05f (diff)
drm/i915: s/update/compute/ for gmch dpll register functions
I was momentarily confused until I've double-checked that these functions really only compute state and don't update the hardware state. They once did that, but since Ander's rework of the dpll computation flow that's no longer the case. Rename them to avoid further confusion. Note that the ilk code already follows the compute_dpll naming scheme for computing the actual register value. DDI code goes with _calc_, but that is close enough. Cc: Ander Conselvan de Oliveira <ander.conselvan.de.oliveira@intel.com> Reviewed-by: Paulo Zanoni <paulo.r.zanoni@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
Diffstat (limited to 'drivers/gpu/drm/i915/intel_display.c')
-rw-r--r--drivers/gpu/drm/i915/intel_display.c40
1 files changed, 20 insertions, 20 deletions
diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
index 1bc217ab3592..060e30be3e39 100644
--- a/drivers/gpu/drm/i915/intel_display.c
+++ b/drivers/gpu/drm/i915/intel_display.c
@@ -7160,8 +7160,8 @@ void intel_dp_set_m_n(struct intel_crtc *crtc, enum link_m_n_set m_n)
7160 intel_cpu_transcoder_set_m_n(crtc, dp_m_n, dp_m2_n2); 7160 intel_cpu_transcoder_set_m_n(crtc, dp_m_n, dp_m2_n2);
7161} 7161}
7162 7162
7163static void vlv_update_pll(struct intel_crtc *crtc, 7163static void vlv_compute_dpll(struct intel_crtc *crtc,
7164 struct intel_crtc_state *pipe_config) 7164 struct intel_crtc_state *pipe_config)
7165{ 7165{
7166 u32 dpll, dpll_md; 7166 u32 dpll, dpll_md;
7167 7167
@@ -7274,8 +7274,8 @@ static void vlv_prepare_pll(struct intel_crtc *crtc,
7274 mutex_unlock(&dev_priv->sb_lock); 7274 mutex_unlock(&dev_priv->sb_lock);
7275} 7275}
7276 7276
7277static void chv_update_pll(struct intel_crtc *crtc, 7277static void chv_compute_dpll(struct intel_crtc *crtc,
7278 struct intel_crtc_state *pipe_config) 7278 struct intel_crtc_state *pipe_config)
7279{ 7279{
7280 pipe_config->dpll_hw_state.dpll = DPLL_SSC_REF_CLOCK_CHV | 7280 pipe_config->dpll_hw_state.dpll = DPLL_SSC_REF_CLOCK_CHV |
7281 DPLL_REFA_CLK_ENABLE_VLV | DPLL_VGA_MODE_DIS | 7281 DPLL_REFA_CLK_ENABLE_VLV | DPLL_VGA_MODE_DIS |
@@ -7414,11 +7414,11 @@ void vlv_force_pll_on(struct drm_device *dev, enum pipe pipe,
7414 }; 7414 };
7415 7415
7416 if (IS_CHERRYVIEW(dev)) { 7416 if (IS_CHERRYVIEW(dev)) {
7417 chv_update_pll(crtc, &pipe_config); 7417 chv_compute_dpll(crtc, &pipe_config);
7418 chv_prepare_pll(crtc, &pipe_config); 7418 chv_prepare_pll(crtc, &pipe_config);
7419 chv_enable_pll(crtc, &pipe_config); 7419 chv_enable_pll(crtc, &pipe_config);
7420 } else { 7420 } else {
7421 vlv_update_pll(crtc, &pipe_config); 7421 vlv_compute_dpll(crtc, &pipe_config);
7422 vlv_prepare_pll(crtc, &pipe_config); 7422 vlv_prepare_pll(crtc, &pipe_config);
7423 vlv_enable_pll(crtc, &pipe_config); 7423 vlv_enable_pll(crtc, &pipe_config);
7424 } 7424 }
@@ -7440,10 +7440,10 @@ void vlv_force_pll_off(struct drm_device *dev, enum pipe pipe)
7440 vlv_disable_pll(to_i915(dev), pipe); 7440 vlv_disable_pll(to_i915(dev), pipe);
7441} 7441}
7442 7442
7443static void i9xx_update_pll(struct intel_crtc *crtc, 7443static void i9xx_compute_dpll(struct intel_crtc *crtc,
7444 struct intel_crtc_state *crtc_state, 7444 struct intel_crtc_state *crtc_state,
7445 intel_clock_t *reduced_clock, 7445 intel_clock_t *reduced_clock,
7446 int num_connectors) 7446 int num_connectors)
7447{ 7447{
7448 struct drm_device *dev = crtc->base.dev; 7448 struct drm_device *dev = crtc->base.dev;
7449 struct drm_i915_private *dev_priv = dev->dev_private; 7449 struct drm_i915_private *dev_priv = dev->dev_private;
@@ -7517,10 +7517,10 @@ static void i9xx_update_pll(struct intel_crtc *crtc,
7517 } 7517 }
7518} 7518}
7519 7519
7520static void i8xx_update_pll(struct intel_crtc *crtc, 7520static void i8xx_compute_dpll(struct intel_crtc *crtc,
7521 struct intel_crtc_state *crtc_state, 7521 struct intel_crtc_state *crtc_state,
7522 intel_clock_t *reduced_clock, 7522 intel_clock_t *reduced_clock,
7523 int num_connectors) 7523 int num_connectors)
7524{ 7524{
7525 struct drm_device *dev = crtc->base.dev; 7525 struct drm_device *dev = crtc->base.dev;
7526 struct drm_i915_private *dev_priv = dev->dev_private; 7526 struct drm_i915_private *dev_priv = dev->dev_private;
@@ -7830,17 +7830,17 @@ static int i9xx_crtc_compute_clock(struct intel_crtc *crtc,
7830 } 7830 }
7831 7831
7832 if (IS_GEN2(dev)) { 7832 if (IS_GEN2(dev)) {
7833 i8xx_update_pll(crtc, crtc_state, 7833 i8xx_compute_dpll(crtc, crtc_state,
7834 has_reduced_clock ? &reduced_clock : NULL, 7834 has_reduced_clock ? &reduced_clock : NULL,
7835 num_connectors); 7835 num_connectors);
7836 } else if (IS_CHERRYVIEW(dev)) { 7836 } else if (IS_CHERRYVIEW(dev)) {
7837 chv_update_pll(crtc, crtc_state); 7837 chv_compute_dpll(crtc, crtc_state);
7838 } else if (IS_VALLEYVIEW(dev)) { 7838 } else if (IS_VALLEYVIEW(dev)) {
7839 vlv_update_pll(crtc, crtc_state); 7839 vlv_compute_dpll(crtc, crtc_state);
7840 } else { 7840 } else {
7841 i9xx_update_pll(crtc, crtc_state, 7841 i9xx_compute_dpll(crtc, crtc_state,
7842 has_reduced_clock ? &reduced_clock : NULL, 7842 has_reduced_clock ? &reduced_clock : NULL,
7843 num_connectors); 7843 num_connectors);
7844 } 7844 }
7845 7845
7846 return 0; 7846 return 0;