Thread (10 messages) 10 messages, 2 authors, 2025-08-07

Re: [PATCH v3 3/4] dt-bindings: trivial-devices: Document max31785 sensors

From: Rob Herring <robh@kernel.org>
Date: 2025-08-06 22:59:47
Also in: linux-aspeed

On Wed, Aug 6, 2025 at 5:04 PM Rob Herring [off-list ref] wrote:
On Wed, Aug 6, 2025 at 1:47 PM Eddie James [off-list ref] wrote:
quoted
Remove the old .txt max31785 documentation and add the compatibles
to trivial-devices.yaml.

Signed-off-by: Eddie James <eajames@linux.ibm.com>
---
 .../devicetree/bindings/hwmon/max31785.txt    | 22 -------------------
 .../devicetree/bindings/trivial-devices.yaml  |  4 ++++
 2 files changed, 4 insertions(+), 22 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/hwmon/max31785.txt
diff --git a/Documentation/devicetree/bindings/hwmon/max31785.txt b/Documentation/devicetree/bindings/hwmon/max31785.txt
deleted file mode 100644
index 106e08c56aaa..000000000000
--- a/Documentation/devicetree/bindings/hwmon/max31785.txt
+++ /dev/null
@@ -1,22 +0,0 @@
-Bindings for the Maxim MAX31785 Intelligent Fan Controller
-==========================================================
-
-Reference:
-
-https://datasheets.maximintegrated.com/en/ds/MAX31785.pdf
-
-The Maxim MAX31785 is a PMBus device providing closed-loop, multi-channel fan
-management with temperature and remote voltage sensing. Various fan control
-features are provided, including PWM frequency control, temperature hysteresis,
-dual tachometer measurements, and fan health monitoring.
While technically the binding is trivial, I don't think this device
really is. It has got 6 PWMs and 6 tach inputs, a reset line, 2
interrupts (alert and fault?), and an I2C master. Not really trivial.

However, better to have this documented as a schema than not, so I'll apply it.
I take that back. You already have 'fan' child nodes, so this isn't a
trivial device/binding.

Rob
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help