cleanup: board: Remove unnecessary fan off/max with custom fan control.

Remove unnecessary fan off/max parameters for some boards.

BUG=none
BRANCH=none
TEST=make buildall

Signed-off-by: Devin Lu <Devin.Lu@quantatw.com>
Change-Id: I230986096bb77f96519e5e233e85a3dc9dbe42c3
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3141334
Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>

8 files changed, 0 insertions, 47 deletions

diff --git a/board/anahera/sensors.c b/board/anahera/sensors.c
index bb86af940b..3d3b58eeae 100644
--- a/board/anahera/sensors.c
+++ b/board/anahera/sensors.c
@@ -82,8 +82,6 @@ static const struct ec_thermal_config thermal_fan = {
 	.temp_host_release = {
 		[EC_TEMP_THRESH_HIGH] = C_TO_K(65),
 	},
-	.temp_fan_off = C_TO_K(35),
-	.temp_fan_max = C_TO_K(50),
 };
 
 /*
@@ -101,8 +99,6 @@ static const struct ec_thermal_config thermal_cpu = {
 	.temp_host_release = {
 		[EC_TEMP_THRESH_HIGH] = C_TO_K(65),
 	},
-	.temp_fan_off = C_TO_K(35),
-	.temp_fan_max = C_TO_K(50),
 };
 
 /*
@@ -116,8 +112,6 @@ static const struct ec_thermal_config thermal_charger = {
 	.temp_host_release = {
 		[EC_TEMP_THRESH_HIGH] = C_TO_K(75),
 	},
-	.temp_fan_off = C_TO_K(40),
-	.temp_fan_max = C_TO_K(55),
 };
 
 /*
@@ -131,8 +125,6 @@ static const struct ec_thermal_config thermal_regulator = {
 	.temp_host_release = {
 		[EC_TEMP_THRESH_HIGH] = C_TO_K(75),
 	},
-	.temp_fan_off = C_TO_K(40),
-	.temp_fan_max = C_TO_K(55),
 };
 
 /* this should really be "const" */
diff --git a/board/eldrid/thermal.c b/board/eldrid/thermal.c
index 4f9ac8796b..e44bcbc75c 100644
--- a/board/eldrid/thermal.c
+++ b/board/eldrid/thermal.c
@@ -35,8 +35,6 @@ const static struct ec_thermal_config thermal_cpu = {
 	.temp_host_release = {
 		[EC_TEMP_THRESH_HIGH] = C_TO_K(65),
 	},
-	.temp_fan_off = C_TO_K(35),
-	.temp_fan_max = C_TO_K(50),
 };
 
 /*
@@ -58,8 +56,6 @@ const static struct ec_thermal_config thermal_inductor = {
 	.temp_host_release = {
 		[EC_TEMP_THRESH_HIGH] = C_TO_K(65),
 	},
-	.temp_fan_off = C_TO_K(40),
-	.temp_fan_max = C_TO_K(55),
 };
 
 struct ec_thermal_config thermal_params[] = {
diff --git a/board/gimble/sensors.c b/board/gimble/sensors.c
index 3468e3eb35..f0818429e4 100644
--- a/board/gimble/sensors.c
+++ b/board/gimble/sensors.c
@@ -324,8 +324,6 @@ static const struct ec_thermal_config thermal_cpu = {
 	.temp_host_release = {
 		[EC_TEMP_THRESH_HIGH] = C_TO_K(65),
 	},
-	.temp_fan_off = C_TO_K(35),
-	.temp_fan_max = C_TO_K(50),
 };
 
 /*
@@ -349,8 +347,6 @@ static const struct ec_thermal_config thermal_inductor = {
 	.temp_host_release = {
 		[EC_TEMP_THRESH_HIGH] = C_TO_K(65),
 	},
-	.temp_fan_off = C_TO_K(40),
-	.temp_fan_max = C_TO_K(55),
 };
 
 /* this should really be "const" */
diff --git a/board/jinlon/board.c b/board/jinlon/board.c
index 470ff37f65..c0022e970f 100644
--- a/board/jinlon/board.c
+++ b/board/jinlon/board.c
@@ -358,8 +358,6 @@ const static struct ec_thermal_config thermal_a = {
 		[EC_TEMP_THRESH_WARN] = 0,
 		[EC_TEMP_THRESH_HIGH] = C_TO_K(65),
 	},
-	.temp_fan_off = C_TO_K(25),
-	.temp_fan_max = C_TO_K(70),
 };
 
 const static struct ec_thermal_config thermal_b = {
@@ -373,8 +371,6 @@ const static struct ec_thermal_config thermal_b = {
 		[EC_TEMP_THRESH_HIGH] = C_TO_K(65),
 		[EC_TEMP_THRESH_HALT] = 0,
 	},
-	.temp_fan_off = C_TO_K(25),
-	.temp_fan_max = C_TO_K(50),
 };
 
 struct ec_thermal_config thermal_params[TEMP_SENSOR_COUNT];
diff --git a/board/morphius/board.c b/board/morphius/board.c
index 8e0577b133..35988b6723 100644
--- a/board/morphius/board.c
+++ b/board/morphius/board.c
@@ -589,8 +589,6 @@ const static struct ec_thermal_config thermal_cpu = {
 	.temp_host_release = {
 		[EC_TEMP_THRESH_HIGH] = C_TO_K(80),
 	},
-	.temp_fan_off = C_TO_K(98),
-	.temp_fan_max = C_TO_K(99),
 };
 
 struct ec_thermal_config thermal_params[TEMP_SENSOR_COUNT];
diff --git a/board/nipperkin/board.c b/board/nipperkin/board.c
index 8dc2f253be..a065dad5c0 100644
--- a/board/nipperkin/board.c
+++ b/board/nipperkin/board.c
@@ -273,9 +273,6 @@ struct ec_thermal_config thermal_params[TEMP_SENSOR_COUNT] = {
 		.temp_host_release = {
 			[EC_TEMP_THRESH_HIGH] = C_TO_K(80),
 		},
-		/* TODO: Setting fan off to 0 so it's allways on */
-		.temp_fan_off = C_TO_K(0),
-		.temp_fan_max = C_TO_K(70),
 	},
 	[TEMP_SENSOR_CHARGER] = {
 		.temp_host = {
@@ -285,8 +282,6 @@ struct ec_thermal_config thermal_params[TEMP_SENSOR_COUNT] = {
 		.temp_host_release = {
 			[EC_TEMP_THRESH_HIGH] = C_TO_K(80),
 		},
-		.temp_fan_off = 0,
-		.temp_fan_max = 0,
 	},
 	[TEMP_SENSOR_MEMORY] = {
 		.temp_host = {
@@ -296,8 +291,6 @@ struct ec_thermal_config thermal_params[TEMP_SENSOR_COUNT] = {
 		.temp_host_release = {
 			[EC_TEMP_THRESH_HIGH] = C_TO_K(80),
 		},
-		.temp_fan_off = 0,
-		.temp_fan_max = 0,
 	},
 	[TEMP_SENSOR_CPU] = {
 		.temp_host = {
@@ -307,12 +300,6 @@ struct ec_thermal_config thermal_params[TEMP_SENSOR_COUNT] = {
 		.temp_host_release = {
 			[EC_TEMP_THRESH_HIGH] = C_TO_K(80),
 		},
-		/*
-		 * CPU temp sensor fan thresholds are high because they are a
-		 * backup for the SOC temp sensor fan thresholds.
-		 */
-		.temp_fan_off = C_TO_K(60),
-		.temp_fan_max = C_TO_K(90),
 	},
 	/*
 	 * Note: Leave ambient entries at 0, both as it does not represent a
diff --git a/board/redrix/sensors.c b/board/redrix/sensors.c
index f720a7cd32..b8446b1c4d 100644
--- a/board/redrix/sensors.c
+++ b/board/redrix/sensors.c
@@ -292,8 +292,6 @@ static const struct ec_thermal_config thermal_ddr = {
 	.temp_host_release = {
 		[EC_TEMP_THRESH_HIGH] = C_TO_K(65),
 	},
-	.temp_fan_off = C_TO_K(35),
-	.temp_fan_max = C_TO_K(50),
 };
 
 /*
@@ -311,8 +309,6 @@ static const struct ec_thermal_config thermal_cpu = {
 	.temp_host_release = {
 		[EC_TEMP_THRESH_HIGH] = C_TO_K(65),
 	},
-	.temp_fan_off = C_TO_K(35),
-	.temp_fan_max = C_TO_K(50),
 };
 
 /*
@@ -326,8 +322,6 @@ static const struct ec_thermal_config thermal_charger = {
 	.temp_host_release = {
 		[EC_TEMP_THRESH_HIGH] = C_TO_K(75),
 	},
-	.temp_fan_off = C_TO_K(40),
-	.temp_fan_max = C_TO_K(55),
 };
 
 /*
@@ -341,8 +335,6 @@ static const struct ec_thermal_config thermal_regulator = {
 	.temp_host_release = {
 		[EC_TEMP_THRESH_HIGH] = C_TO_K(75),
 	},
-	.temp_fan_off = C_TO_K(40),
-	.temp_fan_max = C_TO_K(55),
 };
 
 /* this should really be "const" */
diff --git a/board/woomax/board.c b/board/woomax/board.c
index 7770804402..106f21db59 100644
--- a/board/woomax/board.c
+++ b/board/woomax/board.c
@@ -763,8 +763,6 @@ const static struct ec_thermal_config thermal_thermistor = {
 	.temp_host_release = {
 		[EC_TEMP_THRESH_HIGH] = C_TO_K(90),
 	},
-	.temp_fan_off = C_TO_K(25),
-	.temp_fan_max = C_TO_K(58),
 };
 
 const static struct ec_thermal_config thermal_cpu = {
@@ -775,8 +773,6 @@ const static struct ec_thermal_config thermal_cpu = {
 	.temp_host_release = {
 		[EC_TEMP_THRESH_HIGH] = C_TO_K(90),
 	},
-	.temp_fan_off = C_TO_K(25),
-	.temp_fan_max = C_TO_K(58),
 };
 
 struct ec_thermal_config thermal_params[TEMP_SENSOR_COUNT];