mbox series

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

Message ID 20241001064305.32180-1-Hermes.Wu@ite.com.tw (mailing list archive)
Headers show
Series drm/bridge: it6505: fix HDCP CTS fail items and add MCCS support | expand

Message

Hermes Wu Oct. 1, 2024, 6:42 a.m. UTC
From: Hermes Wu <Hermes.wu@ite.com.tw>

This is a v5 patch-set.

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://lore.kernel.org/all/20240926074755.22176-4-Hermes.Wu@ite.com.tw/
	https://lore.kernel.org/all/20240926075134.22394-1-Hermes.Wu@ite.com.tw/

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(-)

Comments

Dmitry Baryshkov Oct. 14, 2024, 2:05 p.m. UTC | #1
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.

> 
> 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://lore.kernel.org/all/20240926074755.22176-4-Hermes.Wu@ite.com.tw/
> 	https://lore.kernel.org/all/20240926075134.22394-1-Hermes.Wu@ite.com.tw/
> 
> 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
>
Hermes Wu Oct. 15, 2024, 10:44 a.m. UTC | #2
>
>-----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