New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 807390 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Jan 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug
Gfx



Sign in to add a comment

re-enable HDCP logs

Project Member Reported by seanpaul@chromium.org, Jan 30 2018

Issue description

We need https://cgit.freedesktop.org/drm-misc/commit/?h=topic/hdcp&id=bb68922f100e78982515e7593de9fd23b58172d8 from upstream to re-enable HDCP logs for non-repeater setups.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Jan 31 2018

Labels: merge-merged-chromeos-4.4
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/kernel/+/46923e32ae7a35ef76a3eb4b8dfc7642e32a2945

commit 46923e32ae7a35ef76a3eb4b8dfc7642e32a2945
Author: Sean Paul <seanpaul@chromium.org>
Date: Wed Jan 31 01:47:22 2018

UPSTREAM: drm/i915: Restore HDCP DRM_INFO when with no downstream

The commit below returned earlier than before, but failed to move the
info message when authenticating without downstream devices. This patch
restores the message on authentication success.

Changes in v2:
- s/no downstream devices/no repeater present/ (Ram)

Fixes: 87eb3ec818fa ("drm/i915: II stage HDCP auth for repeater only")
Cc: Ramalingam C <ramalingam.c@intel.com>
Cc: Sean Paul <seanpaul@chromium.org>
Cc: Jani Nikula <jani.nikula@linux.intel.com>
Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
Cc: intel-gfx@lists.freedesktop.org
Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Sean Paul <seanpaul@chromium.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20180130142943.168314-1-seanpaul@chromium.org
Link: https://patchwork.freedesktop.org/patch/msgid/20180130142943.168314-1-seanpaul@chromium.org
(cherry picked from commit bb68922f100e78982515e7593de9fd23b58172d8)
Signed-off-by: Sean Paul <seanpaul@chromium.org>

BUG= 807390 
TEST=Tested upstream
Change-Id: Ia0d1beaa61240585ca789f09f7eabd9cf8851ca4
Reviewed-on: https://chromium-review.googlesource.com/893962
Commit-Ready: Sean Paul <seanpaul@google.com>
Tested-by: Sean Paul <seanpaul@google.com>
Reviewed-by: Kristian H. Kristensen <hoegsberg@chromium.org>

[modify] https://crrev.com/46923e32ae7a35ef76a3eb4b8dfc7642e32a2945/drivers/gpu/drm/i915/intel_hdcp.c

Labels: Merge-Request-65
Status: Fixed (was: Started)
Requesting merge for R65, this is a trivial patch which will help with debugging hdcp.
Cc: bhthompson@chromium.org
+bernie for visibility

Comment 4 by gov...@chromium.org, Jan 31 2018

Labels: OS-Chrome
Labels: -Merge-Request-65 Merge-Approved-65
Project Member

Comment 6 by bugdroid1@chromium.org, Jan 31 2018

Labels: merge-merged-release-R65-10323.B-chromeos-4.4
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/kernel/+/2d7de0bde20ae17f934c2a2e44cb24b6a1471dec

commit 2d7de0bde20ae17f934c2a2e44cb24b6a1471dec
Author: Sean Paul <seanpaul@chromium.org>
Date: Wed Jan 31 20:27:22 2018

UPSTREAM: drm/i915: Restore HDCP DRM_INFO when with no downstream

The commit below returned earlier than before, but failed to move the
info message when authenticating without downstream devices. This patch
restores the message on authentication success.

Changes in v2:
- s/no downstream devices/no repeater present/ (Ram)

Fixes: 87eb3ec818fa ("drm/i915: II stage HDCP auth for repeater only")
Cc: Ramalingam C <ramalingam.c@intel.com>
Cc: Sean Paul <seanpaul@chromium.org>
Cc: Jani Nikula <jani.nikula@linux.intel.com>
Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
Cc: intel-gfx@lists.freedesktop.org
Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Sean Paul <seanpaul@chromium.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20180130142943.168314-1-seanpaul@chromium.org
Link: https://patchwork.freedesktop.org/patch/msgid/20180130142943.168314-1-seanpaul@chromium.org
(cherry picked from commit bb68922f100e78982515e7593de9fd23b58172d8)
Signed-off-by: Sean Paul <seanpaul@chromium.org>


BUG= 807390 
TEST=Tested upstream
Change-Id: Ia0d1beaa61240585ca789f09f7eabd9cf8851ca4
Reviewed-on: https://chromium-review.googlesource.com/894222
Reviewed-by: Sean Paul <seanpaul@google.com>
Commit-Queue: Sean Paul <seanpaul@google.com>
Tested-by: Sean Paul <seanpaul@google.com>

[modify] https://crrev.com/2d7de0bde20ae17f934c2a2e44cb24b6a1471dec/drivers/gpu/drm/i915/intel_hdcp.c

Project Member

Comment 7 by sheriffbot@chromium.org, Feb 12 2018

Cc: bhthompson@google.com
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 8 by sheriffbot@chromium.org, Feb 16 2018

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Sign in to add a comment