Re: [PATCH v3 3/4] drm/panel: ili9882t: Break out as separate driver

From: cong yang
Date: Sun Aug 13 2023 - 22:06:25 EST


Hi,Linus

On Mon, Jul 3, 2023 at 9:21 PM Linus Walleij <linus.walleij@xxxxxxxxxx> wrote:
>
> The Starry ILI9882t-based panel should never have been part of the boe
> tv101wum driver, it is clearly based on the Ilitek ILI9882t display
> controller and if you look at the custom command sequences for the
> panel these clearly contain the signature Ilitek page switch (0xff)
> commands. The hardware has nothing in common with the other panels
> supported by this driver.
>
> Break this out into a separate driver and config symbol instead.
>
> If the placement here is out of convenience for using similar code,
> we should consider creating a helper library instead.
>
> Signed-off-by: Linus Walleij <linus.walleij@xxxxxxxxxx>
> ---
> drivers/gpu/drm/panel/Kconfig | 9 +
> drivers/gpu/drm/panel/Makefile | 1 +
> drivers/gpu/drm/panel/panel-boe-tv101wum-nl6.c | 386 -------------
> drivers/gpu/drm/panel/panel-ilitek-ili9882t.c | 739 +++++++++++++++++++++++++
> 4 files changed, 749 insertions(+), 386 deletions(-)
>
> diff --git a/drivers/gpu/drm/panel/Kconfig b/drivers/gpu/drm/panel/Kconfig
> index 1a0fd0754692..c39e949a26eb 100644
> --- a/drivers/gpu/drm/panel/Kconfig
> +++ b/drivers/gpu/drm/panel/Kconfig
> @@ -203,6 +203,15 @@ config DRM_PANEL_ILITEK_ILI9881C
> Say Y if you want to enable support for panels based on the
> Ilitek ILI9881c controller.
>
> +config DRM_PANEL_ILITEK_ILI9882T
> + tristate "Ilitek ILI9882t-based panels"
> + depends on OF
> + depends on DRM_MIPI_DSI
> + depends on BACKLIGHT_CLASS_DEVICE
> + help
> + Say Y if you want to enable support for panels based on the
> + Ilitek ILI9882t controller.
> +
> config DRM_PANEL_INNOLUX_EJ030NA
> tristate "Innolux EJ030NA 320x480 LCD panel"
> depends on OF && SPI
> diff --git a/drivers/gpu/drm/panel/Makefile b/drivers/gpu/drm/panel/Makefile
> index 499e38244253..75c2533d337e 100644
> --- a/drivers/gpu/drm/panel/Makefile
> +++ b/drivers/gpu/drm/panel/Makefile
> @@ -18,6 +18,7 @@ obj-$(CONFIG_DRM_PANEL_HIMAX_HX8394) += panel-himax-hx8394.o
> obj-$(CONFIG_DRM_PANEL_ILITEK_IL9322) += panel-ilitek-ili9322.o
> obj-$(CONFIG_DRM_PANEL_ILITEK_ILI9341) += panel-ilitek-ili9341.o
> obj-$(CONFIG_DRM_PANEL_ILITEK_ILI9881C) += panel-ilitek-ili9881c.o
> +obj-$(CONFIG_DRM_PANEL_ILITEK_ILI9882T) += panel-ilitek-ili9882t.o
> obj-$(CONFIG_DRM_PANEL_INNOLUX_EJ030NA) += panel-innolux-ej030na.o
> obj-$(CONFIG_DRM_PANEL_INNOLUX_P079ZCA) += panel-innolux-p079zca.o
> obj-$(CONFIG_DRM_PANEL_JADARD_JD9365DA_H3) += panel-jadard-jd9365da-h3.o
> diff --git a/drivers/gpu/drm/panel/panel-boe-tv101wum-nl6.c b/drivers/gpu/drm/panel/panel-boe-tv101wum-nl6.c
> index 358918e0f03f..14a0ee95a803 100644
> --- a/drivers/gpu/drm/panel/panel-boe-tv101wum-nl6.c
> +++ b/drivers/gpu/drm/panel/panel-boe-tv101wum-nl6.c
> @@ -1348,361 +1348,6 @@ static int starry_himax83102_j02_init(struct mipi_dsi_device *dsi)
> return 0;
> };
>
> -
> -static int starry_ili9882t_init(struct mipi_dsi_device *dsi)
> -{
> - int ret;
> -
> - msleep(5);
> -
> - mipi_dsi_dcs_write_seq(dsi, 0xFF, 0x98, 0x82, 0x01);
> - mipi_dsi_dcs_write_seq(dsi, 0x00, 0x42);
> - mipi_dsi_dcs_write_seq(dsi, 0x01, 0x11);
> - mipi_dsi_dcs_write_seq(dsi, 0x02, 0x00);
> - mipi_dsi_dcs_write_seq(dsi, 0x03, 0x00);
> -
> - mipi_dsi_dcs_write_seq(dsi, 0x04, 0x01);
> - mipi_dsi_dcs_write_seq(dsi, 0x05, 0x11);
> - mipi_dsi_dcs_write_seq(dsi, 0x06, 0x00);
> - mipi_dsi_dcs_write_seq(dsi, 0x07, 0x00);
> -
> - mipi_dsi_dcs_write_seq(dsi, 0x08, 0x80);
> - mipi_dsi_dcs_write_seq(dsi, 0x09, 0x81);
> - mipi_dsi_dcs_write_seq(dsi, 0x0A, 0x71);
> - mipi_dsi_dcs_write_seq(dsi, 0x0B, 0x00);
> -
> - mipi_dsi_dcs_write_seq(dsi, 0x0C, 0x00);
> - mipi_dsi_dcs_write_seq(dsi, 0x0E, 0x1A);
> -
> - mipi_dsi_dcs_write_seq(dsi, 0x24, 0x00);
> - mipi_dsi_dcs_write_seq(dsi, 0x25, 0x00);
> - mipi_dsi_dcs_write_seq(dsi, 0x26, 0x00);
> - mipi_dsi_dcs_write_seq(dsi, 0x27, 0x00);
> -
> - mipi_dsi_dcs_write_seq(dsi, 0x2C, 0xD4);
> - mipi_dsi_dcs_write_seq(dsi, 0xB9, 0x40);
> -
> - mipi_dsi_dcs_write_seq(dsi, 0xB0, 0x11);
> -
> - mipi_dsi_dcs_write_seq(dsi, 0xE6, 0x32);
> - mipi_dsi_dcs_write_seq(dsi, 0xD1, 0x30);
> -
> - mipi_dsi_dcs_write_seq(dsi, 0xD6, 0x55);
> -
> - mipi_dsi_dcs_write_seq(dsi, 0xD0, 0x01);
> - mipi_dsi_dcs_write_seq(dsi, 0xE3, 0x93);
> - mipi_dsi_dcs_write_seq(dsi, 0xE4, 0x00);
> - mipi_dsi_dcs_write_seq(dsi, 0xE5, 0x80);
> -
> - mipi_dsi_dcs_write_seq(dsi, 0x31, 0x07);
> - mipi_dsi_dcs_write_seq(dsi, 0x32, 0x07);
> - mipi_dsi_dcs_write_seq(dsi, 0x33, 0x07);
> - mipi_dsi_dcs_write_seq(dsi, 0x34, 0x07);
> - mipi_dsi_dcs_write_seq(dsi, 0x35, 0x07);
> - mipi_dsi_dcs_write_seq(dsi, 0x36, 0x01);
> - mipi_dsi_dcs_write_seq(dsi, 0x37, 0x00);
> - mipi_dsi_dcs_write_seq(dsi, 0x38, 0x28);
> - mipi_dsi_dcs_write_seq(dsi, 0x39, 0x29);
> - mipi_dsi_dcs_write_seq(dsi, 0x3A, 0x11);
> - mipi_dsi_dcs_write_seq(dsi, 0x3B, 0x13);
> - mipi_dsi_dcs_write_seq(dsi, 0x3C, 0x15);
> - mipi_dsi_dcs_write_seq(dsi, 0x3D, 0x17);
> - mipi_dsi_dcs_write_seq(dsi, 0x3E, 0x09);
> - mipi_dsi_dcs_write_seq(dsi, 0x3F, 0x0D);
> - mipi_dsi_dcs_write_seq(dsi, 0x40, 0x02);
> - mipi_dsi_dcs_write_seq(dsi, 0x41, 0x02);
> - mipi_dsi_dcs_write_seq(dsi, 0x42, 0x02);
> - mipi_dsi_dcs_write_seq(dsi, 0x43, 0x02);
> - mipi_dsi_dcs_write_seq(dsi, 0x44, 0x02);
> - mipi_dsi_dcs_write_seq(dsi, 0x45, 0x02);
> - mipi_dsi_dcs_write_seq(dsi, 0x46, 0x02);
> -
> - mipi_dsi_dcs_write_seq(dsi, 0x47, 0x07);
> - mipi_dsi_dcs_write_seq(dsi, 0x48, 0x07);
> - mipi_dsi_dcs_write_seq(dsi, 0x49, 0x07);
> - mipi_dsi_dcs_write_seq(dsi, 0x4A, 0x07);
> - mipi_dsi_dcs_write_seq(dsi, 0x4B, 0x07);
> - mipi_dsi_dcs_write_seq(dsi, 0x4C, 0x01);
> - mipi_dsi_dcs_write_seq(dsi, 0x4D, 0x00);
> - mipi_dsi_dcs_write_seq(dsi, 0x4E, 0x28);
> - mipi_dsi_dcs_write_seq(dsi, 0x4F, 0x29);
> - mipi_dsi_dcs_write_seq(dsi, 0x50, 0x10);
> - mipi_dsi_dcs_write_seq(dsi, 0x51, 0x12);
> - mipi_dsi_dcs_write_seq(dsi, 0x52, 0x14);
> - mipi_dsi_dcs_write_seq(dsi, 0x53, 0x16);
> - mipi_dsi_dcs_write_seq(dsi, 0x54, 0x08);
> - mipi_dsi_dcs_write_seq(dsi, 0x55, 0x0C);
> - mipi_dsi_dcs_write_seq(dsi, 0x56, 0x02);
> - mipi_dsi_dcs_write_seq(dsi, 0x57, 0x02);
> - mipi_dsi_dcs_write_seq(dsi, 0x58, 0x02);
> - mipi_dsi_dcs_write_seq(dsi, 0x59, 0x02);
> - mipi_dsi_dcs_write_seq(dsi, 0x5A, 0x02);
> - mipi_dsi_dcs_write_seq(dsi, 0x5B, 0x02);
> - mipi_dsi_dcs_write_seq(dsi, 0x5C, 0x02);
> -
> - mipi_dsi_dcs_write_seq(dsi, 0x61, 0x07);
> - mipi_dsi_dcs_write_seq(dsi, 0x62, 0x07);
> - mipi_dsi_dcs_write_seq(dsi, 0x63, 0x07);
> - mipi_dsi_dcs_write_seq(dsi, 0x64, 0x07);
> - mipi_dsi_dcs_write_seq(dsi, 0x65, 0x07);
> - mipi_dsi_dcs_write_seq(dsi, 0x66, 0x01);
> - mipi_dsi_dcs_write_seq(dsi, 0x67, 0x00);
> - mipi_dsi_dcs_write_seq(dsi, 0x68, 0x28);
> - mipi_dsi_dcs_write_seq(dsi, 0x69, 0x29);
> - mipi_dsi_dcs_write_seq(dsi, 0x6A, 0x16);
> - mipi_dsi_dcs_write_seq(dsi, 0x6B, 0x14);
> - mipi_dsi_dcs_write_seq(dsi, 0x6C, 0x12);
> - mipi_dsi_dcs_write_seq(dsi, 0x6D, 0x10);
> - mipi_dsi_dcs_write_seq(dsi, 0x6E, 0x0C);
> - mipi_dsi_dcs_write_seq(dsi, 0x6F, 0x08);
> - mipi_dsi_dcs_write_seq(dsi, 0x70, 0x02);
> - mipi_dsi_dcs_write_seq(dsi, 0x71, 0x02);
> - mipi_dsi_dcs_write_seq(dsi, 0x72, 0x02);
> - mipi_dsi_dcs_write_seq(dsi, 0x73, 0x02);
> - mipi_dsi_dcs_write_seq(dsi, 0x74, 0x02);
> - mipi_dsi_dcs_write_seq(dsi, 0x75, 0x02);
> - mipi_dsi_dcs_write_seq(dsi, 0x76, 0x02);
> -
> - mipi_dsi_dcs_write_seq(dsi, 0x77, 0x07);
> - mipi_dsi_dcs_write_seq(dsi, 0x78, 0x07);
> - mipi_dsi_dcs_write_seq(dsi, 0x79, 0x07);
> - mipi_dsi_dcs_write_seq(dsi, 0x7A, 0x07);
> - mipi_dsi_dcs_write_seq(dsi, 0x7B, 0x07);
> - mipi_dsi_dcs_write_seq(dsi, 0x7C, 0x01);
> - mipi_dsi_dcs_write_seq(dsi, 0x7D, 0x00);
> - mipi_dsi_dcs_write_seq(dsi, 0x7E, 0x28);
> - mipi_dsi_dcs_write_seq(dsi, 0x7F, 0x29);
> - mipi_dsi_dcs_write_seq(dsi, 0x80, 0x17);
> - mipi_dsi_dcs_write_seq(dsi, 0x81, 0x15);
> - mipi_dsi_dcs_write_seq(dsi, 0x82, 0x13);
> - mipi_dsi_dcs_write_seq(dsi, 0x83, 0x11);
> - mipi_dsi_dcs_write_seq(dsi, 0x84, 0x0D);
> - mipi_dsi_dcs_write_seq(dsi, 0x85, 0x09);
> - mipi_dsi_dcs_write_seq(dsi, 0x86, 0x02);
> - mipi_dsi_dcs_write_seq(dsi, 0x87, 0x07);
> - mipi_dsi_dcs_write_seq(dsi, 0x88, 0x07);
> - mipi_dsi_dcs_write_seq(dsi, 0x89, 0x07);
> - mipi_dsi_dcs_write_seq(dsi, 0x8A, 0x07);
> - mipi_dsi_dcs_write_seq(dsi, 0x8B, 0x07);
> - mipi_dsi_dcs_write_seq(dsi, 0x8C, 0x07);
> -
> - mipi_dsi_dcs_write_seq(dsi, 0xFF, 0x98, 0x82, 0x02);
> - mipi_dsi_dcs_write_seq(dsi, 0x29, 0x3A);
> - mipi_dsi_dcs_write_seq(dsi, 0x2A, 0x3B);
> -
> - mipi_dsi_dcs_write_seq(dsi, 0x06, 0x01);
> - mipi_dsi_dcs_write_seq(dsi, 0x07, 0x01);
> - mipi_dsi_dcs_write_seq(dsi, 0x08, 0x0C);
> - mipi_dsi_dcs_write_seq(dsi, 0x09, 0x44);
> -
> - mipi_dsi_dcs_write_seq(dsi, 0x3C, 0x0A);
> - mipi_dsi_dcs_write_seq(dsi, 0x39, 0x11);
> - mipi_dsi_dcs_write_seq(dsi, 0x3D, 0x00);
> - mipi_dsi_dcs_write_seq(dsi, 0x3A, 0x0C);
> - mipi_dsi_dcs_write_seq(dsi, 0x3B, 0x44);
> -
> - mipi_dsi_dcs_write_seq(dsi, 0x53, 0x1F);
> - mipi_dsi_dcs_write_seq(dsi, 0x5E, 0x40);
> - mipi_dsi_dcs_write_seq(dsi, 0x84, 0x00);
> -
> - mipi_dsi_dcs_write_seq(dsi, 0xFF, 0x98, 0x82, 0x03);
> - mipi_dsi_dcs_write_seq(dsi, 0x20, 0x01);
> - mipi_dsi_dcs_write_seq(dsi, 0x21, 0x3C);
> - mipi_dsi_dcs_write_seq(dsi, 0x22, 0xFA);
> -
> - mipi_dsi_dcs_write_seq(dsi, 0xFF, 0x98, 0x82, 0x0A);
> - mipi_dsi_dcs_write_seq(dsi, 0xE0, 0x01);
> - mipi_dsi_dcs_write_seq(dsi, 0xE2, 0x01);
> - mipi_dsi_dcs_write_seq(dsi, 0xE5, 0x91);
> - mipi_dsi_dcs_write_seq(dsi, 0xE6, 0x3C);
> - mipi_dsi_dcs_write_seq(dsi, 0xE7, 0x00);
> - mipi_dsi_dcs_write_seq(dsi, 0xE8, 0xFA);
> -
> - mipi_dsi_dcs_write_seq(dsi, 0xFF, 0x98, 0x82, 0x12);
> - mipi_dsi_dcs_write_seq(dsi, 0x87, 0x2C);
> -
> - mipi_dsi_dcs_write_seq(dsi, 0xFF, 0x98, 0x82, 0x05);
> - mipi_dsi_dcs_write_seq(dsi, 0x73, 0xE5);
> - mipi_dsi_dcs_write_seq(dsi, 0x7F, 0x6B);
> - mipi_dsi_dcs_write_seq(dsi, 0x6D, 0xA4);
> - mipi_dsi_dcs_write_seq(dsi, 0x79, 0x54);
> - mipi_dsi_dcs_write_seq(dsi, 0x69, 0x97);
> - mipi_dsi_dcs_write_seq(dsi, 0x6A, 0x97);
> - mipi_dsi_dcs_write_seq(dsi, 0xA5, 0x3F);
> - mipi_dsi_dcs_write_seq(dsi, 0x61, 0xDA);
> - mipi_dsi_dcs_write_seq(dsi, 0xA7, 0xF1);
> - mipi_dsi_dcs_write_seq(dsi, 0x5F, 0x01);
> - mipi_dsi_dcs_write_seq(dsi, 0x62, 0x3F);
> - mipi_dsi_dcs_write_seq(dsi, 0x1D, 0x90);
> - mipi_dsi_dcs_write_seq(dsi, 0x86, 0x87);
> -
> - mipi_dsi_dcs_write_seq(dsi, 0xFF, 0x98, 0x82, 0x06);
> - mipi_dsi_dcs_write_seq(dsi, 0xC0, 0x80);
> - mipi_dsi_dcs_write_seq(dsi, 0xC1, 0x07);
> - mipi_dsi_dcs_write_seq(dsi, 0xCA, 0x58);
> - mipi_dsi_dcs_write_seq(dsi, 0xCB, 0x02);
> - mipi_dsi_dcs_write_seq(dsi, 0xCE, 0x58);
> - mipi_dsi_dcs_write_seq(dsi, 0xCF, 0x02);
> - mipi_dsi_dcs_write_seq(dsi, 0x67, 0x60);
> - mipi_dsi_dcs_write_seq(dsi, 0x10, 0x00);
> - mipi_dsi_dcs_write_seq(dsi, 0x92, 0x22);
> - mipi_dsi_dcs_write_seq(dsi, 0xD3, 0x08);
> - mipi_dsi_dcs_write_seq(dsi, 0xD6, 0x55);
> - mipi_dsi_dcs_write_seq(dsi, 0xDC, 0x38);
> -
> - mipi_dsi_dcs_write_seq(dsi, 0xFF, 0x98, 0x82, 0x08);
> - mipi_dsi_dcs_write_seq(dsi, 0xE0, 0x00, 0x10, 0x2A, 0x4D, 0x61, 0x56, 0x6A, 0x6E, 0x79,
> - 0x76, 0x8F, 0x95, 0x98, 0xAE, 0xAA, 0xB2, 0xBB, 0xCE, 0xC6, 0xBD,
> - 0xD5, 0xE2, 0xE8);
> - mipi_dsi_dcs_write_seq(dsi, 0xE1, 0x00, 0x10, 0x2A, 0x4D, 0x61, 0x56, 0x6A, 0x6E, 0x79,
> - 0x76, 0x8F, 0x95, 0x98, 0xAE, 0xAA, 0xB2, 0xBB, 0xCE, 0xC6, 0xBD,
> - 0xD5, 0xE2, 0xE8);
> -
> - mipi_dsi_dcs_write_seq(dsi, 0xFF, 0x98, 0x82, 0x04);
> - mipi_dsi_dcs_write_seq(dsi, 0xBA, 0x81);
> -
> - mipi_dsi_dcs_write_seq(dsi, 0xFF, 0x98, 0x82, 0x0C);
> - mipi_dsi_dcs_write_seq(dsi, 0x00, 0x02);
> - mipi_dsi_dcs_write_seq(dsi, 0x01, 0x00);
> - mipi_dsi_dcs_write_seq(dsi, 0x02, 0x03);
> - mipi_dsi_dcs_write_seq(dsi, 0x03, 0x01);
> - mipi_dsi_dcs_write_seq(dsi, 0x04, 0x03);
> - mipi_dsi_dcs_write_seq(dsi, 0x05, 0x02);
> - mipi_dsi_dcs_write_seq(dsi, 0x06, 0x04);
> - mipi_dsi_dcs_write_seq(dsi, 0x07, 0x03);
> - mipi_dsi_dcs_write_seq(dsi, 0x08, 0x03);
> - mipi_dsi_dcs_write_seq(dsi, 0x09, 0x04);
> - mipi_dsi_dcs_write_seq(dsi, 0x0A, 0x04);
> - mipi_dsi_dcs_write_seq(dsi, 0x0B, 0x05);
> - mipi_dsi_dcs_write_seq(dsi, 0x0C, 0x04);
> - mipi_dsi_dcs_write_seq(dsi, 0x0D, 0x06);
> - mipi_dsi_dcs_write_seq(dsi, 0x0E, 0x05);
> - mipi_dsi_dcs_write_seq(dsi, 0x0F, 0x07);
> - mipi_dsi_dcs_write_seq(dsi, 0x10, 0x04);
> - mipi_dsi_dcs_write_seq(dsi, 0x11, 0x08);
> - mipi_dsi_dcs_write_seq(dsi, 0x12, 0x05);
> - mipi_dsi_dcs_write_seq(dsi, 0x13, 0x09);
> - mipi_dsi_dcs_write_seq(dsi, 0x14, 0x05);
> - mipi_dsi_dcs_write_seq(dsi, 0x15, 0x0A);
> - mipi_dsi_dcs_write_seq(dsi, 0x16, 0x06);
> - mipi_dsi_dcs_write_seq(dsi, 0x17, 0x0B);
> - mipi_dsi_dcs_write_seq(dsi, 0x18, 0x05);
> - mipi_dsi_dcs_write_seq(dsi, 0x19, 0x0C);
> - mipi_dsi_dcs_write_seq(dsi, 0x1A, 0x06);
> - mipi_dsi_dcs_write_seq(dsi, 0x1B, 0x0D);
> - mipi_dsi_dcs_write_seq(dsi, 0x1C, 0x06);
> - mipi_dsi_dcs_write_seq(dsi, 0x1D, 0x0E);
> - mipi_dsi_dcs_write_seq(dsi, 0x1E, 0x07);
> - mipi_dsi_dcs_write_seq(dsi, 0x1F, 0x0F);
> - mipi_dsi_dcs_write_seq(dsi, 0x20, 0x06);
> - mipi_dsi_dcs_write_seq(dsi, 0x21, 0x10);
> - mipi_dsi_dcs_write_seq(dsi, 0x22, 0x07);
> - mipi_dsi_dcs_write_seq(dsi, 0x23, 0x11);
> - mipi_dsi_dcs_write_seq(dsi, 0x24, 0x07);
> - mipi_dsi_dcs_write_seq(dsi, 0x25, 0x12);
> - mipi_dsi_dcs_write_seq(dsi, 0x26, 0x08);
> - mipi_dsi_dcs_write_seq(dsi, 0x27, 0x13);
> - mipi_dsi_dcs_write_seq(dsi, 0x28, 0x07);
> - mipi_dsi_dcs_write_seq(dsi, 0x29, 0x14);
> - mipi_dsi_dcs_write_seq(dsi, 0x2A, 0x08);
> - mipi_dsi_dcs_write_seq(dsi, 0x2B, 0x15);
> - mipi_dsi_dcs_write_seq(dsi, 0x2C, 0x08);
> - mipi_dsi_dcs_write_seq(dsi, 0x2D, 0x16);
> - mipi_dsi_dcs_write_seq(dsi, 0x2E, 0x09);
> - mipi_dsi_dcs_write_seq(dsi, 0x2F, 0x17);
> - mipi_dsi_dcs_write_seq(dsi, 0x30, 0x08);
> - mipi_dsi_dcs_write_seq(dsi, 0x31, 0x18);
> - mipi_dsi_dcs_write_seq(dsi, 0x32, 0x09);
> - mipi_dsi_dcs_write_seq(dsi, 0x33, 0x19);
> - mipi_dsi_dcs_write_seq(dsi, 0x34, 0x09);
> - mipi_dsi_dcs_write_seq(dsi, 0x35, 0x1A);
> - mipi_dsi_dcs_write_seq(dsi, 0x36, 0x0A);
> - mipi_dsi_dcs_write_seq(dsi, 0x37, 0x1B);
> - mipi_dsi_dcs_write_seq(dsi, 0x38, 0x0A);
> - mipi_dsi_dcs_write_seq(dsi, 0x39, 0x1C);
> - mipi_dsi_dcs_write_seq(dsi, 0x3A, 0x0A);
> - mipi_dsi_dcs_write_seq(dsi, 0x3B, 0x1D);
> - mipi_dsi_dcs_write_seq(dsi, 0x3C, 0x0A);
> - mipi_dsi_dcs_write_seq(dsi, 0x3D, 0x1E);
> - mipi_dsi_dcs_write_seq(dsi, 0x3E, 0x0A);
> - mipi_dsi_dcs_write_seq(dsi, 0x3F, 0x1F);
> -
> - mipi_dsi_dcs_write_seq(dsi, 0xFF, 0x98, 0x82, 0x04);
> - mipi_dsi_dcs_write_seq(dsi, 0xBA, 0x01);
> -
> - mipi_dsi_dcs_write_seq(dsi, 0xFF, 0x98, 0x82, 0x0E);
> - mipi_dsi_dcs_write_seq(dsi, 0x02, 0x0C);
> - mipi_dsi_dcs_write_seq(dsi, 0x20, 0x10);
> - mipi_dsi_dcs_write_seq(dsi, 0x25, 0x16);
> - mipi_dsi_dcs_write_seq(dsi, 0x26, 0xE0);
> - mipi_dsi_dcs_write_seq(dsi, 0x27, 0x00);
> - mipi_dsi_dcs_write_seq(dsi, 0x29, 0x71);
> - mipi_dsi_dcs_write_seq(dsi, 0x2A, 0x46);
> - mipi_dsi_dcs_write_seq(dsi, 0x2B, 0x1F);
> - mipi_dsi_dcs_write_seq(dsi, 0x2D, 0xC7);
> - mipi_dsi_dcs_write_seq(dsi, 0x31, 0x02);
> - mipi_dsi_dcs_write_seq(dsi, 0x32, 0xDF);
> - mipi_dsi_dcs_write_seq(dsi, 0x33, 0x5A);
> - mipi_dsi_dcs_write_seq(dsi, 0x34, 0xC0);
> - mipi_dsi_dcs_write_seq(dsi, 0x35, 0x5A);
> - mipi_dsi_dcs_write_seq(dsi, 0x36, 0xC0);
> - mipi_dsi_dcs_write_seq(dsi, 0x38, 0x65);
> - mipi_dsi_dcs_write_seq(dsi, 0x80, 0x3E);
> - mipi_dsi_dcs_write_seq(dsi, 0x81, 0xA0);
> - mipi_dsi_dcs_write_seq(dsi, 0xB0, 0x01);
> - mipi_dsi_dcs_write_seq(dsi, 0xB1, 0xCC);
> - mipi_dsi_dcs_write_seq(dsi, 0xC0, 0x12);
> - mipi_dsi_dcs_write_seq(dsi, 0xC2, 0xCC);
> - mipi_dsi_dcs_write_seq(dsi, 0xC3, 0xCC);
> - mipi_dsi_dcs_write_seq(dsi, 0xC4, 0xCC);
> - mipi_dsi_dcs_write_seq(dsi, 0xC5, 0xCC);
> - mipi_dsi_dcs_write_seq(dsi, 0xC6, 0xCC);
> - mipi_dsi_dcs_write_seq(dsi, 0xC7, 0xCC);
> - mipi_dsi_dcs_write_seq(dsi, 0xC8, 0xCC);
> - mipi_dsi_dcs_write_seq(dsi, 0xC9, 0xCC);
> - mipi_dsi_dcs_write_seq(dsi, 0x30, 0x00);
> - mipi_dsi_dcs_write_seq(dsi, 0x00, 0x81);
> - mipi_dsi_dcs_write_seq(dsi, 0x08, 0x02);
> - mipi_dsi_dcs_write_seq(dsi, 0x09, 0x00);
> - mipi_dsi_dcs_write_seq(dsi, 0x07, 0x21);
> - mipi_dsi_dcs_write_seq(dsi, 0x04, 0x10);
> -
> - mipi_dsi_dcs_write_seq(dsi, 0xFF, 0x98, 0x82, 0x1E);
> - mipi_dsi_dcs_write_seq(dsi, 0x60, 0x00);
> - mipi_dsi_dcs_write_seq(dsi, 0x64, 0x00);
> - mipi_dsi_dcs_write_seq(dsi, 0x6D, 0x00);
> -
> - mipi_dsi_dcs_write_seq(dsi, 0xFF, 0x98, 0x82, 0x0B);
> - mipi_dsi_dcs_write_seq(dsi, 0xA6, 0x44);
> - mipi_dsi_dcs_write_seq(dsi, 0xA7, 0xB6);
> - mipi_dsi_dcs_write_seq(dsi, 0xA8, 0x03);
> - mipi_dsi_dcs_write_seq(dsi, 0xA9, 0x03);
> - mipi_dsi_dcs_write_seq(dsi, 0xAA, 0x51);
> - mipi_dsi_dcs_write_seq(dsi, 0xAB, 0x51);
> - mipi_dsi_dcs_write_seq(dsi, 0xAC, 0x04);
> - mipi_dsi_dcs_write_seq(dsi, 0xBD, 0x92);
> - mipi_dsi_dcs_write_seq(dsi, 0xBE, 0xA1);
> -
> - mipi_dsi_dcs_write_seq(dsi, 0xFF, 0x98, 0x82, 0x05);
> - mipi_dsi_dcs_write_seq(dsi, 0x86, 0x87);
> -
> - mipi_dsi_dcs_write_seq(dsi, 0xFF, 0x98, 0x82, 0x06);
> - mipi_dsi_dcs_write_seq(dsi, 0x92, 0x22);
> -
> - mipi_dsi_dcs_write_seq(dsi, 0xFF, 0x98, 0x82, 0x00);
> -
> - ret = mipi_dsi_dcs_exit_sleep_mode(dsi);
> - if (ret)
> - return ret;
> - msleep(120);
> - ret = mipi_dsi_dcs_set_display_on(dsi);
> - if (ret)
> - return ret;
> - msleep(20);
> -
> - return 0;
> -};
> -
> static inline struct boe_panel *to_boe_panel(struct drm_panel *panel)
> {
> return container_of(panel, struct boe_panel, base);
> @@ -2081,34 +1726,6 @@ static const struct panel_desc starry_himax83102_j02_desc = {
> .lp11_before_reset = true,
> };
>
> -static const struct drm_display_mode starry_ili9882t_default_mode = {
> - .clock = 165280,
> - .hdisplay = 1200,
> - .hsync_start = 1200 + 32,
> - .hsync_end = 1200 + 32 + 30,
> - .htotal = 1200 + 32 + 30 + 32,
> - .vdisplay = 1920,
> - .vsync_start = 1920 + 68,
> - .vsync_end = 1920 + 68 + 2,
> - .vtotal = 1920 + 68 + 2 + 10,
> - .type = DRM_MODE_TYPE_DRIVER | DRM_MODE_TYPE_PREFERRED,
> -};
> -
> -static const struct panel_desc starry_ili9882t_desc = {
> - .modes = &starry_ili9882t_default_mode,
> - .bpc = 8,
> - .size = {
> - .width_mm = 141,
> - .height_mm = 226,
> - },
> - .lanes = 4,
> - .format = MIPI_DSI_FMT_RGB888,
> - .mode_flags = MIPI_DSI_MODE_VIDEO | MIPI_DSI_MODE_VIDEO_SYNC_PULSE |
> - MIPI_DSI_MODE_LPM,
> - .init = starry_ili9882t_init,
> - .lp11_before_reset = true,
> -};
> -
> static int boe_panel_get_modes(struct drm_panel *panel,
> struct drm_connector *connector)
> {
> @@ -2285,9 +1902,6 @@ static const struct of_device_id boe_of_match[] = {
> { .compatible = "starry,himax83102-j02",
> .data = &starry_himax83102_j02_desc
> },
> - { .compatible = "starry,ili9882t",
> - .data = &starry_ili9882t_desc
> - },
> { /* sentinel */ }
> };
> MODULE_DEVICE_TABLE(of, boe_of_match);
> diff --git a/drivers/gpu/drm/panel/panel-ilitek-ili9882t.c b/drivers/gpu/drm/panel/panel-ilitek-ili9882t.c
> new file mode 100644
> index 000000000000..20f3cc37fa83
> --- /dev/null
> +++ b/drivers/gpu/drm/panel/panel-ilitek-ili9882t.c
> @@ -0,0 +1,739 @@
> +// SPDX-License-Identifier: GPL-2.0
> +/*
> + * Panels based on the Ilitek ILI9882T display controller.
> + */
> +#include <linux/delay.h>
> +#include <linux/gpio/consumer.h>
> +#include <linux/module.h>
> +#include <linux/of.h>
> +#include <linux/of_device.h>
> +#include <linux/regulator/consumer.h>
> +
> +#include <drm/drm_connector.h>
> +#include <drm/drm_crtc.h>
> +#include <drm/drm_mipi_dsi.h>
> +#include <drm/drm_panel.h>
> +
> +#include <video/mipi_display.h>
> +
> +/*
> + * Use this descriptor struct to describe different panels using the
> + * Ilitek ILI9882T display controller.
> + */
> +struct panel_desc {
> + const struct drm_display_mode *modes;
> + unsigned int bpc;
> +
> + /**
> + * @width_mm: width of the panel's active display area
> + * @height_mm: height of the panel's active display area
> + */
> + struct {
> + unsigned int width_mm;
> + unsigned int height_mm;
> + } size;
> +
> + unsigned long mode_flags;
> + enum mipi_dsi_pixel_format format;
> + int (*init)(struct mipi_dsi_device *dsi);
> + unsigned int lanes;
> + bool discharge_on_disable;
> + bool lp11_before_reset;
> +};
> +
> +struct ili9882t {
> + struct drm_panel base;
> + struct mipi_dsi_device *dsi;
> +
> + const struct panel_desc *desc;
> +
> + enum drm_panel_orientation orientation;
> + struct regulator *pp3300;
> + struct regulator *pp1800;
> + struct regulator *avee;
> + struct regulator *avdd;
> + struct gpio_desc *enable_gpio;
> +};
> +
> +static int starry_ili9882t_init(struct mipi_dsi_device *dsi)
> +{
> + int ret;
> +
> + msleep(5);
> +
> + mipi_dsi_dcs_write_seq(dsi, 0xFF, 0x98, 0x82, 0x01);
> + mipi_dsi_dcs_write_seq(dsi, 0x00, 0x42);
> + mipi_dsi_dcs_write_seq(dsi, 0x01, 0x11);
> + mipi_dsi_dcs_write_seq(dsi, 0x02, 0x00);
> + mipi_dsi_dcs_write_seq(dsi, 0x03, 0x00);
> +
> + mipi_dsi_dcs_write_seq(dsi, 0x04, 0x01);
> + mipi_dsi_dcs_write_seq(dsi, 0x05, 0x11);
> + mipi_dsi_dcs_write_seq(dsi, 0x06, 0x00);
> + mipi_dsi_dcs_write_seq(dsi, 0x07, 0x00);
> +
> + mipi_dsi_dcs_write_seq(dsi, 0x08, 0x80);
> + mipi_dsi_dcs_write_seq(dsi, 0x09, 0x81);
> + mipi_dsi_dcs_write_seq(dsi, 0x0A, 0x71);
> + mipi_dsi_dcs_write_seq(dsi, 0x0B, 0x00);
> +
> + mipi_dsi_dcs_write_seq(dsi, 0x0C, 0x00);
> + mipi_dsi_dcs_write_seq(dsi, 0x0E, 0x1A);
> +
> + mipi_dsi_dcs_write_seq(dsi, 0x24, 0x00);
> + mipi_dsi_dcs_write_seq(dsi, 0x25, 0x00);
> + mipi_dsi_dcs_write_seq(dsi, 0x26, 0x00);
> + mipi_dsi_dcs_write_seq(dsi, 0x27, 0x00);
> +
> + mipi_dsi_dcs_write_seq(dsi, 0x2C, 0xD4);
> + mipi_dsi_dcs_write_seq(dsi, 0xB9, 0x40);
> +
> + mipi_dsi_dcs_write_seq(dsi, 0xB0, 0x11);
> +
> + mipi_dsi_dcs_write_seq(dsi, 0xE6, 0x32);
> + mipi_dsi_dcs_write_seq(dsi, 0xD1, 0x30);
> +
> + mipi_dsi_dcs_write_seq(dsi, 0xD6, 0x55);
> +
> + mipi_dsi_dcs_write_seq(dsi, 0xD0, 0x01);
> + mipi_dsi_dcs_write_seq(dsi, 0xE3, 0x93);
> + mipi_dsi_dcs_write_seq(dsi, 0xE4, 0x00);
> + mipi_dsi_dcs_write_seq(dsi, 0xE5, 0x80);
> +
> + mipi_dsi_dcs_write_seq(dsi, 0x31, 0x07);
> + mipi_dsi_dcs_write_seq(dsi, 0x32, 0x07);
> + mipi_dsi_dcs_write_seq(dsi, 0x33, 0x07);
> + mipi_dsi_dcs_write_seq(dsi, 0x34, 0x07);
> + mipi_dsi_dcs_write_seq(dsi, 0x35, 0x07);
> + mipi_dsi_dcs_write_seq(dsi, 0x36, 0x01);
> + mipi_dsi_dcs_write_seq(dsi, 0x37, 0x00);
> + mipi_dsi_dcs_write_seq(dsi, 0x38, 0x28);
> + mipi_dsi_dcs_write_seq(dsi, 0x39, 0x29);
> + mipi_dsi_dcs_write_seq(dsi, 0x3A, 0x11);
> + mipi_dsi_dcs_write_seq(dsi, 0x3B, 0x13);
> + mipi_dsi_dcs_write_seq(dsi, 0x3C, 0x15);
> + mipi_dsi_dcs_write_seq(dsi, 0x3D, 0x17);
> + mipi_dsi_dcs_write_seq(dsi, 0x3E, 0x09);
> + mipi_dsi_dcs_write_seq(dsi, 0x3F, 0x0D);
> + mipi_dsi_dcs_write_seq(dsi, 0x40, 0x02);
> + mipi_dsi_dcs_write_seq(dsi, 0x41, 0x02);
> + mipi_dsi_dcs_write_seq(dsi, 0x42, 0x02);
> + mipi_dsi_dcs_write_seq(dsi, 0x43, 0x02);
> + mipi_dsi_dcs_write_seq(dsi, 0x44, 0x02);
> + mipi_dsi_dcs_write_seq(dsi, 0x45, 0x02);
> + mipi_dsi_dcs_write_seq(dsi, 0x46, 0x02);
> +
> + mipi_dsi_dcs_write_seq(dsi, 0x47, 0x07);
> + mipi_dsi_dcs_write_seq(dsi, 0x48, 0x07);
> + mipi_dsi_dcs_write_seq(dsi, 0x49, 0x07);
> + mipi_dsi_dcs_write_seq(dsi, 0x4A, 0x07);
> + mipi_dsi_dcs_write_seq(dsi, 0x4B, 0x07);
> + mipi_dsi_dcs_write_seq(dsi, 0x4C, 0x01);
> + mipi_dsi_dcs_write_seq(dsi, 0x4D, 0x00);
> + mipi_dsi_dcs_write_seq(dsi, 0x4E, 0x28);
> + mipi_dsi_dcs_write_seq(dsi, 0x4F, 0x29);
> + mipi_dsi_dcs_write_seq(dsi, 0x50, 0x10);
> + mipi_dsi_dcs_write_seq(dsi, 0x51, 0x12);
> + mipi_dsi_dcs_write_seq(dsi, 0x52, 0x14);
> + mipi_dsi_dcs_write_seq(dsi, 0x53, 0x16);
> + mipi_dsi_dcs_write_seq(dsi, 0x54, 0x08);
> + mipi_dsi_dcs_write_seq(dsi, 0x55, 0x0C);
> + mipi_dsi_dcs_write_seq(dsi, 0x56, 0x02);
> + mipi_dsi_dcs_write_seq(dsi, 0x57, 0x02);
> + mipi_dsi_dcs_write_seq(dsi, 0x58, 0x02);
> + mipi_dsi_dcs_write_seq(dsi, 0x59, 0x02);
> + mipi_dsi_dcs_write_seq(dsi, 0x5A, 0x02);
> + mipi_dsi_dcs_write_seq(dsi, 0x5B, 0x02);
> + mipi_dsi_dcs_write_seq(dsi, 0x5C, 0x02);
> +
> + mipi_dsi_dcs_write_seq(dsi, 0x61, 0x07);
> + mipi_dsi_dcs_write_seq(dsi, 0x62, 0x07);
> + mipi_dsi_dcs_write_seq(dsi, 0x63, 0x07);
> + mipi_dsi_dcs_write_seq(dsi, 0x64, 0x07);
> + mipi_dsi_dcs_write_seq(dsi, 0x65, 0x07);
> + mipi_dsi_dcs_write_seq(dsi, 0x66, 0x01);
> + mipi_dsi_dcs_write_seq(dsi, 0x67, 0x00);
> + mipi_dsi_dcs_write_seq(dsi, 0x68, 0x28);
> + mipi_dsi_dcs_write_seq(dsi, 0x69, 0x29);
> + mipi_dsi_dcs_write_seq(dsi, 0x6A, 0x16);
> + mipi_dsi_dcs_write_seq(dsi, 0x6B, 0x14);
> + mipi_dsi_dcs_write_seq(dsi, 0x6C, 0x12);
> + mipi_dsi_dcs_write_seq(dsi, 0x6D, 0x10);
> + mipi_dsi_dcs_write_seq(dsi, 0x6E, 0x0C);
> + mipi_dsi_dcs_write_seq(dsi, 0x6F, 0x08);
> + mipi_dsi_dcs_write_seq(dsi, 0x70, 0x02);
> + mipi_dsi_dcs_write_seq(dsi, 0x71, 0x02);
> + mipi_dsi_dcs_write_seq(dsi, 0x72, 0x02);
> + mipi_dsi_dcs_write_seq(dsi, 0x73, 0x02);
> + mipi_dsi_dcs_write_seq(dsi, 0x74, 0x02);
> + mipi_dsi_dcs_write_seq(dsi, 0x75, 0x02);
> + mipi_dsi_dcs_write_seq(dsi, 0x76, 0x02);
> +
> + mipi_dsi_dcs_write_seq(dsi, 0x77, 0x07);
> + mipi_dsi_dcs_write_seq(dsi, 0x78, 0x07);
> + mipi_dsi_dcs_write_seq(dsi, 0x79, 0x07);
> + mipi_dsi_dcs_write_seq(dsi, 0x7A, 0x07);
> + mipi_dsi_dcs_write_seq(dsi, 0x7B, 0x07);
> + mipi_dsi_dcs_write_seq(dsi, 0x7C, 0x01);
> + mipi_dsi_dcs_write_seq(dsi, 0x7D, 0x00);
> + mipi_dsi_dcs_write_seq(dsi, 0x7E, 0x28);
> + mipi_dsi_dcs_write_seq(dsi, 0x7F, 0x29);
> + mipi_dsi_dcs_write_seq(dsi, 0x80, 0x17);
> + mipi_dsi_dcs_write_seq(dsi, 0x81, 0x15);
> + mipi_dsi_dcs_write_seq(dsi, 0x82, 0x13);
> + mipi_dsi_dcs_write_seq(dsi, 0x83, 0x11);
> + mipi_dsi_dcs_write_seq(dsi, 0x84, 0x0D);
> + mipi_dsi_dcs_write_seq(dsi, 0x85, 0x09);
> + mipi_dsi_dcs_write_seq(dsi, 0x86, 0x02);
> + mipi_dsi_dcs_write_seq(dsi, 0x87, 0x07);
> + mipi_dsi_dcs_write_seq(dsi, 0x88, 0x07);
> + mipi_dsi_dcs_write_seq(dsi, 0x89, 0x07);
> + mipi_dsi_dcs_write_seq(dsi, 0x8A, 0x07);
> + mipi_dsi_dcs_write_seq(dsi, 0x8B, 0x07);
> + mipi_dsi_dcs_write_seq(dsi, 0x8C, 0x07);
> +
> + mipi_dsi_dcs_write_seq(dsi, 0xFF, 0x98, 0x82, 0x02);
> + mipi_dsi_dcs_write_seq(dsi, 0x29, 0x3A);
> + mipi_dsi_dcs_write_seq(dsi, 0x2A, 0x3B);
> +
> + mipi_dsi_dcs_write_seq(dsi, 0x06, 0x01);
> + mipi_dsi_dcs_write_seq(dsi, 0x07, 0x01);
> + mipi_dsi_dcs_write_seq(dsi, 0x08, 0x0C);
> + mipi_dsi_dcs_write_seq(dsi, 0x09, 0x44);
> +
> + mipi_dsi_dcs_write_seq(dsi, 0x3C, 0x0A);
> + mipi_dsi_dcs_write_seq(dsi, 0x39, 0x11);
> + mipi_dsi_dcs_write_seq(dsi, 0x3D, 0x00);
> + mipi_dsi_dcs_write_seq(dsi, 0x3A, 0x0C);
> + mipi_dsi_dcs_write_seq(dsi, 0x3B, 0x44);
> +
> + mipi_dsi_dcs_write_seq(dsi, 0x53, 0x1F);
> + mipi_dsi_dcs_write_seq(dsi, 0x5E, 0x40);
> + mipi_dsi_dcs_write_seq(dsi, 0x84, 0x00);
> +
> + mipi_dsi_dcs_write_seq(dsi, 0xFF, 0x98, 0x82, 0x03);
> + mipi_dsi_dcs_write_seq(dsi, 0x20, 0x01);
> + mipi_dsi_dcs_write_seq(dsi, 0x21, 0x3C);
> + mipi_dsi_dcs_write_seq(dsi, 0x22, 0xFA);
> +
> + mipi_dsi_dcs_write_seq(dsi, 0xFF, 0x98, 0x82, 0x0A);
> + mipi_dsi_dcs_write_seq(dsi, 0xE0, 0x01);
> + mipi_dsi_dcs_write_seq(dsi, 0xE2, 0x01);
> + mipi_dsi_dcs_write_seq(dsi, 0xE5, 0x91);
> + mipi_dsi_dcs_write_seq(dsi, 0xE6, 0x3C);
> + mipi_dsi_dcs_write_seq(dsi, 0xE7, 0x00);
> + mipi_dsi_dcs_write_seq(dsi, 0xE8, 0xFA);
> +
> + mipi_dsi_dcs_write_seq(dsi, 0xFF, 0x98, 0x82, 0x12);
> + mipi_dsi_dcs_write_seq(dsi, 0x87, 0x2C);
> +
> + mipi_dsi_dcs_write_seq(dsi, 0xFF, 0x98, 0x82, 0x05);
> + mipi_dsi_dcs_write_seq(dsi, 0x73, 0xE5);
> + mipi_dsi_dcs_write_seq(dsi, 0x7F, 0x6B);
> + mipi_dsi_dcs_write_seq(dsi, 0x6D, 0xA4);
> + mipi_dsi_dcs_write_seq(dsi, 0x79, 0x54);
> + mipi_dsi_dcs_write_seq(dsi, 0x69, 0x97);
> + mipi_dsi_dcs_write_seq(dsi, 0x6A, 0x97);
> + mipi_dsi_dcs_write_seq(dsi, 0xA5, 0x3F);
> + mipi_dsi_dcs_write_seq(dsi, 0x61, 0xDA);
> + mipi_dsi_dcs_write_seq(dsi, 0xA7, 0xF1);
> + mipi_dsi_dcs_write_seq(dsi, 0x5F, 0x01);
> + mipi_dsi_dcs_write_seq(dsi, 0x62, 0x3F);
> + mipi_dsi_dcs_write_seq(dsi, 0x1D, 0x90);
> + mipi_dsi_dcs_write_seq(dsi, 0x86, 0x87);
> +
> + mipi_dsi_dcs_write_seq(dsi, 0xFF, 0x98, 0x82, 0x06);
> + mipi_dsi_dcs_write_seq(dsi, 0xC0, 0x80);
> + mipi_dsi_dcs_write_seq(dsi, 0xC1, 0x07);
> + mipi_dsi_dcs_write_seq(dsi, 0xCA, 0x58);
> + mipi_dsi_dcs_write_seq(dsi, 0xCB, 0x02);
> + mipi_dsi_dcs_write_seq(dsi, 0xCE, 0x58);
> + mipi_dsi_dcs_write_seq(dsi, 0xCF, 0x02);
> + mipi_dsi_dcs_write_seq(dsi, 0x67, 0x60);
> + mipi_dsi_dcs_write_seq(dsi, 0x10, 0x00);
> + mipi_dsi_dcs_write_seq(dsi, 0x92, 0x22);
> + mipi_dsi_dcs_write_seq(dsi, 0xD3, 0x08);
> + mipi_dsi_dcs_write_seq(dsi, 0xD6, 0x55);
> + mipi_dsi_dcs_write_seq(dsi, 0xDC, 0x38);
> +
> + mipi_dsi_dcs_write_seq(dsi, 0xFF, 0x98, 0x82, 0x08);
> + mipi_dsi_dcs_write_seq(dsi, 0xE0, 0x00, 0x10, 0x2A, 0x4D, 0x61, 0x56, 0x6A, 0x6E, 0x79,
> + 0x76, 0x8F, 0x95, 0x98, 0xAE, 0xAA, 0xB2, 0xBB, 0xCE, 0xC6, 0xBD,
> + 0xD5, 0xE2, 0xE8);
> + mipi_dsi_dcs_write_seq(dsi, 0xE1, 0x00, 0x10, 0x2A, 0x4D, 0x61, 0x56, 0x6A, 0x6E, 0x79,
> + 0x76, 0x8F, 0x95, 0x98, 0xAE, 0xAA, 0xB2, 0xBB, 0xCE, 0xC6, 0xBD,
> + 0xD5, 0xE2, 0xE8);
> +
> + mipi_dsi_dcs_write_seq(dsi, 0xFF, 0x98, 0x82, 0x04);
> + mipi_dsi_dcs_write_seq(dsi, 0xBA, 0x81);
> +
> + mipi_dsi_dcs_write_seq(dsi, 0xFF, 0x98, 0x82, 0x0C);
> + mipi_dsi_dcs_write_seq(dsi, 0x00, 0x02);
> + mipi_dsi_dcs_write_seq(dsi, 0x01, 0x00);
> + mipi_dsi_dcs_write_seq(dsi, 0x02, 0x03);
> + mipi_dsi_dcs_write_seq(dsi, 0x03, 0x01);
> + mipi_dsi_dcs_write_seq(dsi, 0x04, 0x03);
> + mipi_dsi_dcs_write_seq(dsi, 0x05, 0x02);
> + mipi_dsi_dcs_write_seq(dsi, 0x06, 0x04);
> + mipi_dsi_dcs_write_seq(dsi, 0x07, 0x03);
> + mipi_dsi_dcs_write_seq(dsi, 0x08, 0x03);
> + mipi_dsi_dcs_write_seq(dsi, 0x09, 0x04);
> + mipi_dsi_dcs_write_seq(dsi, 0x0A, 0x04);
> + mipi_dsi_dcs_write_seq(dsi, 0x0B, 0x05);
> + mipi_dsi_dcs_write_seq(dsi, 0x0C, 0x04);
> + mipi_dsi_dcs_write_seq(dsi, 0x0D, 0x06);
> + mipi_dsi_dcs_write_seq(dsi, 0x0E, 0x05);
> + mipi_dsi_dcs_write_seq(dsi, 0x0F, 0x07);
> + mipi_dsi_dcs_write_seq(dsi, 0x10, 0x04);
> + mipi_dsi_dcs_write_seq(dsi, 0x11, 0x08);
> + mipi_dsi_dcs_write_seq(dsi, 0x12, 0x05);
> + mipi_dsi_dcs_write_seq(dsi, 0x13, 0x09);
> + mipi_dsi_dcs_write_seq(dsi, 0x14, 0x05);
> + mipi_dsi_dcs_write_seq(dsi, 0x15, 0x0A);
> + mipi_dsi_dcs_write_seq(dsi, 0x16, 0x06);
> + mipi_dsi_dcs_write_seq(dsi, 0x17, 0x0B);
> + mipi_dsi_dcs_write_seq(dsi, 0x18, 0x05);
> + mipi_dsi_dcs_write_seq(dsi, 0x19, 0x0C);
> + mipi_dsi_dcs_write_seq(dsi, 0x1A, 0x06);
> + mipi_dsi_dcs_write_seq(dsi, 0x1B, 0x0D);
> + mipi_dsi_dcs_write_seq(dsi, 0x1C, 0x06);
> + mipi_dsi_dcs_write_seq(dsi, 0x1D, 0x0E);
> + mipi_dsi_dcs_write_seq(dsi, 0x1E, 0x07);
> + mipi_dsi_dcs_write_seq(dsi, 0x1F, 0x0F);
> + mipi_dsi_dcs_write_seq(dsi, 0x20, 0x06);
> + mipi_dsi_dcs_write_seq(dsi, 0x21, 0x10);
> + mipi_dsi_dcs_write_seq(dsi, 0x22, 0x07);
> + mipi_dsi_dcs_write_seq(dsi, 0x23, 0x11);
> + mipi_dsi_dcs_write_seq(dsi, 0x24, 0x07);
> + mipi_dsi_dcs_write_seq(dsi, 0x25, 0x12);
> + mipi_dsi_dcs_write_seq(dsi, 0x26, 0x08);
> + mipi_dsi_dcs_write_seq(dsi, 0x27, 0x13);
> + mipi_dsi_dcs_write_seq(dsi, 0x28, 0x07);
> + mipi_dsi_dcs_write_seq(dsi, 0x29, 0x14);
> + mipi_dsi_dcs_write_seq(dsi, 0x2A, 0x08);
> + mipi_dsi_dcs_write_seq(dsi, 0x2B, 0x15);
> + mipi_dsi_dcs_write_seq(dsi, 0x2C, 0x08);
> + mipi_dsi_dcs_write_seq(dsi, 0x2D, 0x16);
> + mipi_dsi_dcs_write_seq(dsi, 0x2E, 0x09);
> + mipi_dsi_dcs_write_seq(dsi, 0x2F, 0x17);
> + mipi_dsi_dcs_write_seq(dsi, 0x30, 0x08);
> + mipi_dsi_dcs_write_seq(dsi, 0x31, 0x18);
> + mipi_dsi_dcs_write_seq(dsi, 0x32, 0x09);
> + mipi_dsi_dcs_write_seq(dsi, 0x33, 0x19);
> + mipi_dsi_dcs_write_seq(dsi, 0x34, 0x09);
> + mipi_dsi_dcs_write_seq(dsi, 0x35, 0x1A);
> + mipi_dsi_dcs_write_seq(dsi, 0x36, 0x0A);
> + mipi_dsi_dcs_write_seq(dsi, 0x37, 0x1B);
> + mipi_dsi_dcs_write_seq(dsi, 0x38, 0x0A);
> + mipi_dsi_dcs_write_seq(dsi, 0x39, 0x1C);
> + mipi_dsi_dcs_write_seq(dsi, 0x3A, 0x0A);
> + mipi_dsi_dcs_write_seq(dsi, 0x3B, 0x1D);
> + mipi_dsi_dcs_write_seq(dsi, 0x3C, 0x0A);
> + mipi_dsi_dcs_write_seq(dsi, 0x3D, 0x1E);
> + mipi_dsi_dcs_write_seq(dsi, 0x3E, 0x0A);
> + mipi_dsi_dcs_write_seq(dsi, 0x3F, 0x1F);
> +
> + mipi_dsi_dcs_write_seq(dsi, 0xFF, 0x98, 0x82, 0x04);
> + mipi_dsi_dcs_write_seq(dsi, 0xBA, 0x01);
> +
> + mipi_dsi_dcs_write_seq(dsi, 0xFF, 0x98, 0x82, 0x0E);
> + mipi_dsi_dcs_write_seq(dsi, 0x02, 0x0C);
> + mipi_dsi_dcs_write_seq(dsi, 0x20, 0x10);
> + mipi_dsi_dcs_write_seq(dsi, 0x25, 0x16);
> + mipi_dsi_dcs_write_seq(dsi, 0x26, 0xE0);
> + mipi_dsi_dcs_write_seq(dsi, 0x27, 0x00);
> + mipi_dsi_dcs_write_seq(dsi, 0x29, 0x71);
> + mipi_dsi_dcs_write_seq(dsi, 0x2A, 0x46);
> + mipi_dsi_dcs_write_seq(dsi, 0x2B, 0x1F);
> + mipi_dsi_dcs_write_seq(dsi, 0x2D, 0xC7);
> + mipi_dsi_dcs_write_seq(dsi, 0x31, 0x02);
> + mipi_dsi_dcs_write_seq(dsi, 0x32, 0xDF);
> + mipi_dsi_dcs_write_seq(dsi, 0x33, 0x5A);
> + mipi_dsi_dcs_write_seq(dsi, 0x34, 0xC0);
> + mipi_dsi_dcs_write_seq(dsi, 0x35, 0x5A);
> + mipi_dsi_dcs_write_seq(dsi, 0x36, 0xC0);
> + mipi_dsi_dcs_write_seq(dsi, 0x38, 0x65);
> + mipi_dsi_dcs_write_seq(dsi, 0x80, 0x3E);
> + mipi_dsi_dcs_write_seq(dsi, 0x81, 0xA0);
> + mipi_dsi_dcs_write_seq(dsi, 0xB0, 0x01);
> + mipi_dsi_dcs_write_seq(dsi, 0xB1, 0xCC);
> + mipi_dsi_dcs_write_seq(dsi, 0xC0, 0x12);
> + mipi_dsi_dcs_write_seq(dsi, 0xC2, 0xCC);
> + mipi_dsi_dcs_write_seq(dsi, 0xC3, 0xCC);
> + mipi_dsi_dcs_write_seq(dsi, 0xC4, 0xCC);
> + mipi_dsi_dcs_write_seq(dsi, 0xC5, 0xCC);
> + mipi_dsi_dcs_write_seq(dsi, 0xC6, 0xCC);
> + mipi_dsi_dcs_write_seq(dsi, 0xC7, 0xCC);
> + mipi_dsi_dcs_write_seq(dsi, 0xC8, 0xCC);
> + mipi_dsi_dcs_write_seq(dsi, 0xC9, 0xCC);
> + mipi_dsi_dcs_write_seq(dsi, 0x30, 0x00);
> + mipi_dsi_dcs_write_seq(dsi, 0x00, 0x81);
> + mipi_dsi_dcs_write_seq(dsi, 0x08, 0x02);
> + mipi_dsi_dcs_write_seq(dsi, 0x09, 0x00);
> + mipi_dsi_dcs_write_seq(dsi, 0x07, 0x21);
> + mipi_dsi_dcs_write_seq(dsi, 0x04, 0x10);
> +
> + mipi_dsi_dcs_write_seq(dsi, 0xFF, 0x98, 0x82, 0x1E);
> + mipi_dsi_dcs_write_seq(dsi, 0x60, 0x00);
> + mipi_dsi_dcs_write_seq(dsi, 0x64, 0x00);
> + mipi_dsi_dcs_write_seq(dsi, 0x6D, 0x00);
> +
> + mipi_dsi_dcs_write_seq(dsi, 0xFF, 0x98, 0x82, 0x0B);
> + mipi_dsi_dcs_write_seq(dsi, 0xA6, 0x44);
> + mipi_dsi_dcs_write_seq(dsi, 0xA7, 0xB6);
> + mipi_dsi_dcs_write_seq(dsi, 0xA8, 0x03);
> + mipi_dsi_dcs_write_seq(dsi, 0xA9, 0x03);
> + mipi_dsi_dcs_write_seq(dsi, 0xAA, 0x51);
> + mipi_dsi_dcs_write_seq(dsi, 0xAB, 0x51);
> + mipi_dsi_dcs_write_seq(dsi, 0xAC, 0x04);
> + mipi_dsi_dcs_write_seq(dsi, 0xBD, 0x92);
> + mipi_dsi_dcs_write_seq(dsi, 0xBE, 0xA1);
> +
> + mipi_dsi_dcs_write_seq(dsi, 0xFF, 0x98, 0x82, 0x05);
> + mipi_dsi_dcs_write_seq(dsi, 0x86, 0x87);
> +
> + mipi_dsi_dcs_write_seq(dsi, 0xFF, 0x98, 0x82, 0x06);
> + mipi_dsi_dcs_write_seq(dsi, 0x92, 0x22);
> +
> + mipi_dsi_dcs_write_seq(dsi, 0xFF, 0x98, 0x82, 0x00);
> +
> + ret = mipi_dsi_dcs_exit_sleep_mode(dsi);
> + if (ret)
> + return ret;
> + msleep(120);
> + ret = mipi_dsi_dcs_set_display_on(dsi);
> + if (ret)
> + return ret;
> + msleep(20);
> +
> + return 0;
> +};
> +
> +static inline struct ili9882t *to_ili9882t(struct drm_panel *panel)
> +{
> + return container_of(panel, struct ili9882t, base);
> +}
> +
> +static int ili9882t_enter_sleep_mode(struct ili9882t *ili)
> +{
> + struct mipi_dsi_device *dsi = ili->dsi;
> + int ret;
> +
> + dsi->mode_flags &= ~MIPI_DSI_MODE_LPM;
> +
> + ret = mipi_dsi_dcs_set_display_off(dsi);
> + if (ret < 0)
> + return ret;
> +
> + ret = mipi_dsi_dcs_enter_sleep_mode(dsi);
> + if (ret < 0)
> + return ret;
> +
> + return 0;
> +}
> +
> +static int ili9882t_disable(struct drm_panel *panel)
> +{
> + struct ili9882t *ili = to_ili9882t(panel);
> + int ret;
> +
> + ret = ili9882t_enter_sleep_mode(ili);
> + if (ret < 0) {
> + dev_err(panel->dev, "failed to set panel off: %d\n", ret);
> + return ret;
> + }
> +
> + msleep(150);
> +
> + return 0;
> +}
> +
> +static int ili9882t_unprepare(struct drm_panel *panel)
> +{
> + struct ili9882t *ili = to_ili9882t(panel);
> +
> + if (ili->desc->discharge_on_disable) {
> + regulator_disable(ili->avee);
> + regulator_disable(ili->avdd);
> + usleep_range(5000, 7000);
> + gpiod_set_value(ili->enable_gpio, 0);
> + usleep_range(5000, 7000);
> + regulator_disable(ili->pp1800);
> + regulator_disable(ili->pp3300);
> + } else {
> + gpiod_set_value(ili->enable_gpio, 0);
> + usleep_range(1000, 2000);
> + regulator_disable(ili->avee);
> + regulator_disable(ili->avdd);
> + usleep_range(5000, 7000);
> + regulator_disable(ili->pp1800);
> + regulator_disable(ili->pp3300);
> + }
> +
> + return 0;
> +}
> +
> +static int ili9882t_prepare(struct drm_panel *panel)
> +{
> + struct ili9882t *ili = to_ili9882t(panel);
> + int ret;
> +
> + gpiod_set_value(ili->enable_gpio, 0);
> + usleep_range(1000, 1500);
> +
> + ret = regulator_enable(ili->pp3300);
> + if (ret < 0)
> + return ret;
> +
> + ret = regulator_enable(ili->pp1800);
> + if (ret < 0)
> + return ret;
> +
> + usleep_range(3000, 5000);
> +
> + ret = regulator_enable(ili->avdd);
> + if (ret < 0)
> + goto poweroff1v8;
> + ret = regulator_enable(ili->avee);
> + if (ret < 0)
> + goto poweroffavdd;
> +
> + usleep_range(10000, 11000);
> +
> + if (ili->desc->lp11_before_reset) {
> + mipi_dsi_dcs_nop(ili->dsi);
> + usleep_range(1000, 2000);
> + }
> + gpiod_set_value(ili->enable_gpio, 1);
> + usleep_range(1000, 2000);
> + gpiod_set_value(ili->enable_gpio, 0);
> + usleep_range(1000, 2000);

Can the RST delay be increased to 40ms-50ms?
At present, we have found that there may be a problem of blurred
screen during fast sleep. The direct cause of the blurred screen
is that the IC does not receive 0x28/0x10. Because of the
particularity of the IC, before the panel enters sleep hid must
stop scanning, i2c_hid_core_suspend before ili9882t_disable.
This doesn't look very spec-compliant. So in order to solve this
problem, the IC can handle it through the exception mechanism when
it cannot receive 0X28/0X10. Handling exceptions requires a reset
50ms delay. Refer to vendor detailed analysis [1].

[1]: https://github.com/ILITEK-LoganLin/Document/tree/main/ILITEK_Power_Sequence

Looking forward for your reply, thank you.

> + gpiod_set_value(ili->enable_gpio, 1);
> + usleep_range(6000, 10000);
> +
> + if (ili->desc->init) {
> + ret = ili->desc->init(ili->dsi);
> + if (ret < 0) {
> + dev_err(panel->dev, "failed to init panel: %d\n", ret);
> + goto poweroff;
> + }
> + }
> +
> + return 0;
> +
> +poweroff:
> + regulator_disable(ili->avee);
> +poweroffavdd:
> + regulator_disable(ili->avdd);
> +poweroff1v8:
> + usleep_range(5000, 7000);
> + regulator_disable(ili->pp1800);
> + gpiod_set_value(ili->enable_gpio, 0);
> +
> + return ret;
> +}
> +
> +static int ili9882t_enable(struct drm_panel *panel)
> +{
> + msleep(130);
> + return 0;
> +}
> +
> +static const struct drm_display_mode starry_ili9882t_default_mode = {
> + .clock = 165280,
> + .hdisplay = 1200,
> + .hsync_start = 1200 + 32,
> + .hsync_end = 1200 + 32 + 30,
> + .htotal = 1200 + 32 + 30 + 32,
> + .vdisplay = 1920,
> + .vsync_start = 1920 + 68,
> + .vsync_end = 1920 + 68 + 2,
> + .vtotal = 1920 + 68 + 2 + 10,
> + .type = DRM_MODE_TYPE_DRIVER | DRM_MODE_TYPE_PREFERRED,
> +};
> +
> +static const struct panel_desc starry_ili9882t_desc = {
> + .modes = &starry_ili9882t_default_mode,
> + .bpc = 8,
> + .size = {
> + .width_mm = 141,
> + .height_mm = 226,
> + },
> + .lanes = 4,
> + .format = MIPI_DSI_FMT_RGB888,
> + .mode_flags = MIPI_DSI_MODE_VIDEO | MIPI_DSI_MODE_VIDEO_SYNC_PULSE |
> + MIPI_DSI_MODE_LPM,
> + .init = starry_ili9882t_init,
> + .lp11_before_reset = true,
> +};
> +
> +static int ili9882t_get_modes(struct drm_panel *panel,
> + struct drm_connector *connector)
> +{
> + struct ili9882t *ili = to_ili9882t(panel);
> + const struct drm_display_mode *m = ili->desc->modes;
> + struct drm_display_mode *mode;
> +
> + mode = drm_mode_duplicate(connector->dev, m);
> + if (!mode) {
> + dev_err(panel->dev, "failed to add mode %ux%u@%u\n",
> + m->hdisplay, m->vdisplay, drm_mode_vrefresh(m));
> + return -ENOMEM;
> + }
> +
> + mode->type = DRM_MODE_TYPE_DRIVER | DRM_MODE_TYPE_PREFERRED;
> + drm_mode_set_name(mode);
> + drm_mode_probed_add(connector, mode);
> +
> + connector->display_info.width_mm = ili->desc->size.width_mm;
> + connector->display_info.height_mm = ili->desc->size.height_mm;
> + connector->display_info.bpc = ili->desc->bpc;
> + /*
> + * TODO: Remove once all drm drivers call
> + * drm_connector_set_orientation_from_panel()
> + */
> + drm_connector_set_panel_orientation(connector, ili->orientation);
> +
> + return 1;
> +}
> +
> +static enum drm_panel_orientation ili9882t_get_orientation(struct drm_panel *panel)
> +{
> + struct ili9882t *ili = to_ili9882t(panel);
> +
> + return ili->orientation;
> +}
> +
> +static const struct drm_panel_funcs ili9882t_funcs = {
> + .disable = ili9882t_disable,
> + .unprepare = ili9882t_unprepare,
> + .prepare = ili9882t_prepare,
> + .enable = ili9882t_enable,
> + .get_modes = ili9882t_get_modes,
> + .get_orientation = ili9882t_get_orientation,
> +};
> +
> +static int ili9882t_add(struct ili9882t *ili)
> +{
> + struct device *dev = &ili->dsi->dev;
> + int err;
> +
> + ili->avdd = devm_regulator_get(dev, "avdd");
> + if (IS_ERR(ili->avdd))
> + return PTR_ERR(ili->avdd);
> +
> + ili->avee = devm_regulator_get(dev, "avee");
> + if (IS_ERR(ili->avee))
> + return PTR_ERR(ili->avee);
> +
> + ili->pp3300 = devm_regulator_get(dev, "pp3300");
> + if (IS_ERR(ili->pp3300))
> + return PTR_ERR(ili->pp3300);
> +
> + ili->pp1800 = devm_regulator_get(dev, "pp1800");
> + if (IS_ERR(ili->pp1800))
> + return PTR_ERR(ili->pp1800);
> +
> + ili->enable_gpio = devm_gpiod_get(dev, "enable", GPIOD_OUT_LOW);
> + if (IS_ERR(ili->enable_gpio)) {
> + dev_err(dev, "cannot get reset-gpios %ld\n",
> + PTR_ERR(ili->enable_gpio));
> + return PTR_ERR(ili->enable_gpio);
> + }
> +
> + gpiod_set_value(ili->enable_gpio, 0);
> +
> + drm_panel_init(&ili->base, dev, &ili9882t_funcs,
> + DRM_MODE_CONNECTOR_DSI);
> + err = of_drm_get_panel_orientation(dev->of_node, &ili->orientation);
> + if (err < 0) {
> + dev_err(dev, "%pOF: failed to get orientation %d\n", dev->of_node, err);
> + return err;
> + }
> +
> + err = drm_panel_of_backlight(&ili->base);
> + if (err)
> + return err;
> +
> + ili->base.funcs = &ili9882t_funcs;
> + ili->base.dev = &ili->dsi->dev;
> +
> + drm_panel_add(&ili->base);
> +
> + return 0;
> +}
> +
> +static int ili9882t_probe(struct mipi_dsi_device *dsi)
> +{
> + struct ili9882t *ili;
> + int ret;
> + const struct panel_desc *desc;
> +
> + ili = devm_kzalloc(&dsi->dev, sizeof(*ili), GFP_KERNEL);
> + if (!ili)
> + return -ENOMEM;
> +
> + desc = of_device_get_match_data(&dsi->dev);
> + dsi->lanes = desc->lanes;
> + dsi->format = desc->format;
> + dsi->mode_flags = desc->mode_flags;
> + ili->desc = desc;
> + ili->dsi = dsi;
> + ret = ili9882t_add(ili);
> + if (ret < 0)
> + return ret;
> +
> + mipi_dsi_set_drvdata(dsi, ili);
> +
> + ret = mipi_dsi_attach(dsi);
> + if (ret)
> + drm_panel_remove(&ili->base);
> +
> + return ret;
> +}
> +
> +static void ili9882t_shutdown(struct mipi_dsi_device *dsi)
> +{
> + struct ili9882t *ili = mipi_dsi_get_drvdata(dsi);
> +
> + drm_panel_disable(&ili->base);
> + drm_panel_unprepare(&ili->base);
> +}
> +
> +static void ili9882t_remove(struct mipi_dsi_device *dsi)
> +{
> + struct ili9882t *ili = mipi_dsi_get_drvdata(dsi);
> + int ret;
> +
> + ili9882t_shutdown(dsi);
> +
> + ret = mipi_dsi_detach(dsi);
> + if (ret < 0)
> + dev_err(&dsi->dev, "failed to detach from DSI host: %d\n", ret);
> +
> + if (ili->base.dev)
> + drm_panel_remove(&ili->base);
> +}
> +
> +static const struct of_device_id ili9882t_of_match[] = {
> + { .compatible = "starry,ili9882t",
> + .data = &starry_ili9882t_desc
> + },
> + { /* sentinel */ }
> +};
> +MODULE_DEVICE_TABLE(of, ili9882t_of_match);
> +
> +static struct mipi_dsi_driver ili9882t_driver = {
> + .driver = {
> + .name = "panel-ili9882t",
> + .of_match_table = ili9882t_of_match,
> + },
> + .probe = ili9882t_probe,
> + .remove = ili9882t_remove,
> + .shutdown = ili9882t_shutdown,
> +};
> +module_mipi_dsi_driver(ili9882t_driver);
> +
> +MODULE_AUTHOR("Linus Walleij <linus.walleij@xxxxxxxxxx>");
> +MODULE_DESCRIPTION("Ilitek ILI9882T-based panels driver");
> +MODULE_LICENSE("GPL");
>
> --
> 2.34.1
>