commit 7c10792f83fea576d0b6044831cbf83d61cd8c74
author TreeHugger Robot <treehugger-gerrit@google.com> Mon Dec 26 16:47:52 2022 +0000
committer Android (Google) Code Review <android-gerrit@google.com> Mon Dec 26 16:47:52 2022 +0000
tree 5b4e79844ceeaba4dec55c96b3adf6529f13c752
parent 43e0cc51ee239263716a12817855f7194da7da68
parent 29fb05b40a241f23273b630e366a4528f27fe277

Merge "libhwc2.1: reset LHBM state while entering AOD mode" into tm-qpr-dev

libhwc2.1/libdevice/BrightnessController.cpp

diff --git a/libhwc2.1/libdevice/BrightnessController.cpp b/libhwc2.1/libdevice/BrightnessController.cpp
index 434e63e..1057629 100644
--- a/libhwc2.1/libdevice/BrightnessController.cpp
+++ b/libhwc2.1/libdevice/BrightnessController.cpp
@@ -382,9 +382,10 @@
 }
 
 void BrightnessController::onClearDisplay() {
+    resetLhbmState();
+
     std::lock_guard<std::recursive_mutex> lock(mBrightnessMutex);
     mEnhanceHbmReq.reset(false);
-    mLhbmReq.reset(false);
     mBrightnessFloatReq.reset(-1);
     mInstantHbmReq.reset(false);
 
@@ -397,9 +398,6 @@
     if (mBrightnessDimmingUsage == BrightnessDimmingUsage::NORMAL) {
         mDimming.store(true);
     }
-    mLhbm.reset(false);
-
-    mLhbmBrightnessAdj = false;
 
     std::lock_guard<std::recursive_mutex> lock1(mCabcModeMutex);
     mCabcMode.reset(false);
@@ -777,6 +775,13 @@
     return ret == NO_ERROR;
 }
 
+void BrightnessController::resetLhbmState() {
+    std::lock_guard<std::recursive_mutex> lock(mBrightnessMutex);
+    mLhbmReq.reset(false);
+    mLhbm.reset(false);
+    mLhbmBrightnessAdj = false;
+}
+
 void BrightnessController::setOutdoorVisibility(LbeState state) {
     std::lock_guard<std::recursive_mutex> lock(mCabcModeMutex);
     mOutdoorVisibility = (state != LbeState::OFF);

libhwc2.1/libdevice/BrightnessController.h

diff --git a/libhwc2.1/libdevice/BrightnessController.h b/libhwc2.1/libdevice/BrightnessController.h
index 97f6ea0..1a19758 100644
--- a/libhwc2.1/libdevice/BrightnessController.h
+++ b/libhwc2.1/libdevice/BrightnessController.h
@@ -116,6 +116,7 @@
     }
     int checkSysfsStatus(const char *file, const std::vector<std::string>& expectedValue,
                          const nsecs_t timeoutNs);
+    void resetLhbmState();
 
     uint32_t getBrightnessLevel() {
         std::lock_guard<std::recursive_mutex> lock(mBrightnessMutex);

libhwc2.1/libmaindisplay/ExynosPrimaryDisplay.cpp

diff --git a/libhwc2.1/libmaindisplay/ExynosPrimaryDisplay.cpp b/libhwc2.1/libmaindisplay/ExynosPrimaryDisplay.cpp
index 7d72633..594495b 100644
--- a/libhwc2.1/libmaindisplay/ExynosPrimaryDisplay.cpp
+++ b/libhwc2.1/libmaindisplay/ExynosPrimaryDisplay.cpp
@@ -343,6 +343,11 @@
         mPowerModeState = mode;
     }
 
+    // LHBM will be disabled in the kernel while entering AOD mode if it's
+    // already enabled. Reset the state to avoid the sync problem.
+    mBrightnessController->resetLhbmState();
+    mLhbmOn = false;
+
     ExynosDisplay::updateRefreshRateHint();
 
     return HWC2_ERROR_NONE;