Re: [PATCH v2 1/2] dt-bindings: hwmon: Add support for ST TSC1641 power monitor

[Guenter Roeck]

On 10/26/25 09:32, Krzysztof Kozlowski wrote:
> On 26/10/2025 07:50, Igor Reznichenko wrote:
> > +properties:
> > +  compatible:
> > +    const: st,tsc1641
>
> Subject: I asked to drop "binding" and not add "support for". "Support
> for" makes little sense in terms of binding. How binding can support
> anything? This is the "ST TSC1641 power monitor" not support.
>
> > +
> > +  reg:
> > +    maxItems: 1
> > +
> > +  shunt-resistor-micro-ohms:
> > +    description: Shunt resistor value in micro-ohms. Since device has internal
> > +      16-bit RSHUNT register with 10 uOhm LSB, the maximum value is capped at
> > +      655.35 mOhm.
> > +    minimum: 100
> > +    default: 1000
> > +    maximum: 655350
> > +
> > +  st,alert-polarity-active-high:
>
> Isn't this just interrupt? You need proper interrupts property and then
> its flag define the type of interrupt.

This is a value to write into the chip. It is orthogonal to how the interrupt
is reported to the interrupt controller. It may be active low by the chip and
inverted, or it may be active high by the chip and inverted. How does one express
an additional inverter in the interrupt signal path in a devicetree property ?
Can you give an example ?

Thanks,
Guenter