On Mon, Feb 16, 2026 at 4:15 AM David Lechner <[email protected]> wrote:
Hi David, > > On 2/14/26 11:21 PM, Archit Anant wrote: > > Hi David, > > > >> Do we actually need a custom driver for these or can we use > >> drivers/gpu/drm/tiny/panel-mipi-dbi.c with a firmware file for > >> init? > > > > That was my initial thought as well, but looking at the staging driver > > (fb_st7789v.c), there is conditional logic in the initialization sequence. > > > > The HSD20_IPS panel requires different PORCTRL (0xB2), GCTRL (0xB7), and > > Gamma (0xE0/E1) settings compared to the standard panel. A static > > firmware blob via panel-mipi-dbi would not allow selecting between > > these configurations based on the compatible string, so a dedicated > > driver seemed necessary to preserve this support. > > > > Best regards, > > Archit > > If two different panels are not 100% compatible with each other, they > should not have the same compatible string. > > It seems like it would be much more efficient to fix the existing > devicetree/driver if needed rather than writing an entire new driver. Agreed. ChenYu Tsai mentioned he is working on refactoring the existing panel-sitronix-st7789v driver to use mipi_dbi. Once that is ready, I can add the HSD20-IPS panel support to that driver using a distinct compatible string and associated configuration data, rather than forking a new driver. I will drop this standalone driver patch and wait for Chen-Yu's series. -- Sincerely, Archit Anant
