Re: [PATCH 0/2] drm/panel: Add support for the Sitronix ST7789V

[Noralf TrÃnnes]

Den 03.02.2017 10.59, skrev Maxime Ripard:
> Hi,
>
> Here is an attempt at supporting the ST7789V LCD controller from Sitronix.

What happens if there's another panel driven by ST7789V that needs
a different controller initialization?
Maybe it's better to name it after the panel, not the controller.

> It is controlled through an SPI bus, with a twist, since each byte sent
> must be prefixed by a bit, which needs an 9-bits-per-word SPI controller,
> which is quite rare. Else, you would need to bitbang it.

It's possible to support SPI controllers that can't do 9-bit by using
8-bit mode and blocks of 9 bytes to send 8x 9-bit words padded with
zeroes. Zero is MIPI DCS no-op command.

Noralf.

> Let me know what you think,
> Maxime
>
> Maxime Ripard (2):
>    dt-bindings: display: panel: Add bindings for the Sitronix ST7789V panel
>    drm/panel: Add driver for sitronix ST7789V panel
>
>   Documentation/devicetree/bindings/display/panel/sitronix,st7789v.txt |  36 ++++++-
>   drivers/gpu/drm/panel/Kconfig                                        |   4 +-
>   drivers/gpu/drm/panel/Makefile                                       |   1 +-
>   drivers/gpu/drm/panel/panel-sitronix-st7789v.c                       | 435 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++-
>   4 files changed, 476 insertions(+), 0 deletions(-)
>   create mode 100644 Documentation/devicetree/bindings/display/panel/sitronix,st7789v.txt
>   create mode 100644 drivers/gpu/drm/panel/panel-sitronix-st7789v.c
>
> base-commit: 2a6e628efb37432a83bb42b00e3c403b1d8873dd