mbox series

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

Message ID 20241230-v7-upstream-v7-0-e0fdd4844703@ite.corp-partner.google.com (mailing list archive)
Headers show
Series drm/bridge: it6505: fix HDCP CTS fail items and add MCCS support | expand

Message

Hermes Wu via B4 Relay Dec. 30, 2024, 10:51 a.m. UTC 
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.

v6->v7
	-[3/10] add header <linux/bitfield.h> included

	links:
	https://lore.kernel.org/all/20241016-upstream-v6-v6-0-4d93a0c46de1@ite.com.tw/

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://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.

To: Andrzej Hajda <andrzej.hajda@intel.com>
To: Neil Armstrong <neil.armstrong@linaro.org>
To: Robert Foss <rfoss@kernel.org>
To: Laurent Pinchart <Laurent.pinchart@ideasonboard.com>
To: Jonas Karlman <jonas@kwiboo.se>
To: Jernej Skrabec <jernej.skrabec@gmail.com>
To: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
To: Maxime Ripard <mripard@kernel.org>
To: Thomas Zimmermann <tzimmermann@suse.de>
To: David Airlie <airlied@gmail.com>
To: Simona Vetter <simona@ffwll.ch>
To: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
To: Hermes Wu <hermes.wu@ite.com.tw>
Cc: dri-devel@lists.freedesktop.org
Cc: linux-kernel@vger.kernel.org
Cc: Pin-yen Lin <treapking@chromium.org>
Cc: Kenneth Hung <Kenneth.Hung@ite.com.tw>
Cc: Pet Weng <Pet.Weng@ite.com.tw>

Change-Id: I7f14eeb18ae70b17ef7e1a8622c590427b0ff2ed
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 | 335 ++++++++++++++++++++++++++++++------
 1 file changed, 278 insertions(+), 57 deletions(-)
---
base-commit: 4d1c251e8889b4629789425fe85adfc6d39666ac
change-id: 20241101-v7-upstream-75ead2f2f567

Best regards,

Comments

AngeloGioacchino Del Regno Jan. 2, 2025, 12:34 p.m. UTC | #1 
Il 30/12/24 11:51, Hermes Wu via B4 Relay ha scritto:
> 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.
> 
> v6->v7
> 	-[3/10] add header <linux/bitfield.h> included
> 
> 	links:
> 	https://lore.kernel.org/all/20241016-upstream-v6-v6-0-4d93a0c46de1@ite.com.tw/
> 
> 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://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.
> 
> To: Andrzej Hajda <andrzej.hajda@intel.com>
> To: Neil Armstrong <neil.armstrong@linaro.org>
> To: Robert Foss <rfoss@kernel.org>
> To: Laurent Pinchart <Laurent.pinchart@ideasonboard.com>
> To: Jonas Karlman <jonas@kwiboo.se>
> To: Jernej Skrabec <jernej.skrabec@gmail.com>
> To: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> To: Maxime Ripard <mripard@kernel.org>
> To: Thomas Zimmermann <tzimmermann@suse.de>
> To: David Airlie <airlied@gmail.com>
> To: Simona Vetter <simona@ffwll.ch>
> To: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
> To: Hermes Wu <hermes.wu@ite.com.tw>
> Cc: dri-devel@lists.freedesktop.org
> Cc: linux-kernel@vger.kernel.org
> Cc: Pin-yen Lin <treapking@chromium.org>
> Cc: Kenneth Hung <Kenneth.Hung@ite.com.tw>
> Cc: Pet Weng <Pet.Weng@ite.com.tw>
> 
> Change-Id: I7f14eeb18ae70b17ef7e1a8622c590427b0ff2ed
> Signed-off-by: Hermes Wu <Hermes.wu@ite.com.tw>

Whole series is

Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Dmitry Baryshkov Jan. 3, 2025, 6:42 p.m. UTC | #2 
On Mon, 30 Dec 2024 18:51:18 +0800, Hermes Wu wrote:
> 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.
> 
> v6->v7
> 	-[3/10] add header <linux/bitfield.h> included
> 
> [...]

Applied to drm-misc-next, thanks!

[01/10] drm/bridge: it6505: Change definition of AUX_FIFO_MAX_SIZE
        commit: c14870218c14532b0f0a7805b96a4d3c92d06fb2
[02/10] drm/bridge: it6505: improve AUX operation for edid read
        commit: d0c97a51cdef2badc8d17f6edfbc0bdc0ba92065
[03/10] drm/bridge: it6505: add AUX operation for HDCP KSV list read
        commit: 235d9bf8a1e1c0a0cf1340c40567e6b4f2b742c6
[04/10] drm/bridge: it6505: Change definition MAX_HDCP_DOWN_STREAM_COUNT
        commit: 85597bc0d70c287ba41f17d14d3d857a38a3d727
[05/10] drm/bridge: it6505: fix HDCP Bstatus check
        commit: 0fd2ff47d8c207fa3173661de04bb9e8201c0ad2
[06/10] drm/bridge: it6505: fix HDCP encryption when R0 ready
        commit: 8c01b0bae2f9e58f2fee0e811cb90d8331986554
[07/10] drm/bridge: it6505: fix HDCP CTS KSV list read with UNIGRAF DPR-100.
        commit: aa97239300e3f2a9ecbccf7ebfa240860bdddafb
[08/10] drm/bridge: it6505: fix HDCP CTS compare V matching
        commit: 0989c02c7a5c887c70afeae80c64d0291624e1a7
[09/10] drm/bridge: it6505: fix HDCP CTS KSV list wait timer
        commit: 9f9eef9ec1a2b57d95a86fe81df758e8253a7766
[10/10] drm/bridge: it6505: add I2C functionality on AUX
        commit: 041d61ad66ea0d88890264e96e8f81437a33a9cf

Best regards,