Re: [PATCH v2 3/3] drm/panel: Add ilitek ili9341 panel driver

From: Noralf Trønnes
Date: Wed Jul 21 2021 - 13:42:53 EST




Den 21.07.2021 09.41, skrev dillon.minfei@xxxxxxxxx:
> From: Dillon Min <dillon.minfei@xxxxxxxxx>
>
> This driver combine tiny/ili9341.c mipi_dbi_interface driver
> with mipi_dpi_interface driver, can support ili9341 with serial
> mode or parallel rgb interface mode by register configuration.
>
> Cc: Linus Walleij <linus.walleij@xxxxxxxxxx>
> Signed-off-by: Dillon Min <dillon.minfei@xxxxxxxxx>
> ---

> +static const struct of_device_id ili9341_of_match[] = {
> + {
> + .compatible = "st,sf-tc240t-9370-t",
> + .data = &ili9341_stm32f429_disco_data,
> + },
> + {
> + /* porting from tiny/ili9341.c
> + * for original mipi dbi compitable
> + */
> + .compatible = "adafruit,yx240qv29",

I don't understand this, now there will be 2 drivers that support the
same display?

AFAICT drm/tiny/ili9341.c is just copied into this driver, is the plan
to remove the tiny/ driver? If so I couldn't see this mentioned anywhere.

Noralf.

> + .data = NULL,
> + },
> +};
> +MODULE_DEVICE_TABLE(of, ili9341_of_match);