From 4c5bbfc702a1c1c2d715bd6a5b8b6b5c0501eb00 Mon Sep 17 00:00:00 2001
From: Darren Hsu <darrenhsu@google.com>
Date: Wed, 24 Aug 2022 16:41:24 +0800
Subject: [PATCH] powerstats: update GPU EnergyConsumer channels

GPU EnergyConsumer should include both S8S_VDD_G3D_L2 and
S2S_VDD_G3D channels.

Bug: 243455584
Test: verified both channels are included
Change-Id: Ia63c89ff5ba24d026a4f0b1e368141521e47b611
Signed-off-by: Darren Hsu <darrenhsu@google.com>
---
 powerstats/Gs201CommonDataProviders.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/powerstats/Gs201CommonDataProviders.cpp b/powerstats/Gs201CommonDataProviders.cpp
index 0e00df71..7a01405d 100644
--- a/powerstats/Gs201CommonDataProviders.cpp
+++ b/powerstats/Gs201CommonDataProviders.cpp
@@ -381,7 +381,7 @@ void addGPU(std::shared_ptr<PowerStats> p) {
         {"848000", 4044}};
 
     p->addEnergyConsumer(PowerStatsEnergyConsumer::createMeterAndAttrConsumer(p,
-            EnergyConsumerType::OTHER, "GPU", {"S8S_VDD_G3D_L2"},
+            EnergyConsumerType::OTHER, "GPU", {"S8S_VDD_G3D_L2", "S2S_VDD_G3D"},
             {{UID_TIME_IN_STATE, "/sys/devices/platform/28000000.mali/uid_time_in_state"}},
             stateCoeffs));