From b14db4c9a642c3bac5b1f5f37d6c51e824e25e02 Mon Sep 17 00:00:00 2001 From: samou Date: Wed, 14 Jul 2021 04:22:48 +0000 Subject: [PATCH] powerstats: correct frequency unit Bug: 192579762 Test: dumpsys android.hardware.power.stats.IPowerStats/default Change-Id: I8f7703c30116622483faa55f9dc16a5558d43c08 --- .../DevfreqStateResidencyDataProvider.cpp | 2 +- powerstats/Gs101CommonDataProviders.cpp | 246 +++++++++--------- 2 files changed, 124 insertions(+), 124 deletions(-) diff --git a/powerstats/DevfreqStateResidencyDataProvider.cpp b/powerstats/DevfreqStateResidencyDataProvider.cpp index 9b21b882..d59e1e5e 100644 --- a/powerstats/DevfreqStateResidencyDataProvider.cpp +++ b/powerstats/DevfreqStateResidencyDataProvider.cpp @@ -97,7 +97,7 @@ std::unordered_map> DevfreqStateResidencyDataPro int32_t id = 0; std::vector states; for (const auto[frequencyHz, totalTimeMs] : timeInState) { - State s = {.id = id++, .name = std::to_string(frequencyHz / 1000) + "KHz"}; + State s = {.id = id++, .name = std::to_string(frequencyHz / 1000) + "MHz"}; states.push_back(s); } diff --git a/powerstats/Gs101CommonDataProviders.cpp b/powerstats/Gs101CommonDataProviders.cpp index 8a1b8386..e8e96e52 100644 --- a/powerstats/Gs101CommonDataProviders.cpp +++ b/powerstats/Gs101CommonDataProviders.cpp @@ -88,99 +88,99 @@ void addDvfsStats(std::shared_ptr p) { std::vector cfgs; cfgs.push_back({"MIF", { - std::make_pair("3172KHz", "3172000"), - std::make_pair("2730KHz", "2730000"), - std::make_pair("2535KHz", "2535000"), - std::make_pair("2288KHz", "2288000"), - std::make_pair("2028KHz", "2028000"), - std::make_pair("1716KHz", "1716000"), - std::make_pair("1539KHz", "1539000"), - std::make_pair("1352KHz", "1352000"), - std::make_pair("1014KHz", "1014000"), - std::make_pair("845KHz", "845000"), - std::make_pair("676KHz", "676000"), - std::make_pair("546KHz", "546000"), - std::make_pair("421KHz", "421000"), - std::make_pair("0KHz", "0"), + std::make_pair("3172MHz", "3172000"), + std::make_pair("2730MHz", "2730000"), + std::make_pair("2535MHz", "2535000"), + std::make_pair("2288MHz", "2288000"), + std::make_pair("2028MHz", "2028000"), + std::make_pair("1716MHz", "1716000"), + std::make_pair("1539MHz", "1539000"), + std::make_pair("1352MHz", "1352000"), + std::make_pair("1014MHz", "1014000"), + std::make_pair("845MHz", "845000"), + std::make_pair("676MHz", "676000"), + std::make_pair("546MHz", "546000"), + std::make_pair("421MHz", "421000"), + std::make_pair("0MHz", "0"), }}); cfgs.push_back({"CL1", { - std::make_pair("2466KHz", "2466000"), - std::make_pair("2393KHz", "2393000"), - std::make_pair("2348KHz", "2348000"), - std::make_pair("2253KHz", "2253000"), - std::make_pair("2130KHz", "2130000"), - std::make_pair("1999KHz", "1999000"), - std::make_pair("1836KHz", "1836000"), - std::make_pair("1663KHz", "1663000"), - std::make_pair("1491KHz", "1491000"), - std::make_pair("1328KHz", "1328000"), - std::make_pair("1197KHz", "1197000"), - std::make_pair("1024KHz", "1024000"), - std::make_pair("910KHz", "910000"), - std::make_pair("799KHz", "799000"), - std::make_pair("696KHz", "696000"), - std::make_pair("533KHz", "533000"), - std::make_pair("400KHz", "400000"), - std::make_pair("0KHz", "0"), + std::make_pair("2466MHz", "2466000"), + std::make_pair("2393MHz", "2393000"), + std::make_pair("2348MHz", "2348000"), + std::make_pair("2253MHz", "2253000"), + std::make_pair("2130MHz", "2130000"), + std::make_pair("1999MHz", "1999000"), + std::make_pair("1836MHz", "1836000"), + std::make_pair("1663MHz", "1663000"), + std::make_pair("1491MHz", "1491000"), + std::make_pair("1328MHz", "1328000"), + std::make_pair("1197MHz", "1197000"), + std::make_pair("1024MHz", "1024000"), + std::make_pair("910MHz", "910000"), + std::make_pair("799MHz", "799000"), + std::make_pair("696MHz", "696000"), + std::make_pair("533MHz", "533000"), + std::make_pair("400MHz", "400000"), + std::make_pair("0MHz", "0"), }}); // B0/B1 chips have different DVFS operating points than A0/A1 SoC const int socRev = android::base::GetIntProperty(kBootHwSoCRev, 0); if (socRev >= 2) { cfgs.push_back({"CL0", { - std::make_pair("2196KHz", "2196000"), - std::make_pair("2098KHz", "2098000"), - std::make_pair("2024KHz", "2024000"), - std::make_pair("1950KHz", "1950000"), - std::make_pair("1803KHz", "1803000"), - std::make_pair("1704KHz", "1704000"), - std::make_pair("1598KHz", "1598000"), - std::make_pair("1401KHz", "1401000"), - std::make_pair("1328KHz", "1328000"), - std::make_pair("1197KHz", "1197000"), - std::make_pair("1098KHz", "1098000"), - std::make_pair("930KHz", "930000"), - std::make_pair("738KHz", "738000"), - std::make_pair("574KHz", "574000"), - std::make_pair("300KHz", "300000"), - std::make_pair("0KHz", "0"), + std::make_pair("2196MHz", "2196000"), + std::make_pair("2098MHz", "2098000"), + std::make_pair("2024MHz", "2024000"), + std::make_pair("1950MHz", "1950000"), + std::make_pair("1803MHz", "1803000"), + std::make_pair("1704MHz", "1704000"), + std::make_pair("1598MHz", "1598000"), + std::make_pair("1401MHz", "1401000"), + std::make_pair("1328MHz", "1328000"), + std::make_pair("1197MHz", "1197000"), + std::make_pair("1098MHz", "1098000"), + std::make_pair("930MHz", "930000"), + std::make_pair("738MHz", "738000"), + std::make_pair("574MHz", "574000"), + std::make_pair("300MHz", "300000"), + std::make_pair("0MHz", "0"), }}); cfgs.push_back({"CL2", { - std::make_pair("3195KHz", "3195000"), - std::make_pair("3097KHz", "3097000"), - std::make_pair("2950KHz", "2950000"), - std::make_pair("2850KHz", "2850000"), - std::make_pair("2802KHz", "2802000"), - std::make_pair("2704KHz", "2704000"), - std::make_pair("2630KHz", "2630000"), - std::make_pair("2507KHz", "2507000"), - std::make_pair("2401KHz", "2401000"), - std::make_pair("2252KHz", "2252000"), - std::make_pair("2188KHz", "2188000"), - std::make_pair("2048KHz", "2048000"), - std::make_pair("1826KHz", "1826000"), - std::make_pair("1745KHz", "1745000"), - std::make_pair("1582KHz", "1582000"), - std::make_pair("1426KHz", "1426000"), - std::make_pair("1277KHz", "1277000"), - std::make_pair("1106KHz", "1106000"), - std::make_pair("984KHz", "984000"), - std::make_pair("851KHz", "851000"), - std::make_pair("500KHz", "500000"), - std::make_pair("0KHz", "0"), + std::make_pair("3195MHz", "3195000"), + std::make_pair("3097MHz", "3097000"), + std::make_pair("2950MHz", "2950000"), + std::make_pair("2850MHz", "2850000"), + std::make_pair("2802MHz", "2802000"), + std::make_pair("2704MHz", "2704000"), + std::make_pair("2630MHz", "2630000"), + std::make_pair("2507MHz", "2507000"), + std::make_pair("2401MHz", "2401000"), + std::make_pair("2252MHz", "2252000"), + std::make_pair("2188MHz", "2188000"), + std::make_pair("2048MHz", "2048000"), + std::make_pair("1826MHz", "1826000"), + std::make_pair("1745MHz", "1745000"), + std::make_pair("1582MHz", "1582000"), + std::make_pair("1426MHz", "1426000"), + std::make_pair("1277MHz", "1277000"), + std::make_pair("1106MHz", "1106000"), + std::make_pair("984MHz", "984000"), + std::make_pair("851MHz", "851000"), + std::make_pair("500MHz", "500000"), + std::make_pair("0MHz", "0"), }}); cfgs.push_back({"TPU", { - std::make_pair("1393KHz", "1393000"), - std::make_pair("1180KHz", "1180000"), - std::make_pair("1024KHz", "1024000"), - std::make_pair("967KHz", "967000"), - std::make_pair("836KHz", "836000"), - std::make_pair("627KHz", "627000"), - std::make_pair("455KHz", "455000"), - std::make_pair("226KHz", "226000"), + std::make_pair("1393MHz", "1393000"), + std::make_pair("1180MHz", "1180000"), + std::make_pair("1024MHz", "1024000"), + std::make_pair("967MHz", "967000"), + std::make_pair("836MHz", "836000"), + std::make_pair("627MHz", "627000"), + std::make_pair("455MHz", "455000"), + std::make_pair("226MHz", "226000"), std::make_pair("RET_SLOW", "6"), std::make_pair("S_OFF", "5"), std::make_pair("S_SLOW", "4"), @@ -191,57 +191,57 @@ void addDvfsStats(std::shared_ptr p) { }}); } else { cfgs.push_back({"CL0", { - std::make_pair("2196KHz", "2196000"), - std::make_pair("2098KHz", "2098000"), - std::make_pair("2024KHz", "2024000"), - std::make_pair("1950KHz", "1950000"), - std::make_pair("1868KHz", "1868000"), - std::make_pair("1745KHz", "1745000"), - std::make_pair("1598KHz", "1598000"), - std::make_pair("1459KHz", "1459000"), - std::make_pair("1328KHz", "1328000"), - std::make_pair("1197KHz", "1197000"), - std::make_pair("1098KHz", "1098000"), - std::make_pair("889KHz", "889000"), - std::make_pair("738KHz", "738000"), - std::make_pair("574KHz", "574000"), - std::make_pair("300KHz", "300000"), - std::make_pair("0KHz", "0"), + std::make_pair("2196MHz", "2196000"), + std::make_pair("2098MHz", "2098000"), + std::make_pair("2024MHz", "2024000"), + std::make_pair("1950MHz", "1950000"), + std::make_pair("1868MHz", "1868000"), + std::make_pair("1745MHz", "1745000"), + std::make_pair("1598MHz", "1598000"), + std::make_pair("1459MHz", "1459000"), + std::make_pair("1328MHz", "1328000"), + std::make_pair("1197MHz", "1197000"), + std::make_pair("1098MHz", "1098000"), + std::make_pair("889MHz", "889000"), + std::make_pair("738MHz", "738000"), + std::make_pair("574MHz", "574000"), + std::make_pair("300MHz", "300000"), + std::make_pair("0MHz", "0"), }}); cfgs.push_back({"CL2", { - std::make_pair("3195KHz", "3195000"), - std::make_pair("3097KHz", "3097000"), - std::make_pair("2999KHz", "2999000"), - std::make_pair("2900KHz", "2900000"), - std::make_pair("2802KHz", "2802000"), - std::make_pair("2704KHz", "2704000"), - std::make_pair("2630KHz", "2630000"), - std::make_pair("2507KHz", "2507000"), - std::make_pair("2302KHz", "2302000"), - std::make_pair("2188KHz", "2188000"), - std::make_pair("2048KHz", "2048000"), - std::make_pair("1901KHz", "1901000"), - std::make_pair("1745KHz", "1745000"), - std::make_pair("1582KHz", "1582000"), - std::make_pair("1426KHz", "1426000"), - std::make_pair("1237KHz", "1237000"), - std::make_pair("1106KHz", "1106000"), - std::make_pair("984KHz", "984000"), - std::make_pair("848KHz", "848000"), - std::make_pair("500KHz", "500000"), - std::make_pair("0KHz", "0"), + std::make_pair("3195MHz", "3195000"), + std::make_pair("3097MHz", "3097000"), + std::make_pair("2999MHz", "2999000"), + std::make_pair("2900MHz", "2900000"), + std::make_pair("2802MHz", "2802000"), + std::make_pair("2704MHz", "2704000"), + std::make_pair("2630MHz", "2630000"), + std::make_pair("2507MHz", "2507000"), + std::make_pair("2302MHz", "2302000"), + std::make_pair("2188MHz", "2188000"), + std::make_pair("2048MHz", "2048000"), + std::make_pair("1901MHz", "1901000"), + std::make_pair("1745MHz", "1745000"), + std::make_pair("1582MHz", "1582000"), + std::make_pair("1426MHz", "1426000"), + std::make_pair("1237MHz", "1237000"), + std::make_pair("1106MHz", "1106000"), + std::make_pair("984MHz", "984000"), + std::make_pair("848MHz", "848000"), + std::make_pair("500MHz", "500000"), + std::make_pair("0MHz", "0"), }}); cfgs.push_back({"TPU", { - std::make_pair("1393KHz", "1393000"), - std::make_pair("1180KHz", "1180000"), - std::make_pair("1049KHz", "1049000"), - std::make_pair("967KHz", "967000"), - std::make_pair("721KHz", "721000"), - std::make_pair("648KHz", "648000"), - std::make_pair("455KHz", "455000"), - std::make_pair("250KHz", "250000"), + std::make_pair("1393MHz", "1393000"), + std::make_pair("1180MHz", "1180000"), + std::make_pair("1049MHz", "1049000"), + std::make_pair("967MHz", "967000"), + std::make_pair("721MHz", "721000"), + std::make_pair("648MHz", "648000"), + std::make_pair("455MHz", "455000"), + std::make_pair("250MHz", "250000"), std::make_pair("RET_SLOW", "6"), std::make_pair("S_OFF", "5"), std::make_pair("S_SLOW", "4"),