[PATCH 02/10] arm64: dts: rockchip: standardize the definition of LEDs for Radxa ROCK Pi E
Krzysztof Kozlowski
krzk at kernel.org
Thu Aug 8 03:52:06 PDT 2024
On 08/08/2024 11:38, FUKAUMI Naoki wrote:
> - sort properties
You mean un-sort?!?!
> - add default-state and function
Commit subject is totally wrong. Sorry, write specific subjects
describing what you are doing. There is no standardization happening
here, whatever that means.
>
> Signed-off-by: FUKAUMI Naoki <naoki at radxa.com>
> ---
> arch/arm64/boot/dts/rockchip/rk3328-rock-pi-e.dtsi | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/rockchip/rk3328-rock-pi-e.dtsi b/arch/arm64/boot/dts/rockchip/rk3328-rock-pi-e.dtsi
> index bb01143dc91a..4e77701f2e01 100644
> --- a/arch/arm64/boot/dts/rockchip/rk3328-rock-pi-e.dtsi
> +++ b/arch/arm64/boot/dts/rockchip/rk3328-rock-pi-e.dtsi
> @@ -49,11 +49,13 @@ gmac_clkin: external-gmac-clock {
>
> leds {
> compatible = "gpio-leds";
> - pinctrl-0 = <&led_pin>;
> pinctrl-names = "default";
> + pinctrl-0 = <&led_pin>;
Nope, that's wrong order.
>
Best regards,
Krzysztof
More information about the Linux-rockchip
mailing list