Message-ID: <abe8d1173fa6ee3b976690ab48897ec3b3b293b9.camel@mediatek.com>
Date: Fri, 15 Aug 2025 08:54:33 +0000
From: CK Hu (胡俊光) <ck.hu@...iatek.com>
To: "p.zabel@...gutronix.de" <p.zabel@...gutronix.de>,
	"chunkuang.hu@...nel.org" <chunkuang.hu@...nel.org>, "AngeloGioacchino Del
 Regno" <angelogioacchino.delregno@...labora.com>, "airlied@...il.com"
	<airlied@...il.com>, Louis-Alexis Eyraud <louisalexis.eyraud@...labora.com>,
	"simona@...ll.ch" <simona@...ll.ch>, "matthias.bgg@...il.com"
	<matthias.bgg@...il.com>
CC: "dri-devel@...ts.freedesktop.org" <dri-devel@...ts.freedesktop.org>,
	"linux-mediatek@...ts.infradead.org" <linux-mediatek@...ts.infradead.org>,
	"linux-arm-kernel@...ts.infradead.org"
	<linux-arm-kernel@...ts.infradead.org>, "kernel@...labora.com"
	<kernel@...labora.com>, "linux-kernel@...r.kernel.org"
	<linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v10 00/10] Add support for MT8195/88 HDMIv2 and DDCv2

Hi, Louis:

In [1], MT8173 HDMI has broken by a patch on which this series depend.
I would like that bug fixed then apply this series.
Please fix that bug first.

[1] https://lists.infradead.org/pipermail/linux-mediatek/2025-August/097094.html

Regards,
CK

On Fri, 2025-08-08 at 12:12 +0200, Louis-Alexis Eyraud wrote:
> External email : Please do not click links or open attachments until you have verified the sender or the content.
> 
> 
> This is a respin of AngeloGioacchino Del Regno's patch series, that adds
> the support of HDMIv2 and DDCv2 for MT8188 and MT8195 in mediatek-drm.
> 
> Changes in v10:
>  - Rebased over next-20250807
>  - Dropped patches from v9 that have been already merged in a previous
>    merge window (1 to 14)
>  - Resolved merge issues with next-20250807 for patches 3, 4 and 9
>  - Added new "drm/mediatek: mtk_hdmi: Drop redundant clock retrieval in
>    mtk_hdmi_get_cec_dev" patch to fix an issue introduced by a previously
>    merged patch from this series
>  - Deleted mtk_hdmi.c.orig file added by "drm/mediatek: mtk_hdmi: Split driver
>    and add common probe function" patch
>  - Fixed in "drm/mediatek: Introduce HDMI/DDC v2 for MT8195/MT8188" patch
>    a bug about EDID reading not properly working in mtk_hdmi_ddc_v2 driver,
>    due to extra byte sent on i2c bus by mtk_ddc_wr_one function when the
>    payload length is equal to 0
>  - Fixed format issues detected by checkpatch in "drm/mediatek: Introduce
>    HDMI/DDC v2 for MT8195/MT8188" patch
>  - Tested on Mediatek Genio 350, 510 and 1200-EVK boards
>  - Link to v9: https://lore.kernel.org/dri-devel/20250415104321.51149-1-angelogioacchino.delregno@collabora.com/
> 
> Changes in v9:
>  - Reordered patch from krzk as first as requested by CK
> 
> Changes in v8:
>  - Dropped DPI patches as those have been applied in the previous merge window
>  - Changed description in mediatek,mt8195-hdmi.yaml as requested by CK
>  - Refactored function mtk_hdmi_v2_hw_gcp_avmute() to include contents
>    of, and delete, mtk_hdmi_v2_hw_reset_av_mute_regs() as requested by CK
>  - Expanded comment before enablement of HDCP reauthentication interrupt
>    to explain that the HW uses this internally as requested by CK
>  - Added comment in mtk_hdmi_v2_hpd_pord_status() explaining why there
>    are three states for cable detection as requested by CK
>  - Moved extra interrupts clearing from ISR to probe function and added
>    comments explaining the reason why those are being cleaned at probe
>    time, as requested by CK.
>  - Added support (and tested on MT8395/8195 and MT8390/8188) for output
>    in both YUV422 and YUV444 colorspaces other than RGB; please note that
>    RGB is still the default, and that the request for using any of the
>    YUV output formats depends on previous component(s) of the display
>    pipeline declaring support for those; should none of them declare any
>    support for YUV formats, only RGB will be available (no errors, the
>    additional ones will be simply ignored).
> 
> Changes in v7:
>  - Split more patches as requested by CK
>  - Changed the order of the interlaced variable addition as requested
>  - Cleanups in DDCv2 as requested by CK
>  - Removed comment from
>    drm/mediatek: mtk_hdmi: Move output init to mtk_hdmi_register_audio_driver()
>    as that was forgotten from reintroduction of print in v5
>  - Some more small nitpicks as pointed out by CK here and there
> 
> Changes in v6:
>  - Split the TVD clock enable/disable calls in a different commit
>  - Changed `is_internal_hdmi` to two different variables, one for
>    DPI input clock from HDMI, and one for AFIFO 1T1P output and
>    conversion (mtk_dpi)
>    - Clarified why MT8195/88 HDMI-reserved DPI1 is different
>  - Moved `input_2p_en` bit to platform data to cleanup DPI vs DPINTF
>    - 1T2P enable bit is different between DPI and DPINTF, but usage
>      is actually the same
>  - Cleaned up headers inclusion in mtk_hdmi_v2.c, mtk_hdmi_ddc_v2.c
>    - Removed some unused headers, added missing bitfield.h header
>  - Split some prints cleanup commits as requested by CK
>  - Split the introduction of mtk_hdmi_conf as requested by CK
>  - Split commit to make CEC optional as requested by CK
>  - Reintroduced forgotten no_capture_mute in codec_pdata (mtk_hdmi_common)
>  - Reintroduced error print for audio clocks enablement failure (mtk_hdmi)
>  - Added cleanup syscon_regmap_lookup_by_phandle commit from Krzysztof K
> 
> Changes in v5:
>  - Rebased over next-20250113
>  - Resolved merge issues with next-20250113
>  - Added bitfield.h inclusion in mtk_dpi in commit [02/33] to resolve
>    build issue from 0day CI
>  - Removed .atomic_check callback from mtk_hdmi_v2 as it is now part
>    of drm_bridge_connector as pointed out by Dmitry B
>  - Removed call to pm_runtime_disable() as the driver uses devm
>  - Tested again :-)
> 
> Changes in v4:
>  - DDCv2 binding erroneously dropped in v3 is included again (oops!)
>  - Added reference to dai-common.yaml in HDMIv2 binding
>  - Dropped pinctrl entries from HDMIv2 binding
>  - Fixed required list in HDMIv2 binding and changed node name to
>    'hdmi' instead of 'hdmi-tx'
>  - Fixed issue in mtk_hdmi derived from wrong commit splitting action
>    from version 3
>  - Exported necessary symbols and added namespaces for those
>  - Fixed module build for both HDMIv1 and HDMIv2
>  - Other cleanups
> 
> Changes in v3:
>  - Added hpd_enable() and hpd_disable() callbacks as suggested by Dmitry B
>  - Removed audio mute call in bridge_enable() as suggested by CK
>  - Reworked commonization commits for mtk_hdmi/mtk_hdmi_common and split
>    out debugfs/abist implementation as suggested by CK
>  - Removed .mode_valid() callback as it is now provided by the bridge
>    API in drm_bridge_connector_helper_funcs
>  - A bit of cleanups here and there
>  - Tested again on HW especially for new hpd_enable/disable callbacks.
> 
> Changes in v2:
>  - Merged series "Add support for MT8195/8188 and Pattern Generator"
>    and "drm/mediatek: Add support for HDMIv2 and DDCv2 IPs" in one
>    as they are directly related, as requested by CK Hu
>  - More commonization: moved some audio functions to mtk_hdmi_common
>  - Fixed a bug in DDCv2 driver to allow sending a message with len=1
>  - Renamed some functions in HDMIv2 to consistently use the prefix
>    mtk_hdmi_v2_ across the driver
>  - Added .mode_valid() callback to HDMIv2
>  - Added .atomic_check() callback to HDMIv2
>  - Reordered drm_bridge_funcs in HDMIv2 driver
>  - Rewritten .edid_read() callback in HDMIv2 to move checking audio
>    availability to bridge_pre_enable() stage, and to stop using the
>    drm_edid_read_ddc() in favor of drm_edid_read()
>  - Added support for API provided HDMI Helpers
>  - Added .tmds_char_rate_valid() callback to HDMIv2 for HDMI helpers
>  - Added .hdmi_{read,write}_infoframe() callback to HDMIv2 for helpers
>  - Added support for Vendor infoframes in HDMIv2
>  - Added missing audio-dai-cells to HDMIv2 binding to fix check error
>  - Added more information to the HDMIv2 binding for clocks and PHY
>  - Added some comments to the HDMIv2 code to clarify why the controller
>    is preconfigured in bridge_pre_enable() instead of bridge_enable()
>  - Added a mention of the differences in HPD between v1 and v2 to the
>    commit introducing the v2 driver (v2 is not using CEC for HPD)
>  - ...and tested again on HW! :-)
> 
> This series adds support for the HDMI-TX v2 Encoder and DDCv2, and for
> the direct connection DPI as found in MT8195, MT8188 and their variants.
> 
> Tested on Genio 700 EVK:
>  - ABIST ON: ok, pattern generated internally from HDMI is shown on
>    HDMI screen at the correct resolution;
>  - ABIST OFF + DPI Pattern Generator ON: ok, pattern coming from DPI is
>    shown on HDMI screen at the correct resolution;
>  - Can negotiate up to 4k60
> 
> and on MT8395 Radxa NIO 12L:
>  - ABIST ON: ok, pattern generated internally from HDMI is shown on
>    HDMI screen at the correct resolution;
>  - ABIST OFF + DPI Pattern Generator ON: ok, pattern coming from DPI is
>    shown on HDMI screen at the correct resolution;
>  - Dual screen usecase validated (DSI + HDMI 3840x2160p 60Hz)
>  - Can negotiate up to 4k60
> 
> Please note that this submission does *not* include support for HDCP
> nor for CECv2, as I want this to be upstream before implementing
> additional features which are not strictly required for simple
> HDMI output.
> 
> Bonus in this series is the addition of support for the Pattern Generator
> found in the DPI HW: since I needed this for debugging during development,
> I had to code in the actual support bits and it looked like a waste of
> time to just remove it.
> I instead decided to clean it up and upstream it, as this will anyway come
> handy for multiple things, of which the most important (imo) are:
>  - Adding support for new SoCs in the future will be less time consuming
>    as this driver already has the pattern generator in;
>  - CI Testing might be able to make use of this to validate that the
>    data that comes out is not garbled (so, to help testing display
>    support in an automated manner).
> 
> --
> 2.49.0
> 
> ---
> AngeloGioacchino Del Regno (9):
>       drm/mediatek: mtk_hdmi: Improve mtk_hdmi_get_all_clk() flexibility
>       drm/mediatek: mtk_hdmi: Add HDMI IP version configuration to pdata
>       drm/mediatek: mtk_hdmi: Split driver and add common probe function
>       drm/mediatek: mtk_hdmi_common: Make CEC support optional
>       drm/mediatek: mtk_hdmi_common: Assign DDC adapter pointer to bridge
>       drm/mediatek: mtk_hdmi_common: Add OP_HDMI if helper funcs assigned
>       drm/mediatek: mtk_hdmi_common: Add var to enable interlaced modes
>       drm/mediatek: Introduce HDMI/DDC v2 for MT8195/MT8188
>       drm/mediatek: mtk_hdmi_v2: Add debugfs ops and implement ABIST
> 
> Louis-Alexis Eyraud (1):
>       drm/mediatek: mtk_hdmi: Drop redundant clock retrieval in mtk_hdmi_get_cec_dev
> 
>  drivers/gpu/drm/mediatek/Kconfig            |   18 +-
>  drivers/gpu/drm/mediatek/Makefile           |    3 +
>  drivers/gpu/drm/mediatek/mtk_hdmi.c         |  539 +---------
>  drivers/gpu/drm/mediatek/mtk_hdmi_common.c  |  440 ++++++++
>  drivers/gpu/drm/mediatek/mtk_hdmi_common.h  |  198 ++++
>  drivers/gpu/drm/mediatek/mtk_hdmi_ddc_v2.c  |  395 +++++++
>  drivers/gpu/drm/mediatek/mtk_hdmi_regs_v2.h |  263 +++++
>  drivers/gpu/drm/mediatek/mtk_hdmi_v2.c      | 1521 +++++++++++++++++++++++++++
>  8 files changed, 2858 insertions(+), 519 deletions(-)
> ---
> base-commit: cd328e98533cd7035c9b3ab126070d88dae36e18
> change-id: 20250806-mediatek-drm-hdmi-v2-cf88f51b8458
> 
> Best regards,
> --
> Louis-Alexis Eyraud <louisalexis.eyraud@...labora.com>
> 

Powered by blists - more mailing lists