From c809762ffa4d2ec1cf46f72c135c72e663e92f81 Mon Sep 17 00:00:00 2001
From: TeYuan Wang <kamewang@google.com>
Date: Fri, 17 Feb 2023 20:31:20 +0800
Subject: [PATCH] thermal: Increase "I" to have better control at target
 temperature

Bug: 269730273
Test: Charge device and confirm thermal log
Change-Id: I6aa02a7963359dd9f75be251f51c4a96da3a3eb7
---
 thermal_info_config_charge_felix.json         | 10 +++++-----
 thermal_info_config_felix.json                | 10 +++++-----
 thermal_info_config_proactive_skin_felix.json | 10 +++++-----
 3 files changed, 15 insertions(+), 15 deletions(-)

diff --git a/thermal_info_config_charge_felix.json b/thermal_info_config_charge_felix.json
index 3b7a8d5..fc256cf 100644
--- a/thermal_info_config_charge_felix.json
+++ b/thermal_info_config_charge_felix.json
@@ -243,7 +243,7 @@
         {
             "Name":"VIRTUAL-SKIN-CHARGE",
             "Type":"UNKNOWN",
-            "Version":"0.2",
+            "Version":"0.3",
             "Hidden":true,
             "VirtualSensor":true,
             "TriggerSensor":"skin_therm",
@@ -258,14 +258,14 @@
             "PIDInfo": {
                 "K_Po":["NAN", "NAN", 1000, "NAN", "NAN", "NAN", "NAN"],
                 "K_Pu":["NAN", "NAN", 30, "NAN", "NAN", "NAN", "NAN"],
-                "K_I":["NAN", "NAN", 5, "NAN", "NAN", "NAN", "NAN"],
+                "K_I":["NAN", "NAN", 40, "NAN", "NAN", "NAN", "NAN"],
                 "K_D":["NAN", "NAN", 0, "NAN", "NAN", "NAN", "NAN"],
                 "I_Max":["NAN", "NAN", 1500, "NAN", "NAN", "NAN", "NAN"],
-                "S_Power":["NAN", "NAN", 3500, "NAN", "NAN", "NAN", "NAN"],
+                "S_Power":["NAN", "NAN", 3000, "NAN", "NAN", "NAN", "NAN"],
                 "MinAllocPower":["NAN", "NAN", 0, "NAN", "NAN", "NAN", "NAN"],
                 "MaxAllocPower":["NAN", "NAN", 10000, "NAN", "NAN", "NAN", "NAN"],
                 "I_Cutoff":["NAN", "NAN", 14, "NAN", "NAN", "NAN", "NAN"],
-                "I_Default":1000
+                "I_Default":1500
             },
             "ExcludedPowerInfo": [
                 {
@@ -278,7 +278,7 @@
                     "CdevRequest": "chg_mdis",
                     "CdevWeightForPID": [1, 1, 1, 1, 1, 1, 1],
                     "MaxReleaseStep": 1,
-                    "MaxThrottleStep": 1,
+                    "MaxThrottleStep": 2,
                     "CdevCeiling": [0, 10, 11, 11, 12, 12, 12],
                     "LimitInfo": [0, 0, 0, 0, 0, 12, 12]
                 }
diff --git a/thermal_info_config_felix.json b/thermal_info_config_felix.json
index 77036e3..3920569 100644
--- a/thermal_info_config_felix.json
+++ b/thermal_info_config_felix.json
@@ -431,7 +431,7 @@
         {
             "Name":"VIRTUAL-SKIN-CHARGE",
             "Type":"UNKNOWN",
-            "Version":"0.2",
+            "Version":"0.3",
             "Hidden":true,
             "VirtualSensor":true,
             "TriggerSensor":"skin_therm",
@@ -446,14 +446,14 @@
             "PIDInfo": {
                 "K_Po":["NAN", "NAN", 1000, "NAN", "NAN", "NAN", "NAN"],
                 "K_Pu":["NAN", "NAN", 30, "NAN", "NAN", "NAN", "NAN"],
-                "K_I":["NAN", "NAN", 5, "NAN", "NAN", "NAN", "NAN"],
+                "K_I":["NAN", "NAN", 40, "NAN", "NAN", "NAN", "NAN"],
                 "K_D":["NAN", "NAN", 0, "NAN", "NAN", "NAN", "NAN"],
                 "I_Max":["NAN", "NAN", 1500, "NAN", "NAN", "NAN", "NAN"],
-                "S_Power":["NAN", "NAN", 3500, "NAN", "NAN", "NAN", "NAN"],
+                "S_Power":["NAN", "NAN", 3000, "NAN", "NAN", "NAN", "NAN"],
                 "MinAllocPower":["NAN", "NAN", 0, "NAN", "NAN", "NAN", "NAN"],
                 "MaxAllocPower":["NAN", "NAN", 10000, "NAN", "NAN", "NAN", "NAN"],
                 "I_Cutoff":["NAN", "NAN", 14, "NAN", "NAN", "NAN", "NAN"],
-                "I_Default":1000
+                "I_Default":1500
             },
             "ExcludedPowerInfo": [
                 {
@@ -466,7 +466,7 @@
                     "CdevRequest": "chg_mdis",
                     "CdevWeightForPID": [1, 1, 1, 1, 1, 1, 1],
                     "MaxReleaseStep": 1,
-                    "MaxThrottleStep": 1,
+                    "MaxThrottleStep": 2,
                     "CdevCeiling": [0, 10, 11, 11, 12, 12, 12],
                     "LimitInfo": [0, 0, 0, 0, 0, 12, 12]
                 }
diff --git a/thermal_info_config_proactive_skin_felix.json b/thermal_info_config_proactive_skin_felix.json
index 0008d87..32e4176 100644
--- a/thermal_info_config_proactive_skin_felix.json
+++ b/thermal_info_config_proactive_skin_felix.json
@@ -431,7 +431,7 @@
         {
             "Name":"VIRTUAL-SKIN-CHARGE",
             "Type":"UNKNOWN",
-            "Version":"0.2",
+            "Version":"0.3",
             "Hidden":true,
             "VirtualSensor":true,
             "TriggerSensor":"skin_therm",
@@ -446,14 +446,14 @@
             "PIDInfo": {
                 "K_Po":["NAN", "NAN", 1000, "NAN", "NAN", "NAN", "NAN"],
                 "K_Pu":["NAN", "NAN", 30, "NAN", "NAN", "NAN", "NAN"],
-                "K_I":["NAN", "NAN", 5, "NAN", "NAN", "NAN", "NAN"],
+                "K_I":["NAN", "NAN", 40, "NAN", "NAN", "NAN", "NAN"],
                 "K_D":["NAN", "NAN", 0, "NAN", "NAN", "NAN", "NAN"],
                 "I_Max":["NAN", "NAN", 1500, "NAN", "NAN", "NAN", "NAN"],
-                "S_Power":["NAN", "NAN", 3500, "NAN", "NAN", "NAN", "NAN"],
+                "S_Power":["NAN", "NAN", 3000, "NAN", "NAN", "NAN", "NAN"],
                 "MinAllocPower":["NAN", "NAN", 0, "NAN", "NAN", "NAN", "NAN"],
                 "MaxAllocPower":["NAN", "NAN", 10000, "NAN", "NAN", "NAN", "NAN"],
                 "I_Cutoff":["NAN", "NAN", 14, "NAN", "NAN", "NAN", "NAN"],
-                "I_Default":1000
+                "I_Default":1500
             },
             "ExcludedPowerInfo": [
                 {
@@ -466,7 +466,7 @@
                     "CdevRequest": "chg_mdis",
                     "CdevWeightForPID": [1, 1, 1, 1, 1, 1, 1],
                     "MaxReleaseStep": 1,
-                    "MaxThrottleStep": 1,
+                    "MaxThrottleStep": 2,
                     "CdevCeiling": [0, 10, 11, 11, 12, 12, 12],
                     "LimitInfo": [0, 0, 0, 0, 0, 12, 12]
                 }