summaryrefslogtreecommitdiff
path: root/meta-openbmc-mods/meta-common/recipes-intel/psu-manager/psu-manager/0001-disable-PSU-cold-redundancy.patch
diff options
context:
space:
mode:
Diffstat (limited to 'meta-openbmc-mods/meta-common/recipes-intel/psu-manager/psu-manager/0001-disable-PSU-cold-redundancy.patch')
-rw-r--r--meta-openbmc-mods/meta-common/recipes-intel/psu-manager/psu-manager/0001-disable-PSU-cold-redundancy.patch73
1 files changed, 0 insertions, 73 deletions
diff --git a/meta-openbmc-mods/meta-common/recipes-intel/psu-manager/psu-manager/0001-disable-PSU-cold-redundancy.patch b/meta-openbmc-mods/meta-common/recipes-intel/psu-manager/psu-manager/0001-disable-PSU-cold-redundancy.patch
deleted file mode 100644
index 23b805b87..000000000
--- a/meta-openbmc-mods/meta-common/recipes-intel/psu-manager/psu-manager/0001-disable-PSU-cold-redundancy.patch
+++ /dev/null
@@ -1,73 +0,0 @@
-From 5829d9e6e1956ebb34ed8a723b0758146529459f Mon Sep 17 00:00:00 2001
-From: AppaRao Puli <apparao.puli@linux.intel.com>
-Date: Wed, 7 Oct 2020 22:42:26 +0530
-Subject: [PATCH] disable PSU cold redundancy
-
-In RP platforms, single PSU also considered as
-valid configuration. We don't have user configuration
-option to enable/disable PSU cold redundancy. So
-it should be disabled by default to avoid issues in
-Rp platforms.
-Also make sure if persistent config already set this
-to true, make it to false.
-
-This avoids unwanted critical event logs and
-unexpected LED status for RP platforms where
-single PSU also considered as valid config.
-
-Tested:
- - Rebooted BMC and observed no CR event logs and
- no amber blocking of status LED.
- - Set the persistent store to true, rebooted
- BMC and value changed back to disabled.
-
-Change-Id: Ie0f1f3f8daa95593af6db698d65ea804cebfee87
-Signed-off-by: AppaRao Puli <apparao.puli@linux.intel.com>
----
- src/cold_redundancy.cpp | 15 ++++++++++++++-
- 1 file changed, 14 insertions(+), 1 deletion(-)
-
-diff --git a/src/cold_redundancy.cpp b/src/cold_redundancy.cpp
-index d64a9e3..3bfd37f 100644
---- a/src/cold_redundancy.cpp
-+++ b/src/cold_redundancy.cpp
-@@ -76,8 +76,9 @@ ColdRedundancy::ColdRedundancy(
- std::cerr << "error initializing assoc interface\n";
- }
-
-+ // For RP platforms, default cold redundancy should be disabled.
-+ powerSupplyRedundancyEnabled(false);
- // set default configuration
-- powerSupplyRedundancyEnabled(true);
- rotationEnabled(true);
- periodOfRotation(7 * oneDay);
- rotationAlgorithm(Algo::bmcSpecific);
-@@ -109,6 +110,14 @@ ColdRedundancy::ColdRedundancy(
- return;
- }
-
-+ // For RP platforms, cold redundancy should be disabled.
-+ // If its already set to true in persistent area, Lets
-+ // override to false during bootup.
-+ if (*redundancyEnabled)
-+ {
-+ *redundancyEnabled = false;
-+ }
-+
- if (*period >= minRotationPeriod && *period <= maxRotationPeriod)
- {
- periodOfRotation(*period);
-@@ -867,6 +876,10 @@ void ColdRedundancy::readPmbus(uint8_t bus, uint8_t slaveAddr, int& value)
-
- void ColdRedundancy::checkRedundancyEvent()
- {
-+ if (!crSupported || !powerSupplyRedundancyEnabled())
-+ {
-+ return;
-+ }
- puRedundantTimer.expires_after(std::chrono::seconds(2));
- puRedundantTimer.async_wait([this](const boost::system::error_code& ec) {
- if (ec == boost::asio::error::operation_aborted)
---
-2.7.4
-