Re: [PATCH V8 1/5] dt-bindings: display: mediatek: add aal binding for MT8183

From: Rex-BC Chen
Date: Thu Mar 17 2022 - 01:34:40 EST


On Wed, 2022-03-16 at 16:18 +0100, Krzysztof Kozlowski wrote:
> On 16/03/2022 12:51, Rex-BC Chen wrote:
> > Add aal binding for MT8183.
> >
> > Signed-off-by: Rex-BC Chen <rex-bc.chen@xxxxxxxxxxxx>
> > ---
> > .../devicetree/bindings/display/mediatek/mediatek,aal.yaml | 4
> > +++-
> > 1 file changed, 3 insertions(+), 1 deletion(-)
> >
> > diff --git
> > a/Documentation/devicetree/bindings/display/mediatek/mediatek,aal.y
> > aml
> > b/Documentation/devicetree/bindings/display/mediatek/mediatek,aal.y
> > aml
> > index 61f0ed1e388f..eb0299b80522 100644
> > ---
> > a/Documentation/devicetree/bindings/display/mediatek/mediatek,aal.y
> > aml
> > +++
> > b/Documentation/devicetree/bindings/display/mediatek/mediatek,aal.y
> > aml
> > @@ -22,7 +22,9 @@ properties:
> > compatible:
> > oneOf:
> > - items:
> > - - const: mediatek,mt8173-disp-aal
> > + - enum:
> > + - mediatek,mt8173-disp-aal
> > + - mediatek,mt8183-disp-aal
>
> This should not be under items. You have just one element here, so
> directly "enum" under oneOf.
>
>
> Best regards,
> Krzysztof

Hello Krzysztof,

OK, I will do this in next version.
Thanks!

BRs,
Rex