[Intel-gfx] [PATCH] drm/i915/hdcp: Fix CP current state

* [Intel-gfx] [PATCH] drm/i915/hdcp: Fix CP current state
@ 2023-04-12  3:48 Suraj Kandpal
  2023-04-12  4:52 ` [Intel-gfx] ✓ Fi.CI.BAT: success for " Patchwork
                   ` (2 more replies)
  0 siblings, 3 replies; 6+ messages in thread
From: Suraj Kandpal @ 2023-04-12  3:48 UTC (permalink / raw)
  To: intel-gfx

In many cases there is a need to change the CP property to desired
even if not modeset has been requested yet on such situation being
lid gets closed while DP is still connected in this case HDCP
authentication needs to be reenabled.
Remove Fix me as we now send a uevent via drm_hdcp_update_property.

Signed-off-by: Suraj Kandpal <suraj.kandpal@intel.com>
---
 drivers/gpu/drm/i915/display/intel_hdcp.c | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_hdcp.c b/drivers/gpu/drm/i915/display/intel_hdcp.c
index 650232c4892b..7b0648e3499c 100644
--- a/drivers/gpu/drm/i915/display/intel_hdcp.c
+++ b/drivers/gpu/drm/i915/display/intel_hdcp.c
@@ -2564,12 +2564,9 @@ void intel_hdcp_atomic_check(struct drm_connector *connector,
 						   new_state->crtc);
 	/*
 	 * Fix the HDCP uapi content protection state in case of modeset.
-	 * FIXME: As per HDCP content protection property uapi doc, an uevent()
-	 * need to be sent if there is transition from ENABLED->DESIRED.
 	 */
-	if (drm_atomic_crtc_needs_modeset(crtc_state) &&
-	    (old_cp == DRM_MODE_CONTENT_PROTECTION_ENABLED &&
-	    new_cp != DRM_MODE_CONTENT_PROTECTION_UNDESIRED))
+	if ((old_cp == DRM_MODE_CONTENT_PROTECTION_ENABLED &&
+	     new_cp != DRM_MODE_CONTENT_PROTECTION_UNDESIRED))
 		new_state->content_protection =
 			DRM_MODE_CONTENT_PROTECTION_DESIRED;
 
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 6+ messages in thread