powerstats: set time limit as timeout for AoC
Bug: 219630658 Test: dumpsys android.hardware.power.stats.IPowerStats/default Change-Id: I9a9eb27fab549f4351bcfae6e335e050c48fc753 Signed-off-by: Darren Hsu <darrenhsu@google.com>
This commit is contained in:
parent
c99aafcbc9
commit
0be3bbd083
1 changed files with 6 additions and 3 deletions
|
@ -45,6 +45,7 @@ using aidl::android::hardware::power::stats::PowerStatsEnergyConsumer;
|
|||
constexpr char kBootHwSoCRev[] = "ro.boot.hw.soc.rev";
|
||||
|
||||
void addAoC(std::shared_ptr<PowerStats> p) {
|
||||
static const uint64_t TIMEOUT_MILLIS = 120;
|
||||
std::string prefix = "/sys/devices/platform/19000000.aoc/control/";
|
||||
|
||||
// Add AoC cores (a32, ff1, hf0, and hf1)
|
||||
|
@ -57,7 +58,7 @@ void addAoC(std::shared_ptr<PowerStats> p) {
|
|||
std::vector<std::pair<std::string, std::string>> coreStates = {
|
||||
{"DWN", "off"}, {"RET", "retention"}, {"WFI", "wfi"}};
|
||||
p->addStateResidencyDataProvider(std::make_unique<AocStateResidencyDataProvider>(coreIds,
|
||||
coreStates));
|
||||
coreStates, TIMEOUT_MILLIS));
|
||||
|
||||
// Add AoC voltage stats
|
||||
std::vector<std::pair<std::string, std::string>> voltageIds = {
|
||||
|
@ -68,7 +69,8 @@ void addAoC(std::shared_ptr<PowerStats> p) {
|
|||
{"UUD", "ultra_underdrive"},
|
||||
{"UD", "underdrive"}};
|
||||
p->addStateResidencyDataProvider(
|
||||
std::make_unique<AocStateResidencyDataProvider>(voltageIds, voltageStates));
|
||||
std::make_unique<AocStateResidencyDataProvider>(voltageIds, voltageStates,
|
||||
TIMEOUT_MILLIS));
|
||||
|
||||
// Add AoC monitor mode
|
||||
std::vector<std::pair<std::string, std::string>> monitorIds = {
|
||||
|
@ -78,7 +80,8 @@ void addAoC(std::shared_ptr<PowerStats> p) {
|
|||
{"MON", "mode"},
|
||||
};
|
||||
p->addStateResidencyDataProvider(
|
||||
std::make_unique<AocStateResidencyDataProvider>(monitorIds, monitorStates));
|
||||
std::make_unique<AocStateResidencyDataProvider>(monitorIds, monitorStates,
|
||||
TIMEOUT_MILLIS));
|
||||
|
||||
// Add AoC restart count
|
||||
const GenericStateResidencyDataProvider::StateResidencyConfig restartCountConfig = {
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue