Messages in this thread | | | Date | Thu, 15 Feb 2024 08:56:54 -0600 | From | Rob Herring <> | Subject | Re: [PATCH] dt-bindings: leds: pwm-multicolour: re-allow active-low |
| |
On Tue, 13 Feb 2024 20:13:41 +0000, Conor Dooley wrote: > From: Conor Dooley <conor.dooley@microchip.com> > > active-low was lifted to the common schema for leds, but it went > unnoticed that the leds-multicolour binding had "additionalProperties: > false" where the other users had "unevaluatedProperties: false", thereby > disallowing active-low for multicolour leds. Explicitly permit it again. > > Fixes: c94d1783136e ("dt-bindings: net: phy: Make LED active-low property common") > Signed-off-by: Conor Dooley <conor.dooley@microchip.com> > --- > > I'm just assuming this is intentionally restrictive, if its not, we > could easily just change this to uneval: false. > > CC: Pavel Machek <pavel@ucw.cz> > CC: Lee Jones <lee@kernel.org> > CC: Rob Herring <robh@kernel.org> > CC: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org> > CC: Conor Dooley <conor+dt@kernel.org> > CC: Sven Schwermer <sven.schwermer@disruptive-technologies.com> > CC: Christian Marangi <ansuelsmth@gmail.com> > CC: linux-leds@vger.kernel.org > CC: devicetree@vger.kernel.org > CC: linux-kernel@vger.kernel.org > --- > Documentation/devicetree/bindings/leds/leds-pwm-multicolor.yaml | 2 ++ > 1 file changed, 2 insertions(+) >
Acked-by: Rob Herring <robh@kernel.org>
| |