diff options
| author | Yichi Chen <yichichen@google.com> | 2022-01-27 22:53:31 +0800 |
|---|---|---|
| committer | Yichi Chen <yichichen@google.com> | 2022-02-08 20:12:51 +0800 |
| commit | e7f859b6f0d94239e582b1a040d354b513db5d0f (patch) | |
| tree | 7ddeb7cf9213e6c59ce7298d40df548267afc6e9 /hwc3/ComposerClient.cpp | |
| parent | eb711fd77622753839047b6d6675214c03ef4828 (diff) | |
hwc: correct the functionality of getDisplayCapabilities
Since BRIGHTNESS and DOZE have been included in getDisplayCapabilities,
we don't need to call the original support function individually.
Bug: 216118058
Test: atest VtsHalGraphicsComposer3_TargetTest
Change-Id: I85932e9d050813c9d2b89bef5783900fbd4c88f9
Diffstat (limited to 'hwc3/ComposerClient.cpp')
| -rw-r--r-- | hwc3/ComposerClient.cpp | 22 |
1 files changed, 9 insertions, 13 deletions
diff --git a/hwc3/ComposerClient.cpp b/hwc3/ComposerClient.cpp index c820a33..96d5f58 100644 --- a/hwc3/ComposerClient.cpp +++ b/hwc3/ComposerClient.cpp @@ -158,23 +158,19 @@ ndk::ScopedAStatus ComposerClient::getDisplayCapabilities(int64_t display, std::vector<DisplayCapability>* caps) { DEBUG_FUNC(); auto err = mHal->getDisplayCapabilities(display, caps); - if (!err) { - bool support = false; - mHal->getRCDLayerSupport(display, support); - if (support) { - caps->push_back(DisplayCapability::DISPLAY_DECORATION); - } + if (err) { return TO_BINDER_STATUS(err); } - bool support; - err = mHal->getDisplayBrightnessSupport(display, support); - if (err == 0 && support) { - caps->push_back(DisplayCapability::BRIGHTNESS); + + bool support = false; + err = mHal->getRCDLayerSupport(display, support); + if (err != ::android::OK) { + LOG(ERROR) << "failed to getRCDLayerSupport: " << err; } - err = mHal->getDozeSupport(display, support); - if (err == 0 && support) { - caps->push_back(DisplayCapability::DOZE); + if (support) { + caps->push_back(DisplayCapability::DISPLAY_DECORATION); } + return TO_BINDER_STATUS(err); } |