[PATCH v7 4/4] drm/mediatek: Add DSI support for mt8188 vdosys0
Jason-JH Lin (林睿祥)
Jason-JH.Lin at mediatek.com
Mon Jul 31 09:08:43 PDT 2023
Hi Angelo,
Thanks for the reviews.
On Fri, 2023-07-28 at 10:14 +0200, AngeloGioacchino Del Regno wrote:
> Il 27/07/23 18:41, Jason-JH.Lin ha scritto:
> > Add DSI as main display output for mt8188 vdosys0.
> >
>
> I would split this commit in two:
>
> 1. drm/mediatek: dsi: Support dynamic connector selection
> 2. drm/mediatek: Support DSI on MT8188 VDOSYS0
>
OK, I'll separate this commit like this.
Regards,
Jason-JH.Lin
> Where the first one sets up the mtk_dsi driver for .encoder_index(),
> and
> where the second one is a one-liner simply adding `{0,
> DDP_COMPONENT_DSI0},`
> to the MT8188 main routes array.
>
> The reason is that if you split it like that, whoever reads the
> commit history
> will immediately understand how to add dynamic connector selection to
> a MTK DRM
> driver, and how to perform the SoC-specific connection....
>
> Please!
>
> Thanks,
> Angelo
>
More information about the Linux-mediatek
mailing list