summaryrefslogtreecommitdiffstats
path: root/powercap.cpp
diff options
context:
space:
mode:
authorGunnar Mills <gmills@us.ibm.com>2018-09-14 14:50:03 -0500
committerBrad Bishop <bradleyb@fuzziesquirrel.com>2018-10-18 16:43:22 -0400
commit94df8c9015798764d40665bb363bdbf27510f285 (patch)
treece26611a482f2f03a582838faec29eeb065a6c47 /powercap.cpp
parent7db89e36d7c5649b826b607cb2d90e38bb02ab86 (diff)
downloadopenpower-occ-control-94df8c9015798764d40665bb363bdbf27510f285.tar.gz
openpower-occ-control-94df8c9015798764d40665bb363bdbf27510f285.zip
clang-format: Update to match docs repo
Update the .clang-format file and run clang-format-6.0. This .clang-format matches the example one in https://github.com/openbmc/docs/blob/master/cpp-style-and-conventions.md#clang-formatting Change-Id: Id6760866dedbaeafd83ea8ef2e0303e30b8955aa Signed-off-by: Gunnar Mills <gmills@us.ibm.com> Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
Diffstat (limited to 'powercap.cpp')
-rw-r--r--powercap.cpp61
1 files changed, 27 insertions, 34 deletions
diff --git a/powercap.cpp b/powercap.cpp
index 4c0c330..bc4d3ca 100644
--- a/powercap.cpp
+++ b/powercap.cpp
@@ -1,5 +1,5 @@
-#include <powercap.hpp>
#include <phosphor-logging/log.hpp>
+#include <powercap.hpp>
namespace open_power
{
@@ -8,7 +8,7 @@ namespace occ
namespace powercap
{
-constexpr auto PCAP_PATH = "/xyz/openbmc_project/control/host0/power_cap";
+constexpr auto PCAP_PATH = "/xyz/openbmc_project/control/host0/power_cap";
constexpr auto PCAP_INTERFACE = "xyz.openbmc_project.Control.Power.Cap";
constexpr auto MAPPER_BUSNAME = "xyz.openbmc_project.ObjectMapper";
@@ -20,21 +20,17 @@ constexpr auto POWER_CAP_ENABLE_PROP = "PowerCapEnable";
using namespace phosphor::logging;
-std::string PowerCap::getService(std::string path,
- std::string interface)
+std::string PowerCap::getService(std::string path, std::string interface)
{
- auto mapper = bus.new_method_call(MAPPER_BUSNAME,
- MAPPER_PATH,
- MAPPER_INTERFACE,
- "GetObject");
+ auto mapper = bus.new_method_call(MAPPER_BUSNAME, MAPPER_PATH,
+ MAPPER_INTERFACE, "GetObject");
mapper.append(path, std::vector<std::string>({interface}));
auto mapperResponseMsg = bus.call(mapper);
if (mapperResponseMsg.is_method_error())
{
- log<level::ERR>("Error in mapper call",
- entry("PATH=%s", path.c_str()),
+ log<level::ERR>("Error in mapper call", entry("PATH=%s", path.c_str()),
entry("INTERFACE=%s", interface.c_str()));
// TODO openbmc/openbmc#851 - Once available, throw returned error
throw std::runtime_error("Error in mapper call");
@@ -64,17 +60,16 @@ uint32_t PowerCap::getOccInput(uint32_t pcap, bool pcapEnabled)
// If pcap is not disabled then just return the pcap with the derating
// factor applied.
- return( (static_cast<uint64_t>(pcap) * PS_DERATING_FACTOR) /100);
+ return ((static_cast<uint64_t>(pcap) * PS_DERATING_FACTOR) / 100);
}
uint32_t PowerCap::getPcap()
{
- auto settingService = getService(PCAP_PATH,PCAP_INTERFACE);
+ auto settingService = getService(PCAP_PATH, PCAP_INTERFACE);
- auto method = this->bus.new_method_call(settingService.c_str(),
- PCAP_PATH,
- "org.freedesktop.DBus.Properties",
- "Get");
+ auto method =
+ this->bus.new_method_call(settingService.c_str(), PCAP_PATH,
+ "org.freedesktop.DBus.Properties", "Get");
method.append(PCAP_INTERFACE, POWER_CAP_PROP);
auto reply = this->bus.call(method);
@@ -92,12 +87,11 @@ uint32_t PowerCap::getPcap()
bool PowerCap::getPcapEnabled()
{
- auto settingService = getService(PCAP_PATH,PCAP_INTERFACE);
+ auto settingService = getService(PCAP_PATH, PCAP_INTERFACE);
- auto method = this->bus.new_method_call(settingService.c_str(),
- PCAP_PATH,
- "org.freedesktop.DBus.Properties",
- "Get");
+ auto method =
+ this->bus.new_method_call(settingService.c_str(), PCAP_PATH,
+ "org.freedesktop.DBus.Properties", "Get");
method.append(PCAP_INTERFACE, POWER_CAP_ENABLE_PROP);
auto reply = this->bus.call(method);
@@ -117,7 +111,7 @@ void PowerCap::writeOcc(uint32_t pcapValue)
{
// Create path out to master occ hwmon entry
std::unique_ptr<fs::path> fileName =
- std::make_unique<fs::path>(OCC_HWMON_PATH);
+ std::make_unique<fs::path>(OCC_HWMON_PATH);
*fileName /= occMasterName;
*fileName /= "/hwmon/";
@@ -129,11 +123,11 @@ void PowerCap::writeOcc(uint32_t pcapValue)
// Append on the hwmon string where we write the user power cap
*fileName /= "/caps1_user";
- auto pcapString {std::to_string(pcapValue)};
+ auto pcapString{std::to_string(pcapValue)};
log<level::INFO>("Writing pcap value to hwmon",
- entry("PCAP_PATH=%s",fileName->c_str()),
- entry("PCAP_VALUE=%s",pcapString.c_str()));
+ entry("PCAP_PATH=%s", fileName->c_str()),
+ entry("PCAP_VALUE=%s", pcapString.c_str()));
// Open the hwmon file and write the power cap
std::ofstream file(*fileName, std::ios::out);
file << pcapString;
@@ -160,8 +154,8 @@ void PowerCap::pcapChanged(sdbusplus::message::message& msg)
auto valPropMap = msgData.find(POWER_CAP_PROP);
if (valPropMap != msgData.end())
{
- pcap = sdbusplus::message::variant_ns::get<uint32_t>(
- valPropMap->second);
+ pcap =
+ sdbusplus::message::variant_ns::get<uint32_t>(valPropMap->second);
pcapEnabled = getPcapEnabled();
}
else
@@ -169,8 +163,8 @@ void PowerCap::pcapChanged(sdbusplus::message::message& msg)
valPropMap = msgData.find(POWER_CAP_ENABLE_PROP);
if (valPropMap != msgData.end())
{
- pcapEnabled = sdbusplus::message::variant_ns::get<bool>(
- valPropMap->second);
+ pcapEnabled =
+ sdbusplus::message::variant_ns::get<bool>(valPropMap->second);
pcap = getPcap();
}
else
@@ -180,9 +174,8 @@ void PowerCap::pcapChanged(sdbusplus::message::message& msg)
}
}
- log<level::INFO>("Power Cap Property Change",
- entry("PCAP=%u",pcap),
- entry("PCAP_ENABLED=%u",pcapEnabled));
+ log<level::INFO>("Power Cap Property Change", entry("PCAP=%u", pcap),
+ entry("PCAP_ENABLED=%u", pcapEnabled));
// Determine desired action to write to occ
@@ -194,8 +187,8 @@ void PowerCap::pcapChanged(sdbusplus::message::message& msg)
return;
}
-} // namespace open_power
+} // namespace powercap
} // namespace occ
-}// namespace powercap
+} // namespace open_power
OpenPOWER on IntegriCloud