Re: [PATCH 2/6] dt-bindings: HID: i2c-hid: elan: add Elan eKTH5015M

From: Johan Hovold
Date: Wed Apr 24 2024 - 03:03:43 EST


On Tue, Apr 23, 2024 at 06:24:39PM +0200, Krzysztof Kozlowski wrote:
> On 23/04/2024 15:46, Johan Hovold wrote:

> > properties:
> > compatible:
> > - items:
> > - - const: elan,ekth6915
> > + oneOf:
> > + - items:
> > + - enum:
> > + - elan,ekth5015m
> > + - const: elan,ekth6915
> > + - items:
>
> Don't re-add the items for this entry. Just const.

Sure. But note that the example schema uses 'items' like this (e.g. for
'compatible' and 'clock-names'):

https://docs.kernel.org/devicetree/bindings/writing-schema.html#annotated-example-schema

Johan