[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <6130bdbc-163a-c950-e2e1-07ef77124833@linaro.org>
Date: Wed, 2 Nov 2022 12:21:44 -0400
From: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
To: Manivannan Sadhasivam <manivannan.sadhasivam@...aro.org>
Cc: andersson@...nel.org, konrad.dybcio@...ainline.org,
robh+dt@...nel.org, krzysztof.kozlowski+dt@...aro.org,
johan+linaro@...nel.org, quic_jprakash@...cinc.com,
linux-arm-msm@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH 09/11] arm64: dts: qcom: sc8280xp-x13s: Add PM8280_{1/2}
VADC channels
On 30/10/2022 03:16, Manivannan Sadhasivam wrote:
>>>
>>>> + reg = <PM8350_ADC7_DIE_TEMP(1)>;
>>>> + label = "pm8280_1_die_temp";
>>>> + qcom,pre-scaling = <1 1>;
>>>> + };
>>>> +
>>>> + sys-therm1@144 {
>>>
>>> I would say sys-therm@... why do we need the "1" suffix in node name?
>>>
>>
>> Suffix represents the index of the thermistor as per the schematics but I guess
>> it can be dropped.
>>
>
> I was wrong. Suffix is necessary since IIO core registers the channel based on
> the node name, so it has to be unique.
>
Makes sense then.
Best regards,
Krzysztof
Powered by blists - more mailing lists