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

From: Conor Dooley

Date: Fri Jun 26 2026 - 12:24:43 EST


On Thu, Jun 25, 2026 at 04:55:03PM -0500, David Lechner wrote:
> From: Kurt Borja <kuurtb@xxxxxxxxx>
>
> Some ADCs have configurable voltage reference sources for each channel.
>
> Signed-off-by: Kurt Borja <kuurtb@xxxxxxxxx>
> Signed-off-by: David Lechner (TI) <dlechner@xxxxxxxxxxxx>
> ---
> 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@xxxxxxxxxxxxx>
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: signature.asc
Description: PGP signature