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

From: Esben Haabendal

Date: Fri Aug 14 2026 - 02:12:40 EST


On Thursday, August 13th, 2026 at 21:20, Rob Herring <robh@xxxxxxxxxx> wrote:

> On Wed, Aug 12, 2026 at 02:41:14PM +0200, Esben Haabendal wrote:
> > "Rob Herring" <robh@xxxxxxxxxx> 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