On Fri, 03 Feb 2017, "Lee, Shawn C" <shawn.c.lee@xxxxxxxxx> wrote:
> From: "Lee, Shawn C" <shawn.c.lee@xxxxxxxxx>
>
> Add the missing INTEL_OUTPUT_DP_MST case in bxt_get_dpll()
> to correctly initialize the crtc_state and port plls when
> link training a DP MST monitor on BXT/APL devices.
>
> Fixes: a277ca7dc01d ("drm/i915: Split bxt_ddi_pll_select()")
> Bugs: https://bugs.freedesktop.org/show_bug.cgi?id=99572
>
> Reviewed-by: Cooper Chiou <cooper.chiou@xxxxxxxxx>
> Reviewed-by: Gary C Wang <gary.c.wang@xxxxxxxxx>
> Reviewed-by: Ciobanu, Nathan D <nathan.d.ciobanu@xxxxxxxxx>
> Reviewed-by: Herbert, Marc <marc.herbert@xxxxxxxxx>
> Reviewed-by: Bride, Jim <jim.bride@xxxxxxxxx>
> Reviewed-by: Navare, Manasi D <manasi.d.navare@xxxxxxxxx>
> Cc: Jani Nikula <jani.nikula@xxxxxxxxx>
>
> Signed-off-by: Lee, Shawn C <shawn.c.lee@xxxxxxxxx>
Pushed to drm-intel-next-queued, thanks for the patch and all the
reviews.
BR,
Jani.
> ---
> drivers/gpu/drm/i915/intel_dpll_mgr.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_dpll_mgr.c b/drivers/gpu/drm/i915/intel_dpll_mgr.c
> index c92a2558beb4..1a1d99d266ed 100644
> --- a/drivers/gpu/drm/i915/intel_dpll_mgr.c
> +++ b/drivers/gpu/drm/i915/intel_dpll_mgr.c
> @@ -1855,7 +1855,8 @@ bool bxt_ddi_dp_set_dpll_hw_state(int clock,
> return NULL;
>
> if ((encoder->type == INTEL_OUTPUT_DP ||
> - encoder->type == INTEL_OUTPUT_EDP) &&
> + encoder->type == INTEL_OUTPUT_EDP ||
> + encoder->type == INTEL_OUTPUT_DP_MST ) &&
> !bxt_ddi_dp_set_dpll_hw_state(clock, &dpll_hw_state))
> return NULL;
--
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
