* [PATCH] arm64: dts: rockchip: arm64: make the system run light heartbeat
@ 2021-07-05 5:06 xiaobo
2021-07-05 8:20 ` Heiko Stuebner
0 siblings, 1 reply; 3+ messages in thread
From: xiaobo @ 2021-07-05 5:06 UTC (permalink / raw)
To: robh+dt, heiko, jagan; +Cc: linux-arm-kernel, linux-rockchip, xiaobo
Make the system run light heartbeat, remove the wrong supply voltage.
Signed-off-by: xiaobo <peterwillcn@gmail.com>
---
.../boot/dts/rockchip/rk3399-nanopi-r4s.dts | 49 ++++++-------------
1 file changed, 15 insertions(+), 34 deletions(-)
diff --git a/arch/arm64/boot/dts/rockchip/rk3399-nanopi-r4s.dts b/arch/arm64/boot/dts/rockchip/rk3399-nanopi-r4s.dts
index fa5809887643..899c6a0fc6a1 100644
--- a/arch/arm64/boot/dts/rockchip/rk3399-nanopi-r4s.dts
+++ b/arch/arm64/boot/dts/rockchip/rk3399-nanopi-r4s.dts
@@ -1,14 +1,15 @@
// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
/*
- * FriendlyElec NanoPC-T4 board device tree source
- *
+ * Copyright (c) 2016 Fuzhou Rockchip Electronics Co., Ltd
* Copyright (c) 2020 FriendlyElec Computer Tech. Co., Ltd.
* (http://www.friendlyarm.com)
*
+ * Copyright (c) 2016 Arm Ltd.
* Copyright (c) 2018 Collabora Ltd.
*
* Copyright (c) 2020 Jensen Huang <jensenhuang@friendlyarm.com>
* Copyright (c) 2020 Marty Jones <mj8263788@gmail.com>
+ * Copyright (C) 2020 Xiaobo Tian <peterwillcn@gmail.com>
* Copyright (c) 2021 Tianling Shen <cnsztl@gmail.com>
*/
@@ -21,28 +22,6 @@ / {
/delete-node/ display-subsystem;
- gpio-leds {
- pinctrl-0 = <&lan_led_pin>, <&sys_led_pin>, <&wan_led_pin>;
-
- /delete-node/ led-0;
-
- lan_led: led-lan {
- gpios = <&gpio1 RK_PA1 GPIO_ACTIVE_HIGH>;
- label = "green:lan";
- };
-
- sys_led: led-sys {
- gpios = <&gpio0 RK_PB5 GPIO_ACTIVE_HIGH>;
- label = "red:sys";
- default-state = "on";
- };
-
- wan_led: led-wan {
- gpios = <&gpio1 RK_PA0 GPIO_ACTIVE_HIGH>;
- label = "green:wan";
- };
- };
-
gpio-keys {
pinctrl-0 = <&reset_button_pin>;
@@ -72,6 +51,18 @@ &i2c4 {
status = "disabled";
};
+&leds {
+ lan_led: led-1 {
+ gpios = <&gpio1 RK_PA1 GPIO_ACTIVE_HIGH>;
+ label = "green:lan";
+ };
+
+ wan_led: led-2 {
+ gpios = <&gpio1 RK_PA0 GPIO_ACTIVE_HIGH>;
+ label = "green:wan";
+ };
+};
+
&pcie0 {
max-link-speed = <1>;
num-lanes = <1>;
@@ -80,16 +71,10 @@ &pcie0 {
&pinctrl {
gpio-leds {
- /delete-node/ status-led-pin;
-
lan_led_pin: lan-led-pin {
rockchip,pins = <1 RK_PA1 RK_FUNC_GPIO &pcfg_pull_none>;
};
- sys_led_pin: sys-led-pin {
- rockchip,pins = <0 RK_PB5 RK_FUNC_GPIO &pcfg_pull_none>;
- };
-
wan_led_pin: wan-led-pin {
rockchip,pins = <1 RK_PA0 RK_FUNC_GPIO &pcfg_pull_none>;
};
@@ -127,7 +112,3 @@ &uart0 {
&usbdrd_dwc3_0 {
dr_mode = "host";
};
-
-&vcc3v3_sys {
- vin-supply = <&vcc5v0_sys>;
-};
--
2.32.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH] arm64: dts: rockchip: arm64: make the system run light heartbeat
2021-07-05 5:06 [PATCH] arm64: dts: rockchip: arm64: make the system run light heartbeat xiaobo
@ 2021-07-05 8:20 ` Heiko Stuebner
0 siblings, 0 replies; 3+ messages in thread
From: Heiko Stuebner @ 2021-07-05 8:20 UTC (permalink / raw)
To: robh+dt, jagan, xiaobo
Cc: linux-arm-kernel, linux-rockchip, xiaobo, Chen-Yu Tsai
Hi,
Am Montag, 5. Juli 2021, 07:06:14 CEST schrieb xiaobo:
> Make the system run light heartbeat, remove the wrong supply voltage.
as mentioned in the other patch, this patch also does too many things
at once ... and includes unrelated changes like removing that FriendlyElec
header from the top comment.
Similarly a commit message should explain the "why" the change is needed.
The file previously defined 3 LEDs, with your change only 2, so does the
board only have 2 LEDs?
And when you're removing the vcc3v3 supply voltage, this should be
a separate patch and describe why defining that voltage is wrong.
Also please version your patches, as I got this patch 2 times with
1 hour distance, without any mention what changed between them.
The way this should look like is:
----- 8< -----
[PATCH v2] arm64: dts: ...
Commit message
---
changes in v2:
- foo
.../boot/dts/rockchip/rk3399-nanopi-r4s.dts | 49 ++++++-------------
1 file changed, 15 insertions(+), 34 deletions(-)
[...]
----- 8< -----
>
> Signed-off-by: xiaobo <peterwillcn@gmail.com>
Also please adjust the Signed-off-by line + patch authorship
to reflect your name :-)
I.e. below we can see
+ * Copyright (C) 2020 Xiaobo Tian <peterwillcn@gmail.com>
so Signed-off-by and author should match this please.
You can set this in the git config globally for future work:
$ git config --global user.name "Xiaobo Tian"
$ git config --global user.email peterwillcn@gmail.com
and also for individual commits that already exist with
$ git commit --amend --author="Xiaobo Tian <peterwillcn@gmail.com>"
Thanks
Heiko
> ---
> .../boot/dts/rockchip/rk3399-nanopi-r4s.dts | 49 ++++++-------------
> 1 file changed, 15 insertions(+), 34 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/rockchip/rk3399-nanopi-r4s.dts b/arch/arm64/boot/dts/rockchip/rk3399-nanopi-r4s.dts
> index fa5809887643..899c6a0fc6a1 100644
> --- a/arch/arm64/boot/dts/rockchip/rk3399-nanopi-r4s.dts
> +++ b/arch/arm64/boot/dts/rockchip/rk3399-nanopi-r4s.dts
> @@ -1,14 +1,15 @@
> // SPDX-License-Identifier: (GPL-2.0+ OR MIT)
> /*
> - * FriendlyElec NanoPC-T4 board device tree source
> - *
> + * Copyright (c) 2016 Fuzhou Rockchip Electronics Co., Ltd
> * Copyright (c) 2020 FriendlyElec Computer Tech. Co., Ltd.
> * (http://www.friendlyarm.com)
> *
> + * Copyright (c) 2016 Arm Ltd.
> * Copyright (c) 2018 Collabora Ltd.
> *
> * Copyright (c) 2020 Jensen Huang <jensenhuang@friendlyarm.com>
> * Copyright (c) 2020 Marty Jones <mj8263788@gmail.com>
> + * Copyright (C) 2020 Xiaobo Tian <peterwillcn@gmail.com>
> * Copyright (c) 2021 Tianling Shen <cnsztl@gmail.com>
> */
>
> @@ -21,28 +22,6 @@ / {
>
> /delete-node/ display-subsystem;
>
> - gpio-leds {
> - pinctrl-0 = <&lan_led_pin>, <&sys_led_pin>, <&wan_led_pin>;
> -
> - /delete-node/ led-0;
> -
> - lan_led: led-lan {
> - gpios = <&gpio1 RK_PA1 GPIO_ACTIVE_HIGH>;
> - label = "green:lan";
> - };
> -
> - sys_led: led-sys {
> - gpios = <&gpio0 RK_PB5 GPIO_ACTIVE_HIGH>;
> - label = "red:sys";
> - default-state = "on";
> - };
> -
> - wan_led: led-wan {
> - gpios = <&gpio1 RK_PA0 GPIO_ACTIVE_HIGH>;
> - label = "green:wan";
> - };
> - };
> -
> gpio-keys {
> pinctrl-0 = <&reset_button_pin>;
>
> @@ -72,6 +51,18 @@ &i2c4 {
> status = "disabled";
> };
>
> +&leds {
> + lan_led: led-1 {
> + gpios = <&gpio1 RK_PA1 GPIO_ACTIVE_HIGH>;
> + label = "green:lan";
> + };
> +
> + wan_led: led-2 {
> + gpios = <&gpio1 RK_PA0 GPIO_ACTIVE_HIGH>;
> + label = "green:wan";
> + };
> +};
> +
> &pcie0 {
> max-link-speed = <1>;
> num-lanes = <1>;
> @@ -80,16 +71,10 @@ &pcie0 {
>
> &pinctrl {
> gpio-leds {
> - /delete-node/ status-led-pin;
> -
> lan_led_pin: lan-led-pin {
> rockchip,pins = <1 RK_PA1 RK_FUNC_GPIO &pcfg_pull_none>;
> };
>
> - sys_led_pin: sys-led-pin {
> - rockchip,pins = <0 RK_PB5 RK_FUNC_GPIO &pcfg_pull_none>;
> - };
> -
> wan_led_pin: wan-led-pin {
> rockchip,pins = <1 RK_PA0 RK_FUNC_GPIO &pcfg_pull_none>;
> };
> @@ -127,7 +112,3 @@ &uart0 {
> &usbdrd_dwc3_0 {
> dr_mode = "host";
> };
> -
> -&vcc3v3_sys {
> - vin-supply = <&vcc5v0_sys>;
> -};
>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH] arm64: dts: rockchip: arm64: make the system run light heartbeat
@ 2021-07-05 4:25 xiaobo
0 siblings, 0 replies; 3+ messages in thread
From: xiaobo @ 2021-07-05 4:25 UTC (permalink / raw)
To: robh+dt, heiko, jagan; +Cc: linux-arm-kernel, linux-rockchip, xiaobo
Make the system run light heartbeat, remove the wrong supply voltage.
Signed-off-by: xiaobo <peterwillcn@gmail.com>
---
.../boot/dts/rockchip/rk3399-nanopi-r4s.dts | 23 +++----------------
1 file changed, 3 insertions(+), 20 deletions(-)
diff --git a/arch/arm64/boot/dts/rockchip/rk3399-nanopi-r4s.dts b/arch/arm64/boot/dts/rockchip/rk3399-nanopi-r4s.dts
index fa5809887643..943f8d7c7909 100644
--- a/arch/arm64/boot/dts/rockchip/rk3399-nanopi-r4s.dts
+++ b/arch/arm64/boot/dts/rockchip/rk3399-nanopi-r4s.dts
@@ -1,7 +1,5 @@
// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
/*
- * FriendlyElec NanoPC-T4 board device tree source
- *
* Copyright (c) 2020 FriendlyElec Computer Tech. Co., Ltd.
* (http://www.friendlyarm.com)
*
@@ -9,6 +7,7 @@
*
* Copyright (c) 2020 Jensen Huang <jensenhuang@friendlyarm.com>
* Copyright (c) 2020 Marty Jones <mj8263788@gmail.com>
+ * Copyright (C) 2020 Xiaobo Tian <peterwillcn@gmail.com>
* Copyright (c) 2021 Tianling Shen <cnsztl@gmail.com>
*/
@@ -24,20 +23,12 @@ / {
gpio-leds {
pinctrl-0 = <&lan_led_pin>, <&sys_led_pin>, <&wan_led_pin>;
- /delete-node/ led-0;
-
- lan_led: led-lan {
+ lan_led: led-1 {
gpios = <&gpio1 RK_PA1 GPIO_ACTIVE_HIGH>;
label = "green:lan";
};
- sys_led: led-sys {
- gpios = <&gpio0 RK_PB5 GPIO_ACTIVE_HIGH>;
- label = "red:sys";
- default-state = "on";
- };
-
- wan_led: led-wan {
+ wan_led: led-2 {
gpios = <&gpio1 RK_PA0 GPIO_ACTIVE_HIGH>;
label = "green:wan";
};
@@ -86,10 +77,6 @@ lan_led_pin: lan-led-pin {
rockchip,pins = <1 RK_PA1 RK_FUNC_GPIO &pcfg_pull_none>;
};
- sys_led_pin: sys-led-pin {
- rockchip,pins = <0 RK_PB5 RK_FUNC_GPIO &pcfg_pull_none>;
- };
-
wan_led_pin: wan-led-pin {
rockchip,pins = <1 RK_PA0 RK_FUNC_GPIO &pcfg_pull_none>;
};
@@ -127,7 +114,3 @@ &uart0 {
&usbdrd_dwc3_0 {
dr_mode = "host";
};
-
-&vcc3v3_sys {
- vin-supply = <&vcc5v0_sys>;
-};
--
2.32.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-07-05 8:25 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-07-05 5:06 [PATCH] arm64: dts: rockchip: arm64: make the system run light heartbeat xiaobo
2021-07-05 8:20 ` Heiko Stuebner
-- strict thread matches above, loose matches on Subject: below --
2021-07-05 4:25 xiaobo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox