From 1f3da1c8b93a8902b9af01adc5d5997601c0d8df Mon Sep 17 00:00:00 2001 From: Julius D'souza Date: Tue, 5 Sep 2017 10:17:06 -0700 Subject: [PATCH] DO NOT MERGE: Dynamic doze check should be required before adjusting sensor rate. Some devices have catastrophic firmware issues in switching the sensor rate; for these devices, the sensor rate switch should be skipped entirely. Change-Id: I04fc3d27c99f95b33f86f012a8d398f817f7713a --- .../android/server/display/AutomaticBrightnessController.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/services/core/java/com/android/server/display/AutomaticBrightnessController.java b/services/core/java/com/android/server/display/AutomaticBrightnessController.java index 65e971cf370d7..66b5e37729740 100644 --- a/services/core/java/com/android/server/display/AutomaticBrightnessController.java +++ b/services/core/java/com/android/server/display/AutomaticBrightnessController.java @@ -275,9 +275,9 @@ class AutomaticBrightnessController { boolean enableSensor = enable && !dozing; if (enableSensor && !mLightSensorEnabled && mActiveDozeLightSensor) { mActiveDozeLightSensor = false; - } else if (!enableSensor && mLightSensorEnabled) { + } else if (!enableSensor && mLightSensorEnabled && mLuxLevels.hasDynamicDozeBrightness()) { // keep the light sensor active until another light sample is taken while dozing - mActiveDozeLightSensor = mLuxLevels.hasDynamicDozeBrightness(); + mActiveDozeLightSensor = true; adjustLightSensorRate(mInitialLightSensorRate); if (mUseNewSensorSamplesForDoze) { mAmbientLightRingBuffer.clear();