Re: [PATCH 1/3] dt-bindings: iio: adc: Update title and enum

[Jonathan Cameron <jic23@kernel.org>]

On Sun,  7 Jun 2026 20:35:40 +0200
Jakub Szczudlo <jakubszczudlo40@gmail.com> wrote:

> From: jszczudlo <jakubszczudlo40@gmail.com>
> 
> Add ads1110 into title and enum compatible for ads1100 driver.
> 
> Signed-off-by: jszczudlo <jakubszczudlo40@gmail.com>
Hi Jakub,

Obviously not related to your patch, but can we switch existing
pair of entries to be in alphabetical order whilst we do this?
Normally I'd ask for a separate patch for that, but given it's
so trivial and that would create churn vs your patch, do it
in one go (but make sure to mention it in the patch description!)



> ---
>  Documentation/devicetree/bindings/iio/adc/ti,ads1100.yaml | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/iio/adc/ti,ads1100.yaml b/Documentation/devicetree/bindings/iio/adc/ti,ads1100.yaml
> index 970ccab15e1e..4c43996b7ec6 100644
> --- a/Documentation/devicetree/bindings/iio/adc/ti,ads1100.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/ti,ads1100.yaml
> @@ -4,7 +4,7 @@
>  $id: http://devicetree.org/schemas/iio/adc/ti,ads1100.yaml#
>  $schema: http://devicetree.org/meta-schemas/core.yaml#
>  
> -title: TI ADS1100/ADS1000 single channel I2C analog to digital converter
> +title: TI ADS1100/ADS1000/ADS1110 single channel I2C analog to digital converter
This is on the edge of the normal problem of never ending title growth
that we cheat to solve by saying something like

title: TI ADS1100 and similar single channel I2C Analog to Digital Converters

When we do that though we tend to include a bulleted list of parts in the
description: section.  Whilst I was looking at that I noticed we only reference
one datasheet which I'd assume doesn't cover this new part?  Perhaps that needs
updating as well?

Thanks,

Jonathan
>  
>  maintainers:
>    - Mike Looijmans <mike.looijmans@topic.nl>
> @@ -17,6 +17,7 @@ properties:
>      enum:
>        - ti,ads1100
>        - ti,ads1000
> +      - ti,ads1110
>  
>    reg:
>      maxItems: 1