Message ID | 20221127203034.54092-2-krzysztof.kozlowski@linaro.org |
---|---|
State | New |
Headers | show |
Series | [v2,1/2] ARM: dts: omap: align LED node names with dtschema | expand |
* Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> [221127 20:30]: > The preferred name suffix for properties with single and multiple GPIOs > is "gpios". Linux GPIO core code supports both. Bindings are going to > expect the "gpios" one: > > omap3-echo.dtb: lp5523A@32: 'enable-gpio' does not match any of the regexes: '^led@[0-8]$', '^multi-led@[0-8]$', 'pinctrl-[0-9]+' Feel free to apply this along with your other clean up patches: Acked-by: Tony Lindgren <tony@atomide.com>
diff --git a/arch/arm/boot/dts/omap3-echo.dts b/arch/arm/boot/dts/omap3-echo.dts index 8f02ff5e7da6..06d2377d28ad 100644 --- a/arch/arm/boot/dts/omap3-echo.dts +++ b/arch/arm/boot/dts/omap3-echo.dts @@ -146,7 +146,7 @@ lp5523A: lp5523A@32 { label = "q1"; reg = <0x32>; clock-mode = /bits/ 8 <0>; /* LP55XX_CLOCK_AUTO */ - enable-gpio = <&gpio4 13 GPIO_ACTIVE_HIGH>; /* GPIO_109 */ + enable-gpios = <&gpio4 13 GPIO_ACTIVE_HIGH>; /* GPIO_109 */ multi-led@0 { #address-cells = <1>;
The preferred name suffix for properties with single and multiple GPIOs is "gpios". Linux GPIO core code supports both. Bindings are going to expect the "gpios" one: omap3-echo.dtb: lp5523A@32: 'enable-gpio' does not match any of the regexes: '^led@[0-8]$', '^multi-led@[0-8]$', 'pinctrl-[0-9]+' Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> --- Changes since v1: 1. Drop old fix for n900 and correct instead Echo board (Sebastian) --- arch/arm/boot/dts/omap3-echo.dts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)