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

fix fading issues #4492

Merged
merged 1 commit into from
Jan 23, 2025
Merged

fix fading issues #4492

merged 1 commit into from
Jan 23, 2025

Conversation

TychoVrahe
Copy link
Contributor

@TychoVrahe TychoVrahe commented Jan 12, 2025

fixes #4392 fixes #4465

The slow fade in bootloader was due to function raising backlight was taking too long, waiting for display TE signal each time. Waiting for TE is removed as its no longer needed due to different mechanism using pending update - so now we will normally wait only once, after the display content is changed. In addition, the backlight fading function is rewritten to ensure constant fading time, at the possible expense of smoothness.

To mitigate the residual image issue, additional wait cycle is added when raising backlight.

As part of this fix i did i minor refactor, fading from python is now done by calling rust functions. The behavior of the fade is unified, so fading from python will behave a bit differently

Additional problem with fading is done before display refresh in progress layouts is fixed.

@TychoVrahe TychoVrahe self-assigned this Jan 12, 2025
Copy link

github-actions bot commented Jan 12, 2025

core UI changes device test click test persistence test
T2T1 Model T test(screens) main(screens) test(screens) main(screens) test(screens) main(screens)
T3B1 Safe 3 test(screens) main(screens) test(screens) main(screens) test(screens) main(screens)
T3T1 Safe 5 test(screens) main(screens) test(screens) main(screens) test(screens) main(screens)
All main(screens)

@TychoVrahe TychoVrahe force-pushed the tychovrahe/bootloader/slow_fade_fix branch 3 times, most recently from 500ecf5 to 525a891 Compare January 13, 2025 08:32
@TychoVrahe TychoVrahe marked this pull request as ready for review January 13, 2025 08:48
@TychoVrahe TychoVrahe requested review from cepetr and removed request for prusnak January 13, 2025 08:48
core/src/apps/base.py Outdated Show resolved Hide resolved
core/src/boot.py Outdated Show resolved Hide resolved
core/tests/test_trezor.ui.display.py Show resolved Hide resolved
core/embed/rust/src/ui/display/mod.rs Outdated Show resolved Hide resolved
Copy link
Contributor

@cepetr cepetr left a comment

Choose a reason for hiding this comment

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

It looks like the problems have been solved. I just added a comment suggesting a minor improvement.

core/embed/rust/src/ui/display/mod.rs Outdated Show resolved Hide resolved
@TychoVrahe TychoVrahe force-pushed the tychovrahe/bootloader/slow_fade_fix branch from 1c7bde1 to a4a5f4c Compare January 23, 2025 13:11
@TychoVrahe TychoVrahe force-pushed the tychovrahe/bootloader/slow_fade_fix branch from a4a5f4c to 5f79785 Compare January 23, 2025 13:19
@TychoVrahe TychoVrahe merged commit 5fd7737 into main Jan 23, 2025
94 checks passed
@TychoVrahe TychoVrahe deleted the tychovrahe/bootloader/slow_fade_fix branch January 23, 2025 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 🤝 Needs QA
Development

Successfully merging this pull request may close these issues.

Display flashes old content before refreshing T3T1 - slow screen fading in bootloader
3 participants