Hi, >Hi, > >On Wed, Oct 16, 2024 at 10:10 PM <[email protected]> wrote: >> >> > >> > >> >-----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/20240926074 >> >> [email protected]/__;!!HtnLE8A!H06km5WNa8VgoR25MvWuN >> >> AGYuJGScCXQx0tQwni_NPHKwOqT4hNpaCrFZPPMVKeqBXXHTRoGj1C2pZazumE8vA$ >> >> >> >> https://urldefense.com/v3/__https://lore.kernel.org/all/20240926075 >> >> 134 >> >> [email protected]/__;!!HtnLE8A!H06km5WNa8VgoR25MvWuNAGY >> >> uJG 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? > >You should add the review tags to the individual patches that have not changed >since the previous review. For this series, I think you can wait a few days >and collect feedbacks from the community first. >>
ok, thanks. >> >Regards, >> >Pin-yen >> >> >> >> BR, >> Hermes > >Pin-yen > BR, Hermes
