Re: [PATCH v3 1/2] dt-bindings: hwmon: add support for MCP998X

[<Victor.Duicu@microchip.com>]

Hi Krzysztof,

> > +
> > +  interrupts:
> > +    items:
> > +      - description: Signal coming from ALERT/THERM pin.
> > +      - description: Signal coming from THERM/ADDR pin.
> > +      - description: Signal coming from SYS_SHDN pin.
> 
> As Guenter pointed out - code is wrong (thanks Guenter!)
> 
> This does not match your if:then:.
> 
> > +
> > +  interrupt-names:
> > +    items:
> > +      - const: alert-therm
> > +      - const: therm-addr
> > +      - const: sys-shutdown
> 
> Neither this.
> 
> 
> ...
> 
> > +    then:
> > +      properties:
> > +        interrupts-names:
> > +          items:
> > +            - const: alert-therm
> > +            - const: sys-shutdown
> 
> So three interrupts, but two AND three interrupt-names? This is mess.

I was mistakenly under the impression that I could set two out of three
possible interrupts depending on the chip. But I understand now that
in the code written as is all chips have three interrupts.

Indeed, this error was present multiple versions in the past and you
identified it in v4 of the IIO driver. I failed to fix it then, I
apologize.

I will use two interrupts only and remove the if:then code regarding
them.

Kind Regards,
Victor