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

[Bug]: eink updates causing packet loss #2356

Closed
andrekir opened this issue Mar 13, 2023 · 4 comments
Closed

[Bug]: eink updates causing packet loss #2356

andrekir opened this issue Mar 13, 2023 · 4 comments
Labels
bug Something isn't working high-priority Issues that affect core functionality or are "show stoppers"

Comments

@andrekir
Copy link
Member

Category

Other

Hardware

Other

Firmware Version

2.1.0+

Description

every packet transmit and receive triggers the e-ink screen to redraw, with the log message "Updating E-Paper..." and looks like nRF52s can't handle well both at the same time.

@caveman99 suggested moving e-ink updates to their own RTOS task.

Relevant log output

No response

@andrekir andrekir added the bug Something isn't working label Mar 13, 2023
@nickpascucci
Copy link

Following on from discussion here: https://discord.com/channels/867578229534359593/871553168369148024/1143163122396450928

I flashed my T-Echo with a build at b47c9c1 with a change on line 35 of src/graphics/EInkDisplay2.h:

bool forceDisplay(uint32_t msecLimit = 100000);

This prevented the display from updating as frequently, and I was able to send 26 messages (A-Z) to a nearby T-Beam with no missed acknowledgments. Previously I would have more misses than acks. This seems to strongly point to the e-ink updates being an issue in a hot loop.

@GUVWAF GUVWAF added the high-priority Issues that affect core functionality or are "show stoppers" label Aug 29, 2023
@GUVWAF
Copy link
Member

GUVWAF commented Aug 29, 2023

Added the high-priority label as it seems to cause ACK failures quite easily, because transmitting triggers the touch button (on a T-Echo) causing multiple e-ink updates.
Conclusion of a discussion on Discord was that the missed packets are caused by the screen update blocking ISR access. Therefore, that needs to happen in a dedicated RTOS task.

@marek22k
Copy link

marek22k commented Dec 6, 2023

Is there a solution or a workaround without having to compile the firmware yourself?

@GUVWAF
Copy link
Member

GUVWAF commented Feb 10, 2024

I think for the T-Echo this is not an issue anymore now that it has partial updates which also causes it not to trigger the touch button anymore.

I can confirm it's still an issue with the Heltec Wireless Paper which only has full updates.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working high-priority Issues that affect core functionality or are "show stoppers"
Projects
None yet
Development

No branches or pull requests

5 participants