commit f1820dbcbd50dabf53b9075ffb4e8cf15f1e873f
author samou <samou@google.com> Thu Jul 29 07:38:05 2021 +0000
committer Thierry Strudel <tstrudel@google.com> Fri Aug 13 00:33:00 2021 +0000
tree 29fb1d7732136e16f31bcca9f664a8daa2512596
parent 4a6d0b7b8ee9ed3439e5c95cc8cf8f88734d753f

powerstats: correct display odpm rail name

Bug: 194651088
Test: dumpsys android.hardware.power.stats.IPowerStats/default
Change-Id: I40b1ab79c64a26008746933fd293064ff69eeebd

powerstats/raven/service.cpp

diff --git a/powerstats/raven/service.cpp b/powerstats/raven/service.cpp
index 6ee27d8..579efa1 100644
--- a/powerstats/raven/service.cpp
+++ b/powerstats/raven/service.cpp
@@ -33,6 +33,13 @@
 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 @@
             "/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},