Re: [RFC PATCH] drm/sitronix: add ST7789V panel driver

[David Lechner]

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.