From 768caced5466e75d09235c29e1973f3f8721bebf Mon Sep 17 00:00:00 2001 From: pyr0ball Date: Sat, 11 Jul 2026 23:00:04 -0700 Subject: [PATCH] docs: disambiguate 'Phase 2' terminology now that LVGL work claimed that name --- docs/build-guide-phase1-notes.md | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/docs/build-guide-phase1-notes.md b/docs/build-guide-phase1-notes.md index 95f881f..d3e34c1 100644 --- a/docs/build-guide-phase1-notes.md +++ b/docs/build-guide-phase1-notes.md @@ -15,7 +15,7 @@ Firmware flashed and running on physical hardware. Boot log confirms: - [ ] Relay bench test with multimeter — user doesn't have relays on hand, will test at office on a breadboard. GPIO wiring assumed correct for now based on clean compile + boot; not electrically verified. - [ ] Full Home Assistant pairing — `ha_api_key` in `secrets.yaml` is currently a placeholder (valid format, not a real key). Touch, sensor, switch, and climate entities not yet confirmed visible/controllable in HA itself. -Proceeding to Phase 2 (LVGL interface design) with the above two items still open — revisit before calling Phase 1 fully signed off. +Proceeding to Phase 2 (LVGL interface design, see `circuitforge-plans/firecrest/superpowers/plans/2026-07-11-phase2-lvgl-ui.md`) with the above two items still open — revisit before calling Phase 1 fully signed off. Phase 2's own implementation is now complete (all 5 tasks reviewed clean, ready to merge) but not yet flashed/verified on hardware — that verification and the two deferred items above are the same outstanding hardware-access gate. Source: Waveshare Wiki, fetched 2026-07-11. - https://www.waveshare.com/wiki/ESP32-S3-Touch-LCD-7B (confirmed this is the "Type B" product page — distinct from the non-B `ESP32-S3-Touch-LCD-7` page, same pinout on both as of this writing) @@ -104,7 +104,7 @@ Options to get 5 relay-driving outputs, in order of how much they cost you: 2. **Also repurpose the TF card SPI pins (GPIO11, 12, 13)** if the thermostat doesn't need an SD card — gets you to 7 free pins, plenty for 5 relays plus headroom. 3. **Add an external I2C GPIO expander** (e.g., PCF8574) on the shared I2C bus (GPIO8/9) instead of touching onboard peripheral pins — keeps RS485/CAN/TF card available for future use, at the cost of one more part. -**Decision (2026-07-11):** Phase 1 proof-of-concept repurposes the TF/SD card SPI pins (GPIO11, 12, 13) and the RS485 pins (GPIO15, 16) for the 5 relay outputs. This is a stopgap — CAN/native-USB (GPIO19, 20) are left untouched for possible future use. **SD card and RS485 are unavailable on this build.** Phase 2 should replace this with an external I2C GPIO expander (e.g. PCF8574) on the shared bus, or a second ESP acting as an I2C/relay satellite, to restore SD card logging without sacrificing GPIO. Tracked as a Phase 2 TODO. +**Decision (2026-07-11):** Phase 1 proof-of-concept repurposes the TF/SD card SPI pins (GPIO11, 12, 13) and the RS485 pins (GPIO15, 16) for the 5 relay outputs. This is a stopgap — CAN/native-USB (GPIO19, 20) are left untouched for possible future use. **SD card and RS485 are unavailable on this build.** A future phase should replace this with an external I2C GPIO expander (e.g. PCF8574) on the shared bus, or a second ESP acting as an I2C/relay satellite, to restore SD card logging without sacrificing GPIO. (Note: "Phase 2" elsewhere in this repo now refers to the LVGL touchscreen UI work — this relay/SD-card item is tracked as a Phase 3+ TODO to avoid confusion, see `circuitforge-plans/firecrest/superpowers/plans/2026-07-11-phase2-lvgl-ui.md` Notes for Phase 3.) ### Relay Outputs | Relay | HVAC Terminal | GPIO | Repurposed from | @@ -116,6 +116,6 @@ Options to get 5 relay-driving outputs, in order of how much they cost you: | 5 | W2 (heat 2) | GPIO16 | RS485 RXD | ## Open Questions -- [x] Relay GPIO source — resolved above (TF + RS485 pins for Phase 1; I2C expander or second ESP planned for Phase 2 to restore SD card) +- [x] Relay GPIO source — resolved above (TF + RS485 pins for Phase 1; I2C expander or second ESP planned for a future phase to restore SD card) - [ ] Confirm exact RGB timing porch/pulse values against schematic or ST7701 init sequence — plan's placeholder timing values are typical-for-1024x600 but unverified against this specific panel -- [ ] Confirm backlight PWM dimming path (Phase 2) — currently backlight is on/off only via CH422G EXIO2, no PWM signal identified for brightness control +- [ ] Confirm backlight PWM dimming path (tracked in Phase 2 plan's Notes for Phase 3) — currently backlight is on/off only via CH422G EXIO2, no PWM signal identified for brightness control