Claude review: dt-bindings: backlight: Add max25014 support

Claude Code Review Bot
2026-04-12T03:25:20Z

Patch Review

This patch adds the YAML binding and MAINTAINERS entry. It has Rob Herring's Reviewed-by.

**Minor observations:**

1. The `maxim,strings` description says "A 4-bit bitfield" but the representation is a 4-element array of `uint32`, not a bitfield. The description is slightly misleading — it's an array of 4 enable/disable flags, not a bitfield. Consider: "An array of 4 values (0 or 1) describing which LED strings to enable."

2. The binding declares `interrupts` and `pwms` properties but the driver doesn't implement either of these features. This is acceptable for forward-compatibility as documented in the cover letter, but it's worth noting.

3. The `default-brightness` property lacks a `$ref` to a type schema. While it will likely be inferred, adding `$ref: /schemas/types.yaml#/definitions/uint32` would be consistent with `maxim,iset`.

**No blocking issues.** The binding is clean and the Rob R-b carries weight.