blob: 4f3c97ad27b5560c8e8baff7caef84f56af2ac98 [file] [log] [blame]
developera20cdc22024-05-31 18:57:31 +08001From 2825cf9ff77fa1065dac5c7a129aad9e8edecdf3 Mon Sep 17 00:00:00 2001
developer9a587882023-07-17 11:11:44 +08002From: Howard Hsu <howard-yh.hsu@mediatek.com>
3Date: Thu, 13 Jul 2023 15:50:00 +0800
developera20cdc22024-05-31 18:57:31 +08004Subject: [PATCH 03/21] wifi: mt76: mt7915: move temperature margin check to
developer9a587882023-07-17 11:11:44 +08005 mt7915_thermal_temp_store()
6
7Originally, we would reduce the 10-degree margin to the restore
8temperature, but the user would not be aware of this when setting it.
9Moving the margin reduction to the user setting check allows the user to
10clearly understand that there is a 10-degree difference between the
11restore and trigger temperature.
12
13Signed-off-by: Howard Hsu <howard-yh.hsu@mediatek.com>
14---
15 mt7915/init.c | 7 ++++---
16 mt7915/mcu.c | 3 +--
17 2 files changed, 5 insertions(+), 5 deletions(-)
18
19diff --git a/mt7915/init.c b/mt7915/init.c
developerdc9eeae2024-04-08 14:36:46 +080020index edf83c4..9fe0524 100644
developer9a587882023-07-17 11:11:44 +080021--- a/mt7915/init.c
22+++ b/mt7915/init.c
23@@ -83,12 +83,13 @@ static ssize_t mt7915_thermal_temp_store(struct device *dev,
24 mutex_lock(&phy->dev->mt76.mutex);
25 val = clamp_val(DIV_ROUND_CLOSEST(val, 1000), 60, 130);
26
27+ /* add a safety margin ~10 */
28 if ((i - 1 == MT7915_CRIT_TEMP_IDX &&
29- val > phy->throttle_temp[MT7915_MAX_TEMP_IDX]) ||
30+ val > phy->throttle_temp[MT7915_MAX_TEMP_IDX] - 10) ||
31 (i - 1 == MT7915_MAX_TEMP_IDX &&
32- val < phy->throttle_temp[MT7915_CRIT_TEMP_IDX])) {
33+ val - 10 < phy->throttle_temp[MT7915_CRIT_TEMP_IDX])) {
34 dev_err(phy->dev->mt76.dev,
35- "temp1_max shall be greater than temp1_crit.");
36+ "temp1_max shall be 10 degrees greater than temp1_crit.");
37 mutex_unlock(&phy->dev->mt76.mutex);
38 return -EINVAL;
39 }
40diff --git a/mt7915/mcu.c b/mt7915/mcu.c
developerdc9eeae2024-04-08 14:36:46 +080041index 7df2162..6e9970c 100644
developer9a587882023-07-17 11:11:44 +080042--- a/mt7915/mcu.c
43+++ b/mt7915/mcu.c
developera46f6132024-03-26 14:09:54 +080044@@ -3226,8 +3226,7 @@ int mt7915_mcu_set_thermal_protect(struct mt7915_phy *phy)
developer9a587882023-07-17 11:11:44 +080045
46 /* set high-temperature trigger threshold */
47 req.ctrl.ctrl_id = THERMAL_PROTECT_ENABLE;
48- /* add a safety margin ~10 */
49- req.restore_temp = cpu_to_le32(phy->throttle_temp[0] - 10);
50+ req.restore_temp = cpu_to_le32(phy->throttle_temp[0]);
51 req.trigger_temp = cpu_to_le32(phy->throttle_temp[1]);
52 req.sustain_time = cpu_to_le16(10);
53
54--
developerbd9fa1e2023-10-16 11:04:00 +0800552.18.0
developer9a587882023-07-17 11:11:44 +080056