diff options
| author | Maarten Lankhorst <maarten.lankhorst@linux.intel.com> | 2018-10-11 06:04:56 -0400 |
|---|---|---|
| committer | Maarten Lankhorst <maarten.lankhorst@linux.intel.com> | 2018-10-16 09:52:33 -0400 |
| commit | b0b62d845e44e5432505ba0f74da41efc1b122d3 (patch) | |
| tree | d31866eb99185b0d7c7379dc96e20213dba98b6a | |
| parent | c5b36facfad802d11e66c3fe5bd2ea6d70c33dfd (diff) | |
drm/i915: Pass crtc_state to ivybridge_update_fdi_bc_bifurcation
We have to look at crtc_state, so pass that instead.
Also cleanup the use of dev vs dev_priv, we really want to pass along
dev_priv.
Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20181011100457.8776-10-maarten.lankhorst@linux.intel.com
| -rw-r--r-- | drivers/gpu/drm/i915/intel_display.c | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c index 8535ffe87925..a2e729fa8d64 100644 --- a/drivers/gpu/drm/i915/intel_display.c +++ b/drivers/gpu/drm/i915/intel_display.c | |||
| @@ -4584,9 +4584,8 @@ static void ironlake_pch_transcoder_set_timings(const struct intel_crtc_state *c | |||
| 4584 | I915_READ(VSYNCSHIFT(cpu_transcoder))); | 4584 | I915_READ(VSYNCSHIFT(cpu_transcoder))); |
| 4585 | } | 4585 | } |
| 4586 | 4586 | ||
| 4587 | static void cpt_set_fdi_bc_bifurcation(struct drm_device *dev, bool enable) | 4587 | static void cpt_set_fdi_bc_bifurcation(struct drm_i915_private *dev_priv, bool enable) |
| 4588 | { | 4588 | { |
| 4589 | struct drm_i915_private *dev_priv = to_i915(dev); | ||
| 4590 | uint32_t temp; | 4589 | uint32_t temp; |
| 4591 | 4590 | ||
| 4592 | temp = I915_READ(SOUTH_CHICKEN1); | 4591 | temp = I915_READ(SOUTH_CHICKEN1); |
| @@ -4605,22 +4604,23 @@ static void cpt_set_fdi_bc_bifurcation(struct drm_device *dev, bool enable) | |||
| 4605 | POSTING_READ(SOUTH_CHICKEN1); | 4604 | POSTING_READ(SOUTH_CHICKEN1); |
| 4606 | } | 4605 | } |
| 4607 | 4606 | ||
| 4608 | static void ivybridge_update_fdi_bc_bifurcation(struct intel_crtc *intel_crtc) | 4607 | static void ivybridge_update_fdi_bc_bifurcation(const struct intel_crtc_state *crtc_state) |
| 4609 | { | 4608 | { |
| 4610 | struct drm_device *dev = intel_crtc->base.dev; | 4609 | struct intel_crtc *crtc = to_intel_crtc(crtc_state->base.crtc); |
| 4610 | struct drm_i915_private *dev_priv = to_i915(crtc->base.dev); | ||
| 4611 | 4611 | ||
| 4612 | switch (intel_crtc->pipe) { | 4612 | switch (crtc->pipe) { |
| 4613 | case PIPE_A: | 4613 | case PIPE_A: |
| 4614 | break; | 4614 | break; |
| 4615 | case PIPE_B: | 4615 | case PIPE_B: |
| 4616 | if (intel_crtc->config->fdi_lanes > 2) | 4616 | if (crtc_state->fdi_lanes > 2) |
| 4617 | cpt_set_fdi_bc_bifurcation(dev, false); | 4617 | cpt_set_fdi_bc_bifurcation(dev_priv, false); |
| 4618 | else | 4618 | else |
| 4619 | cpt_set_fdi_bc_bifurcation(dev, true); | 4619 | cpt_set_fdi_bc_bifurcation(dev_priv, true); |
| 4620 | 4620 | ||
| 4621 | break; | 4621 | break; |
| 4622 | case PIPE_C: | 4622 | case PIPE_C: |
| 4623 | cpt_set_fdi_bc_bifurcation(dev, true); | 4623 | cpt_set_fdi_bc_bifurcation(dev_priv, true); |
| 4624 | 4624 | ||
| 4625 | break; | 4625 | break; |
| 4626 | default: | 4626 | default: |
| @@ -4677,7 +4677,7 @@ static void ironlake_pch_enable(const struct intel_atomic_state *state, | |||
| 4677 | assert_pch_transcoder_disabled(dev_priv, pipe); | 4677 | assert_pch_transcoder_disabled(dev_priv, pipe); |
| 4678 | 4678 | ||
| 4679 | if (IS_IVYBRIDGE(dev_priv)) | 4679 | if (IS_IVYBRIDGE(dev_priv)) |
| 4680 | ivybridge_update_fdi_bc_bifurcation(crtc); | 4680 | ivybridge_update_fdi_bc_bifurcation(crtc_state); |
| 4681 | 4681 | ||
| 4682 | /* Write the TU size bits before fdi link training, so that error | 4682 | /* Write the TU size bits before fdi link training, so that error |
| 4683 | * detection works. */ | 4683 | * detection works. */ |
