Re: [PATCH v2] dt-bindings: iio: light: opt3001: add compatible for opt3004

[Hardevsinh Palaniya <hardevsinh.palaniya@xxxxxxxxxxxxxxxxx>]

Hi Krzysztof,

Thanks for the review.

> >  properties:
> >    compatible:
> > -    enum:
> > -      - ti,opt3001
> > -      - ti,opt3002
> > +    oneOf:
> > +      - enum:
> > +          - ti,opt3001
> > +          - ti,opt3002
> > +      - items:
> > +          - const: ti,opt3001
> > +          - const: ti,opt3004
>
> 3004 is not the fallback.

Do you mean I need to reverse the order?

 items:
          - const: ti,opt3004
          -  const: ti,opt3001

Is it correct?

Best Regards,
Hardev