Re: [PATCH v2 1/3] dt-bindings: display: mediatek: ovl: Modify rules for MT8195/MT8188

["Jason-JH Lin (林睿祥)" <Jason-JH.Lin@mediatek.com>]

On Thu, 2024-12-19 at 12:09 +0100, AngeloGioacchino Del Regno wrote:
> External email : Please do not click links or open attachments until
> you have verified the sender or the content.
> 
> 
> Il 13/12/24 10:50, Jason-JH.Lin ha scritto:
> > From: Hsiao Chien Sung <shawn.sung@mediatek.com>
> > 
> > Modify rules for both MT8195 and MT8188.
> > Hardware capabilities include color formats and AFBC are
> > changed since MT8195, stop using the settings of MT8183.
> > 
> > Acked-by: Rob Herring <robh@kernel.org>
> > Reviewed-by: AngeloGioacchino Del Regno <
> > angelogioacchino.delregno@collabora.com>
> > Reviewed-by: CK Hu <ck.hu@mediatek.com>
> > Signed-off-by: Hsiao Chien Sung <shawn.sung@mediatek.com>
> > Signed-off-by: Jason-JH.Lin <jason-jh.lin@mediatek.com>
> > ---
> >   .../bindings/display/mediatek/mediatek,ovl.yaml          | 9
> > ++++-----
> >   1 file changed, 4 insertions(+), 5 deletions(-)
> > 
> > diff --git
> > a/Documentation/devicetree/bindings/display/mediatek/mediatek,ovl.y
> > aml
> > b/Documentation/devicetree/bindings/display/mediatek/mediatek,ovl.y
> > aml
> > index 9ea796a033b2..8ca8e5966b9a 100644
> > ---
> > a/Documentation/devicetree/bindings/display/mediatek/mediatek,ovl.y
> > aml
> > +++
> > b/Documentation/devicetree/bindings/display/mediatek/mediatek,ovl.y
> > aml
> > @@ -26,6 +26,7 @@ properties:
> >             - mediatek,mt8173-disp-ovl
> >             - mediatek,mt8183-disp-ovl
> >             - mediatek,mt8192-disp-ovl
> > +          - mediatek,mt8195-disp-ovl
> >             - mediatek,mt8195-mdp3-ovl
> >         - items:
> >             - enum:
> > @@ -36,16 +37,14 @@ properties:
> >             - enum:
> >                 - mediatek,mt6795-disp-ovl
> >             - const: mediatek,mt8173-disp-ovl
> > -      - items:
> > -          - enum:
> > -              - mediatek,mt8188-disp-ovl
> > -              - mediatek,mt8195-disp-ovl
> > -          - const: mediatek,mt8183-disp-ovl
> >         - items:
> >             - enum:
> >                 - mediatek,mt8186-disp-ovl
> >                 - mediatek,mt8365-disp-ovl
> >             - const: mediatek,mt8192-disp-ovl
> > +      - items:
> > +          - const: mediatek,mt8188-disp-ovl
> > +          - const: mediatek,mt8195-disp-ovl
> > 
> >     reg:
> >       maxItems: 1
> 
> 
> Jason, while doing MDP3 bindings for MT8188, I noticed that the
> binding for OVL is
> missing; can you please add a commit on top of this series adding the
> following
> snippet?
> 
>        - items:
>            - const: mediatek,mt8188-mdp3-ovl
>            - const: mediatek,mt8195-mdp3-ovl
> 
> ...with a suggested-by tag it's going to be fine :-)
> 

Sure, I'll add one more patch for this in the top of v3.

Regards,
Jason-JH.Lin

> P.S.: Please refer to
> 
https://lore.kernel.org/r/20241218105320.38980-1-angelogioacchino.delregno@collabora.com
> 
> Thanks,
> Angelo