✓ CI.checkpatch: success for drm/i915/dp: Fix the enabling/disabling of audio SDP splitting (rev2)
Patchwork
patchwork at emeril.freedesktop.org
Wed May 21 13:09:26 UTC 2025
== Series Details ==
Series: drm/i915/dp: Fix the enabling/disabling of audio SDP splitting (rev2)
URL : https://patchwork.freedesktop.org/series/149254/
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
202708c00696422fd217223bb679a353a5936e23
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit add661acc8e960ed361be546984f281dfc8bb096
Author: Imre Deak <imre.deak at intel.com>
Date: Tue May 20 17:22:19 2025 +0300
drm/i915/dp: Fix the enabling/disabling of audio SDP splitting
Adjust the enabling/disabling steps of the DP audio SDP splitting
according to a recent Bspec update. This moves the enabling to the audio
codec enable sequence after the transcoder is enabled and disables SDP
splitting explicitly during the audio disable sequence.
Bspec requires waiting for a vblank event after the transcoder is
enabled and before SDP splitting is enabled. There is no need for an
explicit wait for this, since after the transcoder is enabled this
vblank event is guaranteed to have happened via a flip done wait (see
intel_atomic_commit_tail() -> drm_atomic_helper_wait_for_flip_done()).
The bspec update is for LNL+ only, but the HW team clarified that this
has been always the intended sequence on all platforms and bspec will be
updated everywhere accordingly.
The way SDP splitting was originally enabled matched the version of
bspec at that time. Adding here the Fixes: line still, since this
change fixes a FIFO underrun on PTL during output enabling when DSC is
enabled.
Bspec: 49283, 68943
Fixes: 8853750dbad8 ("drm/i915: Enable SDP split for DP2.0")
Cc: Vinod Govindapillai <vinod.govindapillai at intel.com>
Signed-off-by: Imre Deak <imre.deak at intel.com>
Acked-by: Jani Nikula <jani.nikula at intel.com>
Reviewed-by: Vinod Govindapillai <vinod.govindapillai at intel.com>
+ /mt/dim checkpatch 3695d568b260d85e65f9331ca1180a9dd2b5560b drm-intel
add661acc8e9 drm/i915/dp: Fix the enabling/disabling of audio SDP splitting
More information about the Intel-xe
mailing list