psu-ng: Remove Min Power Supplies and cleanup
The psu manager would not prevent a power on based on the number of
power supplies, it's only concerned about the max number so that if the
number of plugged power supplies is less than the specified max, then it
can take action such as log an error.
Therefore there's not a current use case for the min setting, so
removing it.
Also cleanup the function that parses that information from the JSON
config file, like removing passing parameters that are class members,
and adding a doxygen description. In addition, have the structure of
system properties as part of the class instead of individual variables,
since it's duplicate work to set the structure values then set the
individual variables.
Tested: With debug traces verified that maxPowerSupplies was set to 2
which is the value in the JSON config file.
Change-Id: Id98900d213fcbef38dc3879b425161a8af172716
Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>
diff --git a/phosphor-power-supply/psu_manager.hpp b/phosphor-power-supply/psu_manager.hpp
index 06d7c82..f3bba27 100644
--- a/phosphor-power-supply/psu_manager.hpp
+++ b/phosphor-power-supply/psu_manager.hpp
@@ -11,7 +11,6 @@
struct sys_properties
{
- int minPowerSupplies;
int maxPowerSupplies;
};
@@ -47,9 +46,12 @@
PSUManager(sdbusplus::bus::bus& bus, const sdeventplus::Event& e,
const std::string& configfile);
- void getJSONProperties(const std::string& path, sdbusplus::bus::bus& bus,
- sys_properties& p,
- std::vector<std::unique_ptr<PowerSupply>>& psus);
+ /**
+ * @brief Initialize the system properties and PowerSupply objects from
+ * the JSON config file.
+ * @param[in] path - Path to the JSON config file
+ */
+ void getJSONProperties(const std::string& path);
/**
* Initializes the manager.
@@ -188,14 +190,9 @@
}
/**
- * @brief Minimum number of power supplies to operate.
+ * @brief The system properties.
*/
- int minPSUs = 1;
-
- /**
- * @brief Maximum number of power supplies possible.
- */
- int maxPSUs = 1;
+ sys_properties sysProperties;
/**
* @brief The vector for power supplies.