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

On Monday, August 10th, 2026 at 17:50, Rob Herring <[email protected]> wrote:

> On Mon, Aug 10, 2026 at 09:11:13AM +0200, Esben Haabendal wrote:
> > LTR-329ALS-01 is an ambient light sensor similar to LTR-303ALS-01, but
> > without interrupt support.
> >
> > Acked-by: Krzysztof Kozlowski <[email protected]>
> > Acked-by: Kuppuswamy Sathyanarayanan <[email protected]>
> > Signed-off-by: Esben Haabendal <[email protected]>
> > ---
> >  Documentation/devicetree/bindings/iio/light/liteon,ltr501.yaml | 1 +
> >  1 file changed, 1 insertion(+)
> 
> sashiko issue should be addressed.

What about the suggestion about adding `proximity-near-level: false` to the condition. I am thinking something like this is more correct

properties:
  proximity-near-level: false

allOf:
  - if:
      properties:
        compatible:
          contains:
            enum:
              - liteon,ltr501
              - liteon,ltr559
    then:
      properties:
        proximity-near-level: true

As it is only valid for those two chips, and it seems more logical to do a white-list than the opposite.

/Esben