diff options
Diffstat (limited to 'services/powermanager/PowerHalWrapper.cpp')
-rw-r--r-- | services/powermanager/PowerHalWrapper.cpp | 20 |
1 files changed, 14 insertions, 6 deletions
diff --git a/services/powermanager/PowerHalWrapper.cpp b/services/powermanager/PowerHalWrapper.cpp index 95f862370b..4a711ca80b 100644 --- a/services/powermanager/PowerHalWrapper.cpp +++ b/services/powermanager/PowerHalWrapper.cpp @@ -29,12 +29,20 @@ namespace power { // ------------------------------------------------------------------------------------------------- inline HalResult toHalResult(const binder::Status& result) { - return result.isOk() ? HalResult::SUCCESSFUL : HalResult::FAILED; + if (result.isOk()) { + return HalResult::SUCCESSFUL; + } + ALOGE("Power HAL request failed: %s", result.toString8().c_str()); + return HalResult::FAILED; } template <typename T> inline HalResult toHalResult(const hardware::Return<T>& result) { - return result.isOk() ? HalResult::SUCCESSFUL : HalResult::FAILED; + if (result.isOk()) { + return HalResult::SUCCESSFUL; + } + ALOGE("Power HAL request failed: %s", result.description().c_str()); + return HalResult::FAILED; } // ------------------------------------------------------------------------------------------------- @@ -117,8 +125,8 @@ HalResult AidlHalWrapper::setBoost(Boost boost, int32_t durationMs) { bool isSupported = false; auto isSupportedRet = mHandle->isBoostSupported(boost, &isSupported); if (!isSupportedRet.isOk()) { - ALOGV("Skipped setBoost %s because Power HAL is not available to check support", - toString(boost).c_str()); + ALOGE("Skipped setBoost %s because check support failed with: %s", + toString(boost).c_str(), isSupportedRet.toString8().c_str()); return HalResult::FAILED; } @@ -148,8 +156,8 @@ HalResult AidlHalWrapper::setMode(Mode mode, bool enabled) { bool isSupported = false; auto isSupportedRet = mHandle->isModeSupported(mode, &isSupported); if (!isSupportedRet.isOk()) { - ALOGV("Skipped setMode %s because Power HAL is not available to check support", - toString(mode).c_str()); + ALOGE("Skipped setMode %s because check support failed with: %s", + toString(mode).c_str(), isSupportedRet.toString8().c_str()); return HalResult::FAILED; } |