ibm-openbmc / dev

Product Development Project Mgmt and Tracking
16 stars 2 forks source link

Add hysteresis to Everest ambient temp thresholds #3596

Closed spinler closed 1 year ago

spinler commented 2 years ago

In Storm King.json we forgot to add 3 deg of hysteresis to the thresholds on the virtual ambient temp, so we should get that added.

Also, the HardShutdown threshold is the EPOW4 one, which should really be nonreversable but that is supposed to be accomplished with a large hysteresis so the temperature can dip below the threshold within the 15s before shutdown. So we need to use large hysteresis values on the HardShutdown thresholds for both Everest and Rainier.

spinler commented 1 year ago

https://github.com/ibm-openbmc/entity-manager/pull/34

spinler commented 1 year ago

Done.