Re: [PATCH v1 1/4] regulator: dt-bindings: maxim,max8907: Document io-channel-cells property

[Jonathan Cameron <jic23@kernel.org>]

On Sun,  8 Nov 2026 11:56:31 +0200
Svyatoslav Ryhel <clamor95@gmail.com> wrote:

> The Maxim MAX8907 has a built-in 8-channel ADC. It does not require any
> Device Tree (DT) based configuration and is folded into the main PMIC
> node. Document this behavior.

The actual addition here is to allow other devices in DT to consume
the channels.  There is nothing stopping you having an ADC driver
without that which would need nothing at all in DT.

So this patch description is not accurate. The actual change is fine.

> 
> Signed-off-by: Svyatoslav Ryhel <clamor95@gmail.com>
> ---
>  Documentation/devicetree/bindings/regulator/maxim,max8907.yaml | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/regulator/maxim,max8907.yaml b/Documentation/devicetree/bindings/regulator/maxim,max8907.yaml
> index ee1a305d1c210..ccdeb9ba21e26 100644
> --- a/Documentation/devicetree/bindings/regulator/maxim,max8907.yaml
> +++ b/Documentation/devicetree/bindings/regulator/maxim,max8907.yaml
> @@ -27,6 +27,9 @@ properties:
>  
>    system-power-controller: true
>  
> +  "#io-channel-cells":
> +    const: 1
> +
>    mbatt-supply:
>      description:
>        The input supply for MBATT, BBAT, SDBY, VRTC.