Skip to content

bug(v16/pl18/el18): Slow refreshing of screen for all non 180 deg rotated display #6346

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

Open
1 task done
helloradiosky opened this issue Jun 6, 2025 · 4 comments · May be fixed by #6349
Open
1 task done

bug(v16/pl18/el18): Slow refreshing of screen for all non 180 deg rotated display #6346

helloradiosky opened this issue Jun 6, 2025 · 4 comments · May be fixed by #6349
Labels
bug/regression ↩️ A new version of EdgeTX broke something bug 🪲 Something isn't working
Milestone

Comments

@helloradiosky
Copy link
Contributor

helloradiosky commented Jun 6, 2025

Is there an existing issue for this problem?

  • I have searched the existing issues

What part of EdgeTX is the focus of this bug?

Transmitter firmware

Current Behavior

The widget channel information on the main page of the startup screen is slow to refresh. This problem only appeared in 2.11.1. It is normal in 2.11.0. There is no problem with the 2.11.1 T16 display refresh. The difference between the two is that the display data of T16 needs to be rotated 180 degrees, while the display data of V16 is displayed directly. In theory, V16 should be faster.

position:colorlcd lcd.cpp->126

#if !defined(LCD_VERTICAL_INVERT)
  disp_drv.full_refresh = 1;     //V16 
  disp_drv.direct_mode = 1;
#elif defined(RADIO_F16)
  disp_drv.full_refresh = 1;
  disp_drv.direct_mode = (hardwareOptions.pcbrev > 0) ? 1 : 0;
#else
  disp_drv.full_refresh = 0;  //T16
  disp_drv.direct_mode = 0;
#endif

Expected Behavior

Thank you very much for checking and fixing this problem.

Steps To Reproduce

Set up the channel information widget for displaying the homepage

Version

2.11.1

Transmitter

HelloRadioSky V16

Operating System (OS)

No response

OS Version

No response

Anything else?

No response

@helloradiosky helloradiosky added bug 🪲 Something isn't working triage Bug report awaiting review / sorting labels Jun 6, 2025
@richardclli richardclli changed the title HelloradioSky V16 2.11.1 bug(v16/pl18/el18): Slow refreshing of screen for all non 180 deg rotated display Jun 7, 2025
@richardclli richardclli removed the triage Bug report awaiting review / sorting label Jun 7, 2025
@richardclli
Copy link
Collaborator

richardclli commented Jun 7, 2025

Seems it is related to lvgl 8.4 update. Before lvgl 8.4 update, it works properly without full_refresh, and after lvgl 8.4, it is forced to do full_refresh in double buffer. i.e. more CPU overhead than inverted display??!! Not making any sense!

It affects all non-inverted display radios, including:
PL18, PL18EV, PL18U, EL18, NV14, T18, etc.

@richardclli richardclli added this to the 2.11.2 milestone Jun 7, 2025
@richardclli richardclli added the bug/regression ↩️ A new version of EdgeTX broke something label Jun 7, 2025
@philmoz
Copy link
Collaborator

philmoz commented Jun 7, 2025

Please try PR #6349 and report any issues.

@helloradiosky
Copy link
Contributor Author

Please try PR #6349 and report any issues.

I have tested the new PR and everything is working fine now, thank you very much!!!

@helloradiosky
Copy link
Contributor Author

Please try PR #6349 and report any issues.

Sorry, I forgot to describe it clearly, I only verified V16

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug/regression ↩️ A new version of EdgeTX broke something bug 🪲 Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants