Re: [PATCH 0/4] Add support for the mt8167 display pipeline

[AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>]

On 9/18/26 19:55, Luca Leonardo Scorcia wrote:
> This series adds support for (almost) all display blocks of mt8167.
> This SoC supports a DSI or DPI display and has HDMI output via a second
> DPI block wired to the HDMI phy.
> 
> All the hard work was already upstreamed in the past, now all that's left
> is some glue. In the first two patches we add support for the DPI output
> blocks. In the third one we describe all the possible combinations of the
> SoC display pipeline and in the last one we add the required nodes to the
> SoC dtsi.
> 
> The final outputs of the display pipeline in the SoC dtsi are
> intentionally left not connected to allow board dts to specify their
> own output node.
> 
> I said almost all display blocks because HDMI output is not included in
> this patch. I have it tested and working at all supported resolutions, but
> since it needs some fixes in clocks and a new phy driver, I will send it
> in a separate series.
> 
> The series has been tested on a Lenovo Smart Clock 2 (DSI), a Xiaomi Mi
> Smart Clock 2 (DSI), and a Sony Playstation Classic Mini (HDMI via DPI).
> Unfortunately I couldn't find an actual device using a DPI display, but

I don't think that it even exists with your SoC. It's fine though, because
you're anyway testing the DPI interface in one way or another.

> since DPI1-HDMI works, I feel confident saying that the clocks divisors
> are correct.

While series is

Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>

> 
> Luca Leonardo Scorcia (4):
>    dt-bindings: display: mediatek: dpi: Add compatible for mt8167
>    drm/mediatek: Add support for mt8167 Digital Parallel Interface
>    soc: mediatek: mt8167-mmsys: add routes for all display paths
>    arm64: dts: mediatek: mt8167: Add DRM nodes
> 
>   .../display/mediatek/mediatek,dpi.yaml        |   1 +
>   arch/arm64/boot/dts/mediatek/mt8167.dtsi      | 369 ++++++++++++++++++
>   drivers/gpu/drm/mediatek/mtk_dpi.c            |  23 ++
>   drivers/gpu/drm/mediatek/mtk_drm_drv.c        |   2 +
>   drivers/soc/mediatek/mt8167-mmsys.h           | 183 ++++++++-
>   5 files changed, 569 insertions(+), 9 deletions(-)
>