Received-SPF: pass (google.com: domain of linux-kernel+bounces-41323-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Date: Sat, 27 Jan 2024 16:48:04 +0000
From: Conor Dooley <conor@kernel.org>
To: Jonathan Cameron <jic23@kernel.org>
Cc: Peter Rosin <peda@axentia.se>,
	Naresh Solanki <naresh.solanki@9elements.com>,
	Lars-Peter Clausen <lars@metafoo.de>,
	Rob Herring <robh+dt@kernel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	Conor Dooley <conor+dt@kernel.org>, mazziesaccount@gmail.com,
	linux-iio@vger.kernel.org, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH] dt-bindings: iio: afe: voltage-divider: Add
 io-channel-cells
Message-ID: <20240127-capitol-cabbage-7a9b67112a28@spud>
References: <20240126115509.1459425-1-naresh.solanki@9elements.com>
 <20240126-cinnamon-flatware-e042b5773f17@spud>
 <CABqG17hzZf2mme0v7hALhpd6-N3ZHqxdH-AhFg5eF9sbLSC2gw@mail.gmail.com>
 <20240126-scale-serrated-33686467d91b@spud>
 <CABqG17jp6YRGyTmNitz-xDdyhWOPgfT_XpXxw-OJLnXQ777vAA@mail.gmail.com>
 <20240126-blaspheme-calculate-a4134dc1ed68@spud>
 <536971eb-51f0-40e5-d025-7c4c1d683d49@axentia.se>
 <20240127-hunting-wick-fc1eed1af6b1@spud>
 <20240127144920.455b6f0c@jic23-huawei>
Precedence: bulk
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
	protocol="application/pgp-signature"; boundary="GjKLRvODLoSn+Ea7"
Content-Disposition: inline
In-Reply-To: <20240127144920.455b6f0c@jic23-huawei>


--GjKLRvODLoSn+Ea7
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Sat, Jan 27, 2024 at 02:49:20PM +0000, Jonathan Cameron wrote:

> > > It's all in the description of the binding... =20
> >=20
> > Obviously it was not sufficiently clear, it's not as if I didn't look at
> > it...
>=20
> Given this device fits in both categories, perhaps a tiny bit of
> additional documentation would help?

That would be nice.

>   '#io-channels-cells':
>     description:
>       In addition to consuming the measurement services of an ADC,
>       the voltage divider can act as an provider of measurement
>       services to other devices.
>     const: 1

But I am not sure that that covers things. I think an example, like
Peter gave, would be good?

--GjKLRvODLoSn+Ea7
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZbUzwAAKCRB4tDGHoIJi
0gVsAQCylbDj315fDC3IIidM7TlZtqSvlluKUx0X6SSfvlLphQEAqA/uvT4AisrC
flXHy0rxQQ4/TCWb21WrmXXQV3hbVwc=
=7YLY
-----END PGP SIGNATURE-----

--GjKLRvODLoSn+Ea7--