powerstats: correct display odpm rail name am: 3bf4ba165b

Original change: https://googleplex-android-review.googlesource.com/c/device/google/raviole/+/15416441

Change-Id: Id00f24968c1a82849b58c7411ee61eb50eced7d1
This commit is contained in:
samou 2021-08-13 00:45:43 +00:00 committed by Automerger Merge Worker
commit 31e229f365
2 changed files with 34 additions and 2 deletions

View file

@ -31,6 +31,13 @@ using aidl::android::hardware::power::stats::DisplayStateResidencyDataProvider;
using aidl::android::hardware::power::stats::EnergyConsumerType;
using aidl::android::hardware::power::stats::PowerStatsEnergyConsumer;
const char kBootRevision[] = "ro.boot.revision";
std::map<std::string, std::string> displayChannelNames = {
{"PROTO1.0", "PPVAR_VSYS_PWR_DISP"},
{"EVT1.0", "PPVAR_VSYS_PWR_DISP"},
{"EVT1.1", "VSYS_PWR_DISPLAY"},
};
void addDisplay(std::shared_ptr<PowerStats> p) {
// Add display residency stats
std::vector<std::string> states = {
@ -45,13 +52,22 @@ void addDisplay(std::shared_ptr<PowerStats> p) {
"/sys/class/backlight/panel0-backlight/state",
states));
std::string rev = android::base::GetProperty(kBootRevision, "");
std::string channelName;
if (displayChannelNames.find(rev) == displayChannelNames.end()) {
channelName = displayChannelNames["EVT1.1"];
} else {
channelName = displayChannelNames[rev];
}
// Add display energy consumer
/*
* TODO(b/167216667): Add correct display power model here. Must read from display rail
* and include proper coefficients for display states.
*/
p->addEnergyConsumer(PowerStatsEnergyConsumer::createMeterAndEntityConsumer(p,
EnergyConsumerType::DISPLAY, "display", {"PPVAR_VSYS_PWR_DISP"}, "Display",
EnergyConsumerType::DISPLAY, "display", {channelName}, "Display",
{{"LP: 1080x2400@30", 1},
{"On: 1080x2400@60", 2},
{"On: 1080x2400@90", 3},

View file

@ -33,6 +33,13 @@ using aidl::android::hardware::power::stats::EnergyConsumerType;
using aidl::android::hardware::power::stats::GenericStateResidencyDataProvider;
using aidl::android::hardware::power::stats::PowerStatsEnergyConsumer;
const char kBootRevision[] = "ro.boot.revision";
std::map<std::string, std::string> displayChannelNames = {
{"PROTO1.0", "PPVAR_VSYS_PWR_DISP"},
{"EVT1.0", "PPVAR_VSYS_PWR_DISP"},
{"EVT1.1", "VSYS_PWR_DISPLAY"},
};
void addDisplay(std::shared_ptr<PowerStats> p) {
// Add display residency stats
std::vector<std::string> states = {
@ -51,13 +58,22 @@ void addDisplay(std::shared_ptr<PowerStats> p) {
"/sys/class/backlight/panel0-backlight/state",
states));
std::string rev = android::base::GetProperty(kBootRevision, "");
std::string channelName;
if (displayChannelNames.find(rev) == displayChannelNames.end()) {
channelName = displayChannelNames["EVT1.1"];
} else {
channelName = displayChannelNames[rev];
}
// Add display energy consumer
/*
* TODO(b/167216667): Add correct display power model here. Must read from display rail
* and include proper coefficients for display states.
*/
p->addEnergyConsumer(PowerStatsEnergyConsumer::createMeterAndEntityConsumer(p,
EnergyConsumerType::DISPLAY, "display", {"PPVAR_VSYS_PWR_DISP"}, "Display",
EnergyConsumerType::DISPLAY, "display", {channelName}, "Display",
{{"LP: 1440x3120@10", 1},
{"LP: 1440x3120@30", 2},
{"On: 1440x3120@10", 3},