Re: [PATCH v4 6/6] dt-bindings: display: Add SSD132x OLED controllers

From: Javier Martinez Canillas
Date: Fri Oct 20 2023 - 18:29:42 EST


Rob Herring <robh@xxxxxxxxxx> writes:

Hello Rob,

> On Sat, Oct 14, 2023 at 2:15 AM Javier Martinez Canillas
> <javierm@xxxxxxxxxx> wrote:

[...]

>> +maintainers:
>> + - Javier Martinez Canillas <javierm@xxxxxxxxxx>
>> +
>> +properties:
>> + compatible:
>> + - enum:
>
> Now this is a warning in linux-next. The '-' should not be there.
> Please send a fix.
>

Sure, I'll post a fix. Thanks for reporting the issue.

This is a leftover from when the compatible string enum was a 'oneOf'
child and the 'oneOf' got removed during the binding schema review.

> Not sure why there wasn't a report, but most likely it didn't apply to v6.6-rc1.
>

Yes, I'm not sure either. I didn't notice it when testing with:

make dt_binding_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/display/solomon,ssd132x.yaml

> Rob
>

--
Best regards,

Javier Martinez Canillas
Core Platforms
Red Hat