All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] arm64: rk3399: Optimize performance and cleanup
@ 2021-07-05 14:52 Xiaobo Tian
  0 siblings, 0 replies; 3+ messages in thread
From: Xiaobo Tian @ 2021-07-05 14:52 UTC (permalink / raw)
  To: u-boot; +Cc: Xiaobo Tian

1. Delete unused Ethernet aliases
2. Modify the label of the LEDS lamp
3. Inherit the definition of the system status lamp in DSTI
4. Removed error 3.3V incorrect use of 5V supply

Signed-off-by: Xiaobo Tian <peterwillcn@gmail.com>
---
 arch/arm/dts/rk3399-nanopi-r4s.dts | 40 +++++++++---------------------
 1 file changed, 12 insertions(+), 28 deletions(-)

diff --git a/arch/arm/dts/rk3399-nanopi-r4s.dts b/arch/arm/dts/rk3399-nanopi-r4s.dts
index 6f2cf17bf1..a85f4bf491 100644
--- a/arch/arm/dts/rk3399-nanopi-r4s.dts
+++ b/arch/arm/dts/rk3399-nanopi-r4s.dts
@@ -17,10 +17,6 @@
 	model = "FriendlyElec NanoPi R4S";
 	compatible = "friendlyarm,nanopi-r4s", "rockchip,rk3399";
 
-	aliases {
-		ethernet1 = &r8169;
-	};
-
 	vdd_5v: vdd-5v {
 		compatible = "regulator-fixed";
 		regulator-name = "vdd_5v";
@@ -30,6 +26,10 @@
 
 	fan: pwm-fan {
 		compatible = "pwm-fan";
+		/*
+		 * With 20KHz PWM and an EVERCOOL EC4007H12SA fan, these levels
+		 * work out to 0, ~1200, ~3000, and 5000RPM respectively.
+		 */
 		cooling-levels = <0 12 18 255>;
 		#cooling-cells = <2>;
 		fan-supply = <&vdd_5v>;
@@ -73,40 +73,32 @@
 	status = "disabled";
 };
 
+&i2c4 {
+	status = "disabled";
+};
+
 &leds {
 	lan_led: led-1 {
 		gpios = <&gpio1 RK_PA1 GPIO_ACTIVE_HIGH>;
-		label = "nanopi-r4s:green:lan";
+		label = "green:lan";
 	};
 
 	wan_led: led-2 {
 		gpios = <&gpio1 RK_PA0 GPIO_ACTIVE_HIGH>;
-		label = "nanopi-r4s:green:wan";
+		label = "green:wan";
 	};
 };
 
 &leds_gpio {
 	rockchip,pins =
-		<0 RK_PB5 RK_FUNC_GPIO &pcfg_pull_none>,
-		<1 RK_PA0 RK_FUNC_GPIO &pcfg_pull_none>,
-		<1 RK_PA1 RK_FUNC_GPIO &pcfg_pull_none>;
+		<1 RK_PA1 RK_FUNC_GPIO &pcfg_pull_none>,
+		<1 RK_PA0 RK_FUNC_GPIO &pcfg_pull_none>;
 };
 
 &pcie0 {
 	max-link-speed = <1>;
 	num-lanes = <1>;
 	vpcie3v3-supply = <&vcc3v3_sys>;
-
-	pcie@0 {
-		reg = <0x00000000 0 0 0 0>;
-		#address-cells = <3>;
-		#size-cells = <2>;
-
-		r8169: pcie@0,0 {
-			reg = <0x000000 0 0 0 0>;
-			local-mac-address = [ 00 00 00 00 00 00 ];
-		};
-	};
 };
 
 &sdhci {
@@ -117,10 +109,6 @@
 	status = "disabled";
 };
 
-&sdmmc {
-	host-index-min = <1>;
-};
-
 &u2phy0_host {
 	phy-supply = <&vdd_5v>;
 };
@@ -132,7 +120,3 @@
 &usbdrd_dwc3_0 {
 	dr_mode = "host";
 };
-
-&vcc3v3_sys {
-	vin-supply = <&vcc5v0_sys>;
-};
-- 
2.32.0


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] arm64: rk3399: Optimize performance and cleanup
  2021-07-05 14:21 Xiaobo Tian
@ 2021-07-05 14:32 ` Peter Robinson
  0 siblings, 0 replies; 3+ messages in thread
From: Peter Robinson @ 2021-07-05 14:32 UTC (permalink / raw)
  To: Xiaobo Tian; +Cc: u-boot, Kever Yang

I'm not sure what performance optimisation is being done here?

Also we generally sync the rockchip device trees from the Linux kernel
so when that next happens if these aren't upstream in Linux they'll
likely be lost.

Peter

On Mon, Jul 5, 2021 at 3:21 PM Xiaobo Tian <peterwillcn@gmail.com> wrote:
>
> Signed-off-by: Xiaobo Tian <peterwillcn@gmail.com>
> ---
>  arch/arm/dts/rk3399-nanopi-r4s.dts | 40 +++++++++---------------------
>  1 file changed, 12 insertions(+), 28 deletions(-)
>
> diff --git a/arch/arm/dts/rk3399-nanopi-r4s.dts b/arch/arm/dts/rk3399-nanopi-r4s.dts
> index 6f2cf17bf1..a85f4bf491 100644
> --- a/arch/arm/dts/rk3399-nanopi-r4s.dts
> +++ b/arch/arm/dts/rk3399-nanopi-r4s.dts
> @@ -17,10 +17,6 @@
>         model = "FriendlyElec NanoPi R4S";
>         compatible = "friendlyarm,nanopi-r4s", "rockchip,rk3399";
>
> -       aliases {
> -               ethernet1 = &r8169;
> -       };
> -
>         vdd_5v: vdd-5v {
>                 compatible = "regulator-fixed";
>                 regulator-name = "vdd_5v";
> @@ -30,6 +26,10 @@
>
>         fan: pwm-fan {
>                 compatible = "pwm-fan";
> +               /*
> +                * With 20KHz PWM and an EVERCOOL EC4007H12SA fan, these levels
> +                * work out to 0, ~1200, ~3000, and 5000RPM respectively.
> +                */
>                 cooling-levels = <0 12 18 255>;
>                 #cooling-cells = <2>;
>                 fan-supply = <&vdd_5v>;
> @@ -73,40 +73,32 @@
>         status = "disabled";
>  };
>
> +&i2c4 {
> +       status = "disabled";
> +};
> +
>  &leds {
>         lan_led: led-1 {
>                 gpios = <&gpio1 RK_PA1 GPIO_ACTIVE_HIGH>;
> -               label = "nanopi-r4s:green:lan";
> +               label = "green:lan";
>         };
>
>         wan_led: led-2 {
>                 gpios = <&gpio1 RK_PA0 GPIO_ACTIVE_HIGH>;
> -               label = "nanopi-r4s:green:wan";
> +               label = "green:wan";
>         };
>  };
>
>  &leds_gpio {
>         rockchip,pins =
> -               <0 RK_PB5 RK_FUNC_GPIO &pcfg_pull_none>,
> -               <1 RK_PA0 RK_FUNC_GPIO &pcfg_pull_none>,
> -               <1 RK_PA1 RK_FUNC_GPIO &pcfg_pull_none>;
> +               <1 RK_PA1 RK_FUNC_GPIO &pcfg_pull_none>,
> +               <1 RK_PA0 RK_FUNC_GPIO &pcfg_pull_none>;
>  };
>
>  &pcie0 {
>         max-link-speed = <1>;
>         num-lanes = <1>;
>         vpcie3v3-supply = <&vcc3v3_sys>;
> -
> -       pcie@0 {
> -               reg = <0x00000000 0 0 0 0>;
> -               #address-cells = <3>;
> -               #size-cells = <2>;
> -
> -               r8169: pcie@0,0 {
> -                       reg = <0x000000 0 0 0 0>;
> -                       local-mac-address = [ 00 00 00 00 00 00 ];
> -               };
> -       };
>  };
>
>  &sdhci {
> @@ -117,10 +109,6 @@
>         status = "disabled";
>  };
>
> -&sdmmc {
> -       host-index-min = <1>;
> -};
> -
>  &u2phy0_host {
>         phy-supply = <&vdd_5v>;
>  };
> @@ -132,7 +120,3 @@
>  &usbdrd_dwc3_0 {
>         dr_mode = "host";
>  };
> -
> -&vcc3v3_sys {
> -       vin-supply = <&vcc5v0_sys>;
> -};
> --
> 2.32.0
>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [PATCH] arm64: rk3399: Optimize performance and cleanup
@ 2021-07-05 14:21 Xiaobo Tian
  2021-07-05 14:32 ` Peter Robinson
  0 siblings, 1 reply; 3+ messages in thread
From: Xiaobo Tian @ 2021-07-05 14:21 UTC (permalink / raw)
  To: u-boot, kever.yang; +Cc: Xiaobo Tian

Signed-off-by: Xiaobo Tian <peterwillcn@gmail.com>
---
 arch/arm/dts/rk3399-nanopi-r4s.dts | 40 +++++++++---------------------
 1 file changed, 12 insertions(+), 28 deletions(-)

diff --git a/arch/arm/dts/rk3399-nanopi-r4s.dts b/arch/arm/dts/rk3399-nanopi-r4s.dts
index 6f2cf17bf1..a85f4bf491 100644
--- a/arch/arm/dts/rk3399-nanopi-r4s.dts
+++ b/arch/arm/dts/rk3399-nanopi-r4s.dts
@@ -17,10 +17,6 @@
 	model = "FriendlyElec NanoPi R4S";
 	compatible = "friendlyarm,nanopi-r4s", "rockchip,rk3399";
 
-	aliases {
-		ethernet1 = &r8169;
-	};
-
 	vdd_5v: vdd-5v {
 		compatible = "regulator-fixed";
 		regulator-name = "vdd_5v";
@@ -30,6 +26,10 @@
 
 	fan: pwm-fan {
 		compatible = "pwm-fan";
+		/*
+		 * With 20KHz PWM and an EVERCOOL EC4007H12SA fan, these levels
+		 * work out to 0, ~1200, ~3000, and 5000RPM respectively.
+		 */
 		cooling-levels = <0 12 18 255>;
 		#cooling-cells = <2>;
 		fan-supply = <&vdd_5v>;
@@ -73,40 +73,32 @@
 	status = "disabled";
 };
 
+&i2c4 {
+	status = "disabled";
+};
+
 &leds {
 	lan_led: led-1 {
 		gpios = <&gpio1 RK_PA1 GPIO_ACTIVE_HIGH>;
-		label = "nanopi-r4s:green:lan";
+		label = "green:lan";
 	};
 
 	wan_led: led-2 {
 		gpios = <&gpio1 RK_PA0 GPIO_ACTIVE_HIGH>;
-		label = "nanopi-r4s:green:wan";
+		label = "green:wan";
 	};
 };
 
 &leds_gpio {
 	rockchip,pins =
-		<0 RK_PB5 RK_FUNC_GPIO &pcfg_pull_none>,
-		<1 RK_PA0 RK_FUNC_GPIO &pcfg_pull_none>,
-		<1 RK_PA1 RK_FUNC_GPIO &pcfg_pull_none>;
+		<1 RK_PA1 RK_FUNC_GPIO &pcfg_pull_none>,
+		<1 RK_PA0 RK_FUNC_GPIO &pcfg_pull_none>;
 };
 
 &pcie0 {
 	max-link-speed = <1>;
 	num-lanes = <1>;
 	vpcie3v3-supply = <&vcc3v3_sys>;
-
-	pcie@0 {
-		reg = <0x00000000 0 0 0 0>;
-		#address-cells = <3>;
-		#size-cells = <2>;
-
-		r8169: pcie@0,0 {
-			reg = <0x000000 0 0 0 0>;
-			local-mac-address = [ 00 00 00 00 00 00 ];
-		};
-	};
 };
 
 &sdhci {
@@ -117,10 +109,6 @@
 	status = "disabled";
 };
 
-&sdmmc {
-	host-index-min = <1>;
-};
-
 &u2phy0_host {
 	phy-supply = <&vdd_5v>;
 };
@@ -132,7 +120,3 @@
 &usbdrd_dwc3_0 {
 	dr_mode = "host";
 };
-
-&vcc3v3_sys {
-	vin-supply = <&vcc5v0_sys>;
-};
-- 
2.32.0


^ permalink raw reply related	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-07-05 14:52 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-07-05 14:52 [PATCH] arm64: rk3399: Optimize performance and cleanup Xiaobo Tian
  -- strict thread matches above, loose matches on Subject: below --
2021-07-05 14:21 Xiaobo Tian
2021-07-05 14:32 ` Peter Robinson

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.