On Thu, 05 Oct 2023, Gustavo Sousa <[email protected]> wrote: > Quoting Khaled Almahallawy (2023-10-04 21:13:10-03:00) >>Currently, with MFD/pin assignment D, the driver clears the pipe reset bit >>of lane 1 which is not owned by display. This causes the display >>to block S0iX. >> >>By not clearing this bit for lane 1 and keeping whatever default, S0ix >>started to work. This is already what the driver does at the end >>of the phy lane reset sequence (Step#8) >> >>Bspec: 65451 >> > > We should not have blank lines in the trailers section. This could be fixed > while applying.
Agreed. > >>Cc: Mika Kahola <[email protected]> >>Cc: Gustavo Sousa <[email protected]> >>Signed-off-by: Khaled Almahallawy <[email protected]> > > Nice fix. Thanks! > > Reviewed-by: Gustavo Sousa <[email protected]> > >>--- >> drivers/gpu/drm/i915/display/intel_cx0_phy.c | 3 +-- >> 1 file changed, 1 insertion(+), 2 deletions(-) >> >>diff --git a/drivers/gpu/drm/i915/display/intel_cx0_phy.c >>b/drivers/gpu/drm/i915/display/intel_cx0_phy.c >>index abd607b564f1..f653b83a7d4f 100644 >>--- a/drivers/gpu/drm/i915/display/intel_cx0_phy.c >>+++ b/drivers/gpu/drm/i915/display/intel_cx0_phy.c >>@@ -2596,8 +2596,7 @@ static void intel_cx0_phy_lane_reset(struct >>drm_i915_private *i915, >> drm_warn(&i915->drm, "PHY %c failed to bring out of SOC >> reset after %dus.\n", >> phy_name(phy), >> XELPDP_PORT_BUF_SOC_READY_TIMEOUT_US); >> >>- intel_de_rmw(i915, XELPDP_PORT_BUF_CTL2(port), >>- XELPDP_LANE_PIPE_RESET(0) | XELPDP_LANE_PIPE_RESET(1), >>+ intel_de_rmw(i915, XELPDP_PORT_BUF_CTL2(port), lane_pipe_reset, >> lane_pipe_reset); >> >> if (__intel_de_wait_for_register(i915, XELPDP_PORT_BUF_CTL2(port), >>-- >>2.25.1 >> -- Jani Nikula, Intel
