Feature: Add TFT and E-Ink Displays #824
Conversation
…nd switch to a default type to allow general api access rather than specific driver access. similar to v2
…t miss ping and dc
| WS.feedWDT(); | ||
|
|
||
| // Process display controller events, if initialized | ||
| WS._displayController->update(getRSSI(), WS._mqtt->connected()); |
There was a problem hiding this comment.
Mqtt->connected didn't reliably return false when exploring previous issues, it might be worth rethinking or ANDing with wifi status connected.
There was a problem hiding this comment.
Do we have a reliable way to check it?
There was a problem hiding this comment.
Not sure that we do, and we call both runNetFSM and ping earlier in this method so we should be as correct in status as possible (mqtt-connected should be correct, IIRC it resets on failed ping / forced reconnect).
There was a problem hiding this comment.
That's my thinking, if it failed, NetFSM would have picked it up so by the time it gets to this call it would return false.
|
Looks good @brentru, just some minor things. |
This pull request adds support for TFT and E-Ink Displays to the WipperSnapper firmware.
Major Changes:
displaycomponent to WipperSnapperdisplay/controller- Handles routing and management of Protobuf messages between App and Hardware instancesdisplay/hardware- Handles multiple types of display driversdisplay/drivers/display/drivers/dispDrvBase.h- Abstract base class for display drivers, provides a common interface for all display drivers,display/drivers/dispDrvSt7789.h- Driver for ST7789-based TFT displays, such as Adafruit FunHouse and Adafruit Feather ESP32x TFTdisplay/drivers/dispDrvThinkInkGrayscale4Eaamfgn.h- Driver for ThinkInk 2.9" Grayscale 4-level EAAMFGN display (present on the 2025 version of the Adafruit MagTag)display/drivers/dispDrvThinkInkGrayscale4T5.h- Driver for ThinkInk 2.9" Grayscale 4-level T5 display (present on the pre-2025 version of the Adafruit MagTag)Resolves:
#792