Re: [PATCH v4 3/7] dt-bindings: iio: light: ltr501: Add ltr329 compatible

On Thursday, August 13th, 2026 at 21:20, Rob Herring <[email protected]> wrote:

> On Wed, Aug 12, 2026 at 02:41:14PM +0200, Esben Haabendal wrote:
> > "Rob Herring" <[email protected]> writes:
> >
> > > sashiko issue should be addressed.
> >
> > I will.
> >
> > I guess it is best to add the new conditional statement under allOf, but
> > as we already have an allOf block at the top, I would have to add it
> > under that, which would place this new conditional statement related to
> > specific compatible strings before the list of those compatible strings.
> >
> > It should work fine, but it looks a bit odd to me.
> >
> > I could also add it directly  as an if statement, but that would only
> > work as long as it is the only top level if statement, so that looks
> > like a bad idea.
> >
> > Should I just stick it to the existing allOf list?
> 
> You can move the allOf to the end.

Ok, fine with me. I will do that.

/Esben