>
>
>-----Original Message-----
>From: Pin-yen Lin <[email protected]> 
>Sent: Thursday, October 17, 2024 5:52 AM
>To: Hermes Wu (吳佳宏) <[email protected]>
>Cc: Andrzej Hajda <[email protected]>; Neil Armstrong 
><[email protected]>; Robert Foss <[email protected]>; Laurent Pinchart 
><[email protected]>; Jonas Karlman <[email protected]>; Jernej 
>Skrabec <[email protected]>; Maarten Lankhorst 
><[email protected]>; Maxime Ripard <[email protected]>; 
>Thomas Zimmermann <[email protected]>; David Airlie <[email protected]>; 
>Simona Vetter <[email protected]>; AngeloGioacchino Del Regno 
><[email protected]>; [email protected]; 
>[email protected]; Kenneth Hung (洪家倫) <[email protected]>; 
>Pet Weng (翁玉芬) <[email protected]>
>Subject: Re: [PATCH v6 00/10] drm/bridge: it6505: fix HDCP CTS fail items and 
>add MCCS support
>
>Hi Hermes,
>
>On Wed, Oct 16, 2024 at 3:54 AM Hermes Wu via B4 Relay 
><[email protected]> wrote:
>>
>> This is a v6 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.
>>
>> v5->v6:
>>         -keep the commit message wrapped at 72-75 chars.
>>         -[10/10] fix return variable being used without being 
>> initialized
>>
>> 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/[email protected]/__;!!HtnLE8A!H06km5WNa8VgoR25MvWuNAGYuJGScCXQx0tQwni_NPHKwOqT4hNpaCrFZPPMVKeqBXXHTRoGj1C2pZazumE8vA$
>>  
>>         
>> https://urldefense.com/v3/__https://lore.kernel.org/all/20240926075134
>> [email protected]/__;!!HtnLE8A!H06km5WNa8VgoR25MvWuNAGYuJG
>> ScCXQx0tQwni_NPHKwOqT4hNpaCrFZPPMVKeqBXXHTRoGj1C2pZb7YFr4mw$
>>
>> 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
>>
>> Change-Id: Iad0c056d72abf2655081357cf40c3b0d3df916b5
>> Signed-off-by: Hermes Wu <[email protected]>
>> ---
>> 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(-)
>> ---
>> base-commit: b8128f7815ff135f0333c1b46dcdf1543c41b860
>> change-id: 20241015-upstream-v6-9f4b015fecf7
>>
>> Best regards,
>> --
>> Hermes Wu <[email protected]>
>>
>>
>
>I'm not sure if this requires a re-spin, but you are supposed to collect the 
>reviewed-by tags you received when you send out a new version. i.e., put the 
>"Reviewed-by:" lines you received before your "Signed-off-by:" line.

only cover-letter or all patches? need "RESEND" for same v6 patch header?

>Regards,
>Pin-yen



BR,
Hermes

Reply via email to