feat: add madctl param for rotation and BGR/RGB

Author: ritesh006

25.2

@@ -64,6 +64,29 @@ Usage Example
     while True:
         pass
 
+Custom MADCTL Example
+---------------------
+
+You can now override the **MADCTL register** to apply hardware-level rotations or control BGR/RGB order directly.
+This can improve performance when using `OnDiskBitmap` or other rotation-sensitive cases.
+
+.. code-block:: python
+
+    import board
+    import displayio
+    import fourwire
+    import adafruit_ili9341
+
+    spi = board.SPI()
+    tft_cs = board.D9
+    tft_dc = board.D10
+
+    displayio.release_displays()
+    display_bus = fourwire.FourWire(spi, command=tft_dc, chip_select=tft_cs)
+
+    # Pass a custom MADCTL value (example: 0b01011000)
+    display = adafruit_ili9341.ILI9341(display_bus, width=320, height=240, madctl=0b01011000)
+
 Documentation
 =============
 

README.rst

@@ -92,15 +92,48 @@ class ILI9341(BusDisplay):
     ILI9341 display driver
 
     :param FourWire bus: bus that the display is connected to
+    :param bool bgr: Use BGR color order instead of RGB
+    :param bool invert: Invert the display
+    :param int madctl: Optional custom MADCTL value to override rotation/scan order bits
     """
 
-    def __init__(self, bus: FourWire, *, bgr: bool = False, invert: bool = False, **kwargs: Any):
-        init_sequence = _INIT_SEQUENCE
-        if bgr:
-            init_sequence += b"\x36\x01\x30"  # _MADCTL Default rotation plus BGR encoding
-        else:
-            init_sequence += b"\x36\x01\x38"  # _MADCTL Default rotation plus RGB encoding
+    def __init__(
+        self,
+        bus: FourWire,
+        *,
+        bgr: bool = False,
+        invert: bool = False,
+        madctl: int | None = None,
+        **kwargs: Any,
+    ):
+        # Start with base init sequence
+        init_sequence = bytearray(_INIT_SEQUENCE)
+
+        # --- Replace existing MADCTL instead of appending ---
+        # Find MADCTL (0x36) in init sequence
+        try:
+            idx = init_sequence.index(0x36)
+            # Overwrite the length (always 1) + value
+            if madctl is not None:
+                init_sequence[idx + 1] = 0x01
+                init_sequence[idx + 2] = madctl
+            elif bgr:
+                init_sequence[idx + 1] = 0x01
+                init_sequence[idx + 2] = 0x30  # BGR encoding
+            else:
+                init_sequence[idx + 1] = 0x01
+                init_sequence[idx + 2] = 0x38  # RGB encoding
+        except ValueError:
+            # If no MADCTL present, append it
+            if madctl is not None:
+                init_sequence += bytes([0x36, 0x01, madctl])
+            elif bgr:
+                init_sequence += b"\x36\x01\x30"
+            else:
+                init_sequence += b"\x36\x01\x38"
+
+        # Inversion
         if invert:
-            init_sequence += b"\x21\x00"  # _INVON
+            init_sequence += b"\x21\x00"
 
         super().__init__(bus, init_sequence, **kwargs)

adafruit_ili9341.py

@@ -46,3 +46,16 @@ py-modules = ["adafruit_ili9341"]
 [tool.setuptools.dynamic]
 dependencies = {file = ["requirements.txt"]}
 optional-dependencies = {optional = {file = ["optional_requirements.txt"]}}
+
+[tool.ruff]
+# Enable default linting rules
+select = ["E", "F"]
+
+# Ignore specific rules if needed
+ignore = []
+
+# Allow fixing issues automatically with `ruff check --fix`
+fix = true
+
+# Formatting (instead of Black)
+line-length = 88

pyproject.toml

@@ -0,0 +1,49 @@
+# SPDX-FileCopyrightText: 2025 Ritesh
+#
+# SPDX-License-Identifier: MIT
+
+import pytest
+
+from adafruit_ili9341 import ILI9341
+
+
+class DummyBus:
+    """A fake bus to avoid hardware dependency."""
+
+    def __init__(self):
+        self.commands = []
+
+    def send(self, *args, **kwargs):
+        self.commands.append((args, kwargs))
+
+
+def extract_madctl_bytes(init_seq: bytes) -> bytes | None:
+    """
+    Helper to find MADCTL command (0x36) in the init sequence.
+    Returns 2 bytes: [0x36, value] or None if not present.
+    """
+    for i in range(len(init_seq) - 2):
+        if init_seq[i] == 0x36:
+            return init_seq[i : i + 3]
+    return None
+
+
+def test_default_rgb_encoding():
+    bus = DummyBus()
+    disp = ILI9341(bus)
+    madctl = extract_madctl_bytes(disp.init_sequence)
+    assert madctl == b"\x36\x01\x38"  # Default RGB
+
+
+def test_bgr_encoding():
+    bus = DummyBus()
+    disp = ILI9341(bus, bgr=True)
+    madctl = extract_madctl_bytes(disp.init_sequence)
+    assert madctl == b"\x36\x01\x30"  # BGR mode
+
+
+def test_custom_madctl():
+    bus = DummyBus()
+    disp = ILI9341(bus, madctl=0b01011000)
+    madctl = extract_madctl_bytes(disp.init_sequence)
+    assert madctl == bytes([0x36, 0x01, 0b01011000])  # Custom override