On 22/09/2023 13:51, Sricharan R wrote: > From: Sricharan Ramabadhran <quic_srichara@xxxxxxxxxxx> > > IPQ5018 has tsens v1.0 block with 4 sensors and 1 interrupt. Then why do you allow two interrupts? > > Signed-off-by: Sricharan Ramabadhran <quic_srichara@xxxxxxxxxxx> > --- > [v3] Added the tsens-ipq5018 as new binding without rpm > > Documentation/devicetree/bindings/thermal/qcom-tsens.yaml | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml b/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml > index 27e9e16e6455..a02829deeb24 100644 > --- a/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml > +++ b/Documentation/devicetree/bindings/thermal/qcom-tsens.yaml > @@ -44,6 +44,10 @@ properties: > - qcom,qcs404-tsens > - const: qcom,tsens-v1 > > + - description: v1 of TSENS without rpm > + enum: > + - qcom,ipq5018-tsens You miss now description of interrupts, like the other variants have. Best regards, Krzysztof