RE: [PATCH v5 00/10]drm/bridge: it6505: fix HDCP CTS fail items and add MCCS support

From: Hermes.Wu
Date: Tue Oct 15 2024 - 06:45:10 EST


>
>-----Original Message-----
>From: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx>
>Sent: Monday, October 14, 2024 10:06 PM
>To: Hermes Wu (吳佳宏) <Hermes.Wu@xxxxxxxxxx>
>Cc: Pin-yen Lin <treapking@xxxxxxxxxxxx>; Kenneth Hung (洪家倫) <Kenneth.Hung@xxxxxxxxxx>; Pet Weng (翁玉芬) <Pet.Weng@xxxxxxxxxx>; Allen Chen <allen.chen@xxxxxxxxxx>; AngeloGioacchino Del Regno <angelogioacchino.delregno@xxxxxxxxxxxxx>; open list:DRM DRIVERS <dri-devel@xxxxxxxxxxxxxxxxxxxxx>; Jernej Skrabec <jernej.skrabec@xxxxxxxxx>; Jonas Karlman <jonas@xxxxxxxxx>; Laurent Pinchart <Laurent.pinchart@xxxxxxxxxxxxxxxx>; open list <linux-kernel@xxxxxxxxxxxxxxx>; Robert Foss <rfoss@xxxxxxxxxx>
>Subject: Re: [PATCH v5 00/10]drm/bridge: it6505: fix HDCP CTS fail items and add MCCS support
>
>On Tue, Oct 01, 2024 at 02:42:59PM +0800, Hermes Wu wrote:
>> From: Hermes Wu <Hermes.wu@xxxxxxxxxx>
>>
>> This is a v5 patch-set.
>
>Please check the way you are sending your patches. For some reason my
>email client lists patches 0-4 separately, patches 6-8 as a separate
>thread and patches 5, 9, 10 as individual patches. Please use single git
>send-email command to send the whole patchset.

Sending patches use git send-email via smtp server in a short period of time
reach limits.

Next patches will try to use web submission endpoint

>>
>> There are lots of failure items while running HDCP CTS using UNIGRAF DPR-100.
>> In Order to fix those failures, HDCP flow needs to be changed.
>>
>> The DisplayPort AUX protocol supports I2C transport.
>> In Order to support MCCS via the aux channel, the aux-i2c operation is added.
>>
>> v4 ->v5:
>> -add more messages for changes.
>> -[2/10] modified AUX transfer data size judgment.
>> change for-loop to do-while.
>> -[7/10] change for-loop to do-while.
>> -[9/10] change wait timer with timer_after()
>>
>> links:
>> https://urldefense.com/v3/__https://lore.kernel.org/all/20240926074755.22176-4-Hermes.Wu@xxxxxxxxxx/__;!!HtnLE8A!AHGwhLuU6axG3E60-Fc1M5cQVc_EUwgklnN1nc4YeDP3wvtVzTCo8I2tEcw-aFSDLxE-TDDjJgQHw6RYIfeJf4B4JETE$
>> https://urldefense.com/v3/__https://lore.kernel.org/all/20240926075134.22394-1-Hermes.Wu@xxxxxxxxxx/__;!!HtnLE8A!AHGwhLuU6axG3E60-Fc1M5cQVc_EUwgklnN1nc4YeDP3wvtVzTCo8I2tEcw-aFSDLxE-TDDjJgQHw6RYIfeJf1ep_d4X$
>>
>> v3 ->v4:
>> -split changes into patches.
>>
>> v2- > v3:
>> -split aux read KSV function to a patch.
>> -[1/3] new in v3
>> -[2/3] add description of patch
>>
>> v1 -> v2 :
>> - ignored.
>>
>>
>>
>> Hermes Wu (10):
>> drm/bridge: it6505: Change definition of AUX_FIFO_MAX_SIZE
>> drm/bridge: it6505: improve AUX operation for edid read
>> drm/bridge: it6505: add AUX operation for HDCP KSV list read
>> drm/bridge: it6505: Change definition MAX_HDCP_DOWN_STREAM_COUNT
>> drm/bridge: it6505: fix HDCP Bstatus check
>> drm/bridge: it6505: fix HDCP encryption when R0 ready
>> drm/bridge: it6505: fix HDCP CTS KSV list read with UNIGRAF DPR-100.
>> drm/bridge: it6505: fix HDCP CTS compare V matching
>> drm/bridge: it6505: fix HDCP CTS KSV list wait timer
>> drm/bridge: it6505: add I2C functionality on AUX
>>
>> drivers/gpu/drm/bridge/ite-it6505.c | 334 +++++++++++++++++++++++-----
>> 1 file changed, 277 insertions(+), 57 deletions(-)
>>
>> --
>> 2.34.1
>>
>
>--
>With best wishes
>Dmitry
>
>
BR,
Hermes