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@xxxxxxxxxx> wrote: > > > On 30/01/2024 12:56, Naresh Solanki wrote: > > > voltage-divider is always an iio consumer at the same time it is > > > optionally an iio provider. > > > Hence add #io-channel-cells > > > Also update example. > > > > > > > Fix > > wrapping > > and > > proper > > sentences. Each sentence finishes with full stop. > > > > ... > > > output-ohms: > > > description: > > > Resistance Rout over which the output voltage is measured. See full-ohms. > > > @@ -75,12 +82,17 @@ examples: > > > spi-max-frequency = <1000000>; > > > }; > > > }; > > > - sysv { > > > + p12v_vd: sysv { > > > > No, drop label. > > > > > compatible = "voltage-divider"; > > > io-channels = <&maxadc 1>; > > > + #io-channel-cells = <1>; > > > > > > /* Scale the system voltage by 22/222 to fit the ADC range. */ > > > output-ohms = <22>; > > > full-ohms = <222>; /* 200 + 22 */ > > > }; > > > + iio-hwmon { > > > + compatible = "iio-hwmon"; > > > + io-channels = <&p12v_vd 0>; > > > > The same question as for v2. Drop unrelated example. > > 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. Thanks, Conor.
Attachment:
signature.asc
Description: PGP signature