* [PATCH v4 0/3] Support for tft displays based on ilitek,ili9486 @ 2020-01-27 14:24 Kamlesh Gurudasani 2020-01-27 14:25 ` [PATCH v4 1/3] dt-bindings: add vendor prefix for OzzMaker and Waveshare Electronics Kamlesh Gurudasani ` (2 more replies) 0 siblings, 3 replies; 10+ messages in thread From: Kamlesh Gurudasani @ 2020-01-27 14:24 UTC (permalink / raw) To: noralf, Rob Herring, Sam Ravnborg Cc: devicetree, dri-devel, Kamlesh Gurudasani The goal of this series is to get the displays based on ilitek,ili9486 working. Ozzmaker, Piscreen and waveshare,rpi-lcd-35 are such displays. v2 changes: * Changing file from txt to yaml format * removed ilitek,ili9486 from compatible string * assignment of dbi_command before registration * made dc and reset gpio compulsory * typos and unwanted comments removed * changed the name of function which were display specific * arranged the Makefile entries in alphabetical order v3 changes: * added vendor prefix patch to this series instead of separate patch v4 changes: * removing checkpatch warnings Kamlesh Gurudasani (3): dt-bindings: add vendor prefix for OzzMaker and Waveshare Electronics dt-bindings: add binding for tft displays based on ilitek,ili9486 drm/tinydrm: add support for tft displays based on ilitek,ili9486 .../bindings/display/ilitek,ili9486.yaml | 79 ++++++ .../devicetree/bindings/vendor-prefixes.yaml | 4 + MAINTAINERS | 7 + drivers/gpu/drm/tiny/Kconfig | 14 + drivers/gpu/drm/tiny/Makefile | 1 + drivers/gpu/drm/tiny/ili9486.c | 283 +++++++++++++++++++++ 6 files changed, 388 insertions(+) create mode 100644 Documentation/devicetree/bindings/display/ilitek,ili9486.yaml create mode 100644 drivers/gpu/drm/tiny/ili9486.c -- 2.7.4 ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v4 1/3] dt-bindings: add vendor prefix for OzzMaker and Waveshare Electronics 2020-01-27 14:24 [PATCH v4 0/3] Support for tft displays based on ilitek,ili9486 Kamlesh Gurudasani @ 2020-01-27 14:25 ` Kamlesh Gurudasani 2020-01-29 19:48 ` Sam Ravnborg 2020-02-05 17:33 ` Rob Herring 2020-01-27 14:26 ` [PATCH v4 2/3] dt-bindings: add binding for tft displays based on ilitek,ili9486 Kamlesh Gurudasani 2020-01-27 14:26 ` [PATCH v4 3/3] drm/tinydrm: add support " Kamlesh Gurudasani 2 siblings, 2 replies; 10+ messages in thread From: Kamlesh Gurudasani @ 2020-01-27 14:25 UTC (permalink / raw) To: noralf, Rob Herring, Sam Ravnborg Cc: devicetree, dri-devel, Kamlesh Gurudasani Add vendor prefix for OzzMaker [1] and Waveshare Electronics [2] Both are display manufacturers [1] https://ozzmaker.com/about/ [2] https://www.waveshare.com/contact_us Signed-off-by: Kamlesh Gurudasani <kamlesh.gurudasani@gmail.com> --- Documentation/devicetree/bindings/vendor-prefixes.yaml | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/Documentation/devicetree/bindings/vendor-prefixes.yaml b/Documentation/devicetree/bindings/vendor-prefixes.yaml index ac4804d..dfb926d 100644 --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml @@ -721,6 +721,8 @@ patternProperties: description: OmniVision Technologies "^oxsemi,.*": description: Oxford Semiconductor, Ltd. + "^ozzmaker,.*": + description: OzzMaker "^panasonic,.*": description: Panasonic Corporation "^parade,.*": @@ -1038,6 +1040,8 @@ patternProperties: description: Vision Optical Technology Co., Ltd. "^vxt,.*": description: VXT Ltd + "^waveshare,.*": + description: Waveshare Electronics "^wd,.*": description: Western Digital Corp. "^wetek,.*": -- 2.7.4 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH v4 1/3] dt-bindings: add vendor prefix for OzzMaker and Waveshare Electronics 2020-01-27 14:25 ` [PATCH v4 1/3] dt-bindings: add vendor prefix for OzzMaker and Waveshare Electronics Kamlesh Gurudasani @ 2020-01-29 19:48 ` Sam Ravnborg 2020-02-05 17:33 ` Rob Herring 1 sibling, 0 replies; 10+ messages in thread From: Sam Ravnborg @ 2020-01-29 19:48 UTC (permalink / raw) To: Kamlesh Gurudasani; +Cc: noralf, Rob Herring, devicetree, dri-devel Hi Kamlesh On Mon, Jan 27, 2020 at 07:55:33PM +0530, Kamlesh Gurudasani wrote: > Add vendor prefix for OzzMaker [1] and Waveshare Electronics [2] > Both are display manufacturers > > [1] https://ozzmaker.com/about/ > [2] https://www.waveshare.com/contact_us > > Signed-off-by: Kamlesh Gurudasani <kamlesh.gurudasani@gmail.com> Acked-by: Sam Ravnborg <sam@ravnborg.org> > --- > Documentation/devicetree/bindings/vendor-prefixes.yaml | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/Documentation/devicetree/bindings/vendor-prefixes.yaml b/Documentation/devicetree/bindings/vendor-prefixes.yaml > index ac4804d..dfb926d 100644 > --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml > +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml > @@ -721,6 +721,8 @@ patternProperties: > description: OmniVision Technologies > "^oxsemi,.*": > description: Oxford Semiconductor, Ltd. > + "^ozzmaker,.*": > + description: OzzMaker > "^panasonic,.*": > description: Panasonic Corporation > "^parade,.*": > @@ -1038,6 +1040,8 @@ patternProperties: > description: Vision Optical Technology Co., Ltd. > "^vxt,.*": > description: VXT Ltd > + "^waveshare,.*": > + description: Waveshare Electronics > "^wd,.*": > description: Western Digital Corp. > "^wetek,.*": > -- > 2.7.4 ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v4 1/3] dt-bindings: add vendor prefix for OzzMaker and Waveshare Electronics 2020-01-27 14:25 ` [PATCH v4 1/3] dt-bindings: add vendor prefix for OzzMaker and Waveshare Electronics Kamlesh Gurudasani 2020-01-29 19:48 ` Sam Ravnborg @ 2020-02-05 17:33 ` Rob Herring 1 sibling, 0 replies; 10+ messages in thread From: Rob Herring @ 2020-02-05 17:33 UTC (permalink / raw) To: Kamlesh Gurudasani Cc: noralf, Sam Ravnborg, devicetree, dri-devel, Kamlesh Gurudasani On Mon, 27 Jan 2020 19:55:33 +0530, Kamlesh Gurudasani wrote: > Add vendor prefix for OzzMaker [1] and Waveshare Electronics [2] > Both are display manufacturers > > [1] https://ozzmaker.com/about/ > [2] https://www.waveshare.com/contact_us > > Signed-off-by: Kamlesh Gurudasani <kamlesh.gurudasani@gmail.com> > --- > Documentation/devicetree/bindings/vendor-prefixes.yaml | 4 ++++ > 1 file changed, 4 insertions(+) > Applied, thanks. Rob ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v4 2/3] dt-bindings: add binding for tft displays based on ilitek,ili9486 2020-01-27 14:24 [PATCH v4 0/3] Support for tft displays based on ilitek,ili9486 Kamlesh Gurudasani 2020-01-27 14:25 ` [PATCH v4 1/3] dt-bindings: add vendor prefix for OzzMaker and Waveshare Electronics Kamlesh Gurudasani @ 2020-01-27 14:26 ` Kamlesh Gurudasani 2020-01-29 20:14 ` Sam Ravnborg 2020-01-31 14:23 ` Rob Herring 2020-01-27 14:26 ` [PATCH v4 3/3] drm/tinydrm: add support " Kamlesh Gurudasani 2 siblings, 2 replies; 10+ messages in thread From: Kamlesh Gurudasani @ 2020-01-27 14:26 UTC (permalink / raw) To: noralf, Rob Herring, Sam Ravnborg Cc: devicetree, dri-devel, Kamlesh Gurudasani This binding is for the tft displays based on ilitek,ili9486. ozzmaker,piscreen and waveshare,rpi-lcd-35 are such displays Signed-off-by: Kamlesh Gurudasani <kamlesh.gurudasani@gmail.com> --- v2 changes: * Changing file from txt to yaml format * removed ilitek,ili9486 from compatible string v3 changes: * no changes v4 changes: * no changes --- .../bindings/display/ilitek,ili9486.yaml | 79 ++++++++++++++++++++++ 1 file changed, 79 insertions(+) create mode 100644 Documentation/devicetree/bindings/display/ilitek,ili9486.yaml diff --git a/Documentation/devicetree/bindings/display/ilitek,ili9486.yaml b/Documentation/devicetree/bindings/display/ilitek,ili9486.yaml new file mode 100644 index 0000000..dd306c88 --- /dev/null +++ b/Documentation/devicetree/bindings/display/ilitek,ili9486.yaml @@ -0,0 +1,79 @@ +# SPDX-License-Identifier: GPL-2.0-only +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/display/ilitek,ili9486.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Ilitek ILI9486 display panels device tree bindings + +maintainers: + - Kamlesh Gurudasani <kamlesh.gurudasani@gmail.com> + +description: + This binding is for display panels using an Ilitek ILI9486 controller in SPI + mode. + +allOf: + - $ref: panel/panel-common.yaml# + +properties: + compatible: + oneOf: + - description: + Waveshare 3.5" 320x480 Color TFT LCD + items: + - enum: + - waveshare,rpi-lcd-35 + - const: ilitek,ili9486 + - description: + Ozzmaker 3.5" 320x480 Color TFT LCD + items: + - enum: + - ozzmaker,piscreen + - const: ilitek,ili9486 + + spi-max-frequency: + maximum: 32000000 + + dc-gpios: + maxItems: 1 + description: Display data/command selection (D/CX) + + backlight: true + reg: true + reset-gpios: true + rotation: true + +required: + - compatible + - reg + - dc-gpios + - reset-gpios + +additionalProperties: false + +examples: + - | + #include <dt-bindings/gpio/gpio.h> + + backlight: backlight { + compatible = "gpio-backlight"; + gpios = <&gpio 22 GPIO_ACTIVE_HIGH>; + }; + spi { + #address-cells = <1>; + #size-cells = <0>; + + + display@0{ + compatible = "waveshare,rpi-lcd-35"; + reg = <0>; + spi-max-frequency = <32000000>; + dc-gpios = <&gpio0 24 GPIO_ACTIVE_HIGH>; + reset-gpios = <&gpio0 25 GPIO_ACTIVE_HIGH>; + rotation = <180>; + backlight = <&backlight>; + }; + }; + +... -- 2.7.4 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH v4 2/3] dt-bindings: add binding for tft displays based on ilitek,ili9486 2020-01-27 14:26 ` [PATCH v4 2/3] dt-bindings: add binding for tft displays based on ilitek,ili9486 Kamlesh Gurudasani @ 2020-01-29 20:14 ` Sam Ravnborg 2020-01-31 14:23 ` Rob Herring 1 sibling, 0 replies; 10+ messages in thread From: Sam Ravnborg @ 2020-01-29 20:14 UTC (permalink / raw) To: Kamlesh Gurudasani; +Cc: noralf, Rob Herring, devicetree, dri-devel Hi Kamlesh Thanks for the conversion to DT Schema format. On Mon, Jan 27, 2020 at 07:56:14PM +0530, Kamlesh Gurudasani wrote: > This binding is for the tft displays based on ilitek,ili9486. > ozzmaker,piscreen and waveshare,rpi-lcd-35 are such displays > > Signed-off-by: Kamlesh Gurudasani <kamlesh.gurudasani@gmail.com> > --- > > v2 changes: > * Changing file from txt to yaml format > * removed ilitek,ili9486 from compatible string > > v3 changes: > * no changes > > v4 changes: > * no changes > --- > .../bindings/display/ilitek,ili9486.yaml | 79 ++++++++++++++++++++++ > 1 file changed, 79 insertions(+) > create mode 100644 Documentation/devicetree/bindings/display/ilitek,ili9486.yaml > > diff --git a/Documentation/devicetree/bindings/display/ilitek,ili9486.yaml b/Documentation/devicetree/bindings/display/ilitek,ili9486.yaml > new file mode 100644 > index 0000000..dd306c88 > --- /dev/null > +++ b/Documentation/devicetree/bindings/display/ilitek,ili9486.yaml > @@ -0,0 +1,79 @@ > +# SPDX-License-Identifier: GPL-2.0-only > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/display/ilitek,ili9486.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Ilitek ILI9486 display panels device tree bindings > + > +maintainers: > + - Kamlesh Gurudasani <kamlesh.gurudasani@gmail.com> > + > +description: > + This binding is for display panels using an Ilitek ILI9486 controller in SPI > + mode. > + > +allOf: > + - $ref: panel/panel-common.yaml# > + > +properties: > + compatible: > + oneOf: > + - description: > + Waveshare 3.5" 320x480 Color TFT LCD > + items: > + - enum: > + - waveshare,rpi-lcd-35 > + - const: ilitek,ili9486 > + - description: > + Ozzmaker 3.5" 320x480 Color TFT LCD > + items: > + - enum: > + - ozzmaker,piscreen > + - const: ilitek,ili9486 This parts looks a bit too complicated. I think the following would do the trick: compatible: items: - enum: # Waveshare 3.5" 320x480 Color TFT LCD - waveshare,rpi-lcd-35 # Ozzmaker 3.5" 320x480 Color TFT LCD - ozzmaker,piscreen - const: ilitek,ili9486 This makes it more obvious that you try to say that the binding should include one of the enum values and ilitek,ili9486 With this your example would also rightfully fail as it is missing the ilitek,ili9486 compatible. Sam > + > + spi-max-frequency: > + maximum: 32000000 > + > + dc-gpios: > + maxItems: 1 > + description: Display data/command selection (D/CX) > + > + backlight: true > + reg: true > + reset-gpios: true > + rotation: true > + > +required: > + - compatible > + - reg > + - dc-gpios > + - reset-gpios > + > +additionalProperties: false > + > +examples: > + - | > + #include <dt-bindings/gpio/gpio.h> > + > + backlight: backlight { > + compatible = "gpio-backlight"; > + gpios = <&gpio 22 GPIO_ACTIVE_HIGH>; > + }; > + spi { > + #address-cells = <1>; > + #size-cells = <0>; > + > + > + display@0{ > + compatible = "waveshare,rpi-lcd-35"; > + reg = <0>; > + spi-max-frequency = <32000000>; > + dc-gpios = <&gpio0 24 GPIO_ACTIVE_HIGH>; > + reset-gpios = <&gpio0 25 GPIO_ACTIVE_HIGH>; > + rotation = <180>; > + backlight = <&backlight>; > + }; > + }; > + > +... > -- > 2.7.4 ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v4 2/3] dt-bindings: add binding for tft displays based on ilitek,ili9486 2020-01-27 14:26 ` [PATCH v4 2/3] dt-bindings: add binding for tft displays based on ilitek,ili9486 Kamlesh Gurudasani 2020-01-29 20:14 ` Sam Ravnborg @ 2020-01-31 14:23 ` Rob Herring 1 sibling, 0 replies; 10+ messages in thread From: Rob Herring @ 2020-01-31 14:23 UTC (permalink / raw) To: Kamlesh Gurudasani Cc: noralf, Sam Ravnborg, devicetree, dri-devel, Kamlesh Gurudasani On Mon, 27 Jan 2020 19:56:14 +0530, Kamlesh Gurudasani wrote: > This binding is for the tft displays based on ilitek,ili9486. > ozzmaker,piscreen and waveshare,rpi-lcd-35 are such displays > > Signed-off-by: Kamlesh Gurudasani <kamlesh.gurudasani@gmail.com> > --- > > v2 changes: > * Changing file from txt to yaml format > * removed ilitek,ili9486 from compatible string > > v3 changes: > * no changes > > v4 changes: > * no changes > --- > .../bindings/display/ilitek,ili9486.yaml | 79 ++++++++++++++++++++++ > 1 file changed, 79 insertions(+) > create mode 100644 Documentation/devicetree/bindings/display/ilitek,ili9486.yaml > My bot found errors running 'make dt_binding_check' on your patch: Documentation/devicetree/bindings/display/simple-framebuffer.example.dts:21.16-37.11: Warning (chosen_node_is_root): /example-0/chosen: chosen node must be at root node /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/display/ilitek,ili9486.example.dt.yaml: display@0: compatible: ['waveshare,rpi-lcd-35'] is not valid under any of the given schemas (Possible causes of the failure): /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/display/ilitek,ili9486.example.dt.yaml: display@0: compatible: ['waveshare,rpi-lcd-35'] is too short /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/display/ilitek,ili9486.example.dt.yaml: display@0: compatible:0: 'waveshare,rpi-lcd-35' is not one of ['ozzmaker,piscreen'] See https://patchwork.ozlabs.org/patch/1229867 Please check and re-submit. ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v4 3/3] drm/tinydrm: add support for tft displays based on ilitek,ili9486 2020-01-27 14:24 [PATCH v4 0/3] Support for tft displays based on ilitek,ili9486 Kamlesh Gurudasani 2020-01-27 14:25 ` [PATCH v4 1/3] dt-bindings: add vendor prefix for OzzMaker and Waveshare Electronics Kamlesh Gurudasani 2020-01-27 14:26 ` [PATCH v4 2/3] dt-bindings: add binding for tft displays based on ilitek,ili9486 Kamlesh Gurudasani @ 2020-01-27 14:26 ` Kamlesh Gurudasani 2020-01-27 15:13 ` Noralf Trønnes 2020-01-29 20:15 ` Sam Ravnborg 2 siblings, 2 replies; 10+ messages in thread From: Kamlesh Gurudasani @ 2020-01-27 14:26 UTC (permalink / raw) To: noralf, Rob Herring, Sam Ravnborg Cc: devicetree, dri-devel, Kamlesh Gurudasani This adds support fot ilitek,ili9486 based displays with shift register in front of controller. Ozzmaker,Piscreen and Waveshare,rpi-lcd-35 are such displays. Signed-off-by: Kamlesh Gurudasani <kamlesh.gurudasani@gmail.com> --- v2 changes: * assignment of dbi_command before registration * made dc and reset gpio compulsory * typos and unwanted comments removed * changed the name of function which were display specific * arranged the Makefile entries in alphabetical order v3 changes: * no changes v4 changes: * no changes --- MAINTAINERS | 7 + drivers/gpu/drm/tiny/Kconfig | 14 ++ drivers/gpu/drm/tiny/Makefile | 1 + drivers/gpu/drm/tiny/ili9486.c | 283 +++++++++++++++++++++++++++++++++++++++++ 4 files changed, 305 insertions(+) create mode 100644 drivers/gpu/drm/tiny/ili9486.c diff --git a/MAINTAINERS b/MAINTAINERS index 16423f5..30dd396 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -5233,6 +5233,13 @@ S: Maintained F: drivers/gpu/drm/tiny/ili9225.c F: Documentation/devicetree/bindings/display/ilitek,ili9225.txt +DRM DRIVER FOR ILITEK ILI9486 PANELS +M: Kamlesh Gurudasani <kamlesh.gurudasani@gmail.com> +T: git git://anongit.freedesktop.org/drm/drm-misc +S: Maintained +F: drivers/gpu/drm/tiny/ili9486.c +F: Documentation/devicetree/bindings/display/ilitek,ili9486.yaml + DRM DRIVER FOR HX8357D PANELS M: Eric Anholt <eric@anholt.net> T: git git://anongit.freedesktop.org/drm/drm-misc diff --git a/drivers/gpu/drm/tiny/Kconfig b/drivers/gpu/drm/tiny/Kconfig index a866421..4160e74 100644 --- a/drivers/gpu/drm/tiny/Kconfig +++ b/drivers/gpu/drm/tiny/Kconfig @@ -47,6 +47,20 @@ config TINYDRM_ILI9341 If M is selected the module will be called ili9341. +config TINYDRM_ILI9486 + tristate "DRM support for ILI9486 display panels" + depends on DRM && SPI + select DRM_KMS_HELPER + select DRM_KMS_CMA_HELPER + select DRM_MIPI_DBI + select BACKLIGHT_CLASS_DEVICE + help + DRM driver for the following Ilitek ILI9486 panels: + * PISCREEN 3.5" 320x480 TFT (Ozzmaker 3.5") + * RPILCD 3.5" 320x480 TFT (Waveshare 3.5") + + If M is selected the module will be called ili9486. + config TINYDRM_MI0283QT tristate "DRM support for MI0283QT" depends on DRM && SPI diff --git a/drivers/gpu/drm/tiny/Makefile b/drivers/gpu/drm/tiny/Makefile index 896cf31..c96ceee 100644 --- a/drivers/gpu/drm/tiny/Makefile +++ b/drivers/gpu/drm/tiny/Makefile @@ -4,6 +4,7 @@ obj-$(CONFIG_DRM_GM12U320) += gm12u320.o obj-$(CONFIG_TINYDRM_HX8357D) += hx8357d.o obj-$(CONFIG_TINYDRM_ILI9225) += ili9225.o obj-$(CONFIG_TINYDRM_ILI9341) += ili9341.o +obj-$(CONFIG_TINYDRM_ILI9486) += ili9486.o obj-$(CONFIG_TINYDRM_MI0283QT) += mi0283qt.o obj-$(CONFIG_TINYDRM_REPAPER) += repaper.o obj-$(CONFIG_TINYDRM_ST7586) += st7586.o diff --git a/drivers/gpu/drm/tiny/ili9486.c b/drivers/gpu/drm/tiny/ili9486.c new file mode 100644 index 0000000..e960b16 --- /dev/null +++ b/drivers/gpu/drm/tiny/ili9486.c @@ -0,0 +1,283 @@ +// SPDX-License-Identifier: GPL-2.0+ +/* + * DRM driver for Ilitek ILI9486 panels + * + * Copyright 2020 Kamlesh Gurudasani <kamlesh.gurudasani@gmail.com> + */ + +#include <linux/backlight.h> +#include <linux/delay.h> +#include <linux/gpio/consumer.h> +#include <linux/module.h> +#include <linux/property.h> +#include <linux/spi/spi.h> +#include <video/mipi_display.h> + +#include <drm/drm_atomic_helper.h> +#include <drm/drm_drv.h> +#include <drm/drm_fb_helper.h> +#include <drm/drm_gem_cma_helper.h> +#include <drm/drm_gem_framebuffer_helper.h> +#include <drm/drm_mipi_dbi.h> +#include <drm/drm_modeset_helper.h> + +#define ILI9486_ITFCTR1 0xb0 +#define ILI9486_PWCTRL1 0xc2 +#define ILI9486_VMCTRL1 0xc5 +#define ILI9486_PGAMCTRL 0xe0 +#define ILI9486_NGAMCTRL 0xe1 +#define ILI9486_DGAMCTRL 0xe2 +#define ILI9486_MADCTL_BGR BIT(3) +#define ILI9486_MADCTL_MV BIT(5) +#define ILI9486_MADCTL_MX BIT(6) +#define ILI9486_MADCTL_MY BIT(7) + +/* + * The PiScreen/waveshare rpi-lcd-35 has a SPI to 16-bit parallel bus converter + * in front of the display controller. This means that 8-bit values have to be + * transferred as 16-bit. + */ +static int waveshare_command(struct mipi_dbi *mipi, u8 *cmd, u8 *par, size_t num) +{ + struct spi_device *spi = mipi->spi; + void *data = par; + u32 speed_hz; + int i, ret; + u16 *buf; + + buf = kmalloc(32 * sizeof(u16), GFP_KERNEL); + if (!buf) + return -ENOMEM; + + /* + * The displays are Raspberry Pi HATs and connected to the 8-bit only + * SPI controller, so 16-bit command and parameters need byte swapping + * before being transferred as 8-bit on the big endian SPI bus. + * Pixel data bytes have already been swapped before this function is + * called. + */ + buf[0] = cpu_to_be16(*cmd); + gpiod_set_value_cansleep(mipi->dc, 0); + speed_hz = mipi_dbi_spi_cmd_max_speed(spi, 2); + ret = mipi_dbi_spi_transfer(spi, speed_hz, 8, buf, 2); + if (ret || !num) + goto free; + + /* 8-bit configuration data, not 16-bit pixel data */ + if (num <= 32) { + for (i = 0; i < num; i++) + buf[i] = cpu_to_be16(par[i]); + num *= 2; + speed_hz = mipi_dbi_spi_cmd_max_speed(spi, num); + data = buf; + } + + gpiod_set_value_cansleep(mipi->dc, 1); + ret = mipi_dbi_spi_transfer(spi, speed_hz, 8, data, num); + free: + kfree(buf); + + return ret; +} + +static void waveshare_enable(struct drm_simple_display_pipe *pipe, + struct drm_crtc_state *crtc_state, + struct drm_plane_state *plane_state) +{ + struct mipi_dbi_dev *dbidev = drm_to_mipi_dbi_dev(pipe->crtc.dev); + struct mipi_dbi *dbi = &dbidev->dbi; + u8 addr_mode; + int ret, idx; + + if (!drm_dev_enter(pipe->crtc.dev, &idx)) + return; + + DRM_DEBUG_KMS("\n"); + + ret = mipi_dbi_poweron_conditional_reset(dbidev); + if (ret < 0) + goto out_exit; + if (ret == 1) + goto out_enable; + + mipi_dbi_command(dbi, ILI9486_ITFCTR1); + mipi_dbi_command(dbi, MIPI_DCS_EXIT_SLEEP_MODE); + msleep(250); + + mipi_dbi_command(dbi, MIPI_DCS_SET_PIXEL_FORMAT, 0x55); + + mipi_dbi_command(dbi, ILI9486_PWCTRL1, 0x44); + + mipi_dbi_command(dbi, ILI9486_VMCTRL1, 0x00, 0x00, 0x00, 0x00); + + mipi_dbi_command(dbi, ILI9486_PGAMCTRL, + 0x0F, 0x1F, 0x1C, 0x0C, 0x0F, 0x08, 0x48, 0x98, + 0x37, 0x0A, 0x13, 0x04, 0x11, 0x0D, 0x0); + mipi_dbi_command(dbi, ILI9486_NGAMCTRL, + 0x0F, 0x32, 0x2E, 0x0B, 0x0D, 0x05, 0x47, 0x75, + 0x37, 0x06, 0x10, 0x03, 0x24, 0x20, 0x00); + mipi_dbi_command(dbi, ILI9486_DGAMCTRL, + 0x0F, 0x32, 0x2E, 0x0B, 0x0D, 0x05, 0x47, 0x75, + 0x37, 0x06, 0x10, 0x03, 0x24, 0x20, 0x00); + + mipi_dbi_command(dbi, MIPI_DCS_SET_DISPLAY_ON); + msleep(100); + + out_enable: + switch (dbidev->rotation) { + case 90: + addr_mode = ILI9486_MADCTL_MY; + break; + case 180: + addr_mode = ILI9486_MADCTL_MV; + break; + case 270: + addr_mode = ILI9486_MADCTL_MX; + break; + default: + addr_mode = ILI9486_MADCTL_MV | ILI9486_MADCTL_MY | + ILI9486_MADCTL_MX; + break; + } + addr_mode |= ILI9486_MADCTL_BGR; + mipi_dbi_command(dbi, MIPI_DCS_SET_ADDRESS_MODE, addr_mode); + mipi_dbi_enable_flush(dbidev, crtc_state, plane_state); + out_exit: + drm_dev_exit(idx); +} + +static const struct drm_simple_display_pipe_funcs waveshare_pipe_funcs = { + .enable = waveshare_enable, + .disable = mipi_dbi_pipe_disable, + .update = mipi_dbi_pipe_update, + .prepare_fb = drm_gem_fb_simple_display_pipe_prepare_fb, +}; + +static const struct drm_display_mode waveshare_mode = { + DRM_SIMPLE_MODE(480, 320, 73, 49), +}; + +DEFINE_DRM_GEM_CMA_FOPS(ili9486_fops); + +static struct drm_driver ili9486_driver = { + .driver_features = DRIVER_GEM | DRIVER_MODESET | DRIVER_ATOMIC, + .fops = &ili9486_fops, + .release = mipi_dbi_release, + DRM_GEM_CMA_VMAP_DRIVER_OPS, + .debugfs_init = mipi_dbi_debugfs_init, + .name = "ili9486", + .desc = "Ilitek ILI9486", + .date = "20200118", + .major = 1, + .minor = 0, +}; + +static const struct of_device_id ili9486_of_match[] = { + { .compatible = "waveshare,rpi-lcd-35" }, + { .compatible = "ozzmaker,piscreen" }, + {}, +}; +MODULE_DEVICE_TABLE(of, ili9486_of_match); + +static const struct spi_device_id ili9486_id[] = { + { "ili9486", 0 }, + { } +}; +MODULE_DEVICE_TABLE(spi, ili9486_id); + +static int ili9486_probe(struct spi_device *spi) +{ + struct device *dev = &spi->dev; + struct mipi_dbi_dev *dbidev; + struct drm_device *drm; + struct mipi_dbi *dbi; + struct gpio_desc *dc; + u32 rotation = 0; + int ret; + + dbidev = kzalloc(sizeof(*dbidev), GFP_KERNEL); + if (!dbidev) + return -ENOMEM; + + dbi = &dbidev->dbi; + drm = &dbidev->drm; + ret = devm_drm_dev_init(dev, drm, &ili9486_driver); + if (ret) { + kfree(dbidev); + return ret; + } + + drm_mode_config_init(drm); + + dbi->reset = devm_gpiod_get(dev, "reset", GPIOD_OUT_HIGH); + if (IS_ERR(dbi->reset)) { + DRM_DEV_ERROR(dev, "Failed to get gpio 'reset'\n"); + return PTR_ERR(dbi->reset); + } + + dc = devm_gpiod_get(dev, "dc", GPIOD_OUT_LOW); + if (IS_ERR(dc)) { + DRM_DEV_ERROR(dev, "Failed to get gpio 'dc'\n"); + return PTR_ERR(dc); + } + + dbidev->backlight = devm_of_find_backlight(dev); + if (IS_ERR(dbidev->backlight)) + return PTR_ERR(dbidev->backlight); + + device_property_read_u32(dev, "rotation", &rotation); + + ret = mipi_dbi_spi_init(spi, dbi, dc); + if (ret) + return ret; + + dbi->command = waveshare_command; + dbi->read_commands = NULL; + + ret = mipi_dbi_dev_init(dbidev, &waveshare_pipe_funcs, &waveshare_mode, rotation); + if (ret) + return ret; + + drm_mode_config_reset(drm); + + ret = drm_dev_register(drm, 0); + if (ret) + return ret; + + spi_set_drvdata(spi, drm); + + drm_fbdev_generic_setup(drm, 0); + + return 0; +} + +static int ili9486_remove(struct spi_device *spi) +{ + struct drm_device *drm = spi_get_drvdata(spi); + + drm_dev_unplug(drm); + drm_atomic_helper_shutdown(drm); + + return 0; +} + +static void ili9486_shutdown(struct spi_device *spi) +{ + drm_atomic_helper_shutdown(spi_get_drvdata(spi)); +} + +static struct spi_driver ili9486_spi_driver = { + .driver = { + .name = "ili9486", + .of_match_table = ili9486_of_match, + }, + .id_table = ili9486_id, + .probe = ili9486_probe, + .remove = ili9486_remove, + .shutdown = ili9486_shutdown, +}; +module_spi_driver(ili9486_spi_driver); + +MODULE_DESCRIPTION("Ilitek ILI9486 DRM driver"); +MODULE_AUTHOR("Kamlesh Gurudasani <kamlesh.gurudasani@gmail.com>"); +MODULE_LICENSE("GPL"); -- 2.7.4 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH v4 3/3] drm/tinydrm: add support for tft displays based on ilitek,ili9486 2020-01-27 14:26 ` [PATCH v4 3/3] drm/tinydrm: add support " Kamlesh Gurudasani @ 2020-01-27 15:13 ` Noralf Trønnes 2020-01-29 20:15 ` Sam Ravnborg 1 sibling, 0 replies; 10+ messages in thread From: Noralf Trønnes @ 2020-01-27 15:13 UTC (permalink / raw) To: Kamlesh Gurudasani, Rob Herring, Sam Ravnborg; +Cc: devicetree, dri-devel Den 27.01.2020 15.26, skrev Kamlesh Gurudasani: > This adds support fot ilitek,ili9486 based displays with shift register > in front of controller. > Ozzmaker,Piscreen and Waveshare,rpi-lcd-35 are such displays. > > Signed-off-by: Kamlesh Gurudasani <kamlesh.gurudasani@gmail.com> > --- Reviewed-by: Noralf Trønnes <noralf@tronnes.org> When you resend it's important that you add any r-b's or ack's you've already received. This prevents double work (some review a load of patches and can't be expected to remember them all) and it helps the maintainer getting confidence that the patch is solid when others have looked at it. No need to resend again just for this, but if you need another respin, remember to add my r-b. Noralf. ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v4 3/3] drm/tinydrm: add support for tft displays based on ilitek,ili9486 2020-01-27 14:26 ` [PATCH v4 3/3] drm/tinydrm: add support " Kamlesh Gurudasani 2020-01-27 15:13 ` Noralf Trønnes @ 2020-01-29 20:15 ` Sam Ravnborg 1 sibling, 0 replies; 10+ messages in thread From: Sam Ravnborg @ 2020-01-29 20:15 UTC (permalink / raw) To: Kamlesh Gurudasani; +Cc: noralf, Rob Herring, devicetree, dri-devel Hi Kamlesh On Mon, Jan 27, 2020 at 07:56:46PM +0530, Kamlesh Gurudasani wrote: > This adds support fot ilitek,ili9486 based displays with shift register > in front of controller. > Ozzmaker,Piscreen and Waveshare,rpi-lcd-35 are such displays. > > Signed-off-by: Kamlesh Gurudasani <kamlesh.gurudasani@gmail.com> Driver looks good. Can you please replace tinydrm with tiny in the subject. We no longer have a tinydrm folder. With this fixed: Acked-by: Sam Ravnborg <sam@ravnborg.org> > --- > > v2 changes: > * assignment of dbi_command before registration > * made dc and reset gpio compulsory > * typos and unwanted comments removed > * changed the name of function which were display specific > * arranged the Makefile entries in alphabetical order > > v3 changes: > * no changes > > v4 changes: > * no changes > --- > MAINTAINERS | 7 + > drivers/gpu/drm/tiny/Kconfig | 14 ++ > drivers/gpu/drm/tiny/Makefile | 1 + > drivers/gpu/drm/tiny/ili9486.c | 283 +++++++++++++++++++++++++++++++++++++++++ > 4 files changed, 305 insertions(+) > create mode 100644 drivers/gpu/drm/tiny/ili9486.c > > diff --git a/MAINTAINERS b/MAINTAINERS > index 16423f5..30dd396 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -5233,6 +5233,13 @@ S: Maintained > F: drivers/gpu/drm/tiny/ili9225.c > F: Documentation/devicetree/bindings/display/ilitek,ili9225.txt > > +DRM DRIVER FOR ILITEK ILI9486 PANELS > +M: Kamlesh Gurudasani <kamlesh.gurudasani@gmail.com> > +T: git git://anongit.freedesktop.org/drm/drm-misc > +S: Maintained > +F: drivers/gpu/drm/tiny/ili9486.c > +F: Documentation/devicetree/bindings/display/ilitek,ili9486.yaml > + > DRM DRIVER FOR HX8357D PANELS > M: Eric Anholt <eric@anholt.net> > T: git git://anongit.freedesktop.org/drm/drm-misc > diff --git a/drivers/gpu/drm/tiny/Kconfig b/drivers/gpu/drm/tiny/Kconfig > index a866421..4160e74 100644 > --- a/drivers/gpu/drm/tiny/Kconfig > +++ b/drivers/gpu/drm/tiny/Kconfig > @@ -47,6 +47,20 @@ config TINYDRM_ILI9341 > > If M is selected the module will be called ili9341. > > +config TINYDRM_ILI9486 > + tristate "DRM support for ILI9486 display panels" > + depends on DRM && SPI > + select DRM_KMS_HELPER > + select DRM_KMS_CMA_HELPER > + select DRM_MIPI_DBI > + select BACKLIGHT_CLASS_DEVICE > + help > + DRM driver for the following Ilitek ILI9486 panels: > + * PISCREEN 3.5" 320x480 TFT (Ozzmaker 3.5") > + * RPILCD 3.5" 320x480 TFT (Waveshare 3.5") > + > + If M is selected the module will be called ili9486. > + > config TINYDRM_MI0283QT > tristate "DRM support for MI0283QT" > depends on DRM && SPI > diff --git a/drivers/gpu/drm/tiny/Makefile b/drivers/gpu/drm/tiny/Makefile > index 896cf31..c96ceee 100644 > --- a/drivers/gpu/drm/tiny/Makefile > +++ b/drivers/gpu/drm/tiny/Makefile > @@ -4,6 +4,7 @@ obj-$(CONFIG_DRM_GM12U320) += gm12u320.o > obj-$(CONFIG_TINYDRM_HX8357D) += hx8357d.o > obj-$(CONFIG_TINYDRM_ILI9225) += ili9225.o > obj-$(CONFIG_TINYDRM_ILI9341) += ili9341.o > +obj-$(CONFIG_TINYDRM_ILI9486) += ili9486.o > obj-$(CONFIG_TINYDRM_MI0283QT) += mi0283qt.o > obj-$(CONFIG_TINYDRM_REPAPER) += repaper.o > obj-$(CONFIG_TINYDRM_ST7586) += st7586.o > diff --git a/drivers/gpu/drm/tiny/ili9486.c b/drivers/gpu/drm/tiny/ili9486.c > new file mode 100644 > index 0000000..e960b16 > --- /dev/null > +++ b/drivers/gpu/drm/tiny/ili9486.c > @@ -0,0 +1,283 @@ > +// SPDX-License-Identifier: GPL-2.0+ > +/* > + * DRM driver for Ilitek ILI9486 panels > + * > + * Copyright 2020 Kamlesh Gurudasani <kamlesh.gurudasani@gmail.com> > + */ > + > +#include <linux/backlight.h> > +#include <linux/delay.h> > +#include <linux/gpio/consumer.h> > +#include <linux/module.h> > +#include <linux/property.h> > +#include <linux/spi/spi.h> > +#include <video/mipi_display.h> > + > +#include <drm/drm_atomic_helper.h> > +#include <drm/drm_drv.h> > +#include <drm/drm_fb_helper.h> > +#include <drm/drm_gem_cma_helper.h> > +#include <drm/drm_gem_framebuffer_helper.h> > +#include <drm/drm_mipi_dbi.h> > +#include <drm/drm_modeset_helper.h> > + > +#define ILI9486_ITFCTR1 0xb0 > +#define ILI9486_PWCTRL1 0xc2 > +#define ILI9486_VMCTRL1 0xc5 > +#define ILI9486_PGAMCTRL 0xe0 > +#define ILI9486_NGAMCTRL 0xe1 > +#define ILI9486_DGAMCTRL 0xe2 > +#define ILI9486_MADCTL_BGR BIT(3) > +#define ILI9486_MADCTL_MV BIT(5) > +#define ILI9486_MADCTL_MX BIT(6) > +#define ILI9486_MADCTL_MY BIT(7) > + > +/* > + * The PiScreen/waveshare rpi-lcd-35 has a SPI to 16-bit parallel bus converter > + * in front of the display controller. This means that 8-bit values have to be > + * transferred as 16-bit. > + */ > +static int waveshare_command(struct mipi_dbi *mipi, u8 *cmd, u8 *par, size_t num) > +{ > + struct spi_device *spi = mipi->spi; > + void *data = par; > + u32 speed_hz; > + int i, ret; > + u16 *buf; > + > + buf = kmalloc(32 * sizeof(u16), GFP_KERNEL); > + if (!buf) > + return -ENOMEM; > + > + /* > + * The displays are Raspberry Pi HATs and connected to the 8-bit only > + * SPI controller, so 16-bit command and parameters need byte swapping > + * before being transferred as 8-bit on the big endian SPI bus. > + * Pixel data bytes have already been swapped before this function is > + * called. > + */ > + buf[0] = cpu_to_be16(*cmd); > + gpiod_set_value_cansleep(mipi->dc, 0); > + speed_hz = mipi_dbi_spi_cmd_max_speed(spi, 2); > + ret = mipi_dbi_spi_transfer(spi, speed_hz, 8, buf, 2); > + if (ret || !num) > + goto free; > + > + /* 8-bit configuration data, not 16-bit pixel data */ > + if (num <= 32) { > + for (i = 0; i < num; i++) > + buf[i] = cpu_to_be16(par[i]); > + num *= 2; > + speed_hz = mipi_dbi_spi_cmd_max_speed(spi, num); > + data = buf; > + } > + > + gpiod_set_value_cansleep(mipi->dc, 1); > + ret = mipi_dbi_spi_transfer(spi, speed_hz, 8, data, num); > + free: > + kfree(buf); > + > + return ret; > +} > + > +static void waveshare_enable(struct drm_simple_display_pipe *pipe, > + struct drm_crtc_state *crtc_state, > + struct drm_plane_state *plane_state) > +{ > + struct mipi_dbi_dev *dbidev = drm_to_mipi_dbi_dev(pipe->crtc.dev); > + struct mipi_dbi *dbi = &dbidev->dbi; > + u8 addr_mode; > + int ret, idx; > + > + if (!drm_dev_enter(pipe->crtc.dev, &idx)) > + return; > + > + DRM_DEBUG_KMS("\n"); > + > + ret = mipi_dbi_poweron_conditional_reset(dbidev); > + if (ret < 0) > + goto out_exit; > + if (ret == 1) > + goto out_enable; > + > + mipi_dbi_command(dbi, ILI9486_ITFCTR1); > + mipi_dbi_command(dbi, MIPI_DCS_EXIT_SLEEP_MODE); > + msleep(250); > + > + mipi_dbi_command(dbi, MIPI_DCS_SET_PIXEL_FORMAT, 0x55); > + > + mipi_dbi_command(dbi, ILI9486_PWCTRL1, 0x44); > + > + mipi_dbi_command(dbi, ILI9486_VMCTRL1, 0x00, 0x00, 0x00, 0x00); > + > + mipi_dbi_command(dbi, ILI9486_PGAMCTRL, > + 0x0F, 0x1F, 0x1C, 0x0C, 0x0F, 0x08, 0x48, 0x98, > + 0x37, 0x0A, 0x13, 0x04, 0x11, 0x0D, 0x0); > + mipi_dbi_command(dbi, ILI9486_NGAMCTRL, > + 0x0F, 0x32, 0x2E, 0x0B, 0x0D, 0x05, 0x47, 0x75, > + 0x37, 0x06, 0x10, 0x03, 0x24, 0x20, 0x00); > + mipi_dbi_command(dbi, ILI9486_DGAMCTRL, > + 0x0F, 0x32, 0x2E, 0x0B, 0x0D, 0x05, 0x47, 0x75, > + 0x37, 0x06, 0x10, 0x03, 0x24, 0x20, 0x00); > + > + mipi_dbi_command(dbi, MIPI_DCS_SET_DISPLAY_ON); > + msleep(100); > + > + out_enable: > + switch (dbidev->rotation) { > + case 90: > + addr_mode = ILI9486_MADCTL_MY; > + break; > + case 180: > + addr_mode = ILI9486_MADCTL_MV; > + break; > + case 270: > + addr_mode = ILI9486_MADCTL_MX; > + break; > + default: > + addr_mode = ILI9486_MADCTL_MV | ILI9486_MADCTL_MY | > + ILI9486_MADCTL_MX; > + break; > + } > + addr_mode |= ILI9486_MADCTL_BGR; > + mipi_dbi_command(dbi, MIPI_DCS_SET_ADDRESS_MODE, addr_mode); > + mipi_dbi_enable_flush(dbidev, crtc_state, plane_state); > + out_exit: > + drm_dev_exit(idx); > +} > + > +static const struct drm_simple_display_pipe_funcs waveshare_pipe_funcs = { > + .enable = waveshare_enable, > + .disable = mipi_dbi_pipe_disable, > + .update = mipi_dbi_pipe_update, > + .prepare_fb = drm_gem_fb_simple_display_pipe_prepare_fb, > +}; > + > +static const struct drm_display_mode waveshare_mode = { > + DRM_SIMPLE_MODE(480, 320, 73, 49), > +}; > + > +DEFINE_DRM_GEM_CMA_FOPS(ili9486_fops); > + > +static struct drm_driver ili9486_driver = { > + .driver_features = DRIVER_GEM | DRIVER_MODESET | DRIVER_ATOMIC, > + .fops = &ili9486_fops, > + .release = mipi_dbi_release, > + DRM_GEM_CMA_VMAP_DRIVER_OPS, > + .debugfs_init = mipi_dbi_debugfs_init, > + .name = "ili9486", > + .desc = "Ilitek ILI9486", > + .date = "20200118", > + .major = 1, > + .minor = 0, > +}; > + > +static const struct of_device_id ili9486_of_match[] = { > + { .compatible = "waveshare,rpi-lcd-35" }, > + { .compatible = "ozzmaker,piscreen" }, > + {}, > +}; > +MODULE_DEVICE_TABLE(of, ili9486_of_match); > + > +static const struct spi_device_id ili9486_id[] = { > + { "ili9486", 0 }, > + { } > +}; > +MODULE_DEVICE_TABLE(spi, ili9486_id); > + > +static int ili9486_probe(struct spi_device *spi) > +{ > + struct device *dev = &spi->dev; > + struct mipi_dbi_dev *dbidev; > + struct drm_device *drm; > + struct mipi_dbi *dbi; > + struct gpio_desc *dc; > + u32 rotation = 0; > + int ret; > + > + dbidev = kzalloc(sizeof(*dbidev), GFP_KERNEL); > + if (!dbidev) > + return -ENOMEM; > + > + dbi = &dbidev->dbi; > + drm = &dbidev->drm; > + ret = devm_drm_dev_init(dev, drm, &ili9486_driver); > + if (ret) { > + kfree(dbidev); > + return ret; > + } > + > + drm_mode_config_init(drm); > + > + dbi->reset = devm_gpiod_get(dev, "reset", GPIOD_OUT_HIGH); > + if (IS_ERR(dbi->reset)) { > + DRM_DEV_ERROR(dev, "Failed to get gpio 'reset'\n"); > + return PTR_ERR(dbi->reset); > + } > + > + dc = devm_gpiod_get(dev, "dc", GPIOD_OUT_LOW); > + if (IS_ERR(dc)) { > + DRM_DEV_ERROR(dev, "Failed to get gpio 'dc'\n"); > + return PTR_ERR(dc); > + } > + > + dbidev->backlight = devm_of_find_backlight(dev); > + if (IS_ERR(dbidev->backlight)) > + return PTR_ERR(dbidev->backlight); > + > + device_property_read_u32(dev, "rotation", &rotation); > + > + ret = mipi_dbi_spi_init(spi, dbi, dc); > + if (ret) > + return ret; > + > + dbi->command = waveshare_command; > + dbi->read_commands = NULL; > + > + ret = mipi_dbi_dev_init(dbidev, &waveshare_pipe_funcs, &waveshare_mode, rotation); > + if (ret) > + return ret; > + > + drm_mode_config_reset(drm); > + > + ret = drm_dev_register(drm, 0); > + if (ret) > + return ret; > + > + spi_set_drvdata(spi, drm); > + > + drm_fbdev_generic_setup(drm, 0); > + > + return 0; > +} > + > +static int ili9486_remove(struct spi_device *spi) > +{ > + struct drm_device *drm = spi_get_drvdata(spi); > + > + drm_dev_unplug(drm); > + drm_atomic_helper_shutdown(drm); > + > + return 0; > +} > + > +static void ili9486_shutdown(struct spi_device *spi) > +{ > + drm_atomic_helper_shutdown(spi_get_drvdata(spi)); > +} > + > +static struct spi_driver ili9486_spi_driver = { > + .driver = { > + .name = "ili9486", > + .of_match_table = ili9486_of_match, > + }, > + .id_table = ili9486_id, > + .probe = ili9486_probe, > + .remove = ili9486_remove, > + .shutdown = ili9486_shutdown, > +}; > +module_spi_driver(ili9486_spi_driver); > + > +MODULE_DESCRIPTION("Ilitek ILI9486 DRM driver"); > +MODULE_AUTHOR("Kamlesh Gurudasani <kamlesh.gurudasani@gmail.com>"); > +MODULE_LICENSE("GPL"); > -- > 2.7.4 ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2020-02-05 17:33 UTC | newest] Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2020-01-27 14:24 [PATCH v4 0/3] Support for tft displays based on ilitek,ili9486 Kamlesh Gurudasani 2020-01-27 14:25 ` [PATCH v4 1/3] dt-bindings: add vendor prefix for OzzMaker and Waveshare Electronics Kamlesh Gurudasani 2020-01-29 19:48 ` Sam Ravnborg 2020-02-05 17:33 ` Rob Herring 2020-01-27 14:26 ` [PATCH v4 2/3] dt-bindings: add binding for tft displays based on ilitek,ili9486 Kamlesh Gurudasani 2020-01-29 20:14 ` Sam Ravnborg 2020-01-31 14:23 ` Rob Herring 2020-01-27 14:26 ` [PATCH v4 3/3] drm/tinydrm: add support " Kamlesh Gurudasani 2020-01-27 15:13 ` Noralf Trønnes 2020-01-29 20:15 ` Sam Ravnborg
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).