Re: [PATCH V3 1/4] dt-bindings: hwmon: ina3221: Convert to json-schema

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On 26/09/2023 10:18, Conor Dooley wrote:
On Mon, Sep 25, 2023 at 11:46:58AM +0100, Jon Hunter wrote:


On 22/09/2023 22:01, Rob Herring wrote:
On Thu, Sep 21, 2023 at 02:08:15PM +0100, Jon Hunter wrote:
From: Ninad Malwade <nmalwade@xxxxxxxxxx>

Convert the TI INA3221 bindings from the free-form text format to
json-schema.

Note that the INA3221 input channels default to enabled in the chip.
Unless channels are explicitly disabled in device-tree, input
channels will be enabled.

Signed-off-by: Thierry Reding <treding@xxxxxxxxxx>
Signed-off-by: Ninad Malwade <nmalwade@xxxxxxxxxx>
Signed-off-by: Jon Hunter <jonathanh@xxxxxxxxxx>
---
   .../devicetree/bindings/hwmon/ina3221.txt     | 54 ----------
   .../devicetree/bindings/hwmon/ti,ina3221.yaml | 98 +++++++++++++++++++
   2 files changed, 98 insertions(+), 54 deletions(-)
   delete mode 100644 Documentation/devicetree/bindings/hwmon/ina3221.txt
   create mode 100644 Documentation/devicetree/bindings/hwmon/ti,ina3221.yaml


...

+examples:
+  - |
+    i2c {
+        #address-cells = <1>;
+        #size-cells = <0>;
+
+        power-sensor@40 {
+            compatible = "ti,ina3221";
+            reg = <0x40>;
+            #address-cells = <1>;
+            #size-cells = <0>;
+
+            input@0 {
+                reg = <0x0>;
+                status = "disabled";

Examples should be enabled.


Yes normally that would be the case. However, per the discussion with
Guenter and the comment in the changelog, for this device channels are
enabled in the chip by default. And so to disable them, we need to
explicitly disable in device-tree.

Maybe a comment at this location would be good then, to point out that
this is what you are trying to demonstrate with this example?


OK yes I think that that will help convey that this is intentional in this case.

Thanks
Jon

--
nvpublic



[Index of Archives]     [ARM Kernel]     [Linux ARM]     [Linux ARM MSM]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux