Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 1 | /** |
| 2 | * Copyright 2017 Google Inc. |
| 3 | * |
| 4 | * Licensed under the Apache License, Version 2.0 (the "License"); |
| 5 | * you may not use this file except in compliance with the License. |
| 6 | * You may obtain a copy of the License at |
| 7 | * |
| 8 | * http://www.apache.org/licenses/LICENSE-2.0 |
| 9 | * |
| 10 | * Unless required by applicable law or agreed to in writing, software |
| 11 | * distributed under the License is distributed on an "AS IS" BASIS, |
| 12 | * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| 13 | * See the License for the specific language governing permissions and |
| 14 | * limitations under the License. |
| 15 | */ |
| 16 | |
| 17 | /* Configuration. */ |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 18 | #include "zone.hpp" |
| 19 | |
Patrick Venture | da4a5dd | 2018-08-31 09:42:48 -0700 | [diff] [blame] | 20 | #include "conf.hpp" |
James Zheng | 6df8bb5 | 2024-11-27 23:38:47 +0000 | [diff] [blame] | 21 | #include "failsafeloggers/failsafe_logger_utility.hpp" |
Patrick Venture | da4a5dd | 2018-08-31 09:42:48 -0700 | [diff] [blame] | 22 | #include "pid/controller.hpp" |
| 23 | #include "pid/ec/pid.hpp" |
| 24 | #include "pid/fancontroller.hpp" |
James Feist | 22c257a | 2018-08-31 14:07:12 -0700 | [diff] [blame] | 25 | #include "pid/stepwisecontroller.hpp" |
Patrick Venture | da4a5dd | 2018-08-31 09:42:48 -0700 | [diff] [blame] | 26 | #include "pid/thermalcontroller.hpp" |
Patrick Venture | c32e3fc | 2019-02-28 10:01:11 -0800 | [diff] [blame] | 27 | #include "pid/tuning.hpp" |
Patrick Venture | da4a5dd | 2018-08-31 09:42:48 -0700 | [diff] [blame] | 28 | |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 29 | #include <algorithm> |
| 30 | #include <chrono> |
| 31 | #include <cstring> |
| 32 | #include <fstream> |
| 33 | #include <iostream> |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 34 | #include <memory> |
Josh Lehan | 55ccad6 | 2020-09-20 23:57:49 -0700 | [diff] [blame] | 35 | #include <sstream> |
Patrick Venture | 7a98c19 | 2020-08-12 08:35:16 -0700 | [diff] [blame] | 36 | #include <string> |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 37 | |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 38 | using tstamp = std::chrono::high_resolution_clock::time_point; |
| 39 | using namespace std::literals::chrono_literals; |
| 40 | |
Josh Lehan | 55ccad6 | 2020-09-20 23:57:49 -0700 | [diff] [blame] | 41 | // Enforces minimum duration between events |
| 42 | // Rreturns true if event should be allowed, false if disallowed |
| 43 | bool allowThrottle(const tstamp& now, const std::chrono::seconds& pace) |
| 44 | { |
| 45 | static tstamp then; |
| 46 | static bool first = true; |
| 47 | |
| 48 | if (first) |
| 49 | { |
| 50 | // Special case initialization |
| 51 | then = now; |
| 52 | first = false; |
| 53 | |
| 54 | // Initialization, always allow |
| 55 | return true; |
| 56 | } |
| 57 | |
| 58 | auto elapsed = now - then; |
| 59 | if (elapsed < pace) |
| 60 | { |
| 61 | // Too soon since last time, disallow |
| 62 | return false; |
| 63 | } |
| 64 | |
| 65 | // It has been long enough, allow |
| 66 | then = now; |
| 67 | return true; |
| 68 | } |
| 69 | |
| 70 | namespace pid_control |
| 71 | { |
| 72 | |
Patrick Venture | 597ebd6 | 2020-08-11 08:48:19 -0700 | [diff] [blame] | 73 | double DbusPidZone::getMaxSetPointRequest(void) const |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 74 | { |
Patrick Venture | f7a2dd5 | 2019-07-16 14:31:13 -0700 | [diff] [blame] | 75 | return _maximumSetPoint; |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 76 | } |
| 77 | |
Patrick Venture | 597ebd6 | 2020-08-11 08:48:19 -0700 | [diff] [blame] | 78 | bool DbusPidZone::getManualMode(void) const |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 79 | { |
| 80 | return _manualMode; |
| 81 | } |
| 82 | |
Patrick Venture | 597ebd6 | 2020-08-11 08:48:19 -0700 | [diff] [blame] | 83 | void DbusPidZone::setManualMode(bool mode) |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 84 | { |
| 85 | _manualMode = mode; |
Josh Lehan | a4146eb | 2020-10-01 11:49:09 -0700 | [diff] [blame] | 86 | |
| 87 | // If returning to automatic mode, need to restore PWM from PID loop |
| 88 | if (!mode) |
| 89 | { |
| 90 | _redundantWrite = true; |
| 91 | } |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 92 | } |
| 93 | |
Patrick Venture | 597ebd6 | 2020-08-11 08:48:19 -0700 | [diff] [blame] | 94 | bool DbusPidZone::getFailSafeMode(void) const |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 95 | { |
| 96 | // If any keys are present at least one sensor is in fail safe mode. |
| 97 | return !_failSafeSensors.empty(); |
| 98 | } |
| 99 | |
Harvey Wu | a427007 | 2024-05-29 16:11:13 +0800 | [diff] [blame] | 100 | FailSafeSensorsMap DbusPidZone::getFailSafeSensors(void) const |
| 101 | { |
| 102 | return _failSafeSensors; |
| 103 | } |
| 104 | |
| 105 | void DbusPidZone::markSensorMissing(const std::string& name, |
| 106 | const std::string& failReason) |
Josh Lehan | 3f0f7bc | 2023-02-13 01:45:29 -0800 | [diff] [blame] | 107 | { |
| 108 | if (_missingAcceptable.find(name) != _missingAcceptable.end()) |
| 109 | { |
| 110 | // Disallow sensors in MissingIsAcceptable list from causing failsafe |
James Zheng | 6df8bb5 | 2024-11-27 23:38:47 +0000 | [diff] [blame] | 111 | outputFailsafeLogWithZone(_zoneId, this->getFailSafeMode(), name, |
| 112 | "The sensor is missing but is acceptable."); |
Josh Lehan | 3f0f7bc | 2023-02-13 01:45:29 -0800 | [diff] [blame] | 113 | return; |
| 114 | } |
| 115 | |
Harvey Wu | 92f9f3c | 2023-11-07 09:23:35 +0800 | [diff] [blame] | 116 | if (_sensorFailSafePercent[name] == 0) |
| 117 | { |
Harvey Wu | a427007 | 2024-05-29 16:11:13 +0800 | [diff] [blame] | 118 | _failSafeSensors[name] = std::pair(failReason, _zoneFailSafePercent); |
Harvey Wu | 92f9f3c | 2023-11-07 09:23:35 +0800 | [diff] [blame] | 119 | } |
| 120 | else |
| 121 | { |
Harvey Wu | a427007 | 2024-05-29 16:11:13 +0800 | [diff] [blame] | 122 | _failSafeSensors[name] = |
| 123 | std::pair(failReason, _sensorFailSafePercent[name]); |
Harvey Wu | 92f9f3c | 2023-11-07 09:23:35 +0800 | [diff] [blame] | 124 | } |
| 125 | |
| 126 | if (debugEnabled) |
| 127 | { |
| 128 | std::cerr << "Sensor " << name << " marked missing\n"; |
| 129 | } |
Josh Lehan | 3f0f7bc | 2023-02-13 01:45:29 -0800 | [diff] [blame] | 130 | } |
| 131 | |
Patrick Venture | 597ebd6 | 2020-08-11 08:48:19 -0700 | [diff] [blame] | 132 | int64_t DbusPidZone::getZoneID(void) const |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 133 | { |
| 134 | return _zoneId; |
| 135 | } |
| 136 | |
Nirav Shah | ccc8bb6 | 2022-02-17 21:06:51 -0800 | [diff] [blame] | 137 | void DbusPidZone::addSetPoint(double setPoint, const std::string& name) |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 138 | { |
ykchiu | 7c6d35d | 2023-05-10 17:01:46 +0800 | [diff] [blame] | 139 | /* exclude disabled pidloop from _maximumSetPoint calculation*/ |
| 140 | if (!isPidProcessEnabled(name)) |
| 141 | { |
| 142 | return; |
| 143 | } |
| 144 | |
Delphine CC Chiu | 9788963 | 2023-11-06 11:32:46 +0800 | [diff] [blame] | 145 | auto profileName = name; |
| 146 | if (getAccSetPoint()) |
| 147 | { |
| 148 | /* |
| 149 | * If the name of controller is Linear_Temp_CPU0. |
| 150 | * The profile name will be Temp_CPU0. |
| 151 | */ |
| 152 | profileName = name.substr(name.find("_") + 1); |
| 153 | _SetPoints[profileName] += setPoint; |
| 154 | } |
| 155 | else |
| 156 | { |
| 157 | if (_SetPoints[profileName] < setPoint) |
| 158 | { |
| 159 | _SetPoints[profileName] = setPoint; |
| 160 | } |
| 161 | } |
| 162 | |
Nirav Shah | ccc8bb6 | 2022-02-17 21:06:51 -0800 | [diff] [blame] | 163 | /* |
| 164 | * if there are multiple thermal controllers with the same |
| 165 | * value, pick the first one in the iterator |
| 166 | */ |
Delphine CC Chiu | 9788963 | 2023-11-06 11:32:46 +0800 | [diff] [blame] | 167 | if (_maximumSetPoint < _SetPoints[profileName]) |
Nirav Shah | ccc8bb6 | 2022-02-17 21:06:51 -0800 | [diff] [blame] | 168 | { |
Delphine CC Chiu | 9788963 | 2023-11-06 11:32:46 +0800 | [diff] [blame] | 169 | _maximumSetPoint = _SetPoints[profileName]; |
| 170 | _maximumSetPointName = profileName; |
Nirav Shah | ccc8bb6 | 2022-02-17 21:06:51 -0800 | [diff] [blame] | 171 | } |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 172 | } |
| 173 | |
Patrick Venture | 597ebd6 | 2020-08-11 08:48:19 -0700 | [diff] [blame] | 174 | void DbusPidZone::addRPMCeiling(double ceiling) |
James Feist | 608304d | 2019-02-25 10:01:42 -0800 | [diff] [blame] | 175 | { |
| 176 | _RPMCeilings.push_back(ceiling); |
| 177 | } |
| 178 | |
Patrick Venture | 597ebd6 | 2020-08-11 08:48:19 -0700 | [diff] [blame] | 179 | void DbusPidZone::clearRPMCeilings(void) |
James Feist | 608304d | 2019-02-25 10:01:42 -0800 | [diff] [blame] | 180 | { |
| 181 | _RPMCeilings.clear(); |
| 182 | } |
| 183 | |
Patrick Venture | 597ebd6 | 2020-08-11 08:48:19 -0700 | [diff] [blame] | 184 | void DbusPidZone::clearSetPoints(void) |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 185 | { |
Patrick Venture | 9bbf333 | 2019-07-16 10:50:37 -0700 | [diff] [blame] | 186 | _SetPoints.clear(); |
Nirav Shah | ccc8bb6 | 2022-02-17 21:06:51 -0800 | [diff] [blame] | 187 | _maximumSetPoint = 0; |
ykchiu | 7c6d35d | 2023-05-10 17:01:46 +0800 | [diff] [blame] | 188 | _maximumSetPointName.clear(); |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 189 | } |
| 190 | |
Harvey Wu | 92f9f3c | 2023-11-07 09:23:35 +0800 | [diff] [blame] | 191 | double DbusPidZone::getFailSafePercent(void) |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 192 | { |
Eric Yang | d59ccac | 2025-06-05 19:18:03 +0800 | [diff] [blame^] | 193 | if (_failSafeSensors.empty()) |
| 194 | { |
| 195 | return _zoneFailSafePercent; |
| 196 | } |
| 197 | |
Harvey Wu | a427007 | 2024-05-29 16:11:13 +0800 | [diff] [blame] | 198 | FailSafeSensorsMap::iterator maxData = std::max_element( |
Harvey Wu | 92f9f3c | 2023-11-07 09:23:35 +0800 | [diff] [blame] | 199 | _failSafeSensors.begin(), _failSafeSensors.end(), |
Harvey Wu | a427007 | 2024-05-29 16:11:13 +0800 | [diff] [blame] | 200 | [](const FailSafeSensorPair firstData, |
| 201 | const FailSafeSensorPair secondData) { |
| 202 | return firstData.second.second < secondData.second.second; |
Harvey Wu | 92f9f3c | 2023-11-07 09:23:35 +0800 | [diff] [blame] | 203 | }); |
| 204 | |
| 205 | // In dbus/dbusconfiguration.cpp, the default sensor failsafepercent is 0 if |
| 206 | // there is no setting in json. |
| 207 | // Therfore, if the max failsafe duty in _failSafeSensors is 0, set final |
| 208 | // failsafe duty to _zoneFailSafePercent. |
Harvey Wu | a427007 | 2024-05-29 16:11:13 +0800 | [diff] [blame] | 209 | if ((*maxData).second.second == 0) |
Harvey Wu | 92f9f3c | 2023-11-07 09:23:35 +0800 | [diff] [blame] | 210 | { |
| 211 | return _zoneFailSafePercent; |
| 212 | } |
| 213 | else |
| 214 | { |
Harvey Wu | a427007 | 2024-05-29 16:11:13 +0800 | [diff] [blame] | 215 | return (*maxData).second.second; |
Harvey Wu | 92f9f3c | 2023-11-07 09:23:35 +0800 | [diff] [blame] | 216 | } |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 217 | } |
| 218 | |
Nirav Shah | ccc8bb6 | 2022-02-17 21:06:51 -0800 | [diff] [blame] | 219 | double DbusPidZone::getMinThermalSetPoint(void) const |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 220 | { |
James Feist | 3484bed | 2019-02-25 13:28:18 -0800 | [diff] [blame] | 221 | return _minThermalOutputSetPt; |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 222 | } |
| 223 | |
Bonnie Lo | 0e8fc39 | 2022-10-05 10:20:55 +0800 | [diff] [blame] | 224 | uint64_t DbusPidZone::getCycleIntervalTime(void) const |
| 225 | { |
| 226 | return _cycleTime.cycleIntervalTimeMS; |
| 227 | } |
| 228 | |
| 229 | uint64_t DbusPidZone::getUpdateThermalsCycle(void) const |
| 230 | { |
| 231 | return _cycleTime.updateThermalsTimeMS; |
| 232 | } |
| 233 | |
Patrick Venture | 597ebd6 | 2020-08-11 08:48:19 -0700 | [diff] [blame] | 234 | void DbusPidZone::addFanPID(std::unique_ptr<Controller> pid) |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 235 | { |
| 236 | _fans.push_back(std::move(pid)); |
| 237 | } |
| 238 | |
Patrick Venture | 597ebd6 | 2020-08-11 08:48:19 -0700 | [diff] [blame] | 239 | void DbusPidZone::addThermalPID(std::unique_ptr<Controller> pid) |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 240 | { |
| 241 | _thermals.push_back(std::move(pid)); |
| 242 | } |
| 243 | |
Patrick Venture | 597ebd6 | 2020-08-11 08:48:19 -0700 | [diff] [blame] | 244 | double DbusPidZone::getCachedValue(const std::string& name) |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 245 | { |
Josh Lehan | b300575 | 2022-02-22 20:48:07 -0800 | [diff] [blame] | 246 | return _cachedValuesByName.at(name).scaled; |
| 247 | } |
| 248 | |
| 249 | ValueCacheEntry DbusPidZone::getCachedValues(const std::string& name) |
| 250 | { |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 251 | return _cachedValuesByName.at(name); |
| 252 | } |
| 253 | |
Josh Lehan | b300575 | 2022-02-22 20:48:07 -0800 | [diff] [blame] | 254 | void DbusPidZone::setOutputCache(std::string_view name, |
| 255 | const ValueCacheEntry& values) |
| 256 | { |
| 257 | _cachedFanOutputs[std::string{name}] = values; |
| 258 | } |
| 259 | |
Josh Lehan | 3f0f7bc | 2023-02-13 01:45:29 -0800 | [diff] [blame] | 260 | void DbusPidZone::addFanInput(const std::string& fan, bool missingAcceptable) |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 261 | { |
| 262 | _fanInputs.push_back(fan); |
Josh Lehan | 3f0f7bc | 2023-02-13 01:45:29 -0800 | [diff] [blame] | 263 | |
| 264 | if (missingAcceptable) |
| 265 | { |
| 266 | _missingAcceptable.emplace(fan); |
| 267 | } |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 268 | } |
| 269 | |
Josh Lehan | 3f0f7bc | 2023-02-13 01:45:29 -0800 | [diff] [blame] | 270 | void DbusPidZone::addThermalInput(const std::string& therm, |
| 271 | bool missingAcceptable) |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 272 | { |
Delphine CC Chiu | 9788963 | 2023-11-06 11:32:46 +0800 | [diff] [blame] | 273 | /* |
| 274 | * One sensor may have stepwise and PID at the same time. |
| 275 | * Searching the sensor name before inserting it to avoid duplicated sensor |
| 276 | * names. |
| 277 | */ |
| 278 | if (std::find(_thermalInputs.begin(), _thermalInputs.end(), therm) == |
| 279 | _thermalInputs.end()) |
| 280 | { |
| 281 | _thermalInputs.push_back(therm); |
| 282 | } |
Josh Lehan | 3f0f7bc | 2023-02-13 01:45:29 -0800 | [diff] [blame] | 283 | |
| 284 | if (missingAcceptable) |
| 285 | { |
| 286 | _missingAcceptable.emplace(therm); |
| 287 | } |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 288 | } |
| 289 | |
Josh Lehan | 55ccad6 | 2020-09-20 23:57:49 -0700 | [diff] [blame] | 290 | // Updates desired RPM setpoint from optional text file |
| 291 | // Returns true if rpmValue updated, false if left unchanged |
| 292 | static bool fileParseRpm(const std::string& fileName, double& rpmValue) |
| 293 | { |
| 294 | static constexpr std::chrono::seconds throttlePace{3}; |
| 295 | |
| 296 | std::string errText; |
| 297 | |
| 298 | try |
| 299 | { |
| 300 | std::ifstream ifs; |
| 301 | ifs.open(fileName); |
| 302 | if (ifs) |
| 303 | { |
| 304 | int value; |
| 305 | ifs >> value; |
| 306 | |
| 307 | if (value <= 0) |
| 308 | { |
| 309 | errText = "File content could not be parsed to a number"; |
| 310 | } |
| 311 | else if (value <= 100) |
| 312 | { |
| 313 | errText = "File must contain RPM value, not PWM value"; |
| 314 | } |
| 315 | else |
| 316 | { |
| 317 | rpmValue = static_cast<double>(value); |
| 318 | return true; |
| 319 | } |
| 320 | } |
| 321 | } |
| 322 | catch (const std::exception& e) |
| 323 | { |
| 324 | errText = "Exception: "; |
| 325 | errText += e.what(); |
| 326 | } |
| 327 | |
| 328 | // The file is optional, intentionally not an error if file not found |
| 329 | if (!(errText.empty())) |
| 330 | { |
| 331 | tstamp now = std::chrono::high_resolution_clock::now(); |
| 332 | if (allowThrottle(now, throttlePace)) |
| 333 | { |
| 334 | std::cerr << "Unable to read from '" << fileName << "': " << errText |
| 335 | << "\n"; |
| 336 | } |
| 337 | } |
| 338 | |
| 339 | return false; |
| 340 | } |
| 341 | |
Patrick Venture | 597ebd6 | 2020-08-11 08:48:19 -0700 | [diff] [blame] | 342 | void DbusPidZone::determineMaxSetPointRequest(void) |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 343 | { |
Patrick Venture | 5f59c0f | 2018-11-11 12:55:14 -0800 | [diff] [blame] | 344 | std::vector<double>::iterator result; |
Nirav Shah | ccc8bb6 | 2022-02-17 21:06:51 -0800 | [diff] [blame] | 345 | double minThermalThreshold = getMinThermalSetPoint(); |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 346 | |
James Feist | 608304d | 2019-02-25 10:01:42 -0800 | [diff] [blame] | 347 | if (_RPMCeilings.size() > 0) |
| 348 | { |
| 349 | result = std::min_element(_RPMCeilings.begin(), _RPMCeilings.end()); |
Nirav Shah | ccc8bb6 | 2022-02-17 21:06:51 -0800 | [diff] [blame] | 350 | // if Max set point is larger than the lowest ceiling, reset to lowest |
| 351 | // ceiling. |
| 352 | if (*result < _maximumSetPoint) |
| 353 | { |
| 354 | _maximumSetPoint = *result; |
| 355 | // When using lowest ceiling, controller name is ceiling. |
| 356 | _maximumSetPointName = "Ceiling"; |
| 357 | } |
James Feist | 608304d | 2019-02-25 10:01:42 -0800 | [diff] [blame] | 358 | } |
| 359 | |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 360 | /* |
Delphine CC Chiu | 9788963 | 2023-11-06 11:32:46 +0800 | [diff] [blame] | 361 | * Combine the maximum SetPoint Name if the controllers have same profile |
| 362 | * name. e.g., PID_BB_INLET_TEMP_C + Stepwise_BB_INLET_TEMP_C. |
| 363 | */ |
| 364 | if (getAccSetPoint()) |
| 365 | { |
| 366 | auto profileName = _maximumSetPointName; |
| 367 | _maximumSetPointName = ""; |
| 368 | |
| 369 | for (auto& p : _thermals) |
| 370 | { |
| 371 | auto controllerID = p->getID(); |
| 372 | auto found = controllerID.find(profileName); |
| 373 | if (found != std::string::npos) |
| 374 | { |
| 375 | if (_maximumSetPointName.empty()) |
| 376 | { |
| 377 | _maximumSetPointName = controllerID; |
| 378 | } |
| 379 | else |
| 380 | { |
| 381 | _maximumSetPointName += " + " + controllerID; |
| 382 | } |
| 383 | } |
| 384 | } |
| 385 | } |
| 386 | |
| 387 | /* |
Patrick Venture | 7280e27 | 2019-02-11 10:45:32 -0800 | [diff] [blame] | 388 | * If the maximum RPM setpoint output is below the minimum RPM |
| 389 | * setpoint, set it to the minimum. |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 390 | */ |
Nirav Shah | ccc8bb6 | 2022-02-17 21:06:51 -0800 | [diff] [blame] | 391 | if (minThermalThreshold >= _maximumSetPoint) |
| 392 | { |
| 393 | _maximumSetPoint = minThermalThreshold; |
ykchiu | 7c6d35d | 2023-05-10 17:01:46 +0800 | [diff] [blame] | 394 | _maximumSetPointName = "Minimum"; |
Nirav Shah | ccc8bb6 | 2022-02-17 21:06:51 -0800 | [diff] [blame] | 395 | } |
| 396 | else if (_maximumSetPointName.compare(_maximumSetPointNamePrev)) |
| 397 | { |
| 398 | std::cerr << "PID Zone " << _zoneId << " max SetPoint " |
| 399 | << _maximumSetPoint << " requested by " |
| 400 | << _maximumSetPointName; |
| 401 | for (const auto& sensor : _failSafeSensors) |
| 402 | { |
Harvey Wu | 92f9f3c | 2023-11-07 09:23:35 +0800 | [diff] [blame] | 403 | if (sensor.first.find("Fan") == std::string::npos) |
Nirav Shah | ccc8bb6 | 2022-02-17 21:06:51 -0800 | [diff] [blame] | 404 | { |
Harvey Wu | 92f9f3c | 2023-11-07 09:23:35 +0800 | [diff] [blame] | 405 | std::cerr << " " << sensor.first; |
Nirav Shah | ccc8bb6 | 2022-02-17 21:06:51 -0800 | [diff] [blame] | 406 | } |
| 407 | } |
| 408 | std::cerr << "\n"; |
| 409 | _maximumSetPointNamePrev.assign(_maximumSetPointName); |
| 410 | } |
Patrick Venture | de79ee0 | 2019-05-08 14:50:00 -0700 | [diff] [blame] | 411 | if (tuningEnabled) |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 412 | { |
Patrick Venture | c32e3fc | 2019-02-28 10:01:11 -0800 | [diff] [blame] | 413 | /* |
| 414 | * We received no setpoints from thermal sensors. |
| 415 | * This is a case experienced during tuning where they only specify |
| 416 | * fan sensors and one large fan PID for all the fans. |
| 417 | */ |
| 418 | static constexpr auto setpointpath = "/etc/thermal.d/setpoint"; |
Patrick Venture | df766f2 | 2018-10-13 09:30:58 -0700 | [diff] [blame] | 419 | |
Nirav Shah | ccc8bb6 | 2022-02-17 21:06:51 -0800 | [diff] [blame] | 420 | fileParseRpm(setpointpath, _maximumSetPoint); |
Josh Lehan | 55ccad6 | 2020-09-20 23:57:49 -0700 | [diff] [blame] | 421 | |
| 422 | // Allow per-zone setpoint files to override overall setpoint file |
| 423 | std::ostringstream zoneSuffix; |
| 424 | zoneSuffix << ".zone" << _zoneId; |
| 425 | std::string zoneSetpointPath = setpointpath + zoneSuffix.str(); |
| 426 | |
Nirav Shah | ccc8bb6 | 2022-02-17 21:06:51 -0800 | [diff] [blame] | 427 | fileParseRpm(zoneSetpointPath, _maximumSetPoint); |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 428 | } |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 429 | return; |
| 430 | } |
| 431 | |
Patrick Venture | 597ebd6 | 2020-08-11 08:48:19 -0700 | [diff] [blame] | 432 | void DbusPidZone::initializeLog(void) |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 433 | { |
Patrick Venture | 5f02ad2 | 2018-04-24 10:18:40 -0700 | [diff] [blame] | 434 | /* Print header for log file: |
Josh Lehan | b300575 | 2022-02-22 20:48:07 -0800 | [diff] [blame] | 435 | * epoch_ms,setpt,fan1,fan1_raw,fan1_pwm,fan1_pwm_raw,fan2,fan2_raw,fan2_pwm,fan2_pwm_raw,fanN,fanN_raw,fanN_pwm,fanN_pwm_raw,sensor1,sensor1_raw,sensor2,sensor2_raw,sensorN,sensorN_raw,failsafe |
Patrick Venture | 5f02ad2 | 2018-04-24 10:18:40 -0700 | [diff] [blame] | 436 | */ |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 437 | |
Nirav Shah | ccc8bb6 | 2022-02-17 21:06:51 -0800 | [diff] [blame] | 438 | _log << "epoch_ms,setpt,requester"; |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 439 | |
Patrick Venture | 4a2dc4d | 2018-10-23 09:02:55 -0700 | [diff] [blame] | 440 | for (const auto& f : _fanInputs) |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 441 | { |
Josh Lehan | b300575 | 2022-02-22 20:48:07 -0800 | [diff] [blame] | 442 | _log << "," << f << "," << f << "_raw"; |
| 443 | _log << "," << f << "_pwm," << f << "_pwm_raw"; |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 444 | } |
Patrick Venture | 4a2dc4d | 2018-10-23 09:02:55 -0700 | [diff] [blame] | 445 | for (const auto& t : _thermalInputs) |
Patrick Venture | 5f02ad2 | 2018-04-24 10:18:40 -0700 | [diff] [blame] | 446 | { |
Josh Lehan | b300575 | 2022-02-22 20:48:07 -0800 | [diff] [blame] | 447 | _log << "," << t << "," << t << "_raw"; |
Patrick Venture | 5f02ad2 | 2018-04-24 10:18:40 -0700 | [diff] [blame] | 448 | } |
Josh Lehan | b300575 | 2022-02-22 20:48:07 -0800 | [diff] [blame] | 449 | |
Patrick Venture | 5f02ad2 | 2018-04-24 10:18:40 -0700 | [diff] [blame] | 450 | _log << ",failsafe"; |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 451 | _log << std::endl; |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 452 | } |
| 453 | |
Patrick Venture | 7a98c19 | 2020-08-12 08:35:16 -0700 | [diff] [blame] | 454 | void DbusPidZone::writeLog(const std::string& value) |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 455 | { |
Patrick Venture | 7a98c19 | 2020-08-12 08:35:16 -0700 | [diff] [blame] | 456 | _log << value; |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 457 | } |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 458 | |
| 459 | /* |
| 460 | * TODO(venture) This is effectively updating the cache and should check if the |
| 461 | * values they're using to update it are new or old, or whatnot. For instance, |
| 462 | * if we haven't heard from the host in X time we need to detect this failure. |
| 463 | * |
| 464 | * I haven't decided if the Sensor should have a lastUpdated method or whether |
| 465 | * that should be for the ReadInterface or etc... |
| 466 | */ |
| 467 | |
| 468 | /** |
| 469 | * We want the PID loop to run with values cached, so this will get all the |
| 470 | * fan tachs for the loop. |
| 471 | */ |
Patrick Venture | 597ebd6 | 2020-08-11 08:48:19 -0700 | [diff] [blame] | 472 | void DbusPidZone::updateFanTelemetry(void) |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 473 | { |
| 474 | /* TODO(venture): Should I just make _log point to /dev/null when logging |
| 475 | * is disabled? I think it's a waste to try and log things even if the |
| 476 | * data is just being dropped though. |
| 477 | */ |
Tom Tung | df1f183 | 2022-11-14 19:26:52 +0800 | [diff] [blame] | 478 | const auto now = std::chrono::high_resolution_clock::now(); |
Patrick Venture | de79ee0 | 2019-05-08 14:50:00 -0700 | [diff] [blame] | 479 | if (loggingEnabled) |
Patrick Venture | c32e3fc | 2019-02-28 10:01:11 -0800 | [diff] [blame] | 480 | { |
Patrick Venture | c32e3fc | 2019-02-28 10:01:11 -0800 | [diff] [blame] | 481 | _log << std::chrono::duration_cast<std::chrono::milliseconds>( |
| 482 | now.time_since_epoch()) |
| 483 | .count(); |
Patrick Venture | f7a2dd5 | 2019-07-16 14:31:13 -0700 | [diff] [blame] | 484 | _log << "," << _maximumSetPoint; |
Nirav Shah | ccc8bb6 | 2022-02-17 21:06:51 -0800 | [diff] [blame] | 485 | _log << "," << _maximumSetPointName; |
Patrick Venture | c32e3fc | 2019-02-28 10:01:11 -0800 | [diff] [blame] | 486 | } |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 487 | |
Tom Tung | df1f183 | 2022-11-14 19:26:52 +0800 | [diff] [blame] | 488 | processSensorInputs</* fanSensorLogging */ true>(_fanInputs, now); |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 489 | |
Patrick Venture | de79ee0 | 2019-05-08 14:50:00 -0700 | [diff] [blame] | 490 | if (loggingEnabled) |
Patrick Venture | 5f02ad2 | 2018-04-24 10:18:40 -0700 | [diff] [blame] | 491 | { |
Patrick Venture | c32e3fc | 2019-02-28 10:01:11 -0800 | [diff] [blame] | 492 | for (const auto& t : _thermalInputs) |
| 493 | { |
Josh Lehan | b300575 | 2022-02-22 20:48:07 -0800 | [diff] [blame] | 494 | const auto& v = _cachedValuesByName[t]; |
| 495 | _log << "," << v.scaled << "," << v.unscaled; |
Patrick Venture | c32e3fc | 2019-02-28 10:01:11 -0800 | [diff] [blame] | 496 | } |
Patrick Venture | 5f02ad2 | 2018-04-24 10:18:40 -0700 | [diff] [blame] | 497 | } |
Patrick Venture | 5f02ad2 | 2018-04-24 10:18:40 -0700 | [diff] [blame] | 498 | |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 499 | return; |
| 500 | } |
| 501 | |
Patrick Venture | 597ebd6 | 2020-08-11 08:48:19 -0700 | [diff] [blame] | 502 | void DbusPidZone::updateSensors(void) |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 503 | { |
Tom Tung | df1f183 | 2022-11-14 19:26:52 +0800 | [diff] [blame] | 504 | processSensorInputs</* fanSensorLogging */ false>( |
| 505 | _thermalInputs, std::chrono::high_resolution_clock::now()); |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 506 | |
| 507 | return; |
| 508 | } |
| 509 | |
Patrick Venture | 597ebd6 | 2020-08-11 08:48:19 -0700 | [diff] [blame] | 510 | void DbusPidZone::initializeCache(void) |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 511 | { |
Josh Lehan | 3f0f7bc | 2023-02-13 01:45:29 -0800 | [diff] [blame] | 512 | auto nan = std::numeric_limits<double>::quiet_NaN(); |
| 513 | |
Patrick Venture | 4a2dc4d | 2018-10-23 09:02:55 -0700 | [diff] [blame] | 514 | for (const auto& f : _fanInputs) |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 515 | { |
Josh Lehan | 3f0f7bc | 2023-02-13 01:45:29 -0800 | [diff] [blame] | 516 | _cachedValuesByName[f] = {nan, nan}; |
| 517 | _cachedFanOutputs[f] = {nan, nan}; |
Will Liang | ded0ab5 | 2019-05-15 17:10:06 +0800 | [diff] [blame] | 518 | |
| 519 | // Start all fans in fail-safe mode. |
Harvey Wu | a427007 | 2024-05-29 16:11:13 +0800 | [diff] [blame] | 520 | markSensorMissing(f, ""); |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 521 | } |
| 522 | |
Patrick Venture | 4a2dc4d | 2018-10-23 09:02:55 -0700 | [diff] [blame] | 523 | for (const auto& t : _thermalInputs) |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 524 | { |
Josh Lehan | 3f0f7bc | 2023-02-13 01:45:29 -0800 | [diff] [blame] | 525 | _cachedValuesByName[t] = {nan, nan}; |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 526 | |
| 527 | // Start all sensors in fail-safe mode. |
Harvey Wu | a427007 | 2024-05-29 16:11:13 +0800 | [diff] [blame] | 528 | markSensorMissing(t, ""); |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 529 | } |
| 530 | } |
| 531 | |
Patrick Venture | 597ebd6 | 2020-08-11 08:48:19 -0700 | [diff] [blame] | 532 | void DbusPidZone::dumpCache(void) |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 533 | { |
| 534 | std::cerr << "Cache values now: \n"; |
Patrick Venture | 2a50eda | 2020-08-16 08:26:35 -0700 | [diff] [blame] | 535 | for (const auto& [name, value] : _cachedValuesByName) |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 536 | { |
Josh Lehan | b300575 | 2022-02-22 20:48:07 -0800 | [diff] [blame] | 537 | std::cerr << name << ": " << value.scaled << " " << value.unscaled |
| 538 | << "\n"; |
| 539 | } |
| 540 | |
| 541 | std::cerr << "Fan outputs now: \n"; |
| 542 | for (const auto& [name, value] : _cachedFanOutputs) |
| 543 | { |
| 544 | std::cerr << name << ": " << value.scaled << " " << value.unscaled |
| 545 | << "\n"; |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 546 | } |
| 547 | } |
| 548 | |
Patrick Venture | 597ebd6 | 2020-08-11 08:48:19 -0700 | [diff] [blame] | 549 | void DbusPidZone::processFans(void) |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 550 | { |
| 551 | for (auto& p : _fans) |
| 552 | { |
James Feist | 22c257a | 2018-08-31 14:07:12 -0700 | [diff] [blame] | 553 | p->process(); |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 554 | } |
Josh Lehan | a4146eb | 2020-10-01 11:49:09 -0700 | [diff] [blame] | 555 | |
| 556 | if (_redundantWrite) |
| 557 | { |
| 558 | // This is only needed once |
| 559 | _redundantWrite = false; |
| 560 | } |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 561 | } |
| 562 | |
Patrick Venture | 597ebd6 | 2020-08-11 08:48:19 -0700 | [diff] [blame] | 563 | void DbusPidZone::processThermals(void) |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 564 | { |
| 565 | for (auto& p : _thermals) |
| 566 | { |
James Feist | 22c257a | 2018-08-31 14:07:12 -0700 | [diff] [blame] | 567 | p->process(); |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 568 | } |
| 569 | } |
| 570 | |
Patrick Venture | 597ebd6 | 2020-08-11 08:48:19 -0700 | [diff] [blame] | 571 | Sensor* DbusPidZone::getSensor(const std::string& name) |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 572 | { |
Patrick Venture | fe75b19 | 2018-06-08 11:19:43 -0700 | [diff] [blame] | 573 | return _mgr.getSensor(name); |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 574 | } |
| 575 | |
James Zheng | 6df8bb5 | 2024-11-27 23:38:47 +0000 | [diff] [blame] | 576 | std::vector<std::string> DbusPidZone::getSensorNames(void) |
| 577 | { |
| 578 | return _thermalInputs; |
| 579 | } |
| 580 | |
Josh Lehan | a4146eb | 2020-10-01 11:49:09 -0700 | [diff] [blame] | 581 | bool DbusPidZone::getRedundantWrite(void) const |
| 582 | { |
| 583 | return _redundantWrite; |
| 584 | } |
| 585 | |
Patrick Venture | 597ebd6 | 2020-08-11 08:48:19 -0700 | [diff] [blame] | 586 | bool DbusPidZone::manual(bool value) |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 587 | { |
| 588 | std::cerr << "manual: " << value << std::endl; |
| 589 | setManualMode(value); |
| 590 | return ModeObject::manual(value); |
| 591 | } |
| 592 | |
Patrick Venture | 597ebd6 | 2020-08-11 08:48:19 -0700 | [diff] [blame] | 593 | bool DbusPidZone::failSafe() const |
Patrick Venture | d801218 | 2018-03-08 08:21:38 -0800 | [diff] [blame] | 594 | { |
| 595 | return getFailSafeMode(); |
| 596 | } |
Patrick Venture | a076487 | 2020-08-08 07:48:43 -0700 | [diff] [blame] | 597 | |
Harvey Wu | 3718006 | 2023-10-02 09:42:50 +0800 | [diff] [blame] | 598 | void DbusPidZone::addPidControlProcess(std::string name, std::string type, |
| 599 | double setpoint, sdbusplus::bus_t& bus, |
ykchiu | 7c6d35d | 2023-05-10 17:01:46 +0800 | [diff] [blame] | 600 | std::string objPath, bool defer) |
| 601 | { |
| 602 | _pidsControlProcess[name] = std::make_unique<ProcessObject>( |
| 603 | bus, objPath.c_str(), |
| 604 | defer ? ProcessObject::action::defer_emit |
| 605 | : ProcessObject::action::emit_object_added); |
| 606 | // Default enable setting = true |
| 607 | _pidsControlProcess[name]->enabled(true); |
Harvey Wu | 3718006 | 2023-10-02 09:42:50 +0800 | [diff] [blame] | 608 | _pidsControlProcess[name]->setpoint(setpoint); |
| 609 | |
| 610 | if (type == "temp") |
| 611 | { |
| 612 | _pidsControlProcess[name]->classType("Temperature"); |
| 613 | } |
| 614 | else if (type == "margin") |
| 615 | { |
| 616 | _pidsControlProcess[name]->classType("Margin"); |
| 617 | } |
| 618 | else if (type == "power") |
| 619 | { |
| 620 | _pidsControlProcess[name]->classType("Power"); |
| 621 | } |
| 622 | else if (type == "powersum") |
| 623 | { |
| 624 | _pidsControlProcess[name]->classType("PowerSum"); |
| 625 | } |
ykchiu | 7c6d35d | 2023-05-10 17:01:46 +0800 | [diff] [blame] | 626 | } |
| 627 | |
| 628 | bool DbusPidZone::isPidProcessEnabled(std::string name) |
| 629 | { |
| 630 | return _pidsControlProcess[name]->enabled(); |
| 631 | } |
| 632 | |
Harvey Wu | 92f9f3c | 2023-11-07 09:23:35 +0800 | [diff] [blame] | 633 | void DbusPidZone::addPidFailSafePercent(std::vector<std::string> inputs, |
| 634 | double percent) |
ykchiu | 9fe3a3c | 2023-05-11 13:43:54 +0800 | [diff] [blame] | 635 | { |
Harvey Wu | 92f9f3c | 2023-11-07 09:23:35 +0800 | [diff] [blame] | 636 | for (const auto& sensorName : inputs) |
ykchiu | 9fe3a3c | 2023-05-11 13:43:54 +0800 | [diff] [blame] | 637 | { |
Harvey Wu | 92f9f3c | 2023-11-07 09:23:35 +0800 | [diff] [blame] | 638 | if (_sensorFailSafePercent.find(sensorName) != |
| 639 | _sensorFailSafePercent.end()) |
| 640 | { |
| 641 | _sensorFailSafePercent[sensorName] = |
| 642 | std::max(_sensorFailSafePercent[sensorName], percent); |
| 643 | if (debugEnabled) |
| 644 | { |
| 645 | std::cerr << "Sensor " << sensorName |
| 646 | << " failsafe percent updated to " |
| 647 | << _sensorFailSafePercent[sensorName] << "\n"; |
| 648 | } |
| 649 | } |
| 650 | else |
| 651 | { |
| 652 | _sensorFailSafePercent[sensorName] = percent; |
| 653 | if (debugEnabled) |
| 654 | { |
| 655 | std::cerr << "Sensor " << sensorName |
| 656 | << " failsafe percent set to " << percent << "\n"; |
| 657 | } |
| 658 | } |
ykchiu | 9fe3a3c | 2023-05-11 13:43:54 +0800 | [diff] [blame] | 659 | } |
ykchiu | 9fe3a3c | 2023-05-11 13:43:54 +0800 | [diff] [blame] | 660 | } |
| 661 | |
Harvey Wu | cc0232a | 2023-02-09 14:58:55 +0800 | [diff] [blame] | 662 | std::string DbusPidZone::leader() const |
| 663 | { |
| 664 | return _maximumSetPointName; |
| 665 | } |
| 666 | |
Patrick Williams | bd63bca | 2024-08-16 15:21:10 -0400 | [diff] [blame] | 667 | void DbusPidZone::updateThermalPowerDebugInterface( |
| 668 | std::string pidName, std::string leader, double input, double output) |
Harvey Wu | 3718006 | 2023-10-02 09:42:50 +0800 | [diff] [blame] | 669 | { |
| 670 | if (leader.empty()) |
| 671 | { |
| 672 | _pidsControlProcess[pidName]->output(output); |
| 673 | } |
| 674 | else |
| 675 | { |
| 676 | _pidsControlProcess[pidName]->leader(leader); |
| 677 | _pidsControlProcess[pidName]->input(input); |
| 678 | } |
| 679 | } |
| 680 | |
Delphine CC Chiu | 9788963 | 2023-11-06 11:32:46 +0800 | [diff] [blame] | 681 | bool DbusPidZone::getAccSetPoint(void) const |
| 682 | { |
| 683 | return _accumulateSetPoint; |
| 684 | } |
| 685 | |
Patrick Venture | a076487 | 2020-08-08 07:48:43 -0700 | [diff] [blame] | 686 | } // namespace pid_control |