Re: [PATCH v2 1/8] dt-bindings: iio: adc: Add reference-sources property

[Conor Dooley <conor@kernel.org>]

[-- Attachment #1: Type: text/plain, Size: 1647 bytes --]

On Thu, Jun 25, 2026 at 04:55:03PM -0500, David Lechner wrote:
> From: Kurt Borja <kuurtb@gmail.com>
> 
> Some ADCs have configurable voltage reference sources for each channel.
> 
> Signed-off-by: Kurt Borja <kuurtb@gmail.com>
> Signed-off-by: David Lechner (TI) <dlechner@baylibre.com>
> ---
> v2 changes (compared to Kurt's RFC v2):
> * Drop maxItems.
> * Drop 2nd paragraph of description.
> * Make reference-sources plural since it is an array.
> ---
>  Documentation/devicetree/bindings/iio/adc/adc.yaml | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/iio/adc/adc.yaml b/Documentation/devicetree/bindings/iio/adc/adc.yaml
> index b9bc02b5b07a..12800737d4fb 100644
> --- a/Documentation/devicetree/bindings/iio/adc/adc.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/adc.yaml
> @@ -73,6 +73,14 @@ properties:
>        device design and can interact with other characteristics such as
>        settling time.
>  
> +  reference-sources:
> +    $ref: /schemas/types.yaml#/definitions/string-array

Either Kurt didn't use strings, or I didn't notice they did, but the
property is much clearer about how custom it is with undefined strings
than with numbers.

Acked-by: Conor Dooley <conor.dooley@microchip.com>
pw-bot: not-applicable

> +    minItems: 1
> +    description:
> +      Indicates the voltage reference source or sources for this channel. Some
> +      ADCs usually allow choosing between internal reference sources or a pair
> +      of external pins.
> +
>  anyOf:
>    - oneOf:
>        - required:
> 
> -- 
> 2.43.0
> 
> 

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]