diff mbox series

[v4] arm64: dts: meson: add gpio-fan control to GS-King-X

Message ID 20220611170852.19487-1-f.kardame@manjaro.org (mailing list archive)
State New, archived
Headers show
Series [v4] arm64: dts: meson: add gpio-fan control to GS-King-X | expand

Commit Message

Furkan Kardame June 11, 2022, 5:08 p.m. UTC
GS-King-X has a single speed GPIO fan which is always-on by default. 
If we add controls for the fan and a trip point,
the fan stays off most of the time, reducing background 
noise from the unit.

Signed-off-by: Furkan Kardame <f.kardame@manjaro.org>
---
Changelog:
V1:
* Using tabs instead of space.

V2:
* Remove unused map.

V3:
* Fixed speed-map indentation
* Aligned brackets
* Changed cooling-device
* Increased temperature to 70°C

V4:
* Fixed syntax error.
 .../boot/dts/amlogic/meson-g12b-gsking-x.dts  | 25 +++++++++++++++++++
 1 file changed, 25 insertions(+)

Comments

Neil Armstrong June 13, 2022, 8:18 a.m. UTC | #1
On 11/06/2022 19:08, Furkan Kardame wrote:
> GS-King-X has a single speed GPIO fan which is always-on by default.
> If we add controls for the fan and a trip point,
> the fan stays off most of the time, reducing background
> noise from the unit.
> 
> Signed-off-by: Furkan Kardame <f.kardame@manjaro.org>
> ---
> Changelog:
> V1:
> * Using tabs instead of space.
> 
> V2:
> * Remove unused map.
> 
> V3:
> * Fixed speed-map indentation
> * Aligned brackets
> * Changed cooling-device
> * Increased temperature to 70°C
> 
> V4:
> * Fixed syntax error.
>   .../boot/dts/amlogic/meson-g12b-gsking-x.dts  | 25 +++++++++++++++++++
>   1 file changed, 25 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/amlogic/meson-g12b-gsking-x.dts b/arch/arm64/boot/dts/amlogic/meson-g12b-gsking-x.dts
> index 6c7bfacba..23592ae21 100644
> --- a/arch/arm64/boot/dts/amlogic/meson-g12b-gsking-x.dts
> +++ b/arch/arm64/boot/dts/amlogic/meson-g12b-gsking-x.dts
> @@ -20,6 +20,14 @@ aliases {
>   		rtc1 = &vrtc;
>   	};
>   
> +	gpio_fan: gpio-fan {
> +		compatible = "gpio-fan";
> +		gpios = <&gpio GPIOH_5 GPIO_ACTIVE_HIGH>;
> +		/* Using Dummy Speed */
> +		gpio-fan,speed-map = <0 0>, <1 1>;
> +		#cooling-cells = <2>;
> +	};
> +
>   	gpio-keys-polled {
>   		compatible = "gpio-keys-polled";
>   		#address-cells = <1>;
> @@ -96,6 +104,23 @@ &clkc_audio {
>   	status = "okay";
>   };
>   
> +&cpu_thermal {
> +	trips {
> +		cpu_active: cpu-active {
> +			temperature = <70000>; /* millicelsius */
> +			hysteresis = <2000>; /* millicelsius */
> +			type = "active";
> +		};
> +	};
> +
> +	cooling-maps {
> +		map {
> +			trip = <&cpu_active>;
> +			cooling-device = <&gpio_fan THERMAL_NO_LIMIT THERMAL_NO_LIMIT>;
> +		};
> +	};
> +};
> +
>   &frddr_a {
>   	status = "okay";
>   };

Reviewed-by: Neil Armstrong <narmstrong@baylibre.com>
Neil Armstrong June 13, 2022, 8:19 a.m. UTC | #2
Hi,

On Sat, 11 Jun 2022 20:08:52 +0300, Furkan Kardame wrote:
> GS-King-X has a single speed GPIO fan which is always-on by default.
> If we add controls for the fan and a trip point,
> the fan stays off most of the time, reducing background
> noise from the unit.
> 
> 

Thanks, Applied to https://git.kernel.org/pub/scm/linux/kernel/git/amlogic/linux.git (v5.20/arm64-dt)

[1/1] arm64: dts: meson: add gpio-fan control to GS-King-X
      https://git.kernel.org/amlogic/c/b7bfbbbf72fcf95c9a58d6ad0e905de539fba025

These changes has been applied on the intermediate git tree [1].

The v5.20/arm64-dt branch will then be sent via a formal Pull Request to the Linux SoC maintainers
for inclusion in their intermediate git branches in order to be sent to Linus during
the next merge window, or sooner if it's a set of fixes.

In the cases of fixes, those will be merged in the current release candidate
kernel and as soon they appear on the Linux master branch they will be
backported to the previous Stable and Long-Stable kernels [2].

The intermediate git branches are merged daily in the linux-next tree [3],
people are encouraged testing these pre-release kernels and report issues on the
relevant mailing-lists.

If problems are discovered on those changes, please submit a signed-off-by revert
patch followed by a corrective changeset.

[1] https://git.kernel.org/pub/scm/linux/kernel/git/amlogic/linux.git
[2] https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
[3] https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/amlogic/meson-g12b-gsking-x.dts b/arch/arm64/boot/dts/amlogic/meson-g12b-gsking-x.dts
index 6c7bfacba..23592ae21 100644
--- a/arch/arm64/boot/dts/amlogic/meson-g12b-gsking-x.dts
+++ b/arch/arm64/boot/dts/amlogic/meson-g12b-gsking-x.dts
@@ -20,6 +20,14 @@  aliases {
 		rtc1 = &vrtc;
 	};
 
+	gpio_fan: gpio-fan {
+		compatible = "gpio-fan";
+		gpios = <&gpio GPIOH_5 GPIO_ACTIVE_HIGH>;
+		/* Using Dummy Speed */
+		gpio-fan,speed-map = <0 0>, <1 1>;
+		#cooling-cells = <2>;
+	};
+
 	gpio-keys-polled {
 		compatible = "gpio-keys-polled";
 		#address-cells = <1>;
@@ -96,6 +104,23 @@  &clkc_audio {
 	status = "okay";
 };
 
+&cpu_thermal {
+	trips {
+		cpu_active: cpu-active {
+			temperature = <70000>; /* millicelsius */
+			hysteresis = <2000>; /* millicelsius */
+			type = "active";
+		};
+	};
+
+	cooling-maps {
+		map {
+			trip = <&cpu_active>;
+			cooling-device = <&gpio_fan THERMAL_NO_LIMIT THERMAL_NO_LIMIT>;
+		};
+	};
+};
+
 &frddr_a {
 	status = "okay";
 };