From: <Hermes.Wu@ite.com.tw>
To: <dmitry.baryshkov@linaro.org>
Cc: <andrzej.hajda@intel.com>, <neil.armstrong@linaro.org>,
<rfoss@kernel.org>, <Laurent.pinchart@ideasonboard.com>,
<jonas@kwiboo.se>, <jernej.skrabec@gmail.com>,
<maarten.lankhorst@linux.intel.com>, <mripard@kernel.org>,
<tzimmermann@suse.de>, <airlied@gmail.com>, <simona@ffwll.ch>,
<angelogioacchino.delregno@collabora.com>,
<treapking@chromium.org>, <dri-devel@lists.freedesktop.org>,
<linux-kernel@vger.kernel.org>, <Kenneth.Hung@ite.com.tw>,
<Pet.Weng@ite.com.tw>
Subject: RE: [PATCH v6 00/10] drm/bridge: it6505: fix HDCP CTS fail items and add MCCS support
Date: Mon, 21 Oct 2024 06:33:09 +0000 [thread overview]
Message-ID: <2302452d3a6e446d9d288241958ec086@ite.com.tw> (raw)
In-Reply-To: <e6ksqhssu55ox4ffzyko7fsahvk6i6aoj4jo2rbe2gv6sclvzp@elpidkcflh3s>
hi
>-----Original Message-----
>From: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
>Sent: Sunday, October 20, 2024 9:57 PM
>To: Hermes Wu (吳佳宏) <Hermes.Wu@ite.com.tw>
>Cc: Andrzej Hajda <andrzej.hajda@intel.com>; Neil Armstrong <neil.armstrong@linaro.org>; Robert Foss <rfoss@kernel.org>; Laurent Pinchart <Laurent.pinchart@ideasonboard.com>; Jonas Karlman <jonas@kwiboo.se>; Jernej Skrabec <jernej.skrabec@gmail.com>; Maarten Lankhorst <maarten.lankhorst@linux.intel.com>; Maxime Ripard <mripard@kernel.org>; Thomas Zimmermann <tzimmermann@suse.de>; David Airlie <airlied@gmail.com>; Simona Vetter <simona@ffwll.ch>; AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>; Pin-yen Lin <treapking@chromium.org>; dri-devel@lists.freedesktop.org; linux-kernel@vger.kernel.org; Kenneth Hung (洪家倫) <Kenneth.Hung@ite.com.tw>; Pet Weng (翁玉芬) <Pet.Weng@ite.com.tw>
>Subject: Re: [PATCH v6 00/10] drm/bridge: it6505: fix HDCP CTS fail items and add MCCS support
>
>On Wed, Oct 16, 2024 at 03:54:12PM +0800, Hermes Wu via B4 Relay 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
>
>I have reviewed the patches that I could review, but I lack deep HDCP knowledge, so I can not review patches 5, 6, 8, 9. Hopefully somebody else can step up.
>
thank you.
>> 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!HOwy05_egTbj5U7vrjfPhJC21VK_Jxll8UWNOcGEzNSDxnKQjE6QSLhamd-IROE-SxGOrzFS_SLfixtORjLLBTjdL3SV$
>>
>> https://urldefense.com/v3/__https://lore.kernel.org/all/20240926075134
>> .22394-1-Hermes.Wu@ite.com.tw/__;!!HtnLE8A!HOwy05_egTbj5U7vrjfPhJC21VK
>> _Jxll8UWNOcGEzNSDxnKQjE6QSLhamd-IROE-SxGOrzFS_SLfixtORjLLBTc_Yg8R$
>>
>> 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.wu@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.wu@ite.com.tw>
>>
>>
>
>--
>With best wishes
>Dmitry
>
BR,
Hermes
prev parent reply other threads:[~2024-10-21 6:33 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-10-16 7:54 [PATCH v6 00/10] drm/bridge: it6505: fix HDCP CTS fail items and add MCCS support Hermes Wu via B4 Relay
2024-10-16 7:54 ` [PATCH v6 01/10] drm/bridge: it6505: Change definition of AUX_FIFO_MAX_SIZE Hermes Wu via B4 Relay
2024-10-18 12:56 ` Dmitry Baryshkov
2024-10-16 7:54 ` [PATCH v6 02/10] drm/bridge: it6505: improve AUX operation for edid read Hermes Wu via B4 Relay
2024-10-20 13:53 ` Dmitry Baryshkov
2024-10-16 7:54 ` [PATCH v6 03/10] drm/bridge: it6505: add AUX operation for HDCP KSV list read Hermes Wu via B4 Relay
2024-10-19 23:30 ` kernel test robot
2024-10-20 0:52 ` kernel test robot
2024-10-20 13:53 ` Dmitry Baryshkov
2024-10-16 7:54 ` [PATCH v6 04/10] drm/bridge: it6505: Change definition MAX_HDCP_DOWN_STREAM_COUNT Hermes Wu via B4 Relay
2024-10-20 13:53 ` Dmitry Baryshkov
2024-10-16 7:54 ` [PATCH v6 05/10] drm/bridge: it6505: fix HDCP Bstatus check Hermes Wu via B4 Relay
2024-10-16 7:54 ` [PATCH v6 06/10] drm/bridge: it6505: fix HDCP encryption when R0 ready Hermes Wu via B4 Relay
2024-10-16 7:54 ` [PATCH v6 07/10] drm/bridge: it6505: fix HDCP CTS KSV list read with UNIGRAF DPR-100 Hermes Wu via B4 Relay
2024-10-20 13:54 ` Dmitry Baryshkov
2024-10-16 7:54 ` [PATCH v6 08/10] drm/bridge: it6505: fix HDCP CTS compare V matching Hermes Wu via B4 Relay
2024-10-16 7:54 ` [PATCH v6 09/10] drm/bridge: it6505: fix HDCP CTS KSV list wait timer Hermes Wu via B4 Relay
2024-10-16 7:54 ` [PATCH v6 10/10] drm/bridge: it6505: add I2C functionality on AUX Hermes Wu via B4 Relay
2024-10-20 13:54 ` Dmitry Baryshkov
2024-10-16 21:52 ` [PATCH v6 00/10] drm/bridge: it6505: fix HDCP CTS fail items and add MCCS support Pin-yen Lin
2024-10-17 2:10 ` Hermes.Wu
2024-10-17 2:44 ` Pin-yen Lin
2024-10-17 2:50 ` Hermes.Wu
2024-10-20 13:57 ` Dmitry Baryshkov
2024-10-21 6:33 ` Hermes.Wu [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=2302452d3a6e446d9d288241958ec086@ite.com.tw \
--to=hermes.wu@ite.com.tw \
--cc=Kenneth.Hung@ite.com.tw \
--cc=Laurent.pinchart@ideasonboard.com \
--cc=Pet.Weng@ite.com.tw \
--cc=airlied@gmail.com \
--cc=andrzej.hajda@intel.com \
--cc=angelogioacchino.delregno@collabora.com \
--cc=dmitry.baryshkov@linaro.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=jernej.skrabec@gmail.com \
--cc=jonas@kwiboo.se \
--cc=linux-kernel@vger.kernel.org \
--cc=maarten.lankhorst@linux.intel.com \
--cc=mripard@kernel.org \
--cc=neil.armstrong@linaro.org \
--cc=rfoss@kernel.org \
--cc=simona@ffwll.ch \
--cc=treapking@chromium.org \
--cc=tzimmermann@suse.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox