[PATCH v2 2/5] dt-bindings: thermal: generic-adc: Document #io-channel-cells

Luca Weiss posted 5 patches 1 month, 1 week ago
[PATCH v2 2/5] dt-bindings: thermal: generic-adc: Document #io-channel-cells
Posted by Luca Weiss 1 month, 1 week ago
Document the property to allow using this sensor with io-channels.

Signed-off-by: Luca Weiss <luca.weiss@fairphone.com>
---
 Documentation/devicetree/bindings/thermal/generic-adc-thermal.yaml | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/Documentation/devicetree/bindings/thermal/generic-adc-thermal.yaml b/Documentation/devicetree/bindings/thermal/generic-adc-thermal.yaml
index 12e6418dc24d..317fc39ed46a 100644
--- a/Documentation/devicetree/bindings/thermal/generic-adc-thermal.yaml
+++ b/Documentation/devicetree/bindings/thermal/generic-adc-thermal.yaml
@@ -24,6 +24,9 @@ properties:
   '#thermal-sensor-cells':
     const: 0
 
+  "#io-channel-cells":
+    const: 0
+
   io-channels:
     maxItems: 1
 
@@ -58,6 +61,7 @@ examples:
     thermal-sensor {
         compatible = "generic-adc-thermal";
         #thermal-sensor-cells = <0>;
+        #io-channel-cells = <0>;
         io-channels = <&ads1015 1>;
         io-channel-names = "sensor-channel";
         temperature-lookup-table = <

-- 
2.53.0
Re: [PATCH v2 2/5] dt-bindings: thermal: generic-adc: Document #io-channel-cells
Posted by Rob Herring 1 month, 1 week ago
On Fri, Feb 20, 2026 at 10:19:06AM +0100, Luca Weiss wrote:
> Document the property to allow using this sensor with io-channels.

Why? On the surface, it looks like you are adding a provider property to 
a consumer. Maybe that's right? 

> 
> Signed-off-by: Luca Weiss <luca.weiss@fairphone.com>
> ---
>  Documentation/devicetree/bindings/thermal/generic-adc-thermal.yaml | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/thermal/generic-adc-thermal.yaml b/Documentation/devicetree/bindings/thermal/generic-adc-thermal.yaml
> index 12e6418dc24d..317fc39ed46a 100644
> --- a/Documentation/devicetree/bindings/thermal/generic-adc-thermal.yaml
> +++ b/Documentation/devicetree/bindings/thermal/generic-adc-thermal.yaml
> @@ -24,6 +24,9 @@ properties:
>    '#thermal-sensor-cells':
>      const: 0
>  
> +  "#io-channel-cells":
> +    const: 0
> +
>    io-channels:
>      maxItems: 1
>  
> @@ -58,6 +61,7 @@ examples:
>      thermal-sensor {
>          compatible = "generic-adc-thermal";
>          #thermal-sensor-cells = <0>;
> +        #io-channel-cells = <0>;
>          io-channels = <&ads1015 1>;
>          io-channel-names = "sensor-channel";
>          temperature-lookup-table = <
> 
> -- 
> 2.53.0
>
Re: [PATCH v2 2/5] dt-bindings: thermal: generic-adc: Document #io-channel-cells
Posted by Luca Weiss 1 month, 1 week ago
On Mon Feb 23, 2026 at 7:48 PM CET, Rob Herring wrote:
> On Fri, Feb 20, 2026 at 10:19:06AM +0100, Luca Weiss wrote:
>> Document the property to allow using this sensor with io-channels.
>
> Why? On the surface, it looks like you are adding a provider property to 
> a consumer. Maybe that's right? 

Correct, commit 3762f5851ac5a65dcccadf73dbe853b1b346f561
("thermal/drivers/thermal-generic-adc: Add temperature sensor channel")
added support for this in the driver, this documents the dt parts of it.

Regards
Luca

>
>> 
>> Signed-off-by: Luca Weiss <luca.weiss@fairphone.com>
>> ---
>>  Documentation/devicetree/bindings/thermal/generic-adc-thermal.yaml | 4 ++++
>>  1 file changed, 4 insertions(+)
>> 
>> diff --git a/Documentation/devicetree/bindings/thermal/generic-adc-thermal.yaml b/Documentation/devicetree/bindings/thermal/generic-adc-thermal.yaml
>> index 12e6418dc24d..317fc39ed46a 100644
>> --- a/Documentation/devicetree/bindings/thermal/generic-adc-thermal.yaml
>> +++ b/Documentation/devicetree/bindings/thermal/generic-adc-thermal.yaml
>> @@ -24,6 +24,9 @@ properties:
>>    '#thermal-sensor-cells':
>>      const: 0
>>  
>> +  "#io-channel-cells":
>> +    const: 0
>> +
>>    io-channels:
>>      maxItems: 1
>>  
>> @@ -58,6 +61,7 @@ examples:
>>      thermal-sensor {
>>          compatible = "generic-adc-thermal";
>>          #thermal-sensor-cells = <0>;
>> +        #io-channel-cells = <0>;
>>          io-channels = <&ads1015 1>;
>>          io-channel-names = "sensor-channel";
>>          temperature-lookup-table = <
>> 
>> -- 
>> 2.53.0
>>