Messages in this thread | | | Date | Fri, 2 Feb 2024 13:07:51 +0000 | From | Conor Dooley <> | Subject | Re: [PATCH v3] dt-bindings: iio: afe: voltage-divider: Add io-channel-cells |
| |
On Fri, Feb 02, 2024 at 12:49:26PM +0100, Peter Rosin wrote: > 2024-02-02 at 11:43, Naresh Solanki wrote: > > On Wed, 31 Jan 2024 at 22:24, Conor Dooley <conor@kernel.org> wrote: > >> On Wed, Jan 31, 2024 at 04:35:16PM +0000, Jonathan Cameron wrote: > >>> On Wed, 31 Jan 2024 09:29:59 +0100 > >>> Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> wrote: > >>>> On 30/01/2024 12:56, Naresh Solanki wrote: > >>> Conor requested an example of the device acting as a consumer and a provider. > >>> Might have meant in the patch description? > >>> > >>> Conor? > >> > >> I wanted it in the property description to help with understanding when > >> to use it. I don't think the extra example nodes actually help you > >> understand what it is doing, only how to write one yourself once you > >> know you need it. > > > > I'm not sure if I get it right but what I understood is that a > > voltage-divider can > > also be a provider to other devices & hence the property. > > Also do you want me to put a complete example of it in description ? > > My understanding is the requested example in the description should not > be exactly /how/ to hook up the voltage-divider as a provider, but > instead have some words about why it is interesting to do so at all. And > those words would also make it clear that is even possible. The latter > is something which, to be honest, is perhaps not all that obvious. It > has always been totally obvious to me of course, sorry for not being > clearer when I wrote the binding...
Yeah, you're right about what I was looking for Peter.
In my original request, which I think I already linked to in this thread, I said that I would like an example like the one that Peter had used to explain to me the scenario in which someone would want to use this feature: https://lore.kernel.org/all/536971eb-51f0-40e5-d025-7c4c1d683d49@axentia.se/
Cheers, Conor. [unhandled content-type:application/pgp-signature] | |