So following up on this:
We should state in the commit message that this driver is for all displays using the Sitronix ST770x display controllers. The driver should be named panel-sitronix-st770x.c.
On Thu, Mar 19, 2020 at 3:08 PM Linus Walleij linus.walleij@linaro.org wrote:
+/* Manufacturer specific Commands send via DSI */ +#define ST7703_CMD_ALL_PIXEL_OFF 0x22 +#define ST7703_CMD_ALL_PIXEL_ON 0x23 +#define ST7703_CMD_SETDISP 0xB2 +#define ST7703_CMD_SETRGBIF 0xB3 +#define ST7703_CMD_SETCYC 0xB4 +#define ST7703_CMD_SETBGP 0xB5 +#define ST7703_CMD_SETVCOM 0xB6 +#define ST7703_CMD_SETOTP 0xB7 +#define ST7703_CMD_SETPOWER_EXT 0xB8 +#define ST7703_CMD_SETEXTC 0xB9 +#define ST7703_CMD_SETMIPI 0xBA +#define ST7703_CMD_SETVDC 0xBC +#define ST7703_CMD_SETSCR 0xC0 +#define ST7703_CMD_SETPOWER 0xC1 +#define ST7703_CMD_UNK_C6 0xC6 +#define ST7703_CMD_SETPANEL 0xCC +#define ST7703_CMD_SETGAMMA 0xE0 +#define ST7703_CMD_SETEQ 0xE3 +#define ST7703_CMD_SETGIP1 0xE9 +#define ST7703_CMD_SETGIP2 0xEA
I should have seen the ST7703 prefix shouldn't I...
This actually looks very much like an Ilitek display controller. Some commands are clearly identical to Ilitek ILI9342: http://www.ampdisplay.com/documents/pdf/ILI9342_DS_V008_20100331.pdf
I'm still wondering about the apparent similarity between ST770x and Ilitek ILI9342, haha :D
- Try to determine what the actual display controller
is. I think it is some Ilitek.
OK so this is Sitronix ST770x.
- Write a panel-ilitek-ili9342.c (if that is the actual controller)
and parameterize it for this display controller the same way we do in e.g. panel-novatek-nt35510.c or panel-ilitek-ili9322.c, so you use the compatible string to set up the actual per-display settings for this display controller.
This should be panel-sitronix-st770x.c
Yours, Linus Walleij