Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 2242f723 authored by Ady Abraham's avatar Ady Abraham
Browse files

SurfaceFlinger: Dispsync changePhaseOffset 

When changing the phase offset of DispSync there is no need
to adjust the last event time beyond the diff between the old
offset and the new one.

Test: default <-> early offset transition
Test: default <-> earlyGL offset transition
Bug: 135215384
Change-Id: I436c63914e53ddb53963c590493ee160efb62db8
parent 45e4e363

services/surfaceflinger/Scheduler/DispSync.cpp

+0 −11
Original line number Original line Diff line number Diff line
@@ -288,17 +288,6 @@ public: 
                // new offset to allow for a seamless offset change without double-firing or

                // new offset to allow for a seamless offset change without double-firing or

                // skipping.

                // skipping.

                nsecs_t diff = oldPhase - phase;

                nsecs_t diff = oldPhase - phase;

                if (diff > mPeriod / 2) {

                    diff -= mPeriod;

                } else if (diff < -mPeriod / 2) {

                    diff += mPeriod;

                }



                if (phase < 0 && oldPhase > 0) {

                    diff += mPeriod;

                } else if (phase > 0 && oldPhase < 0) {

                    diff -= mPeriod;

                }

                eventListener.mLastEventTime -= diff;

                eventListener.mLastEventTime -= diff;

                eventListener.mLastCallbackTime -= diff;

                eventListener.mLastCallbackTime -= diff;

                mCond.signal();

                mCond.signal();