Re: [PATCH v10 20/24] drm/mediatek: Add Padding to OVL adaptor

From: Shawn Sung (宋孝謙)
Date: Thu Oct 19 2023 - 07:51:14 EST


Hi Angelo,

On Thu, 2023-10-19 at 11:55 +0200, AngeloGioacchino Del Regno wrote:
>
> That's like many other components in MediaTek, so we can keep the
> mtk-disp-padding
> name.... in devicetree, we will anyway use "mediatek,mt8195-mdp3-
> padding" as one of
> the compatible string(s).
>
> This is the only way that we have to actually distinguish between
> components used
> for MDP3 and components used for the display subsystem, if we keep
> them "generic"
> we won't understand what's going on in case of issues.
>
> The driver name should contain "disp" for consistency with all of the
> component
> drivers in mediatek-drm; if this wasn't in this folder, we could've
> dropped the
> "disp" in the name, but that's not the case.
>
> Consistency is #1.
>
> Cheers,
> Angelo
>

Got it. Thank you for making that clear.
Will change it in the next version.

Cheers,
Shawn