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

DIY external power pin #2398

Merged
merged 4 commits into from
Apr 2, 2023
Merged

Conversation

osmanovv
Copy link
Contributor

This fixes various issues such as unexpected node sleep, WiFi disconnection.

On some boards (DIY boards with ESP32 DevKit and LILYGO® TTGO T-Energy T18) we don't have a power management chip (like AXPxxxx) so we use GPIO pin EXT_PWR_DETECT to detect the external power supply.

To detect an external power supply, we need to pull up the EXT_PWR_DETECT pin on the board.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 31, 2023

🤖 Pull request artifacts

file commit
pr2398-firmware-2.1.7.a4ee6a7.zip a4ee6a7

thebentern added a commit to meshtastic/artifacts that referenced this pull request Mar 31, 2023
thebentern added a commit to meshtastic/artifacts that referenced this pull request Mar 31, 2023
thebentern added a commit to meshtastic/artifacts that referenced this pull request Apr 2, 2023
@caveman99 caveman99 merged commit 1398611 into meshtastic:master Apr 2, 2023
@osmanovv osmanovv deleted the diy-external-power-pin branch April 2, 2023 09:45
@osmanovv
Copy link
Contributor Author

osmanovv commented Apr 2, 2023

@caveman99, thank you for fixing formatting. I build on Windows machine where trunk extension is not available. Is there other way to check code style without this Trunk extension on Windows?

@caveman99
Copy link
Member

@osmanovv i use windows too, i have a WSL2 window running ubuntu and use trunk in there. the windows filesystems are available inside WSL under /mnt/

@osmanovv
Copy link
Contributor Author

osmanovv commented Apr 2, 2023

Thank you! I have just managed how to run it in WSL:

image

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