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

T-Deck fix: revert KB_POWERON changes #2847

Merged
merged 1 commit into from
Sep 30, 2023
Merged

T-Deck fix: revert KB_POWERON changes #2847

merged 1 commit into from
Sep 30, 2023

Conversation

mverch67
Copy link
Collaborator

The change in #2786 to disable the T-Deck's keyboard during screen blanking to save 20mAh power seems to have a negative side effect on the LoRa TX interrupt which leads to a reboot when sending mesh packets after the screen blanking.
This PR reverts the change in PowerFSM.cpp.

INFO  | ??:??:?? 201 [Screen] From Radio onread
INFO  | ??:??:?? 201 [Screen] getFromRadio=STATE_SEND_PACKETS
DEBUG | ??:??:?? 201 [Screen] encoding toPhone packet to phone variant=11, 12 bytes
INFO  | ??:??:?? 201 [Screen] From Radio onread
WARN  | ??:??:?? 201 [RadioIf] Can not send yet, busyTx
ERROR | ??:??:?? 201 [RadioIf] Hardware Failure! busyTx for more than 60s
ERROR | ??:??:?? 201 [RadioIf] NOTE! Recording critical error 8 at src/mesh/RadioLibInterface.cpp:88
INFO  | ??:??:?? 201 Rebooting

[20:46:07.013] Disconnected
[20:46:08.014] Warning: Could not open tty device (Permission denied)
[20:46:08.014] Waiting for tty device..
[20:46:09.016] Connected
��@INFO  | ??:??:?? 2 

//\ E S H T /\ S T / C

@github-actions
Copy link
Contributor

🤖 Pull request artifacts

file commit
pr2847-firmware-2.2.10.75e1e97.zip 75e1e97

thebentern added a commit to meshtastic/artifacts that referenced this pull request Sep 30, 2023
@thebentern thebentern merged commit e9215a5 into master Sep 30, 2023
@caveman99 caveman99 deleted the revert-kb-power branch November 26, 2023 10:13
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.

2 participants