[PATCH v4 3/3] drm/panel-fannal-c3004: Add fannal c3004 DSI panel

Linus Walleij linus.walleij at linaro.org
Thu Jun 22 08:22:42 UTC 2023


On Wed, Jun 21, 2023 at 5:09 PM Paulo Pavacic <pavacic.p at gmail.com> wrote:

> A lot of modifications to st7701 are required. I believe it would
> result in a driver that doesn't look or work the same. e.g compare
> delays between initialization sequences of panel-fannal-c3004 and
> panel-st7701. I think it would be optimal to create st7701s driver and
> have special handling for st7701s panels. If there was a flag for
> whether panel is st7701 or st7701s it would end up looking like a
> mess.

What matters is if the original authors of the old st7701 driver are
around and reviewing and testing patches at all. What we need is
active maintainers. (Added Jagan, Marek & Maya).

I buy the reasoning that the st7701s is perhaps substantially different
from st7701.

If st7701s is very different then I suppose it needs a separate driver,
then all we need to to name the driver properly, i.e.
panel-sitronix-st7701s.c.

Yours,
Linus Walleij


More information about the dri-devel mailing list