public inbox for linux-hwmon@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2 23/52] dt-bindings: hwmon: Add IIO HWMON binding
       [not found] <20210901091852.479202-1-maxime@cerno.tech>
@ 2021-09-01  9:18 ` Maxime Ripard
  2021-09-03 19:33   ` Rob Herring
  2021-09-04 15:03   ` Guenter Roeck
  0 siblings, 2 replies; 3+ messages in thread
From: Maxime Ripard @ 2021-09-01  9:18 UTC (permalink / raw)
  To: devicetree, Rob Herring, Frank Rowand, Chen-Yu Tsai,
	Maxime Ripard, Jernej Škrabec
  Cc: linux-arm-kernel, linux-sunxi, Guenter Roeck, Jean Delvare,
	Jonathan Cameron, linux-hwmon

Even though we had the iio-hwmon driver for some time and a number of
boards using it already, we never had a binding for it. Let's add it
based on what the driver expects and the boards are providing.

Cc: Guenter Roeck <linux@roeck-us.net>
Cc: Jean Delvare <jdelvare@suse.com>
Cc: Jonathan Cameron <jic23@kernel.org>
Cc: linux-hwmon@vger.kernel.org
Signed-off-by: Maxime Ripard <maxime@cerno.tech>

---

Changes from v1:
  - Reduced the maximum number of io-channels, and added a comment for
    the limitation
---
 .../devicetree/bindings/hwmon/iio-hwmon.yaml  | 37 +++++++++++++++++++
 1 file changed, 37 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/hwmon/iio-hwmon.yaml

diff --git a/Documentation/devicetree/bindings/hwmon/iio-hwmon.yaml b/Documentation/devicetree/bindings/hwmon/iio-hwmon.yaml
new file mode 100644
index 000000000000..f5a6cc3efd33
--- /dev/null
+++ b/Documentation/devicetree/bindings/hwmon/iio-hwmon.yaml
@@ -0,0 +1,37 @@
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: "http://devicetree.org/schemas/hwmon/iio-hwmon.yaml#"
+$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+
+title: ADC-attached Hardware Sensor Device Tree Bindings
+
+maintainers:
+  - Jonathan Cameron <jic23@kernel.org>
+
+description: >
+  Bindings for hardware monitoring devices connected to ADC controllers
+  supporting the Industrial I/O bindings.
+
+properties:
+  compatible:
+    const: iio-hwmon
+
+  io-channels:
+    minItems: 1
+    maxItems: 8 # Should be enough
+    description: >
+      List of phandles to ADC channels to read the monitoring values
+
+required:
+  - compatible
+  - io-channels
+
+additionalProperties: false
+
+examples:
+  - |
+      iio-hwmon {
+          compatible = "iio-hwmon";
+          io-channels = <&adc 1>, <&adc 2>;
+      };
-- 
2.31.1


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

* Re: [PATCH v2 23/52] dt-bindings: hwmon: Add IIO HWMON binding
  2021-09-01  9:18 ` [PATCH v2 23/52] dt-bindings: hwmon: Add IIO HWMON binding Maxime Ripard
@ 2021-09-03 19:33   ` Rob Herring
  2021-09-04 15:03   ` Guenter Roeck
  1 sibling, 0 replies; 3+ messages in thread
From: Rob Herring @ 2021-09-03 19:33 UTC (permalink / raw)
  To: Maxime Ripard
  Cc: Jonathan Cameron, linux-sunxi, Chen-Yu Tsai, devicetree,
	Jean Delvare, linux-arm-kernel, Jernej Škrabec, Rob Herring,
	linux-hwmon, Guenter Roeck, Frank Rowand

On Wed, 01 Sep 2021 11:18:23 +0200, Maxime Ripard wrote:
> Even though we had the iio-hwmon driver for some time and a number of
> boards using it already, we never had a binding for it. Let's add it
> based on what the driver expects and the boards are providing.
> 
> Cc: Guenter Roeck <linux@roeck-us.net>
> Cc: Jean Delvare <jdelvare@suse.com>
> Cc: Jonathan Cameron <jic23@kernel.org>
> Cc: linux-hwmon@vger.kernel.org
> Signed-off-by: Maxime Ripard <maxime@cerno.tech>
> 
> ---
> 
> Changes from v1:
>   - Reduced the maximum number of io-channels, and added a comment for
>     the limitation
> ---
>  .../devicetree/bindings/hwmon/iio-hwmon.yaml  | 37 +++++++++++++++++++
>  1 file changed, 37 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/hwmon/iio-hwmon.yaml
> 

Reviewed-by: Rob Herring <robh@kernel.org>

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

* Re: [PATCH v2 23/52] dt-bindings: hwmon: Add IIO HWMON binding
  2021-09-01  9:18 ` [PATCH v2 23/52] dt-bindings: hwmon: Add IIO HWMON binding Maxime Ripard
  2021-09-03 19:33   ` Rob Herring
@ 2021-09-04 15:03   ` Guenter Roeck
  1 sibling, 0 replies; 3+ messages in thread
From: Guenter Roeck @ 2021-09-04 15:03 UTC (permalink / raw)
  To: Maxime Ripard
  Cc: devicetree, Rob Herring, Frank Rowand, Chen-Yu Tsai,
	Jernej Škrabec, linux-arm-kernel, linux-sunxi, Jean Delvare,
	Jonathan Cameron, linux-hwmon

On Wed, Sep 01, 2021 at 11:18:23AM +0200, Maxime Ripard wrote:
> Even though we had the iio-hwmon driver for some time and a number of
> boards using it already, we never had a binding for it. Let's add it
> based on what the driver expects and the boards are providing.
> 
> Cc: Guenter Roeck <linux@roeck-us.net>
> Cc: Jean Delvare <jdelvare@suse.com>
> Cc: Jonathan Cameron <jic23@kernel.org>
> Cc: linux-hwmon@vger.kernel.org
> Signed-off-by: Maxime Ripard <maxime@cerno.tech>
> Reviewed-by: Rob Herring <robh@kernel.org>

Applied (assuming this is supposed to be pushed through hwmon).

Thanks,
Guenter

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

end of thread, other threads:[~2021-09-04 15:03 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <20210901091852.479202-1-maxime@cerno.tech>
2021-09-01  9:18 ` [PATCH v2 23/52] dt-bindings: hwmon: Add IIO HWMON binding Maxime Ripard
2021-09-03 19:33   ` Rob Herring
2021-09-04 15:03   ` Guenter Roeck

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