From: Ramalingam C <ramalingam.c@intel.com>
To: Sean Paul <seanpaul@chromium.org>
Cc: daniel.vetter@ffwll.ch, intel-gfx@lists.freedesktop.org,
tomas.winkler@intel.com, rodrigo.vivi@intel.com
Subject: Re: [PATCH 23/43] drm/i915: wrapping all hdcp var into intel_hdcp
Date: Mon, 26 Feb 2018 11:35:41 +0530 [thread overview]
Message-ID: <b328f2e4-c23a-df2f-b808-c8242d956133@intel.com> (raw)
In-Reply-To: <20180222144739.GB223881@art_vandelay>
On Thursday 22 February 2018 08:17 PM, Sean Paul wrote:
> On Wed, Feb 14, 2018 at 07:43:38PM +0530, Ramalingam C wrote:
>> Considering the upcoming significant no HDCP2.2 variables, it will
>> be clean to have separate struct fo HDCP.
>>
>> New structure called intel_hdcp is introduced.
>>
>> Signed-off-by: Ramalingam C <ramalingam.c@intel.com>
>> ---
>> drivers/gpu/drm/i915/intel_display.c | 9 ++-
>> drivers/gpu/drm/i915/intel_dp.c | 4 +-
>> drivers/gpu/drm/i915/intel_drv.h | 20 +++--
>> drivers/gpu/drm/i915/intel_hdcp.c | 141 ++++++++++++++++++++---------------
>> 4 files changed, 100 insertions(+), 74 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
>> index 048d60b5143b..22097349529b 100644
>> --- a/drivers/gpu/drm/i915/intel_display.c
>> +++ b/drivers/gpu/drm/i915/intel_display.c
>> @@ -15314,10 +15314,11 @@ static void intel_hpd_poll_fini(struct drm_device *dev)
>> for_each_intel_connector_iter(connector, &conn_iter) {
>> if (connector->modeset_retry_work.func)
>> cancel_work_sync(&connector->modeset_retry_work);
>> - if (connector->hdcp_shim) {
>> - cancel_delayed_work_sync(&connector->hdcp_check_work);
>> - cancel_work_sync(&connector->hdcp_prop_work);
>> - cancel_work_sync(&connector->hdcp_enable_work);
>> + if (connector->hdcp && connector->hdcp->hdcp_shim) {
>> + cancel_delayed_work_sync(
>> + &connector->hdcp->hdcp_check_work);
>> + cancel_work_sync(&connector->hdcp->hdcp_prop_work);
>> + cancel_work_sync(&connector->hdcp->hdcp_enable_work);
>> }
>> }
>> drm_connector_list_iter_end(&conn_iter);
>> diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c
>> index f20b25f98e5a..80476689754f 100644
>> --- a/drivers/gpu/drm/i915/intel_dp.c
>> +++ b/drivers/gpu/drm/i915/intel_dp.c
>> @@ -5445,7 +5445,9 @@ intel_dp_hpd_pulse(struct intel_digital_port *intel_dig_port, bool long_hpd)
>> WARN(iret, "Acquiring modeset locks failed with %i\n", iret);
>>
>> /* Short pulse can signify loss of hdcp authentication */
>> - intel_hdcp_check_link(intel_dp->attached_connector);
>> + if (intel_dp->attached_connector->hdcp)
>> + intel_hdcp_check_link(
>> + intel_dp->attached_connector->hdcp);
>>
>> if (!handled) {
>> intel_dp->detect_done = false;
>> diff --git a/drivers/gpu/drm/i915/intel_drv.h b/drivers/gpu/drm/i915/intel_drv.h
>> index 7b9e5f70826f..5b170ff7ec14 100644
>> --- a/drivers/gpu/drm/i915/intel_drv.h
>> +++ b/drivers/gpu/drm/i915/intel_drv.h
>> @@ -375,6 +375,17 @@ struct intel_hdcp_shim {
>> bool *hdcp_capable);
>> };
>>
>> +struct intel_hdcp {
>> + struct intel_connector *connector;
> You don't need a backpointer, use container_of if necessary.
ok sure.
>
>> +
>> + const struct intel_hdcp_shim *hdcp_shim;
>> + struct mutex hdcp_mutex;
>> + uint64_t hdcp_value; /* protected by hdcp_mutex */
>> + struct delayed_work hdcp_check_work;
>> + struct work_struct hdcp_prop_work;
>> + struct work_struct hdcp_enable_work;
>> +};
>> +
>> struct intel_connector {
>> struct drm_connector base;
>> /*
>> @@ -407,12 +418,7 @@ struct intel_connector {
>> /* Work struct to schedule a uevent on link train failure */
>> struct work_struct modeset_retry_work;
>>
>> - const struct intel_hdcp_shim *hdcp_shim;
>> - struct mutex hdcp_mutex;
>> - uint64_t hdcp_value; /* protected by hdcp_mutex */
>> - struct delayed_work hdcp_check_work;
>> - struct work_struct hdcp_prop_work;
>> - struct work_struct hdcp_enable_work;
>> + struct intel_hdcp *hdcp;
>> };
>>
>> struct intel_digital_connector_state {
>> @@ -1853,7 +1859,7 @@ int intel_hdcp_init(struct intel_connector *connector,
>> const struct intel_hdcp_shim *hdcp_shim);
>> int intel_hdcp_enable(struct intel_connector *connector);
>> int intel_hdcp_disable(struct intel_connector *connector);
>> -int intel_hdcp_check_link(struct intel_connector *connector);
>> +int intel_hdcp_check_link(struct intel_hdcp *hdcp);
>> bool is_hdcp_supported(struct drm_i915_private *dev_priv, enum port port);
>>
>> /* intel_psr.c */
>> diff --git a/drivers/gpu/drm/i915/intel_hdcp.c b/drivers/gpu/drm/i915/intel_hdcp.c
>> index e03bd376d92c..9147fb17a9fc 100644
>> --- a/drivers/gpu/drm/i915/intel_hdcp.c
>> +++ b/drivers/gpu/drm/i915/intel_hdcp.c
>> @@ -545,8 +545,9 @@ struct intel_digital_port *conn_to_dig_port(struct intel_connector *connector)
>> return enc_to_dig_port(&intel_attached_encoder(&connector->base)->base);
>> }
>>
>> -static int _intel_hdcp_disable(struct intel_connector *connector)
>> +static int _intel_hdcp_disable(struct intel_hdcp *hdcp)
>> {
>> + struct intel_connector *connector = hdcp->connector;
> I'm not sure why we need to change all of the function arguments to intel_hdcp
> if the first thing we do is fetch the connector.
>
>> struct drm_i915_private *dev_priv = connector->base.dev->dev_private;
>> struct intel_digital_port *intel_dig_port = conn_to_dig_port(connector);
>> enum port port = intel_dig_port->base.port;
>> @@ -562,7 +563,7 @@ static int _intel_hdcp_disable(struct intel_connector *connector)
>> return -ETIMEDOUT;
>> }
>>
>> - ret = connector->hdcp_shim->toggle_signalling(intel_dig_port, false);
>> + ret = hdcp->hdcp_shim->toggle_signalling(intel_dig_port, false);
>> if (ret) {
>> DRM_ERROR("Failed to disable HDCP signalling\n");
>> return ret;
>> @@ -572,8 +573,9 @@ static int _intel_hdcp_disable(struct intel_connector *connector)
>> return 0;
>> }
>>
>> -static int _intel_hdcp_enable(struct intel_connector *connector)
>> +static int _intel_hdcp_enable(struct intel_hdcp *hdcp)
>> {
>> + struct intel_connector *connector = hdcp->connector;
>> struct drm_i915_private *dev_priv = connector->base.dev->dev_private;
>> int i, ret, tries = 3;
>>
>> @@ -599,12 +601,12 @@ static int _intel_hdcp_enable(struct intel_connector *connector)
>> /* Incase of authentication failures, HDCP spec expects reauth. */
>> for (i = 0; i < tries; i++) {
>> ret = intel_hdcp_auth(conn_to_dig_port(connector),
>> - connector->hdcp_shim);
>> + hdcp->hdcp_shim);
>> if (!ret) {
>> - connector->hdcp_value =
>> + hdcp->hdcp_value =
>> DRM_MODE_CONTENT_PROTECTION_ENABLED;
>> - schedule_work(&connector->hdcp_prop_work);
>> - schedule_delayed_work(&connector->hdcp_check_work,
>> + schedule_work(&hdcp->hdcp_prop_work);
>> + schedule_delayed_work(&hdcp->hdcp_check_work,
>> DRM_HDCP_CHECK_PERIOD_MS);
>> return 0;
>> }
>> @@ -612,7 +614,7 @@ static int _intel_hdcp_enable(struct intel_connector *connector)
>> DRM_DEBUG_KMS("HDCP Auth failure (%d)\n", ret);
>>
>> /* Ensuring HDCP encryption and signalling are stopped. */
>> - _intel_hdcp_disable(connector);
>> + _intel_hdcp_disable(hdcp);
>> }
>>
>> DRM_ERROR("HDCP authentication failed (%d tries/%d)\n", tries, ret);
>> @@ -621,47 +623,45 @@ static int _intel_hdcp_enable(struct intel_connector *connector)
>>
>> static void intel_hdcp_enable_work(struct work_struct *work)
>> {
>> - struct intel_connector *connector = container_of(work,
>> - struct intel_connector,
>> - hdcp_enable_work);
>> + struct intel_hdcp *hdcp = container_of(work, struct intel_hdcp,
>> + hdcp_enable_work);
>>
>> - mutex_lock(&connector->hdcp_mutex);
>> - _intel_hdcp_enable(connector);
>> - mutex_unlock(&connector->hdcp_mutex);
>> + mutex_lock(&hdcp->hdcp_mutex);
>> + _intel_hdcp_enable(hdcp);
>> + mutex_unlock(&hdcp->hdcp_mutex);
>> }
>>
>> static void intel_hdcp_check_work(struct work_struct *work)
>> {
>> - struct intel_connector *connector = container_of(to_delayed_work(work),
>> - struct intel_connector,
>> - hdcp_check_work);
>> - if (!intel_hdcp_check_link(connector))
>> - schedule_delayed_work(&connector->hdcp_check_work,
>> + struct intel_hdcp *hdcp = container_of(to_delayed_work(work),
>> + struct intel_hdcp,
>> + hdcp_check_work);
>> + if (!intel_hdcp_check_link(hdcp))
>> + schedule_delayed_work(&hdcp->hdcp_check_work,
>> DRM_HDCP_CHECK_PERIOD_MS);
>> }
>>
>> static void intel_hdcp_prop_work(struct work_struct *work)
>> {
>> - struct intel_connector *connector = container_of(work,
>> - struct intel_connector,
>> - hdcp_prop_work);
>> - struct drm_device *dev = connector->base.dev;
>> + struct intel_hdcp *hdcp = container_of(work, struct intel_hdcp,
>> + hdcp_prop_work);
>> + struct drm_device *dev = hdcp->connector->base.dev;
>> struct drm_connector_state *state;
>>
>> drm_modeset_lock(&dev->mode_config.connection_mutex, NULL);
>> - mutex_lock(&connector->hdcp_mutex);
>> + mutex_lock(&hdcp->hdcp_mutex);
>>
>> /*
>> * This worker is only used to flip between ENABLED/DESIRED. Either of
>> * those to UNDESIRED is handled by core. If hdcp_value == UNDESIRED,
>> * we're running just after hdcp has been disabled, so just exit
>> */
>> - if (connector->hdcp_value != DRM_MODE_CONTENT_PROTECTION_UNDESIRED) {
>> - state = connector->base.state;
>> - state->content_protection = connector->hdcp_value;
>> + if (hdcp->hdcp_value != DRM_MODE_CONTENT_PROTECTION_UNDESIRED) {
>> + state = hdcp->connector->base.state;
>> + state->content_protection = hdcp->hdcp_value;
>> }
>>
>> - mutex_unlock(&connector->hdcp_mutex);
>> + mutex_unlock(&hdcp->hdcp_mutex);
>> drm_modeset_unlock(&dev->mode_config.connection_mutex);
>> }
>>
>> @@ -676,48 +676,64 @@ int intel_hdcp_init(struct intel_connector *connector,
>> const struct intel_hdcp_shim *hdcp_shim)
>> {
>> int ret;
>> + struct intel_hdcp *hdcp;
>> +
>> + hdcp = kzalloc(sizeof(struct intel_hdcp), GFP_KERNEL);
> You're leaking this. You probably don't need it to be dynamically allocated.
didn't want the mem spared for struct hdcp, incase of non hdcp capable
connectors.
May be its ok to have small struct for all connectors. i will move it
from dynamic alloc.
>
>> + if (!hdcp) {
>> + ret = -ENOMEM;
>> + goto err;
>> + }
>>
>> ret = drm_connector_attach_content_protection_property(
>> &connector->base);
>> if (ret)
>> - return ret;
>> + goto err;
>> +
>> + hdcp->hdcp_shim = hdcp_shim;
>> + mutex_init(&hdcp->hdcp_mutex);
>> + INIT_DELAYED_WORK(&hdcp->hdcp_check_work, intel_hdcp_check_work);
>> + INIT_WORK(&hdcp->hdcp_prop_work, intel_hdcp_prop_work);
>> + INIT_WORK(&hdcp->hdcp_enable_work, intel_hdcp_enable_work);
>>
>> - connector->hdcp_shim = hdcp_shim;
>> - mutex_init(&connector->hdcp_mutex);
>> - INIT_DELAYED_WORK(&connector->hdcp_check_work, intel_hdcp_check_work);
>> - INIT_WORK(&connector->hdcp_prop_work, intel_hdcp_prop_work);
>> - INIT_WORK(&connector->hdcp_enable_work, intel_hdcp_enable_work);
>> + connector->hdcp = hdcp;
>> return 0;
>> +
>> +err:
>> + kfree(hdcp);
>> + return ret;
>> }
>>
>> int intel_hdcp_enable(struct intel_connector *connector)
>> {
>> - if (!connector->hdcp_shim)
>> + struct intel_hdcp *hdcp = connector->hdcp;
>> +
>> + if (!hdcp || !hdcp->hdcp_shim)
> Is it possible for hdcp to be != NULL, but hdcp_shim == NULL?
Along with hdcp2.2, this could be a possibility. Anyway by moving to
static allocation for hdcp, no more check for hdcp ptr :)
--Ram
>
>> return -ENOENT;
>>
>> - mutex_lock(&connector->hdcp_mutex);
>> - schedule_work(&connector->hdcp_enable_work);
>> - mutex_unlock(&connector->hdcp_mutex);
>> + mutex_lock(&hdcp->hdcp_mutex);
>> + schedule_work(&hdcp->hdcp_enable_work);
>> + mutex_unlock(&hdcp->hdcp_mutex);
>>
>> return 0;
>> }
>>
>> int intel_hdcp_disable(struct intel_connector *connector)
>> {
>> + struct intel_hdcp *hdcp = connector->hdcp;
>> int ret = 0;
>>
>> - if (!connector->hdcp_shim)
>> + if (!hdcp || !hdcp->hdcp_shim)
>> return -ENOENT;
>>
>> - mutex_lock(&connector->hdcp_mutex);
>> + mutex_lock(&hdcp->hdcp_mutex);
>>
>> - if (connector->hdcp_value != DRM_MODE_CONTENT_PROTECTION_UNDESIRED) {
>> - connector->hdcp_value = DRM_MODE_CONTENT_PROTECTION_UNDESIRED;
>> - ret = _intel_hdcp_disable(connector);
>> + if (hdcp->hdcp_value != DRM_MODE_CONTENT_PROTECTION_UNDESIRED) {
>> + hdcp->hdcp_value = DRM_MODE_CONTENT_PROTECTION_UNDESIRED;
>> + ret = _intel_hdcp_disable(hdcp);
>> }
>>
>> - mutex_unlock(&connector->hdcp_mutex);
>> - cancel_delayed_work_sync(&connector->hdcp_check_work);
>> + mutex_unlock(&hdcp->hdcp_mutex);
>> + cancel_delayed_work_sync(&hdcp->hdcp_check_work);
>> return ret;
>> }
>>
>> @@ -755,19 +771,20 @@ void intel_hdcp_atomic_check(struct drm_connector *connector,
>> }
>>
>> /* Implements Part 3 of the HDCP authorization procedure */
>> -int intel_hdcp_check_link(struct intel_connector *connector)
>> +int intel_hdcp_check_link(struct intel_hdcp *hdcp)
>> {
>> + struct intel_connector *connector = hdcp->connector;
>> struct drm_i915_private *dev_priv = connector->base.dev->dev_private;
>> struct intel_digital_port *intel_dig_port = conn_to_dig_port(connector);
>> enum port port = intel_dig_port->base.port;
>> int ret = 0;
>>
>> - if (!connector->hdcp_shim)
>> + if (!hdcp->hdcp_shim)
>> return -ENOENT;
>>
>> - mutex_lock(&connector->hdcp_mutex);
>> + mutex_lock(&hdcp->hdcp_mutex);
>>
>> - if (connector->hdcp_value == DRM_MODE_CONTENT_PROTECTION_UNDESIRED)
>> + if (hdcp->hdcp_value == DRM_MODE_CONTENT_PROTECTION_UNDESIRED)
>> goto out;
>>
>> if (!(I915_READ(PORT_HDCP_STATUS(port)) & HDCP_STATUS_ENC)) {
>> @@ -775,17 +792,17 @@ int intel_hdcp_check_link(struct intel_connector *connector)
>> connector->base.name, connector->base.base.id,
>> I915_READ(PORT_HDCP_STATUS(port)));
>> ret = -ENXIO;
>> - connector->hdcp_value = DRM_MODE_CONTENT_PROTECTION_DESIRED;
>> - schedule_work(&connector->hdcp_prop_work);
>> + hdcp->hdcp_value = DRM_MODE_CONTENT_PROTECTION_DESIRED;
>> + schedule_work(&hdcp->hdcp_prop_work);
>> goto out;
>> }
>>
>> - if (connector->hdcp_shim->check_link(intel_dig_port)) {
>> - if (connector->hdcp_value !=
>> + if (hdcp->hdcp_shim->check_link(intel_dig_port)) {
>> + if (hdcp->hdcp_value !=
>> DRM_MODE_CONTENT_PROTECTION_UNDESIRED) {
>> - connector->hdcp_value =
>> + hdcp->hdcp_value =
>> DRM_MODE_CONTENT_PROTECTION_ENABLED;
>> - schedule_work(&connector->hdcp_prop_work);
>> + schedule_work(&hdcp->hdcp_prop_work);
>> }
>> goto out;
>> }
>> @@ -793,23 +810,23 @@ int intel_hdcp_check_link(struct intel_connector *connector)
>> DRM_DEBUG_KMS("[%s:%d] HDCP link failed, retrying authentication\n",
>> connector->base.name, connector->base.base.id);
>>
>> - ret = _intel_hdcp_disable(connector);
>> + ret = _intel_hdcp_disable(hdcp);
>> if (ret) {
>> DRM_ERROR("Failed to disable hdcp (%d)\n", ret);
>> - connector->hdcp_value = DRM_MODE_CONTENT_PROTECTION_DESIRED;
>> - schedule_work(&connector->hdcp_prop_work);
>> + hdcp->hdcp_value = DRM_MODE_CONTENT_PROTECTION_DESIRED;
>> + schedule_work(&hdcp->hdcp_prop_work);
>> goto out;
>> }
>>
>> - ret = _intel_hdcp_enable(connector);
>> + ret = _intel_hdcp_enable(hdcp);
>> if (ret) {
>> DRM_ERROR("Failed to enable hdcp (%d)\n", ret);
>> - connector->hdcp_value = DRM_MODE_CONTENT_PROTECTION_DESIRED;
>> - schedule_work(&connector->hdcp_prop_work);
>> + hdcp->hdcp_value = DRM_MODE_CONTENT_PROTECTION_DESIRED;
>> + schedule_work(&hdcp->hdcp_prop_work);
>> goto out;
>> }
>>
>> out:
>> - mutex_unlock(&connector->hdcp_mutex);
>> + mutex_unlock(&hdcp->hdcp_mutex);
>> return ret;
>> }
>> --
>> 2.7.4
>>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
next prev parent reply other threads:[~2018-02-26 6:11 UTC|newest]
Thread overview: 79+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-02-14 14:13 [PATCH 00/43] drm/i915: Implement HDCP2.2 Ramalingam C
2018-02-14 14:13 ` [PATCH 01/43] drm: hdcp2.2 authentication msg definitions Ramalingam C
2018-02-14 15:15 ` Winkler, Tomas
2018-02-14 19:40 ` Jani Nikula
2018-02-22 5:24 ` Ramalingam C
2018-02-22 5:29 ` Ramalingam C
2018-02-14 14:13 ` [PATCH 02/43] drm: HDMI and DP specific HDCP2.2 defines Ramalingam C
2018-02-14 14:13 ` [PATCH 03/43] mei: bus: whitelist hdcp client Ramalingam C
2018-02-14 14:13 ` [PATCH 04/43] mei: me: add gemini lake devices ids Ramalingam C
2018-02-14 14:45 ` Winkler, Tomas
2018-02-22 5:22 ` Ramalingam C
2018-02-14 14:13 ` [PATCH 05/43] misc/mei/hdcp: Client driver for HDCP application Ramalingam C
2018-02-14 14:54 ` Winkler, Tomas
2018-02-14 14:57 ` Ramalingam C
2018-02-14 14:13 ` [PATCH 06/43] misc/mei/hdcp: Add KBuild for mei hdcp driver Ramalingam C
2018-02-14 14:54 ` Winkler, Tomas
2018-02-14 14:58 ` Ramalingam C
2018-02-28 15:43 ` Ramalingam C
2018-02-28 16:11 ` Winkler, Tomas
2018-02-14 14:13 ` [PATCH 07/43] misc/mei/hdcp: Verify mei client device status Ramalingam C
2018-02-14 14:13 ` [PATCH 08/43] misc/mei/hdcp: Get & Put for mei cl_device Ramalingam C
2018-02-14 14:13 ` [PATCH 09/43] misc/mei/hdcp: Define ME FW interface for HDCP2.2 Ramalingam C
2018-02-14 14:13 ` [PATCH 10/43] linux/mei: Header for mei_hdcp driver interface Ramalingam C
2018-02-14 14:13 ` [PATCH 11/43] misc/mei/hdcp: Initiate Wired HDCP2.2 Tx Session Ramalingam C
2018-02-14 14:13 ` [PATCH 12/43] misc/mei/hdcp: Verify Receiver Cert and prepare km Ramalingam C
2018-02-14 14:13 ` [PATCH 13/43] misc/mei/hdcp: Verify H_prime Ramalingam C
2018-02-14 14:13 ` [PATCH 14/43] misc/mei/hdcp: Store the HDCP Pairing info Ramalingam C
2018-02-14 14:13 ` [PATCH 15/43] misc/mei/hdcp: Initiate Locality check Ramalingam C
2018-02-14 14:13 ` [PATCH 16/43] misc/mei/hdcp: Verify L_prime Ramalingam C
2018-02-14 14:13 ` [PATCH 17/43] misc/mei/hdcp: Prepare Session Key Ramalingam C
2018-02-14 14:13 ` [PATCH 18/43] misc/mei/hdcp: Repeater topology verifcation and ack Ramalingam C
2018-02-14 14:13 ` [PATCH 19/43] misc/mei/hdcp: Verify M_prime Ramalingam C
2018-02-14 14:13 ` [PATCH 20/43] misc/mei/hdcp: Enabling the HDCP authentication Ramalingam C
2018-02-14 14:13 ` [PATCH 21/43] misc/mei/hdcp: Closing wired HDCP2.2 Tx Session Ramalingam C
2018-02-14 14:13 ` [PATCH 22/43] drm/i915: Async execution of hdcp authentication Ramalingam C
2018-02-22 14:39 ` Sean Paul
2018-02-26 6:32 ` Ramalingam C
2018-02-14 14:13 ` [PATCH 23/43] drm/i915: wrapping all hdcp var into intel_hdcp Ramalingam C
2018-02-22 14:47 ` Sean Paul
2018-02-26 6:05 ` Ramalingam C [this message]
2018-02-14 14:13 ` [PATCH 24/43] drm/i915: wait for cp_irq Ramalingam C
2018-02-22 15:46 ` Sean Paul
2018-02-26 5:49 ` Ramalingam C
2018-02-14 14:13 ` [PATCH 25/43] drm/i915: Define HDCP2.2 related variables Ramalingam C
2018-02-14 14:36 ` Chris Wilson
2018-02-14 14:56 ` Ramalingam C
2018-02-22 14:59 ` Sean Paul
2018-02-26 5:46 ` Ramalingam C
2018-02-14 14:13 ` [PATCH 26/43] drm/i915: Define Intel HDCP2.2 registers Ramalingam C
2018-02-22 15:43 ` Sean Paul
2018-02-26 5:33 ` Ramalingam C
2018-02-14 14:13 ` [PATCH 27/43] drm/i915: Wrappers for mei HDCP2.2 services Ramalingam C
2018-02-14 14:13 ` [PATCH 28/43] drm/i915: Implement HDCP2.2 receiver authentication Ramalingam C
2018-02-14 14:13 ` [PATCH 29/43] drm/i915: Implement HDCP2.2 repeater authentication Ramalingam C
2018-02-14 14:13 ` [PATCH 30/43] drm/i915: Enable and Disable HDCP2.2 port encryption Ramalingam C
2018-02-14 14:38 ` Chris Wilson
2018-02-14 14:52 ` Ramalingam C
2018-02-14 14:13 ` [PATCH 31/43] drm/i915: Implement HDCP2.2 En/Dis-able Ramalingam C
2018-02-14 14:13 ` [PATCH 32/43] drm/i915: Implement HDCP2.2 link integrity check Ramalingam C
2018-02-14 14:13 ` [PATCH 33/43] drm/i915: Handle HDCP2.2 downstream topology change Ramalingam C
2018-02-14 14:13 ` [PATCH 34/43] drm/i915: Pullout the bksv read and validation Ramalingam C
2018-02-14 14:13 ` [PATCH 35/43] drm/i915: Enable HDCP version that is best capable Ramalingam C
2018-02-14 14:42 ` Chris Wilson
2018-02-14 14:51 ` Ramalingam C
2018-02-14 15:00 ` Chris Wilson
2018-02-14 15:00 ` Ramalingam C
2018-02-14 14:13 ` [PATCH 36/43] drm/i915: Enable HDCP1.4 incase of HDCP2.2 failure Ramalingam C
2018-02-14 14:13 ` [PATCH 37/43] drm/i915: Initialize HDCP2.2 and its MEI interface Ramalingam C
2018-02-14 14:45 ` Chris Wilson
2018-02-14 14:45 ` Ramalingam C
2018-02-14 14:13 ` [PATCH 38/43] drm/i915: Implement gmbus burst read Ramalingam C
2018-02-14 14:13 ` [PATCH 39/43] drm/i915: Implement the HDCP2.2 support for DP Ramalingam C
2018-02-14 14:13 ` [PATCH 40/43] drm/i915: Implement the HDCP2.2 support for HDMI Ramalingam C
2018-02-14 14:13 ` [PATCH 41/43] drm/i915: Add HDCP2.2 support for DP connectors Ramalingam C
2018-02-14 14:13 ` [PATCH 42/43] drm/i915: Add HDCP2.2 support for HDMI connectors Ramalingam C
2018-02-14 14:13 ` [PATCH 43/43] drm/i915: Invoke check link on CP_IRQ of DP Ramalingam C
2018-02-14 15:07 ` ✗ Fi.CI.CHECKPATCH: warning for drm/i915: Implement HDCP2.2 Patchwork
2018-02-14 15:13 ` ✗ Fi.CI.SPARSE: " Patchwork
2018-02-14 15:20 ` ✗ Fi.CI.BAT: failure " Patchwork
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=b328f2e4-c23a-df2f-b808-c8242d956133@intel.com \
--to=ramalingam.c@intel.com \
--cc=daniel.vetter@ffwll.ch \
--cc=intel-gfx@lists.freedesktop.org \
--cc=rodrigo.vivi@intel.com \
--cc=seanpaul@chromium.org \
--cc=tomas.winkler@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox