Skip to content

Add ST7123 integrated touch support to M5Stack Tab5#616

Draft
Copilot wants to merge 7 commits into
mainfrom
copilot/fix-tab5-st7123-touch
Draft

Add ST7123 integrated touch support to M5Stack Tab5#616
Copilot wants to merge 7 commits into
mainfrom
copilot/fix-tab5-st7123-touch

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 23, 2026

  • Prior session: Created st7123touch component and added touch support to m5stack-tab5
  • Add C++23 TouchDriverConcept + type-erased ITouchDriver/TouchDriverAdapter/make_touch_driver to st7123touch.hpp
  • Replace two typed pointers in m5stack-tab5.hpp with one std::shared_ptr<espp::ITouchDriver>
  • Update touchpad.cpp to use concept-based single pointer (cleaner, no if/else branching)
  • Create st7123touch example (CMakeLists.txt, sdkconfig.defaults, Kconfig.projbuild, README.md, st7123touch_example.cpp)
  • Add example to build.yml (between st25dv and state_machine, target esp32s3)
  • Add example to Doxyfile EXAMPLE_PATH
  • Update doc/en/input/st7123touch.rst with toctree entry and concept description
  • Create doc/en/input/st7123touch_example.md

Copilot AI linked an issue May 23, 2026 that may be closed by this pull request
2 tasks
Agent-Logs-Url: https://github.com/esp-cpp/espp/sessions/3ad85902-651b-43d5-8f1d-d4c119604286

Co-authored-by: finger563 <213467+finger563@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix touch availability for m5stack tab5 ST7123 Add ST7123 integrated touch support to M5Stack Tab5 May 23, 2026
Copilot AI requested a review from finger563 May 23, 2026 03:50
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 23, 2026

✅Static analysis result - no issues found! ✅

…er interface

Agent-Logs-Url: https://github.com/esp-cpp/espp/sessions/7358df43-1e1a-4119-bdc3-37852bfec05e

Co-authored-by: finger563 <213467+finger563@users.noreply.github.com>
@finger563
Copy link
Copy Markdown
Contributor

@Shadowtrance LMK if you're able to test the code on this branch on your hardware :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

m5stack tab5 ST7123 touch

2 participants