Re: [PATCH] dt-bindings: display: mediatek: dpi: Add compatible for MediaTek MT8188

[liangxu.xu <liangxu.xu@mediatek.com>]

On Mon, 2022-09-26 at 08:26 +0200, Krzysztof Kozlowski wrote:
> On 26/09/2022 04:53, liangxu.xu wrote:
> > On Fri, 2022-09-23 at 13:16 +0200, Krzysztof Kozlowski wrote:
> > > On 23/09/2022 03:42, liangxu.xu@mediatek.com wrote:
> > > > From: liangxu xu <liangxu.xu@mediatek.com>
> > > > 
> > > > Add dt-binding documentation of dpi for MediaTek MT8188 SoC.
> > > > 
> > > > Signed-off-by: liangxu xu <liangxu.xu@mediatek.com>
> > > 
> > > Where is the DTS? Where are driver changes?
> > > 
> > > Best regards,
> > > Krzysztof
> > > 
> > 
> > Hi Krzysztof:
> > 
> > If you want to see the synchronous changes of dts and binding
> > files,
> > then I will attach this binding file changes when sending dts
> > later.
> 
> Keep discussions public. You submit a lot of unused compatibles. I
> want
> users to come with them.
> 
> > 
> > The driver change of mt8188 reuses that of mt8195. The driver link
> > is
> > as follows:
> > 
> > 
https://urldefense.com/v3/__https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/drivers/gpu/drm/mediatek/mtk_dpi.c?id=48f4230642ee32a97ddf4be492838ce96089f040__;!!CTRNKA9wMg0ARbw!1Rd4E1huTfB_u1FjLXMIKlqMciGL24xsYInu1vg1z9c026cL0PiITklArLfwrMsi3x4$
> >  
> 
> Driver does not support that compa
> > 
> > Best Regards,
> > LiangXu
> > 
> 
> Best regards,
> Krzysztof


Hi Krzysztof:

Can I modify it to the following form:
properties:
  compatible:
    oneOf:
      - enum:
        - mediatek,mt2701-dpi
        - mediatek,mt7623-dpi
        - mediatek,mt8173-dpi
        - mediatek,mt8183-dpi
        - mediatek,mt8186-dpi
        - mediatek,mt8192-dpi
        - mediatek,mt8195-dp-intf
      - items:
          - enum:
              - mediatek,mt8188-dp-intf
          - const: mediatek,mt8195-dp-intf

This means that mt8188 reuses mt8195, so there is no need to add
compatible to the driver.

Best Regards,
LiangXu
>