| Message ID | 1548917996-28081-13-git-send-email-ramalingam.c@intel.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | drm/i915: Implement HDCP2.2 | expand |
>-----Original Message----- >From: C, Ramalingam >Sent: Thursday, January 31, 2019 12:29 PM >To: intel-gfx@lists.freedesktop.org; dri-devel@lists.freedesktop.org; >daniel.vetter@ffwll.ch; Winkler, Tomas <tomas.winkler@intel.com>; Shankar, >Uma <uma.shankar@intel.com> >Cc: C, Ramalingam <ramalingam.c@intel.com> >Subject: [PATCH v10 12/40] drm: HDCP2.2 link check period > >Time period for HDCP2.2 link check. > >Signed-off-by: Ramalingam C <ramalingam.c@intel.com> >Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch> Not sure if we need a separate patch for this. Could be merged where check_link is introduced for hdcp2.2. If there is a valid reasoning, no hard objection and it looks ok in general. So Reviewed-by: Uma Shankar <uma.shankar@intel.com> >--- > include/drm/drm_hdcp.h | 1 + > 1 file changed, 1 insertion(+) > >diff --git a/include/drm/drm_hdcp.h b/include/drm/drm_hdcp.h index >7260b31af276..d4e98b11b4aa 100644 >--- a/include/drm/drm_hdcp.h >+++ b/include/drm/drm_hdcp.h >@@ -13,6 +13,7 @@ > > /* Period of hdcp checks (to ensure we're still authenticated) */ > #define DRM_HDCP_CHECK_PERIOD_MS (128 * 16) >+#define DRM_HDCP2_CHECK_PERIOD_MS 500 > > /* Shared lengths/masks between HDMI/DVI/DisplayPort */ > #define DRM_HDCP_AN_LEN 8 >-- >2.7.4
On 2/4/2019 7:54 PM, Shankar, Uma wrote: > >> -----Original Message----- >> From: C, Ramalingam >> Sent: Thursday, January 31, 2019 12:29 PM >> To: intel-gfx@lists.freedesktop.org; dri-devel@lists.freedesktop.org; >> daniel.vetter@ffwll.ch; Winkler, Tomas <tomas.winkler@intel.com>; Shankar, >> Uma <uma.shankar@intel.com> >> Cc: C, Ramalingam <ramalingam.c@intel.com> >> Subject: [PATCH v10 12/40] drm: HDCP2.2 link check period >> >> Time period for HDCP2.2 link check. >> >> Signed-off-by: Ramalingam C <ramalingam.c@intel.com> >> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch> > Not sure if we need a separate patch for this. Could be merged where check_link is > introduced for hdcp2.2. > > If there is a valid reasoning, no hard objection and it looks ok in general. So drm level change is convenient to have in separate patch to get approval from dri-devel. --Ram > > Reviewed-by: Uma Shankar <uma.shankar@intel.com> > >> --- >> include/drm/drm_hdcp.h | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/include/drm/drm_hdcp.h b/include/drm/drm_hdcp.h index >> 7260b31af276..d4e98b11b4aa 100644 >> --- a/include/drm/drm_hdcp.h >> +++ b/include/drm/drm_hdcp.h >> @@ -13,6 +13,7 @@ >> >> /* Period of hdcp checks (to ensure we're still authenticated) */ >> #define DRM_HDCP_CHECK_PERIOD_MS (128 * 16) >> +#define DRM_HDCP2_CHECK_PERIOD_MS 500 >> >> /* Shared lengths/masks between HDMI/DVI/DisplayPort */ >> #define DRM_HDCP_AN_LEN 8 >> -- >> 2.7.4
diff --git a/include/drm/drm_hdcp.h b/include/drm/drm_hdcp.h index 7260b31af276..d4e98b11b4aa 100644 --- a/include/drm/drm_hdcp.h +++ b/include/drm/drm_hdcp.h @@ -13,6 +13,7 @@ /* Period of hdcp checks (to ensure we're still authenticated) */ #define DRM_HDCP_CHECK_PERIOD_MS (128 * 16) +#define DRM_HDCP2_CHECK_PERIOD_MS 500 /* Shared lengths/masks between HDMI/DVI/DisplayPort */ #define DRM_HDCP_AN_LEN 8