Skip to content

Commit

Permalink
hw-mgmt: thermal: Revert "Set pwm update worker immediate run only fo…
Browse files Browse the repository at this point in the history
…r high PWM trend"

1. Revert change ""Set pwm update worker immediate run only for high PWM trend"
2. Removed error reading of psu{X}_fan1_speed_get from error cases
TC not using psu1_fan1_speed_get value in PWM calculation. TC only shows
PSU FAN speed "just for information" in the log and should not be considered as thermal  error case.

Signed-off-by: Oleksandr Shamray <[email protected]>
  • Loading branch information
sholeksandr committed Aug 31, 2023
1 parent 73f8ebc commit a9677e2
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions usr/usr/bin/hw_management_thermal_control.py
Original file line number Diff line number Diff line change
Expand Up @@ -1522,7 +1522,8 @@ def handle_input(self, thermal_table, flow_dir, amb_tmp):
self.update_value(value)
self.log.debug("{} value {}".format(self.name, self.value))
except BaseException:
self.handle_reading_file_err(rpm_file_name)
self.update_value(-1)
pass
return

# ----------------------------------------------------------------------
Expand Down Expand Up @@ -2386,7 +2387,7 @@ def _set_pwm(self, pwm, reason=""):
self._update_psu_fan_speed(pwm)
self.pwm_target = pwm
if self.pwm_worker_timer:
self.pwm_worker_timer.start( self.pwm_target > self.pwm )
self.pwm_worker_timer.start(True)
else:
self.pwm = pwm
self._update_chassis_fan_speed(self.pwm)
Expand Down

0 comments on commit a9677e2

Please sign in to comment.