Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Auto brightness fix for oneplus 7 series #694

Closed
wants to merge 2 commits into from
Closed

Auto brightness fix for oneplus 7 series #694

wants to merge 2 commits into from

Conversation

mukesh22584
Copy link

No description provided.

ritujb and others added 2 commits July 22, 2021 01:17
AOSPA:
* Bugfix for when mProperty.setValue() is not called due to the animation amount being smaller than EPSILON (BrightnessSynchronizer@L52)

Change-Id: I99ca859e5c021992bfbeece3bfe011df4e87be36
Signed-off-by: Mukesh Singh <mks22584@gmail.com>
The brightness slider is animated at a fixed max duration of
3000ms. If the brightness ramp rate is reduced to a value that is
much lower than the default rate, the brightness slider animation
can be considerably out of sync with the screen brightness
transitions.

Override the slider animator duration scale by the ratio of the
default ramp rate to the actual rate in use. The maximum scale is
clamped to 3x. Note that the purpose is not to make the slider
perfectly in sync with the screen, as the original design was
not like that either.

Signed-off-by: Chenyang Zhong <zhongcy95@gmail.com>
Change-Id: I980a8976d1d8e6fe9c295be9a06ee305a66c9448
Signed-off-by: Mukesh Singh <mks22584@gmail.com>
Copy link
Member

@neobuddy89 neobuddy89 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

config_brightnessRampRateSlowFloat not defined.

@mukesh22584
Copy link
Author

mukesh22584 commented Jul 22, 2021

config_brightnessRampRateSlowFloat not defined.

Isn't it device specific overlay ?

@neobuddy89
Copy link
Member

How do you overlay it if it's not present in source?

Moreover, the code compilation will fail since you are looking for the undefined symbol in source whether or not defined in device tree .

@neobuddy89 neobuddy89 closed this Jul 22, 2021
@mukesh22584
Copy link
Author

How do you overlay it if it's not present in source?

Moreover, the code compilation will fail since you are looking for the undefined symbol in source whether or not defined in device tree .

Ahh ! My bad, I've missed commits.

@neobuddy89
Copy link
Member

Added.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants