Skip to content

examples/lvgldemo: Add optional touchscreen diagnostics. - #3659

Open
aviralgarg05 wants to merge 1 commit into
apache:masterfrom
aviralgarg05:gsoc/lvgldemo-touch-diag-pr9
Open

examples/lvgldemo: Add optional touchscreen diagnostics.#3659
aviralgarg05 wants to merge 1 commit into
apache:masterfrom
aviralgarg05:gsoc/lvgldemo-touch-diag-pr9

Conversation

@aviralgarg05

@aviralgarg05 aviralgarg05 commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Note: Please adhere to Contributing Guidelines.

Summary

Add an opt-in LVGL touchscreen diagnostic screen and make the framebuffer
device path configurable for framebuffer-backed targets.

The default demo selection and LCD path are unchanged. When
CONFIG_EXAMPLES_LVGLDEMO_TOUCH_DIAGNOSTIC is enabled, the screen reports
whether the input device opened and displays live press state and
coordinates. This separates driver-availability problems from coordinate
mapping problems without requiring serial-only diagnostics.

Impact

  • New feature: YES, opt-in diagnostic mode.
  • User adaptation: NO; existing configurations retain current behavior.
  • Build: YES only when the new option is selected.
  • Hardware: NO driver change.
  • Documentation: YES; Kconfig help documents the options.
  • Security: NO new security boundary or privilege change.
  • Compatibility: Existing LCD and demo paths remain unchanged.
  • Other: NO.

Testing

  • Build host: macOS 26.5, arm64, GCC 14.2.0.
  • Target: Xtensa ESP32-S3, Waveshare ESP32-S3-Touch-LCD-7.
  • git diff --check
  • NuttX tools/checkpatch.sh
  • ESP32-S3 framebuffer/touchscreen build
  • Physical touchscreen press/coordinate observation on /dev/input0
  • Partial LVGL initialization is released on display-open failure

Testing logs before change:

The base configuration has no opt-in touchscreen diagnostic screen.

Testing logs after change:

/dev/fb0 and /dev/input0 opened on the ESP32-S3 target.
Live touch press state and coordinates were observed on the display.

PR verification Self-Check

  • This PR contains one focused diagnostic change.
  • I have updated all required description fields above.
  • I have reviewed and signed the commit.
  • The branch passes local style checks.
  • This PR adheres to the current contribution and coding guidelines.
  • My PR is still work in progress.
  • My PR is ready for review and can be safely merged.

Rebased onto current master (2aebae740); the diff is limited to
examples/lvgldemo/.

Make the framebuffer device path configurable for framebuffer-backed LVGL targets. Keep the existing LCD path and demo selection behavior unchanged by default.

Add an opt-in touchscreen diagnostic screen that reports device-open failure, press state, and coordinates. This helps distinguish driver availability from coordinate-mapping problems without making the hardware-specific diagnostic replace the general LVGL demo.

Assisted-by: OpenAI Codex:gpt-5.6-sol
Signed-off-by: aviralgarg05 <gargaviral99@gmail.com>
@aviralgarg05
aviralgarg05 force-pushed the gsoc/lvgldemo-touch-diag-pr9 branch from 326f094 to debc05b Compare August 6, 2026 16:35
@aviralgarg05
aviralgarg05 marked this pull request as ready for review August 6, 2026 16:37
@aviralgarg05 aviralgarg05 changed the title [WIP] examples/lvgldemo: Add optional touchscreen diagnostics. examples/lvgldemo: Add optional touchscreen diagnostics. Aug 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants