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

[<Hermes.Wu@ite.com.tw>]

>
>-----Original Message-----
>From: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> 
>Sent: Monday, October 14, 2024 10:06 PM
>To: Hermes Wu (吳佳宏) <Hermes.Wu@ite.com.tw>
>Cc: Pin-yen Lin <treapking@chromium.org>; Kenneth Hung (洪家倫) <Kenneth.Hung@ite.com.tw>; Pet Weng (翁玉芬) <Pet.Weng@ite.com.tw>; Allen Chen <allen.chen@ite.com.tw>; AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>; open list:DRM DRIVERS <dri-devel@lists.freedesktop.org>; Jernej Skrabec <jernej.skrabec@gmail.com>; Jonas Karlman <jonas@kwiboo.se>; Laurent Pinchart <Laurent.pinchart@ideasonboard.com>; open list <linux-kernel@vger.kernel.org>; Robert Foss <rfoss@kernel.org>
>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@ite.com.tw>
>> 
>> 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@ite.com.tw/__;!!HtnLE8A!AHGwhLuU6axG3E60-Fc1M5cQVc_EUwgklnN1nc4YeDP3wvtVzTCo8I2tEcw-aFSDLxE-TDDjJgQHw6RYIfeJf4B4JETE$ 
>> 	https://urldefense.com/v3/__https://lore.kernel.org/all/20240926075134.22394-1-Hermes.Wu@ite.com.tw/__;!!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