fan_config: minipack3ba: update fan service config with incremental PID logic#590
fan_config: minipack3ba: update fan service config with incremental PID logic#590jeankuo wants to merge 1 commit into
Conversation
| "shutdownCondition": { | ||
| "numOvertempSensorForShutdown": 1, | ||
| "conditions": [ | ||
| { | ||
| "sensorName": "SMB_TH5_TEMP", | ||
| "overtempThreshold": 110.0, | ||
| "slidingWindowSize": 1 | ||
| } | ||
| ] | ||
| }, |
There was a problem hiding this comment.
How many levels of hardware protection (without reliance on software) do we have? If we have sufficient protection in hardware itself, we dont need this here.
There was a problem hiding this comment.
How many levels of hardware protection (without reliance on software) do we have? If we have sufficient protection in hardware itself, we dont need this here.
Currently, we have a hardware protection mechanism built into the TH5:
it automatically shuts down the device and enters reset mode when any of the
15 PVTMON sensors reaches 125°C.
Our current software shutdown threshold is set to 110°C.
This allows us to shut down the system earlier than the hardware limit,
which provides an additional safety margin.
If we want to maintain this earlier shutdown behavior,
we would still need to keep the software shutdown command in place.
Thanks.
There was a problem hiding this comment.
This is the protection provided by TH5 hardware itself. Do you have any mechanism in CPLD/FPGA which will shut down TH5 power supply when the ASIC temperature exceeds a threshold? This is without any involvement from user space software.
There was a problem hiding this comment.
This is the protection provided by TH5 hardware itself. Do you have any mechanism in CPLD/FPGA which will shut down TH5 power supply when the ASIC temperature exceeds a threshold? This is without any involvement from user space software.
Hi @somasun ,
At present, the SMB CPLD does not implement any hardware protection mechanism that would automatically shut down the TH5 power supply when the ASIC temperature exceeds a certain threshold. Such control is not handled at the CPLD level.

Description:
Incremental PID logic has been verified by thermal team. Update the fan service config data according to thermal's tuning result.
Changes:
Test Plan:
Test Log:
1_mp3ba_platform_manager_smb_change_to_0x33.txt
2_mp3ba_thermal_team_fan_service_35C.txt
3_mp3ba_thermal_team_fan_service_35C.xlsx
4_mp3ba_thermal_team_fan_service_35C_fan3_one_rotor_failed.txt
5_mp3ba_thermal_team_fan_service_35C_fan3_one_rotor_failed.xlsx
6_mp3ba_th5_pwr_en_test_log.txt