
<html>

<head>

<meta http-equiv="Content-Type" content="text/html; charset=big5">

<style type="text/css" style="display:none;"><!-- P {margin-top:0;margin-bottom:0;} --></style>

</head>

<body dir="ltr">

<div id="divtagdefaultwrapper" dir="ltr" style="font-size: 12pt; color: rgb(0, 0, 0); font-family: Calibri, Helvetica, sans-serif, EmojiFont, "Apple Color Emoji", "Segoe UI Emoji", NotoColorEmoji, "Segoe UI Symbol", "Android Emoji", EmojiSymbols;">

<p>Hi All,</p>

<p><br>

</p>

<p>This issue is related to the repository <a href="https://github.com/openbmc/phosphor-pid-control" class="OWAAutoLink">

phosphor-pid-control</a>.</p>

<p><br>

</p>

<p>It is in commit <span><a href="https://github.com/openbmc/phosphor-pid-control/commit/75eb769d351434547899186f73ff70ae00d7934a" class="OWAAutoLink">dbuspassive: allow scaling</a> that one added scaling mechanism. However when it comes to

<a href="https://github.com/openbmc/phosphor-pid-control/blob/master/pid/ec/pid.cpp" class="OWAAutoLink">

pid.cpp</a>, it may encounter scaling mismatch. That is to say, (in fan control stage) the pid takes the scaled values as inputs, but it utilizes unscaled <span>setpoints resulting wrong error terms. Or just change the setpoint value?</span></span></p>

<p><span><span><br>

</span></span></p>

<p><span><span>Thanks,</span></span></p>

<p><span><span><br>

</span></span></p>

<p>Hank Liou</p>

<p>Quanta Computer Inc.</p>

</div>

</body>

</html>