i2cvr: restrict apply time to OnReset
Applying VR firmware updates with Immediate ApplyTime may trigger
power fluctuations in a running system and lead to unstable or
abnormal behavior. To ensure safe updates, the ApplyTime for VR
devices is restricted to OnReset. This guarantees updates occur
only during a system reset, when power rails are stable.
Change-Id: If7e4bc5d8fb5bb187b3baacd9e6f782f41738bcd
Signed-off-by: Kevin Tung <kevin.tung.openbmc@gmail.com>
diff --git a/i2c-vr/i2cvr_software_manager.cpp b/i2c-vr/i2cvr_software_manager.cpp
index 29e198f..691a20d 100644
--- a/i2c-vr/i2cvr_software_manager.cpp
+++ b/i2c-vr/i2cvr_software_manager.cpp
@@ -91,10 +91,7 @@
software->setVersion(std::format("{:X}", sum),
SoftwareInf::SoftwareVersion::VersionPurpose::Other);
- std::set<RequestedApplyTimes> allowedApplyTime = {
- RequestedApplyTimes::Immediate, RequestedApplyTimes::OnReset};
-
- software->enableUpdate(allowedApplyTime);
+ software->enableUpdate({RequestedApplyTimes::OnReset});
i2cDevice->softwareCurrent = std::move(software);