[Intel-gfx] [PATCH v8 12/35] drm/i915: Implement the HDCP2.2 support for DP

[C, Ramalingam]

On 11/27/2018 11:07 PM, Daniel Vetter wrote:
> On Tue, Nov 27, 2018 at 04:54:15PM +0000, Bloomfield, Jon wrote:
>> I'm not formally reviewing this series, but while glancing at it, I noticed....
>>
>>> -----Original Message-----
>>> From: Intel-gfx <intel-gfx-bounces at lists.freedesktop.org> On Behalf Of
>>> Ramalingam C
>>> Sent: Tuesday, November 27, 2018 2:43 AM
>>> To: intel-gfx at lists.freedesktop.org; dri-devel at lists.freedesktop.org;
>>> daniel.vetter at ffwll.ch; Winkler, Tomas <tomas.winkler at intel.com>
>>> Subject: [Intel-gfx] [PATCH v8 12/35] drm/i915: Implement the HDCP2.2
>>> support for DP
>>>
>>> Implements the DP adaptation specific HDCP2.2 functions.
>>>
>>> These functions perform the DPCD read and write for communicating the
>>> HDCP2.2 auth message back and forth.
>>>
>>> v2:
>>>    wait for cp_irq is merged with this patch. Rebased.
>>> v3:
>>>    wait_queue is used for wait for cp_irq [Chris Wilson]
>>> v4:
>>>    Style fixed.
>>>    %s/PARING/PAIRING
>>>    Few style fixes [Uma]
>>> v5:
>>>    Lookup table for DP HDCP2.2 msg details [Daniel].
>>>    Extra lines are removed.
>>> v6:
>>>    Rebased.
>>> v7:
>>>    Fixed some regression introduced at v5. [Ankit]
>>>    Macro HDCP_2_2_RX_CAPS_VERSION_VAL is reused [Uma]
>>>    Converted a function to inline [Uma]
>>>    %s/uintxx_t/uxx
>>> v8:
>>>    Error due to the sinks are reported as DEBUG logs.
>>>    Adjust to the new mei interface.
>>>
>>> Signed-off-by: Ramalingam C <ramalingam.c at intel.com>
>>> Signed-off-by: Ankit K Nautiyal <ankit.k.nautiyal at intel.com>
>>> Reviewed-by: Uma Shankar <uma.shankar at intel.com>
>>> ---
>>>   drivers/gpu/drm/i915/intel_dp.c   | 338
>>> ++++++++++++++++++++++++++++++++++++++
>>>   drivers/gpu/drm/i915/intel_drv.h  |   7 +
>>>   drivers/gpu/drm/i915/intel_hdcp.c |   6 +
>>>   3 files changed, 351 insertions(+)
>>>
>>> diff --git a/drivers/gpu/drm/i915/intel_dp.c
>>> b/drivers/gpu/drm/i915/intel_dp.c
>>> index ecc4706db7dc..1cc82e490999 100644
>>> --- a/drivers/gpu/drm/i915/intel_dp.c
>>> +++ b/drivers/gpu/drm/i915/intel_dp.c
>>> @@ -31,6 +31,7 @@
>>>   #include <linux/types.h>
>>>   #include <linux/notifier.h>
>>>   #include <linux/reboot.h>
>>> +#include <linux/mei_hdcp.h>
>>>   #include <asm/byteorder.h>
>>>   #include <drm/drmP.h>
>>>   #include <drm/drm_atomic_helper.h>
>>> @@ -5347,6 +5348,27 @@ void intel_dp_encoder_suspend(struct
>>> intel_encoder *intel_encoder)
>>>   	pps_unlock(intel_dp);
>>>   }
>>>
>> <SNIP>
>>> +
>>> +static struct hdcp2_dp_msg_data *get_hdcp2_dp_msg_data(u8 msg_id)
>>> +{
>>> +	int i;
>>> +
>>> +	for (i = 0; i < sizeof(hdcp2_msg_data); i++)
>> Should be
>> 	sizeof(hdcp2_msg_data) / sizeof(*hdcp2_msg_data)
>> or equivalent.
> ARRAY_SIZE is probably what you want here.

yes. missed it as authentication flow always provides the valid msg_id. 
Will fix it.

--Ram

> -Daniel
>
>> Has the failure return been tested?
>>
>>> +		if (hdcp2_msg_data[i].msg_id == msg_id)
>>> +			return &hdcp2_msg_data[i];
>>> +
>>> +	return NULL;
>>> +}
>>> +
>> <SNIP>
>>> --
>>> 2.7.4
>>>
>>> _______________________________________________
>>> Intel-gfx mailing list
>>> Intel-gfx at lists.freedesktop.org
>>> https://lists.freedesktop.org/mailman/listinfo/intel-gfx