Silence H752 build warnings#18
Merged
Merged
Conversation
Adds the pre-"H752-01" revision of the LilyGO T5 E-Paper S3 Pro (v1.0-240810, 4.7" ED047TC1, 960x540). It has no PCA9535/TPS65185, so it can't use epdiy's epd_board_v7 path and instead drives the panel with the vendored FastEPD library. All of it is behind a new model + display-backend Kconfig (DRAFTLING_MODEL_LILYGO_T5_EPD_S3_PRO_H752 / DRAFTLING_DISPLAY_H752_EPD), selected only by sdkconfig.h752, so the existing boards are unaffected. Hardware-verified on the board. - FastEPD display backend (components/fastepd vendored + display_h752.cpp): full 960x540 1-bit plane, partialUpdate for flicker-free local refresh on typing/deletes, full refresh on demand / every N partials. Includes a fix for a FastEPD bbepPartialUpdate DMA ping-pong bug (clear BOTH neutral row halves) that otherwise leaves full-height vertical streaks under edited text. - Touch: GT911 calibrated for this panel (reports 540x960 portrait; swap_xy + mirror_x). Front capacitive touch-key wired via a new touchscreen button callback to act as Back/Esc. Coexists with the existing GT911 address-drift recovery. - Power/keys: GPIO48 side key = Menu (injects F1). GPIO48 is not an RTC IO on the ESP32-S3, so standby uses light-sleep + gpio_wakeup + esp_restart instead of EXT0 deep sleep. - Backlight: H752 added to the <=5% extra-dim front-light option. - Battery indicator: on H752 only, drop the periodic 5 s battery-label timer (each rewrite drove a FastEPD refresh -> visible ghosting) in favour of a pull model that refreshes the label at natural redraw points and only when the value changed. Gated on the H752 model; every other board keeps the original timer unchanged. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Copilot created this pull request from a session on behalf of
clackups
June 11, 2026 18:33
View session
…ing-field-initializers
Copilot
AI
changed the title
Refactor PR #16 (H752 board support) per review
Silence H752 build warnings
Jun 11, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
H752 builds emitted three
-Wwarnings: twounused-functionfort5_isolate_unused_gpios/t5_release_held_gpios_after_wakeinmain.cpp, and onemissing-field-initializersfrom the vendoredFastEPD.inl.Changes
main/main.cpp-- the two T5 GPIO helpers live inside the widerPRO || PRO_H752guard but are only invoked from#if ...PROsections (the Pro-only deep-sleep teardown and wake-up restore). Wrap both definitions in a narrower#if defined(CONFIG_DRAFTLING_MODEL_LILYGO_T5_EPD_S3_PRO)so they are not compiled on H752.components/fastepd/CMakeLists.txt--FastEPD.inlis vendored third-party code that leaves neweresp_lcd_i80_bus_config_tfields (e.g.sram_trans_align) default-initialized. Addtarget_compile_options(... PRIVATE -Wno-missing-field-initializers), gated onCONFIG_DRAFTLING_DISPLAY_H752_EPDso the suppression stays scoped to this component and does not mask the warning elsewhere.