✓ CI.checkpatch: success for drm/i915/dp_mst: Fix encoder HW state readout for UHBR MST
Patchwork
patchwork at emeril.freedesktop.org
Mon Feb 24 19:44:09 UTC 2025
== Series Details ==
Series: drm/i915/dp_mst: Fix encoder HW state readout for UHBR MST
URL : https://patchwork.freedesktop.org/series/145328/
State : success
== Summary ==
+ KERNEL=/kernel
+ git clone https://gitlab.freedesktop.org/drm/maintainer-tools mt
Cloning into 'mt'...
warning: redirecting to https://gitlab.freedesktop.org/drm/maintainer-tools.git/
+ git -C mt rev-list -n1 origin/master
22f9cda3436b4fe965b5c5f31d2f2c1bcb483189
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 25176bac2fb118e475772c63843a86c2de96ce81
Author: Imre Deak <imre.deak at intel.com>
Date: Mon Feb 24 11:32:42 2025 +0200
drm/i915/dp_mst: Fix encoder HW state readout for UHBR MST
The encoder HW/SW state verification should use a SW state which stays
unchanged while the encoder/output is active. The intel_dp::is_mst flag
used during state computation to choose between the DP SST/MST modes can
change while the output is active, if the sink gets disconnected or the
MST topology is removed for another reason. A subsequent state
verification using intel_dp::is_mst leads then to a mismatch if the
output is disabled/re-enabled without recomputing its state.
Use the encoder's active MST link count instead, which will be always
non-zero for an active MST output and will be zero for SST.
Fixes: 35d2e4b75649 ("drm/i915/ddi: start distinguishing 128b/132b SST and MST at state readout")
Fixes: 40d489fac0e8 ("drm/i915/ddi: handle 128b/132b SST in intel_ddi_read_func_ctl()")
Cc: Jani Nikula <jani.nikula at intel.com>
Signed-off-by: Imre Deak <imre.deak at intel.com>
+ /mt/dim checkpatch 9c30a334a46272b014f3923d3d5c29ba0f962de7 drm-intel
25176bac2fb1 drm/i915/dp_mst: Fix encoder HW state readout for UHBR MST
More information about the Intel-xe
mailing list