Re: [PATCH v2 3/3] dt-bindings: leds: backlight: Convert TPS65217 to DT schema
Eddie <[email protected]> Mon, 20 Jul 2026 17:07:19 +0300
| Newsgroups | org.kernel.vger.linux-omap,org.freedesktop.lists.dri-devel,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-input,org.kernel.vger.linux-kernel,org.kernel.vger.linux-leds |
|---|---|
| Message-ID | <CAEFOc61Tan85xQhz33BuGSnJHyACvjRc+6YMFYav85CzxMwiwA@mail.gmail.com> |
On Mon, Jul 20, 2026 at 09:54, Krzysztof Kozlowski <[email protected]> wrote: > > On Sat, Jul 18, 2026 at 09:56:32PM +0000, Eduard Bostina wrote: > + > > + regulator-compatible: > > + $ref: /schemas/types.yaml#/definitions/string > > You should not need this property, it is already in regulator.yaml. I did not find it in regulator.yaml, or in any other schema, but I found out that it has been deprecated in 2012 by commit 13511def87b9 ("regulator: deprecate regulator-compatible DT property") and was never carried over during the conversion. I agree it should not be in a new binding, so I will drop it. It needs a dts change too, tps65217.dtsi names the children [email protected]@6, while the driver matches dcdc1, dcdc2, dcdc3, ldo1…ldo4. of_regulator_match() looks at regulator-compatible first and falls back to the node name, so without the property nothing matches and the regulators stop probing. So I will rename the nodes to dcdc1..ldo4 and drop regulator-compatible, reg and the address cells. I will send that as a separate patch in v3 along with the other fixes. Thanks, Eduard