On 8/22/25 09:26, Daniel Thompson wrote:
On Tue, Aug 19, 2025 at 12:58:59PM +0200, Maud Spierings via B4 Relay wrote:
From: Maud Spierings <maudspieri...@gocontroll.com>

The Maxim MAX25014 is a 4-channel automotive grade backlight driver IC
with intgrated boost controller.

Signed-off-by: Maud Spierings <maudspieri...@gocontroll.com>
---
  .../bindings/leds/backlight/maxim,max25014.yaml    | 79 ++++++++++++++++++++++
  MAINTAINERS                                        |  5 ++
  2 files changed, 84 insertions(+)

diff --git 
a/Documentation/devicetree/bindings/leds/backlight/maxim,max25014.yaml 
b/Documentation/devicetree/bindings/leds/backlight/maxim,max25014.yaml
new file mode 100644
index 
0000000000000000000000000000000000000000..30b591152fa31d5e43243cac44c72028b05b5f8a
--- /dev/null
+++ b/Documentation/devicetree/bindings/leds/backlight/maxim,max25014.yaml
@@ -0,0 +1,79 @@
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/leds/backlight/maxim,max25014.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Maxim max25014 backlight controller
+
+maintainers:
+  - Maud Spierings <maudspieri...@gocontroll.com>
+
+allOf:
+  - $ref: common.yaml#
+
+properties:
+  compatible:
+    enum:
+      - maxim,max25014
+
+  reg:
+    maxItems: 1
+
+  enable-gpios:
+    maxItems: 1
+
+  interrupts:
+    maxItems: 1

Is the interrupt useful for anything?

AFAIK its perfectly OK for DT bindings to describe hardware features the
driver does not use... but is the driver missing out on error reporting
(for example) by not using this interrupt for anything?

The interrupt is indeed for error reporting, it is not wired up in the driver right now (although there is already a function for scanning the error registers, I am not sure how I would properly hook that up to an irq. This is my first driver so I thought to keep things simple for now, these extra functions can be added later), same with the pwms for example. But jeah I wanted to fully describe the hardware in the bindings as from what I understand that is what should be done.

Kind regards,
Maud

Reply via email to