diff --git a/device-comet.mk b/device-comet.mk index da0ab35..74e8416 100644 --- a/device-comet.mk +++ b/device-comet.mk @@ -79,6 +79,8 @@ PRODUCT_DEFAULT_PROPERTY_OVERRIDES += ro.vendor.primarydisplay.vrr.expected_pres PRODUCT_DEFAULT_PROPERTY_OVERRIDES += ro.vendor.primarydisplay.vrr.expected_present.timeout_ns=500000000 endif +PRODUCT_DEFAULT_PROPERTY_OVERRIDES += ro.vendor.primarydisplay.powerstats.entity_name=Inner-Display + PRODUCT_VENDOR_PROPERTIES += \ vendor.primarydisplay.op.hs_hz=120 \ vendor.primarydisplay.op.ns_hz=60 diff --git a/powerstats/comet/service.cpp b/powerstats/comet/service.cpp index 346f30b..98e2966 100644 --- a/powerstats/comet/service.cpp +++ b/powerstats/comet/service.cpp @@ -32,36 +32,12 @@ using aidl::android::hardware::power::stats::DisplayStateResidencyDataProvider; using aidl::android::hardware::power::stats::EnergyConsumerType; using aidl::android::hardware::power::stats::PowerStatsEnergyConsumer; -void addDisplay(std::shared_ptr p) { - // Add display residency stats for inner display - struct stat primaryBuffer; - if (!stat("/sys/class/drm/card0/device/primary-panel/time_in_state", &primaryBuffer)) { - // time_in_state exists - addDisplayMrrByEntity(p, "Inner Display", "/sys/class/drm/card0/device/primary-panel/"); - } else { - // time_in_state doesn't exist - std::vector inner_states = { - "Off", - "LP: 2152x2076@1", - "LP: 2152x2076@30", - "On: 2152x2076@1", - "On: 2152x2076@10", - "On: 2152x2076@60", - "On: 2152x2076@120", - "HBM: 2152x2076@60", - "HBM: 2152x2076@120"}; - - p->addStateResidencyDataProvider(std::make_unique( - "Inner Display", - "/sys/class/backlight/panel0-backlight/state", - inner_states)); - } - +void addOuterDisplay(std::shared_ptr p) { // Add display residency stats for outer display struct stat secondaryBuffer; if (!stat("/sys/class/drm/card0/device/secondary-panel/time_in_state", &secondaryBuffer)) { // time_in_state exists - addDisplayMrrByEntity(p, "Outer Display", "/sys/class/drm/card0/device/secondary-panel/"); + addDisplayMrrByEntity(p, "Outer-Display", "/sys/class/drm/card0/device/secondary-panel/"); } else { // time_in_state doesn't exist std::vector outer_states = { @@ -73,7 +49,7 @@ void addDisplay(std::shared_ptr p) { "HBM: 1080x2424@120"}; p->addStateResidencyDataProvider(std::make_unique( - "Outer Display", + "Outer-Display", "/sys/class/backlight/panel1-backlight/state", outer_states)); } @@ -94,8 +70,9 @@ int main() { std::shared_ptr p = ndk::SharedRefBase::make(); + addDisplayVrr(p, "Inner-Display"); addZumaProCommonDataProviders(p); - addDisplay(p); + addOuterDisplay(p); const std::string instance = std::string() + PowerStats::descriptor + "/default"; binder_status_t status = AServiceManager_addService(p->asBinder().get(), instance.c_str());