Skip to content

diy: waveshare lcd touch 2 improvements#230

Closed
trevarj wants to merge 4 commits into
Blockstream:masterfrom
trevarj:lcdtouch2-improvements
Closed

diy: waveshare lcd touch 2 improvements#230
trevarj wants to merge 4 commits into
Blockstream:masterfrom
trevarj:lcdtouch2-improvements

Conversation

@trevarj
Copy link
Copy Markdown
Contributor

@trevarj trevarj commented May 9, 2025

Got pretty excited that there is a confirmed one whole extra person using this board besides me, so I finally put my head down and finished a few things that were bothering me and improving the device's usability.

Battery and backlight are taken from Waveshare's examples and adjusted slightly.

  • Added battery voltage and status
  • Added USB connectivity check (guess from ADC voltage)
  • Added sleep/wake using BOOT button
  • Enabled display brightness setting

- Added battery voltage and status
- Added USB connectivity check (guess from ADC voltage)
- Added sleep/wake using BOOT button
- Enabled display brightness setting
@JamieDriver
Copy link
Copy Markdown
Collaborator

Cheers - I'll take a look next week. 👍

@harinworks
Copy link
Copy Markdown
Contributor

Many thanks to @trevarj for the initial contribution.
I've built upon that work and submitted a follow-up PR #232 to further improve the code.
Please take a look when you have time!

@trevarj
Copy link
Copy Markdown
Contributor Author

trevarj commented May 22, 2025

@harinworks Thanks for the fixes! Would you mind putting that PR up against my repo instead? Then I can merge it there and it will show up on this PR.

@harinworks
Copy link
Copy Markdown
Contributor

@trevarj Done! Please check your repo :)

Refactor wslcdtouch2.inc for better code clarity
@JamieDriver
Copy link
Copy Markdown
Collaborator

Sorry for the delay on my part - I've run the latest through our internal ci and all seems good.
Are you guys happy you've done all you want to in this area for now, and it's good to merge to master ?

@trevarj
Copy link
Copy Markdown
Contributor Author

trevarj commented May 27, 2025

@JamieDriver all good on my end. Been testing and everything seems to be working fine. Thanks a lot!

@JamieDriver
Copy link
Copy Markdown
Collaborator

Rebased and merged to master at a5c9eab.

Many thanks!

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.

3 participants