Re: [PATCH i-g-t v1 0/3] Revert HDCP per crtc infrastructure

[Jani Nikula <jani.nikula@intel.com>]

On Thu, 14 May 2026, Santhosh Reddy Guddati <santhosh.reddy.guddati@intel.com> wrote:
> Reverts below commits
>
> b4036b1a0 tests/kms_content_protection: Add FB cleanup for MST tests
> 8d7c2453d tests/kms_content_protection: Pass crtc parameter and use per-CRTC FBs
> 925b14d43 tests/kms_content_protection: Add per-CRTC framebuffer infrastructure
>
> The per-CRTC fb introduced in the commits caused regression especially
> when the eDP panel is connected with 2 4k Panels connected through a
> DP-MST hub with "-ENOSPC" and all the content protection tests are
> failing.
>
> The MST subtests are skipping when subtests are executed with "--r
> dp-mst*" , as the MST changes introduced did not clean up properly
> across subtests.
>
> The create_fbs() now iterated through all crtcs and eDP (2k) claims the first
> crtc before the larger output and fb is created with smaller size. Later when DP-4 maps to
> the smae CRTC (it has differnt fb dimensions now 2k fb than the mode
> required) , the kernel rejects the commits  with ENOSPC because of
> invalid coordinates.

The cover letter won't be committed to git.

The commit messages for the reverts themselves need the explanation for
posterity.

You can't have empty commit messages that just have the "This reverts
commit ..."

BR,
Jani.

>
> Santhosh Reddy Guddati (3):
>   Revert "tests/kms_content_protection: Add per-CRTC framebuffer
>     infrastructure"
>   Revert "tests/kms_content_protection: Pass crtc parameter and use
>     per-CRTC FBs"
>   Revert "tests/kms_content_protection: Add FB cleanup for MST tests"
>
>  tests/kms_content_protection.c | 168 +++++++++------------------------
>  1 file changed, 47 insertions(+), 121 deletions(-)

-- 
Jani Nikula, Intel