From: Imre Deak Date: Thu, 20 Nov 2025 17:23:54 +0000 (+0200) Subject: drm/i915/cx0: Fix port to PLL ID mapping on BMG X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=370f45b1cea859484691128ec205abc7e9402fb7;p=thirdparty%2Fkernel%2Flinux.git drm/i915/cx0: Fix port to PLL ID mapping on BMG The intel_port_to_tc() call in mtl_port_to_pll_id() assumed that all TypeC DDI ports are connected to a TypeC PHY. This is not true on BMG where all the ports - including the PORT_TC1..4 TypeC DDI ports - are connected to a non-TypeC PHY. For these ports intel_port_to_tc() returns TC_PORT_NONE, which results in an incorrect port -> PLL ID mapping. Fix this up by using the expected PORT_TC1..4 port -> TC_PORT_1..4 tc_port mapping on BMG as well. Fixes: ca1eda5cd476c ("drm/i915/cx0: Add MTL+ .get_dplls hook") Cc: Suraj Kandpal Reviewed-by: Mika Kahola Signed-off-by: Imre Deak Link: https://patch.msgid.link/20251120172358.1282765-1-imre.deak@intel.com --- diff --git a/drivers/gpu/drm/i915/display/intel_ddi.c b/drivers/gpu/drm/i915/display/intel_ddi.c index 96fcad6dbb2f4..8471bdab5c62f 100644 --- a/drivers/gpu/drm/i915/display/intel_ddi.c +++ b/drivers/gpu/drm/i915/display/intel_ddi.c @@ -4294,9 +4294,10 @@ static void mtl_ddi_cx0_get_config(struct intel_encoder *encoder, } /* - * Get the configuration for either a port using a C10 PHY PLL, or in the case of - * the PTL port B eDP on TypeC PHY case the configuration of a port using a C20 - * PHY PLL. + * Get the configuration for either a port using a C10 PHY PLL, or a port using a + * C20 PHY PLL in the cases of: + * - BMG port A/B + * - PTL port B eDP over TypeC PHY */ static void mtl_ddi_non_tc_phy_get_config(struct intel_encoder *encoder, struct intel_crtc_state *crtc_state) diff --git a/drivers/gpu/drm/i915/display/intel_dpll_mgr.c b/drivers/gpu/drm/i915/display/intel_dpll_mgr.c index 6d7d5394713d6..8ae8cc7ad79d3 100644 --- a/drivers/gpu/drm/i915/display/intel_dpll_mgr.c +++ b/drivers/gpu/drm/i915/display/intel_dpll_mgr.c @@ -206,7 +206,7 @@ enum intel_dpll_id icl_tc_port_to_pll_id(enum tc_port tc_port) enum intel_dpll_id mtl_port_to_pll_id(struct intel_display *display, enum port port) { if (port >= PORT_TC1) - return icl_tc_port_to_pll_id(intel_port_to_tc(display, port)); + return icl_tc_port_to_pll_id(port - PORT_TC1 + TC_PORT_1); switch (port) { case PORT_A: @@ -3507,9 +3507,10 @@ err_unreference_tbt_pll: } /* - * Get the PLL for either a port using a C10 PHY PLL, or in the - * PTL port B eDP over TypeC PHY case, the PLL for a port using - * a C20 PHY PLL. + * Get the PLL for either a port using a C10 PHY PLL, or for a port using a + * C20 PHY PLL in the cases of: + * - BMG port A/B + * - PTL port B eDP over TypeC PHY */ static int mtl_get_non_tc_phy_dpll(struct intel_atomic_state *state, struct intel_crtc *crtc,