mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH v3 05/17] dt-bindings: iio: adc: ad7768-1: document regulator provider property
@ 2025-02-11 23:47 Jonathan Santos
  2025-02-12  7:17 ` Krzysztof Kozlowski
  0 siblings, 1 reply; 3+ messages in thread
From: Jonathan Santos @ 2025-02-11 23:47 UTC (permalink / raw)
  To: linux-iio, devicetree, linux-kernel
  Cc: Jonathan Santos, lars, Michael.Hennerich, marcelo.schmitt, jic23,
	robh, krzk+dt, conor+dt, jonath4nns, marcelo.schmitt1

The AD7768-1 provides a buffered common-mode voltage output
on the VCM pin that can be used to bias analog input signals.

Add regulators property to enable the use of the VCM output,
referenced here as vcm_output, by any other device.

Signed-off-by: Jonathan Santos <Jonathan.Santos@analog.com>
---
v3 Changes:
* VCM is now provided as a regulator within the device, instead of a 
  custom property.

v2 Changes:
* New patch in v2.
---
 .../bindings/iio/adc/adi,ad7768-1.yaml        | 21 +++++++++++++++++++
 1 file changed, 21 insertions(+)

diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7768-1.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7768-1.yaml
index e2f9782b5fc8..38f7bb0a0e20 100644
--- a/Documentation/devicetree/bindings/iio/adc/adi,ad7768-1.yaml
+++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7768-1.yaml
@@ -59,6 +59,19 @@ properties:
       in any way, for example if the filter decimation rate changes.
       As the line is active low, it should be marked GPIO_ACTIVE_LOW.
 
+  regulators:
+    type: object
+    description:
+      list of regulators provided by this controller.
+
+    properties:
+      vcm_output:
+        $ref: /schemas/regulator/regulator.yaml#
+        type: object
+        unevaluatedProperties: false
+
+    additionalProperties: false
+
   reset-gpios:
     maxItems: 1
 
@@ -152,6 +165,14 @@ examples:
                 reg = <0>;
                 label = "channel_0";
             };
+
+            regulators {
+              vcm_reg: vcm_output {
+                regulator-name = "vcm_output";
+                regulator-min-microvolt = <900000>;
+                regulator-max-microvolt = <2500000>;
+              };
+            };
         };
     };
 ...
-- 
2.34.1


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH v3 05/17] dt-bindings: iio: adc: ad7768-1: document regulator provider property
  2025-02-11 23:47 [PATCH v3 05/17] dt-bindings: iio: adc: ad7768-1: document regulator provider property Jonathan Santos
@ 2025-02-12  7:17 ` Krzysztof Kozlowski
  2025-02-12 15:01   ` Jonathan Santos
  0 siblings, 1 reply; 3+ messages in thread
From: Krzysztof Kozlowski @ 2025-02-12  7:17 UTC (permalink / raw)
  To: Jonathan Santos
  Cc: linux-iio, devicetree, linux-kernel, lars, Michael.Hennerich,
	marcelo.schmitt, jic23, robh, krzk+dt, conor+dt, jonath4nns,
	marcelo.schmitt1

On Tue, Feb 11, 2025 at 08:47:16PM -0300, Jonathan Santos wrote:
> The AD7768-1 provides a buffered common-mode voltage output
> on the VCM pin that can be used to bias analog input signals.
> 
> Add regulators property to enable the use of the VCM output,
> referenced here as vcm_output, by any other device.
> 
> Signed-off-by: Jonathan Santos <Jonathan.Santos@analog.com>
> ---
> v3 Changes:
> * VCM is now provided as a regulator within the device, instead of a 
>   custom property.
> 
> v2 Changes:
> * New patch in v2.

There is some mess in this posting. Nothing looks threaded and b4 does
not see entire posting.

Sorry, don't make it complicated to us. Send proper submission - see
numorous guides or submitting-patches - so reviewing will be
straightforward.

Best regards,
Krzysztof


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH v3 05/17] dt-bindings: iio: adc: ad7768-1: document regulator provider property
  2025-02-12  7:17 ` Krzysztof Kozlowski
@ 2025-02-12 15:01   ` Jonathan Santos
  0 siblings, 0 replies; 3+ messages in thread
From: Jonathan Santos @ 2025-02-12 15:01 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Jonathan Santos, linux-iio, devicetree, linux-kernel, lars,
	Michael.Hennerich, marcelo.schmitt, jic23, robh, krzk+dt,
	conor+dt, marcelo.schmitt1

On 02/12, Krzysztof Kozlowski wrote:
> On Tue, Feb 11, 2025 at 08:47:16PM -0300, Jonathan Santos wrote:
> > The AD7768-1 provides a buffered common-mode voltage output
> > on the VCM pin that can be used to bias analog input signals.
> > 
> > Add regulators property to enable the use of the VCM output,
> > referenced here as vcm_output, by any other device.
> > 
> > Signed-off-by: Jonathan Santos <Jonathan.Santos@analog.com>
> > ---
> > v3 Changes:
> > * VCM is now provided as a regulator within the device, instead of a 
> >   custom property.
> > 
> > v2 Changes:
> > * New patch in v2.
> 
> There is some mess in this posting. Nothing looks threaded and b4 does
> not see entire posting.
> 
> Sorry, don't make it complicated to us. Send proper submission - see
> numorous guides or submitting-patches - so reviewing will be
> straightforward.
> 
> Best regards,
> Krzysztof

I am sorry about that, my mistake. I missed the --thread option
when creating the patches. I will resend them properly. 

Apologies,
Jonathan S.

> 

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2025-02-12 15:01 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-02-11 23:47 [PATCH v3 05/17] dt-bindings: iio: adc: ad7768-1: document regulator provider property Jonathan Santos
2025-02-12  7:17 ` Krzysztof Kozlowski
2025-02-12 15:01   ` Jonathan Santos

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®