[Intel-gfx] [PATCH 6/7] drm/i915: make SDVO TV-out work for multifunction devices
Jani Nikula
jani.nikula at linux.intel.com
Mon May 6 15:36:31 CEST 2013
On Tue, 30 Apr 2013, Daniel Vetter <daniel.vetter at ffwll.ch> wrote:
> We need to track this correctly. While at it shovel the boolean
> to track whether the sdvo is in tv mode or not into pipe_config.
>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=36997
Bug scrubbing, should this be:
https://bugs.freedesktop.org/show_bug.cgi?id=63609 ?
Jani.
> Tested-by: Pierre Assal <pierre.assal at verint.com>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=36997
> Tested-by: cancan,feng <cancan.feng at intel.com>
> Signed-off-by: Daniel Vetter <daniel.vetter at ffwll.ch>
> ---
> drivers/gpu/drm/i915/intel_display.c | 8 +++-----
> drivers/gpu/drm/i915/intel_drv.h | 5 ++++-
> drivers/gpu/drm/i915/intel_sdvo.c | 8 ++------
> 3 files changed, 9 insertions(+), 12 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
> index 44bcfae..ef0d27b 100644
> --- a/drivers/gpu/drm/i915/intel_display.c
> +++ b/drivers/gpu/drm/i915/intel_display.c
> @@ -4546,7 +4546,7 @@ static void i9xx_update_pll(struct intel_crtc *crtc,
> if (INTEL_INFO(dev)->gen >= 4)
> dpll |= (6 << PLL_LOAD_PULSE_PHASE_SHIFT);
>
> - if (is_sdvo && intel_pipe_has_type(&crtc->base, INTEL_OUTPUT_TVOUT))
> + if (crtc->config.sdvo_tv_clock)
> dpll |= PLL_REF_INPUT_TVCLKINBC;
> else if (intel_pipe_has_type(&crtc->base, INTEL_OUTPUT_LVDS) &&
> intel_panel_use_ssc(dev_priv) && num_connectors < 2)
> @@ -5590,7 +5590,7 @@ static uint32_t ironlake_compute_dpll(struct intel_crtc *intel_crtc,
> struct intel_encoder *intel_encoder;
> uint32_t dpll;
> int factor, num_connectors = 0;
> - bool is_lvds = false, is_sdvo = false, is_tv = false;
> + bool is_lvds = false, is_sdvo = false;
>
> for_each_encoder_on_crtc(dev, crtc, intel_encoder) {
> switch (intel_encoder->type) {
> @@ -5600,8 +5600,6 @@ static uint32_t ironlake_compute_dpll(struct intel_crtc *intel_crtc,
> case INTEL_OUTPUT_SDVO:
> case INTEL_OUTPUT_HDMI:
> is_sdvo = true;
> - if (intel_encoder->needs_tv_clock)
> - is_tv = true;
> break;
> }
>
> @@ -5615,7 +5613,7 @@ static uint32_t ironlake_compute_dpll(struct intel_crtc *intel_crtc,
> dev_priv->lvds_ssc_freq == 100) ||
> (HAS_PCH_IBX(dev) && intel_is_dual_link_lvds(dev)))
> factor = 25;
> - } else if (is_sdvo && is_tv)
> + } else if (intel_crtc->config.sdvo_tv_clock)
> factor = 20;
>
> if (ironlake_needs_fb_cb_tune(&intel_crtc->config.dpll, factor))
> diff --git a/drivers/gpu/drm/i915/intel_drv.h b/drivers/gpu/drm/i915/intel_drv.h
> index 766afcf..be196ff 100644
> --- a/drivers/gpu/drm/i915/intel_drv.h
> +++ b/drivers/gpu/drm/i915/intel_drv.h
> @@ -120,7 +120,6 @@ struct intel_encoder {
> struct intel_crtc *new_crtc;
>
> int type;
> - bool needs_tv_clock;
> /*
> * Intel hw has only one MUX where encoders could be clone, hence a
> * simple flag is enough to compute the possible_clones mask.
> @@ -223,6 +222,10 @@ struct intel_crtc_config {
> /* Controls for the clock computation, to override various stages. */
> bool clock_set;
>
> + /* SDVO TV has a bunch of special case. To make multifunction encoders
> + * work correctly, we need to track this at runtime.*/
> + bool sdvo_tv_clock;
> +
> /*
> * crtc bandwidth limit, don't increase pipe bpp or clock if not really
> * required. This is set in the 2nd loop of calling encoder's
> diff --git a/drivers/gpu/drm/i915/intel_sdvo.c b/drivers/gpu/drm/i915/intel_sdvo.c
> index f6bf9fc..3a1d710 100644
> --- a/drivers/gpu/drm/i915/intel_sdvo.c
> +++ b/drivers/gpu/drm/i915/intel_sdvo.c
> @@ -1092,6 +1092,7 @@ static bool intel_sdvo_compute_config(struct intel_encoder *encoder,
> (void) intel_sdvo_get_preferred_input_mode(intel_sdvo,
> mode,
> adjusted_mode);
> + pipe_config->sdvo_tv_clock = true;
> } else if (intel_sdvo->is_lvds) {
> if (!intel_sdvo_set_output_timings_from_mode(intel_sdvo,
> intel_sdvo->sdvo_lvds_fixed_mode))
> @@ -1655,12 +1656,9 @@ intel_sdvo_detect(struct drm_connector *connector, bool force)
> if (ret == connector_status_connected) {
> intel_sdvo->is_tv = false;
> intel_sdvo->is_lvds = false;
> - intel_sdvo->base.needs_tv_clock = false;
>
> - if (response & SDVO_TV_MASK) {
> + if (response & SDVO_TV_MASK)
> intel_sdvo->is_tv = true;
> - intel_sdvo->base.needs_tv_clock = true;
> - }
> if (response & SDVO_LVDS_MASK)
> intel_sdvo->is_lvds = intel_sdvo->sdvo_lvds_fixed_mode != NULL;
> }
> @@ -2357,7 +2355,6 @@ intel_sdvo_tv_init(struct intel_sdvo *intel_sdvo, int type)
> intel_sdvo_connector->output_flag = type;
>
> intel_sdvo->is_tv = true;
> - intel_sdvo->base.needs_tv_clock = true;
>
> intel_sdvo_connector_init(intel_sdvo_connector, intel_sdvo);
>
> @@ -2445,7 +2442,6 @@ static bool
> intel_sdvo_output_setup(struct intel_sdvo *intel_sdvo, uint16_t flags)
> {
> intel_sdvo->is_tv = false;
> - intel_sdvo->base.needs_tv_clock = false;
> intel_sdvo->is_lvds = false;
>
> /* SDVO requires XXX1 function may not exist unless it has XXX0 function.*/
> --
> 1.7.11.7
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
More information about the Intel-gfx
mailing list