diff options
| author | Hridya Valsaraju <hridya@google.com> | 2018-01-17 23:02:56 -0800 |
|---|---|---|
| committer | Hridya Valsaraju <hridya@google.com> | 2018-01-18 14:51:26 -0800 |
| commit | 79d38ebcaf25dd90b58dac018bc3532db908e9b0 (patch) | |
| tree | 8ca4a7d09f1ea210650e1a754de1ea850c1e127a | |
| parent | 4bd6183721292a1304686b0fe60cd1075dae09f7 (diff) | |
| download | system_core-79d38ebcaf25dd90b58dac018bc3532db908e9b0.tar.gz system_core-79d38ebcaf25dd90b58dac018bc3532db908e9b0.tar.bz2 system_core-79d38ebcaf25dd90b58dac018bc3532db908e9b0.zip | |
health: Modify IHealthInfo Callback interface to return V2.0 HealthInfo
Bug: 71860528
Test: vts-tradefed run vts -m VtsHalHealthV2_0
Change-Id: I7de39ffa4c21772d8878c9a3f054bc6dbbde2fb9
| -rw-r--r-- | healthd/HealthServiceCommon.cpp | 6 | ||||
| -rw-r--r-- | storaged/include/storaged.h | 2 | ||||
| -rw-r--r-- | storaged/storaged.cpp | 4 |
3 files changed, 6 insertions, 6 deletions
diff --git a/healthd/HealthServiceCommon.cpp b/healthd/HealthServiceCommon.cpp index e8fbcb004..a2ca44b05 100644 --- a/healthd/HealthServiceCommon.cpp +++ b/healthd/HealthServiceCommon.cpp @@ -30,7 +30,7 @@ using android::hardware::IPCThreadState; using android::hardware::configureRpcThreadpool; using android::hardware::handleTransportPoll; using android::hardware::setupTransportPolling; -using android::hardware::health::V1_0::HealthInfo; +using android::hardware::health::V2_0::HealthInfo; using android::hardware::health::V1_0::hal_conversion::convertToHealthInfo; using android::hardware::health::V2_0::IHealth; using android::hardware::health::V2_0::implementation::Health; @@ -71,8 +71,8 @@ void healthd_mode_service_2_0_heartbeat(void) { void healthd_mode_service_2_0_battery_update(struct android::BatteryProperties* prop) { HealthInfo info; - convertToHealthInfo(prop, info); - Health::getImplementation()->notifyListeners(info); + convertToHealthInfo(prop, info.legacy); + Health::getImplementation()->notifyListeners(&info); } static struct healthd_mode_ops healthd_mode_service_2_0_ops = { diff --git a/storaged/include/storaged.h b/storaged/include/storaged.h index 7b27988b4..400e734ca 100644 --- a/storaged/include/storaged.h +++ b/storaged/include/storaged.h @@ -136,7 +136,7 @@ class storaged_t : public android::hardware::health::V2_0::IHealthInfoCallback, void remove_user_ce(userid_t user_id); virtual ::android::hardware::Return<void> healthInfoChanged( - const ::android::hardware::health::V1_0::HealthInfo& info); + const ::android::hardware::health::V2_0::HealthInfo& info); void serviceDied(uint64_t cookie, const wp<::android::hidl::base::V1_0::IBase>& who); void report_storage_info(); diff --git a/storaged/storaged.cpp b/storaged/storaged.cpp index 6807cd920..bf8b448ab 100644 --- a/storaged/storaged.cpp +++ b/storaged/storaged.cpp @@ -66,9 +66,9 @@ const uint32_t storaged_t::current_version = 4; using android::hardware::interfacesEqual; using android::hardware::Return; using android::hardware::health::V1_0::BatteryStatus; -using android::hardware::health::V1_0::HealthInfo; using android::hardware::health::V1_0::toString; using android::hardware::health::V2_0::get_health_service; +using android::hardware::health::V2_0::HealthInfo; using android::hardware::health::V2_0::IHealth; using android::hardware::health::V2_0::Result; using android::hidl::manager::V1_0::IServiceManager; @@ -80,7 +80,7 @@ inline charger_stat_t is_charger_on(BatteryStatus prop) { } Return<void> storaged_t::healthInfoChanged(const HealthInfo& props) { - mUidm.set_charger_state(is_charger_on(props.batteryStatus)); + mUidm.set_charger_state(is_charger_on(props.legacy.batteryStatus)); return android::hardware::Void(); } |
