From c9b3e21137b9fd447a7e9676e340758bd6b7ee25 Mon Sep 17 00:00:00 2001
From: jeffreyjacques <44651844+jeffreyjacques@users.noreply.github.com>
Date: Sat, 5 Sep 2026 09:58:47 -0400
Subject: [PATCH] rev C: honor a STARTMODE config option for power-on display

The revision C protocol already defines STARTMODE_DEFAULT/IMAGE/VIDEO
(byte 0 of the OPTIONS payload), but SetOrientation always sent DEFAULT,
which blanks the panel at power-on until the host connects. The panel
keeps this setting across power cycles.

Read an optional display.STARTMODE config value (DEFAULT/IMAGE/VIDEO) so
a display fitted with an SD card can instead replay the last image or
video it was sent. Defaults to DEFAULT, so behavior is unchanged unless
the option is set; unknown values warn and fall back to DEFAULT.

config.yaml is intentionally not modified (guarded by CI); maintainers
can add the documented 'STARTMODE: DEFAULT' key to the shipped template.
---
 library/lcd/lcd_comm_rev_c.py | 25 +++++++++++++++++++++++--
 1 file changed, 23 insertions(+), 2 deletions(-)

diff --git a/library/lcd/lcd_comm_rev_c.py b/library/lcd/lcd_comm_rev_c.py
index 5de42bfa0..41c150c20 100644
--- a/library/lcd/lcd_comm_rev_c.py
+++ b/library/lcd/lcd_comm_rev_c.py
@@ -31,6 +31,7 @@
 from PIL import Image
 from serial.tools.list_ports import comports
 
+import library.config as config
 from library.lcd.lcd_comm import Orientation, LcdComm
 from library.lcd.serialize import image_to_BGRA, image_to_BGR, chunked
 from library.log import logger
@@ -311,12 +312,32 @@ def SetOrientation(self, orientation: Orientation = Orientation.PORTRAIT):
         # logger.info(f"Call SetOrientation to: {self.orientation.name}")
 
         # if self.orientation == Orientation.REVERSE_LANDSCAPE or self.orientation == Orientation.REVERSE_PORTRAIT:
-        #   b = Command.STARTMODE_DEFAULT.value + Padding.NULL.value + Command.FLIP_180.value + SleepInterval.OFF.value
+        #   b = self._start_mode() + Padding.NULL.value + Command.FLIP_180.value + SleepInterval.OFF.value
         #   self._send_command(Command.OPTIONS, payload=b)
         # else:
-        b = Command.STARTMODE_DEFAULT.value + Padding.NULL.value + Command.NO_FLIP.value + SleepInterval.OFF.value
+        b = self._start_mode() + Padding.NULL.value + Command.NO_FLIP.value + SleepInterval.OFF.value
         self._send_command(Command.OPTIONS, payload=b)
 
+    @staticmethod
+    def _start_mode() -> bytearray:
+        # Byte 0 of the OPTIONS payload selects what the panel shows at power-on,
+        # a choice the firmware keeps across power cycles until it is changed
+        # again. This driver always sent DEFAULT, which blanks the panel until
+        # the host connects; the STARTMODE config option lets the panel instead
+        # replay the last image or video it was sent (useful with an SD card).
+        modes = {
+            "DEFAULT": Command.STARTMODE_DEFAULT,  # show nothing until the host connects
+            "IMAGE": Command.STARTMODE_IMAGE,      # replay the last image sent to the panel
+            "VIDEO": Command.STARTMODE_VIDEO,      # replay the last video sent to the panel
+        }
+        name = str(config.CONFIG_DATA["display"].get("STARTMODE", "DEFAULT")).upper()
+        if name not in modes:
+            logger.warning("Unknown STARTMODE '%s' in config.yaml, using DEFAULT" % name)
+            name = "DEFAULT"
+        if name != "DEFAULT":
+            logger.debug("Display start mode at power-on: %s" % name)
+        return modes[name].value
+
     def DisplayPILImage(
             self,
             image: Image.Image,