> > >-----Original Message----- >From: Pin-yen Lin <treapk...@chromium.org> >Sent: Thursday, October 17, 2024 5:52 AM >To: Hermes Wu (吳佳宏) <hermes...@ite.com.tw> >Cc: Andrzej Hajda <andrzej.ha...@intel.com>; Neil Armstrong ><neil.armstr...@linaro.org>; Robert Foss <rf...@kernel.org>; Laurent Pinchart ><laurent.pinch...@ideasonboard.com>; Jonas Karlman <jo...@kwiboo.se>; Jernej >Skrabec <jernej.skra...@gmail.com>; Maarten Lankhorst ><maarten.lankho...@linux.intel.com>; Maxime Ripard <mrip...@kernel.org>; >Thomas Zimmermann <tzimmerm...@suse.de>; David Airlie <airl...@gmail.com>; >Simona Vetter <sim...@ffwll.ch>; AngeloGioacchino Del Regno ><angelogioacchino.delre...@collabora.com>; dri-devel@lists.freedesktop.org; >linux-ker...@vger.kernel.org; Kenneth Hung (洪家倫) <kenneth.h...@ite.com.tw>; >Pet Weng (翁玉芬) <pet.w...@ite.com.tw> >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 ><devnull+hermes.wu.ite.com...@kernel.org> 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/20240926074755.22176-4-hermes...@ite.com.tw/__;!!HtnLE8A!H06km5WNa8VgoR25MvWuNAGYuJGScCXQx0tQwni_NPHKwOqT4hNpaCrFZPPMVKeqBXXHTRoGj1C2pZazumE8vA$ >> >> >> https://urldefense.com/v3/__https://lore.kernel.org/all/20240926075134 >> .22394-1-hermes...@ite.com.tw/__;!!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 <hermes...@ite.com.tw> >> --- >> 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 <hermes...@ite.com.tw> >> >> > >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
