* [PATCH v5 0/8] Enable ili9341 and l3gd20 on stm32f429-disco [not found] <broonie@kernel.org> @ 2020-05-25 3:40 ` dillon.minfei 2020-05-25 3:40 ` [PATCH v5 1/8] ARM: dts: stm32: Add dma config for spi5 dillon.minfei ` (8 more replies) 0 siblings, 9 replies; 15+ messages in thread From: dillon.minfei @ 2020-05-25 3:40 UTC (permalink / raw) To: robh+dt, p.zabel, mcoquelin.stm32, alexandre.torgue, thierry.reding, sam, airlied, daniel, mturquette, sboyd Cc: devicetree, linux-arm-kernel, linux-kernel, linux-spi, linux-stm32, dri-devel, linux-clk, dillon min From: dillon min <dillon.minfei@gmail.com> V5's update based on Mark Brown's suggestion, use 'SPI_MASTER_MUST_RX' for SPI_SIMPLEX_RX mode on stm32 spi controller. V5: 1 instead of add send dummy data out under SIMPLEX_RX mode, add flags 'SPI_CONTROLLER_MUST_TX' for stm32 spi driver 2 bypass 'SPI_CONTROLLER_MUST_TX' and 'SPI_CONTROLLER_MUST_RX' under 'SPI_3WIRE' mode V4: According to alexandre torgue's suggestion, combine ili9341 and l3gd20's modification on stm32f429-disco board to one patchset. Changes: ili9341: 1 update ili9341 panel driver according to Linus's suggestion 2 drop V1's No.5 patch, sumbit new changes for clk-stm32f4 3 merge l3gd20's change to this patchset V3: 1 merge original tiny/ili9341.c driver to panel/panel-ilitek-ili9341.c to support serial spi & parallel rgb interface in one driver. 2 update ilitek,ili9341.yaml dts binding documentation. 3 update stm32f429-disco dts binding V2: 1 verify ilitek,ili9341.yaml with make O=../linux-stm32 dt_binding_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/display/panel/ ilitek,ili9341.yaml V1: 1 add ili9341 drm panel driver 2 add ltdc, spi5 controller for stm32f429-disco 3 add ltdc, spi5 pin map for stm32f429-disco 4 add docs about ili9341 5 fix ltdc driver loading hang in clk set rate bug L3gd20: V3: 1 merge stm32f429-disco dtbs binding with ili9341 part V2: 1 insert blank line at stm32f420-disco.dts line 143 2 add more description for l3gd20 in commit message V1: 1 enable spi5 controller on stm32f429-disco (dts) 2 add spi5 pinmap for stm32f429-disco (dts) 3 add SPI_SIMPLEX_RX, SPI_3WIRE_RX support for stm32f4 dillon min (8): ARM: dts: stm32: Add dma config for spi5 ARM: dts: stm32: Add pin map for ltdc & spi5 on stm32f429-disco board ARM: dts: stm32: enable ltdc binding with ili9341, gyro l3gd20 on stm32429-disco board dt-bindings: display: panel: Add ilitek ili9341 panel bindings clk: stm32: Fix stm32f429's ltdc driver hang in set clock rate, fix duplicated ltdc clock register to 'clk_core' case ltdc's clock turn off by clk_disable_unused() drm/panel: Add ilitek ili9341 panel driver spi: stm32: Add 'SPI_SIMPLEX_RX', 'SPI_3WIRE_RX' support for stm32f4 spi: flags 'SPI_CONTROLLER_MUST_RX' and 'SPI_CONTROLLER_MUST_TX' can't be coexit with 'SPI_3WIRE' mode .../bindings/display/panel/ilitek,ili9341.yaml | 69 ++ arch/arm/boot/dts/stm32f4-pinctrl.dtsi | 67 + arch/arm/boot/dts/stm32f429-disco.dts | 48 + arch/arm/boot/dts/stm32f429.dtsi | 3 + drivers/clk/clk-stm32f4.c | 7 +- drivers/gpu/drm/panel/Kconfig | 12 + drivers/gpu/drm/panel/Makefile | 1 + drivers/gpu/drm/panel/panel-ilitek-ili9341.c | 1301 ++++++++++++++++++++ drivers/spi/spi-stm32.c | 19 +- drivers/spi/spi.c | 3 +- 10 files changed, 1521 insertions(+), 9 deletions(-) create mode 100644 Documentation/devicetree/bindings/display/panel/ilitek,ili9341.yaml create mode 100644 drivers/gpu/drm/panel/panel-ilitek-ili9341.c -- 2.7.4 ^ permalink raw reply [flat|nested] 15+ messages in thread
* [PATCH v5 1/8] ARM: dts: stm32: Add dma config for spi5 2020-05-25 3:40 ` [PATCH v5 0/8] Enable ili9341 and l3gd20 on stm32f429-disco dillon.minfei @ 2020-05-25 3:40 ` dillon.minfei 2020-05-25 3:40 ` [PATCH v5 2/8] ARM: dts: stm32: Add pin map for ltdc & spi5 on stm32f429-disco board dillon.minfei ` (7 subsequent siblings) 8 siblings, 0 replies; 15+ messages in thread From: dillon.minfei @ 2020-05-25 3:40 UTC (permalink / raw) To: robh+dt, p.zabel, mcoquelin.stm32, alexandre.torgue, thierry.reding, sam, airlied, daniel, mturquette, sboyd Cc: devicetree, linux-arm-kernel, linux-kernel, linux-spi, linux-stm32, dri-devel, linux-clk, dillon min From: dillon min <dillon.minfei@gmail.com> Enable spi5's dma configuration. for graphics data output to ilitek ili9341 panel via mipi dbi interface Signed-off-by: dillon min <dillon.minfei@gmail.com> --- arch/arm/boot/dts/stm32f429.dtsi | 3 +++ 1 file changed, 3 insertions(+) diff --git a/arch/arm/boot/dts/stm32f429.dtsi b/arch/arm/boot/dts/stm32f429.dtsi index d777069..5820b11 100644 --- a/arch/arm/boot/dts/stm32f429.dtsi +++ b/arch/arm/boot/dts/stm32f429.dtsi @@ -660,6 +660,9 @@ reg = <0x40015000 0x400>; interrupts = <85>; clocks = <&rcc 0 STM32F4_APB2_CLOCK(SPI5)>; + dmas = <&dma2 3 2 0x400 0x0>, + <&dma2 4 2 0x400 0x0>; + dma-names = "rx", "tx"; status = "disabled"; }; -- 2.7.4 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH v5 2/8] ARM: dts: stm32: Add pin map for ltdc & spi5 on stm32f429-disco board 2020-05-25 3:40 ` [PATCH v5 0/8] Enable ili9341 and l3gd20 on stm32f429-disco dillon.minfei 2020-05-25 3:40 ` [PATCH v5 1/8] ARM: dts: stm32: Add dma config for spi5 dillon.minfei @ 2020-05-25 3:40 ` dillon.minfei 2020-05-25 3:40 ` [PATCH v5 3/8] ARM: dts: stm32: enable ltdc binding with ili9341, gyro l3gd20 on stm32429-disco board dillon.minfei ` (6 subsequent siblings) 8 siblings, 0 replies; 15+ messages in thread From: dillon.minfei @ 2020-05-25 3:40 UTC (permalink / raw) To: robh+dt, p.zabel, mcoquelin.stm32, alexandre.torgue, thierry.reding, sam, airlied, daniel, mturquette, sboyd Cc: devicetree, linux-arm-kernel, linux-kernel, linux-spi, linux-stm32, dri-devel, linux-clk, dillon min From: dillon min <dillon.minfei@gmail.com> This patch adds the pin configuration for ltdc and spi5 controller on stm32f429-disco board. Signed-off-by: dillon min <dillon.minfei@gmail.com> --- arch/arm/boot/dts/stm32f4-pinctrl.dtsi | 67 ++++++++++++++++++++++++++++++++++ 1 file changed, 67 insertions(+) diff --git a/arch/arm/boot/dts/stm32f4-pinctrl.dtsi b/arch/arm/boot/dts/stm32f4-pinctrl.dtsi index 392fa14..0eb107f 100644 --- a/arch/arm/boot/dts/stm32f4-pinctrl.dtsi +++ b/arch/arm/boot/dts/stm32f4-pinctrl.dtsi @@ -316,6 +316,73 @@ }; }; + ltdc_pins_f429_disco: ltdc-1 { + pins { + pinmux = <STM32_PINMUX('C', 6, AF14)>, + /* LCD_HSYNC */ + <STM32_PINMUX('A', 4, AF14)>, + /* LCD_VSYNC */ + <STM32_PINMUX('G', 7, AF14)>, + /* LCD_CLK */ + <STM32_PINMUX('C', 10, AF14)>, + /* LCD_R2 */ + <STM32_PINMUX('B', 0, AF9)>, + /* LCD_R3 */ + <STM32_PINMUX('A', 11, AF14)>, + /* LCD_R4 */ + <STM32_PINMUX('A', 12, AF14)>, + /* LCD_R5 */ + <STM32_PINMUX('B', 1, AF9)>, + /* LCD_R6*/ + <STM32_PINMUX('G', 6, AF14)>, + /* LCD_R7 */ + <STM32_PINMUX('A', 6, AF14)>, + /* LCD_G2 */ + <STM32_PINMUX('G', 10, AF9)>, + /* LCD_G3 */ + <STM32_PINMUX('B', 10, AF14)>, + /* LCD_G4 */ + <STM32_PINMUX('D', 6, AF14)>, + /* LCD_B2 */ + <STM32_PINMUX('G', 11, AF14)>, + /* LCD_B3*/ + <STM32_PINMUX('B', 11, AF14)>, + /* LCD_G5 */ + <STM32_PINMUX('C', 7, AF14)>, + /* LCD_G6 */ + <STM32_PINMUX('D', 3, AF14)>, + /* LCD_G7 */ + <STM32_PINMUX('G', 12, AF9)>, + /* LCD_B4 */ + <STM32_PINMUX('A', 3, AF14)>, + /* LCD_B5 */ + <STM32_PINMUX('B', 8, AF14)>, + /* LCD_B6 */ + <STM32_PINMUX('B', 9, AF14)>, + /* LCD_B7 */ + <STM32_PINMUX('F', 10, AF14)>; + /* LCD_DE */ + slew-rate = <2>; + }; + }; + + spi5_pins: spi5-0 { + pins1 { + pinmux = <STM32_PINMUX('F', 7, AF5)>, + /* SPI5_CLK */ + <STM32_PINMUX('F', 9, AF5)>; + /* SPI5_MOSI */ + bias-disable; + drive-push-pull; + slew-rate = <0>; + }; + pins2 { + pinmux = <STM32_PINMUX('F', 8, AF5)>; + /* SPI5_MISO */ + bias-disable; + }; + }; + dcmi_pins: dcmi-0 { pins { pinmux = <STM32_PINMUX('A', 4, AF13)>, /* DCMI_HSYNC */ -- 2.7.4 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH v5 3/8] ARM: dts: stm32: enable ltdc binding with ili9341, gyro l3gd20 on stm32429-disco board 2020-05-25 3:40 ` [PATCH v5 0/8] Enable ili9341 and l3gd20 on stm32f429-disco dillon.minfei 2020-05-25 3:40 ` [PATCH v5 1/8] ARM: dts: stm32: Add dma config for spi5 dillon.minfei 2020-05-25 3:40 ` [PATCH v5 2/8] ARM: dts: stm32: Add pin map for ltdc & spi5 on stm32f429-disco board dillon.minfei @ 2020-05-25 3:40 ` dillon.minfei 2020-05-25 3:40 ` [PATCH v5 4/8] dt-bindings: display: panel: Add ilitek ili9341 panel bindings dillon.minfei ` (5 subsequent siblings) 8 siblings, 0 replies; 15+ messages in thread From: dillon.minfei @ 2020-05-25 3:40 UTC (permalink / raw) To: robh+dt, p.zabel, mcoquelin.stm32, alexandre.torgue, thierry.reding, sam, airlied, daniel, mturquette, sboyd Cc: devicetree, linux-arm-kernel, linux-kernel, linux-spi, linux-stm32, dri-devel, linux-clk, dillon min From: dillon min <dillon.minfei@gmail.com> Enable the ltdc & ili9341, gyro l3gd20 on stm32429-disco board. Signed-off-by: dillon min <dillon.minfei@gmail.com> --- arch/arm/boot/dts/stm32f429-disco.dts | 48 +++++++++++++++++++++++++++++++++++ 1 file changed, 48 insertions(+) diff --git a/arch/arm/boot/dts/stm32f429-disco.dts b/arch/arm/boot/dts/stm32f429-disco.dts index 30c0f67..365d16f 100644 --- a/arch/arm/boot/dts/stm32f429-disco.dts +++ b/arch/arm/boot/dts/stm32f429-disco.dts @@ -49,6 +49,8 @@ #include "stm32f429.dtsi" #include "stm32f429-pinctrl.dtsi" #include <dt-bindings/input/input.h> +#include <dt-bindings/interrupt-controller/irq.h> +#include <dt-bindings/gpio/gpio.h> / { model = "STMicroelectronics STM32F429i-DISCO board"; @@ -127,3 +129,49 @@ pinctrl-names = "default"; status = "okay"; }; + +<dc { + status = "okay"; + pinctrl-0 = <<dc_pins_f429_disco>; + pinctrl-names = "default"; + + port { + ltdc_out_rgb: endpoint { + remote-endpoint = <&panel_in_rgb>; + }; + }; +}; + +&spi5 { + status = "okay"; + pinctrl-0 = <&spi5_pins>; + pinctrl-names = "default"; + #address-cells = <1>; + #size-cells = <0>; + cs-gpios = <&gpioc 1 GPIO_ACTIVE_LOW>, <&gpioc 2 GPIO_ACTIVE_LOW>; + + l3gd20: l3gd20@0 { + compatible = "st,l3gd20-gyro"; + spi-max-frequency = <10000000>; + st,drdy-int-pin = <2>; + interrupt-parent = <&gpioa>; + interrupts = <1 IRQ_TYPE_EDGE_RISING>, + <2 IRQ_TYPE_EDGE_RISING>; + reg = <0>; + status = "okay"; + }; + + display: display@1{ + /* Connect panel-ilitek-9341 to ltdc */ + compatible = "st,sf-tc240t-9370-t"; + reg = <1>; + spi-3wire; + spi-max-frequency = <10000000>; + dc-gpios = <&gpiod 13 0>; + port { + panel_in_rgb: endpoint { + remote-endpoint = <<dc_out_rgb>; + }; + }; + }; +}; -- 2.7.4 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH v5 4/8] dt-bindings: display: panel: Add ilitek ili9341 panel bindings 2020-05-25 3:40 ` [PATCH v5 0/8] Enable ili9341 and l3gd20 on stm32f429-disco dillon.minfei ` (2 preceding siblings ...) 2020-05-25 3:40 ` [PATCH v5 3/8] ARM: dts: stm32: enable ltdc binding with ili9341, gyro l3gd20 on stm32429-disco board dillon.minfei @ 2020-05-25 3:40 ` dillon.minfei 2020-05-25 3:40 ` [PATCH v5 5/8] clk: stm32: Fix stm32f429's ltdc driver hang in set clock rate, fix duplicated ltdc clock register to 'clk_core' case ltdc's clock turn off by clk_disable_unused() dillon.minfei ` (4 subsequent siblings) 8 siblings, 0 replies; 15+ messages in thread From: dillon.minfei @ 2020-05-25 3:40 UTC (permalink / raw) To: robh+dt, p.zabel, mcoquelin.stm32, alexandre.torgue, thierry.reding, sam, airlied, daniel, mturquette, sboyd Cc: devicetree, linux-arm-kernel, linux-kernel, linux-spi, linux-stm32, dri-devel, linux-clk, dillon min From: dillon min <dillon.minfei@gmail.com> Add documentation for "ilitek,ili9341" panel. Signed-off-by: dillon min <dillon.minfei@gmail.com> --- .../bindings/display/panel/ilitek,ili9341.yaml | 69 ++++++++++++++++++++++ 1 file changed, 69 insertions(+) create mode 100644 Documentation/devicetree/bindings/display/panel/ilitek,ili9341.yaml diff --git a/Documentation/devicetree/bindings/display/panel/ilitek,ili9341.yaml b/Documentation/devicetree/bindings/display/panel/ilitek,ili9341.yaml new file mode 100644 index 0000000..2172f88 --- /dev/null +++ b/Documentation/devicetree/bindings/display/panel/ilitek,ili9341.yaml @@ -0,0 +1,69 @@ +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/display/panel/ilitek,ili9341.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Ilitek-9341 Display Panel + +maintainers: + - Dillon Min <dillon.minfei@gmail.com> + +description: | + Ilitek ILI9341 TFT panel driver with SPI control bus + This is a driver for 320x240 TFT panels, accepting a rgb input + streams with 16 bits or 18 bits. + +allOf: + - $ref: panel-common.yaml# + +properties: + compatible: + items: + - enum: + # ili9341 240*320 Color on stm32f429-disco board + - st,sf-tc240t-9370-t + - const: ilitek,ili9341 + + reg: true + + dc-gpios: + maxItems: 1 + description: Display data/command selection (D/CX) + + spi-3wire: true + + spi-max-frequency: + const: 10000000 + + port: true + +additionalProperties: false + +required: + - compatible + - reg + - dc-gpios + - port + +examples: + - |+ + spi { + #address-cells = <1>; + #size-cells = <0>; + panel: display@0 { + compatible = "st,sf-tc240t-9370-t", + "ilitek,ili9341"; + reg = <0>; + spi-3wire; + spi-max-frequency = <10000000>; + dc-gpios = <&gpiod 13 0>; + port { + panel_in: endpoint { + remote-endpoint = <&display_out>; + }; + }; + }; + }; +... + -- 2.7.4 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH v5 5/8] clk: stm32: Fix stm32f429's ltdc driver hang in set clock rate, fix duplicated ltdc clock register to 'clk_core' case ltdc's clock turn off by clk_disable_unused() 2020-05-25 3:40 ` [PATCH v5 0/8] Enable ili9341 and l3gd20 on stm32f429-disco dillon.minfei ` (3 preceding siblings ...) 2020-05-25 3:40 ` [PATCH v5 4/8] dt-bindings: display: panel: Add ilitek ili9341 panel bindings dillon.minfei @ 2020-05-25 3:40 ` dillon.minfei 2020-05-25 3:41 ` [PATCH v5 6/8] drm/panel: Add ilitek ili9341 panel driver dillon.minfei ` (3 subsequent siblings) 8 siblings, 0 replies; 15+ messages in thread From: dillon.minfei @ 2020-05-25 3:40 UTC (permalink / raw) To: robh+dt, p.zabel, mcoquelin.stm32, alexandre.torgue, thierry.reding, sam, airlied, daniel, mturquette, sboyd Cc: devicetree, linux-arm-kernel, linux-kernel, linux-spi, linux-stm32, dri-devel, linux-clk, dillon min From: dillon min <dillon.minfei@gmail.com> ltdc set clock rate crashed 'post_div_data[]''s pll_num is PLL_I2S, PLL_SAI (number is 1,2). but, as pll_num is offset of 'clks[]' input to clk_register_pll_div(), which is FCLK, CLK_LSI, defined in 'include/dt-bindings/clock/stm32fx-clock.h' so, this is a null object at the register time. then, in ltdc's clock is_enabled(), enable(), will call to_clk_gate(). will return a null object, cause kernel crashed. need change pll_num to PLL_VCO_I2S, PLL_VCO_SAI for 'post_div_data[]' duplicated ltdc clock 'stm32f429_gates[]' has a member 'ltdc' register to 'clk_core', but no upper driver use it, ltdc driver use the lcd-tft defined in 'stm32f429_aux_clk[]'. after system startup, as stm32f429_gates[]'s ltdc enable_count is zero, so turn off by clk_disable_unused() Changes since V3: 1 drop last wrong changes about 'CLK_IGNORE_UNUSED' patch 2 fix PLL_SAI mismatch with PLL_VCO_SAI Signed-off-by: dillon min <dillon.minfei@gmail.com> --- drivers/clk/clk-stm32f4.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers/clk/clk-stm32f4.c b/drivers/clk/clk-stm32f4.c index 18117ce..fa62e99 100644 --- a/drivers/clk/clk-stm32f4.c +++ b/drivers/clk/clk-stm32f4.c @@ -129,7 +129,6 @@ static const struct stm32f4_gate_data stm32f429_gates[] __initconst = { { STM32F4_RCC_APB2ENR, 20, "spi5", "apb2_div" }, { STM32F4_RCC_APB2ENR, 21, "spi6", "apb2_div" }, { STM32F4_RCC_APB2ENR, 22, "sai1", "apb2_div" }, - { STM32F4_RCC_APB2ENR, 26, "ltdc", "apb2_div" }, }; static const struct stm32f4_gate_data stm32f469_gates[] __initconst = { @@ -557,13 +556,13 @@ static const struct clk_div_table post_divr_table[] = { #define MAX_POST_DIV 3 static const struct stm32f4_pll_post_div_data post_div_data[MAX_POST_DIV] = { - { CLK_I2SQ_PDIV, PLL_I2S, "plli2s-q-div", "plli2s-q", + { CLK_I2SQ_PDIV, PLL_VCO_I2S, "plli2s-q-div", "plli2s-q", CLK_SET_RATE_PARENT, STM32F4_RCC_DCKCFGR, 0, 5, 0, NULL}, - { CLK_SAIQ_PDIV, PLL_SAI, "pllsai-q-div", "pllsai-q", + { CLK_SAIQ_PDIV, PLL_VCO_SAI, "pllsai-q-div", "pllsai-q", CLK_SET_RATE_PARENT, STM32F4_RCC_DCKCFGR, 8, 5, 0, NULL }, - { NO_IDX, PLL_SAI, "pllsai-r-div", "pllsai-r", CLK_SET_RATE_PARENT, + { NO_IDX, PLL_VCO_SAI, "pllsai-r-div", "pllsai-r", CLK_SET_RATE_PARENT, STM32F4_RCC_DCKCFGR, 16, 2, 0, post_divr_table }, }; -- 2.7.4 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH v5 6/8] drm/panel: Add ilitek ili9341 panel driver 2020-05-25 3:40 ` [PATCH v5 0/8] Enable ili9341 and l3gd20 on stm32f429-disco dillon.minfei ` (4 preceding siblings ...) 2020-05-25 3:40 ` [PATCH v5 5/8] clk: stm32: Fix stm32f429's ltdc driver hang in set clock rate, fix duplicated ltdc clock register to 'clk_core' case ltdc's clock turn off by clk_disable_unused() dillon.minfei @ 2020-05-25 3:41 ` dillon.minfei 2020-05-25 3:41 ` [PATCH v5 7/8] spi: stm32: Add 'SPI_SIMPLEX_RX', 'SPI_3WIRE_RX' support for stm32f4 dillon.minfei ` (2 subsequent siblings) 8 siblings, 0 replies; 15+ messages in thread From: dillon.minfei @ 2020-05-25 3:41 UTC (permalink / raw) To: robh+dt, p.zabel, mcoquelin.stm32, alexandre.torgue, thierry.reding, sam, airlied, daniel, mturquette, sboyd Cc: devicetree, linux-arm-kernel, linux-kernel, linux-spi, linux-stm32, dri-devel, linux-clk, dillon min From: dillon min <dillon.minfei@gmail.com> 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. Changes since V3: accoding to Linus Walleij's suggestion. 1 add more comments to driver. 2 reduce magic number usage in the driver. 3 move panel configuration from common place to system configuration. 4 reuse MIPI_DCS_* as more as possible. Signed-off-by: dillon min <dillon.minfei@gmail.com> --- drivers/gpu/drm/panel/Kconfig | 12 + drivers/gpu/drm/panel/Makefile | 1 + drivers/gpu/drm/panel/panel-ilitek-ili9341.c | 1301 ++++++++++++++++++++++++++ 3 files changed, 1314 insertions(+) create mode 100644 drivers/gpu/drm/panel/panel-ilitek-ili9341.c diff --git a/drivers/gpu/drm/panel/Kconfig b/drivers/gpu/drm/panel/Kconfig index a1723c1..c938bee 100644 --- a/drivers/gpu/drm/panel/Kconfig +++ b/drivers/gpu/drm/panel/Kconfig @@ -95,6 +95,18 @@ config DRM_PANEL_ILITEK_IL9322 Say Y here if you want to enable support for Ilitek IL9322 QVGA (320x240) RGB, YUV and ITU-T BT.656 panels. +config DRM_PANEL_ILITEK_ILI9341 + tristate "Ilitek ILI9341 240x320 QVGA panels" + depends on OF && SPI + depends on DRM_KMS_HELPER + depends on DRM_KMS_CMA_HELPER + depends on BACKLIGHT_CLASS_DEVICE + select DRM_MIPI_DBI + help + Say Y here if you want to enable support for Ilitek IL9341 + QVGA (240x320) RGB panels. support serial & parallel rgb + interface. + config DRM_PANEL_ILITEK_ILI9881C tristate "Ilitek ILI9881C-based panels" depends on OF diff --git a/drivers/gpu/drm/panel/Makefile b/drivers/gpu/drm/panel/Makefile index 96a883c..16947d7 100644 --- a/drivers/gpu/drm/panel/Makefile +++ b/drivers/gpu/drm/panel/Makefile @@ -8,6 +8,7 @@ obj-$(CONFIG_DRM_PANEL_ELIDA_KD35T133) += panel-elida-kd35t133.o obj-$(CONFIG_DRM_PANEL_FEIXIN_K101_IM2BA02) += panel-feixin-k101-im2ba02.o obj-$(CONFIG_DRM_PANEL_FEIYANG_FY07024DI26A30D) += panel-feiyang-fy07024di26a30d.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_INNOLUX_P079ZCA) += panel-innolux-p079zca.o obj-$(CONFIG_DRM_PANEL_JDI_LT070ME05000) += panel-jdi-lt070me05000.o diff --git a/drivers/gpu/drm/panel/panel-ilitek-ili9341.c b/drivers/gpu/drm/panel/panel-ilitek-ili9341.c new file mode 100644 index 0000000..dd6f860 --- /dev/null +++ b/drivers/gpu/drm/panel/panel-ilitek-ili9341.c @@ -0,0 +1,1301 @@ +// SPDX-License-Identifier: GPL-2.0-only +/* + * Ilitek ILI9341 TFT LCD drm_panel driver. + * + * This panel can be configured to support: + * - 16-bit parallel RGB interface + * - 18-bit parallel RGB interface + * - 4-line serial spi interface + * + * Copyright (C) 2020 Dillon Min <dillon.minfei@gmail.com> + * Derived from drivers/drm/gpu/panel/panel-ilitek-ili9322.c + */ + +#include <linux/bitops.h> +#include <linux/gpio/consumer.h> +#include <linux/module.h> +#include <linux/of_device.h> +#include <linux/regulator/consumer.h> +#include <linux/spi/spi.h> +#include <linux/delay.h> +#include <video/mipi_display.h> +#include <drm/drm_mipi_dbi.h> +#include <drm/drm_gem_framebuffer_helper.h> +#include <drm/drm_gem_cma_helper.h> +#include <drm/drm_fb_helper.h> +#include <drm/drm_atomic_helper.h> + +#include <drm/drm_drv.h> +#include <drm/drm_modes.h> +#include <drm/drm_panel.h> +#include <drm/drm_print.h> + +#define ILI9341_RGB_INTERFACE 0xb0 /* RGB Interface Signal Control */ +#define ILI9341_FRC 0xb1 /* Frame Rate Control register */ +#define ILI9341_DFC 0xb6 /* Display Function Control register */ +#define ILI9341_POWER1 0xc0 /* Power Control 1 register */ +#define ILI9341_POWER2 0xc1 /* Power Control 2 register */ +#define ILI9341_VCOM1 0xc5 /* VCOM Control 1 register */ +#define ILI9341_VCOM2 0xc7 /* VCOM Control 2 register */ +#define ILI9341_POWERA 0xcb /* Power control A register */ +#define ILI9341_POWERB 0xcf /* Power control B register */ +#define ILI9341_PGAMMA 0xe0 /* Positive Gamma Correction register */ +#define ILI9341_NGAMMA 0xe1 /* Negative Gamma Correction register */ +#define ILI9341_DTCA 0xe8 /* Driver timing control A */ +#define ILI9341_DTCB 0xea /* Driver timing control B */ +#define ILI9341_POWER_SEQ 0xed /* Power on sequence register */ +#define ILI9341_3GAMMA_EN 0xf2 /* 3 Gamma enable register */ +#define ILI9341_INTERFACE 0xf6 /* Interface control register */ +#define ILI9341_PRC 0xf7 /* Pump ratio control register */ +#define ILI9341_ETMOD 0xb7 /* Entry mode set */ + +#define ILI9341_MADCTL_BGR BIT(3) +#define ILI9341_MADCTL_MV BIT(5) +#define ILI9341_MADCTL_MX BIT(6) +#define ILI9341_MADCTL_MY BIT(7) + + +#define ILI9341_POWER_B_LEN 3 +#define ILI9341_POWER_SEQ_LEN 4 +#define ILI9341_DTCA_LEN 3 +#define ILI9341_DTCB_LEN 2 +#define ILI9341_POWER_A_LEN 5 +#define ILI9341_DFC_1_LEN 2 +#define ILI9341_FRC_LEN 2 +#define ILI9341_VCOM_1_LEN 2 +#define ILI9341_DFC_2_LEN 4 +#define ILI9341_COLUMN_ADDR_LEN 4 +#define ILI9341_PAGE_ADDR_LEN 4 +#define ILI9341_INTERFACE_LEN 3 +#define ILI9341_PGAMMA_LEN 15 +#define ILI9341_NGAMMA_LEN 15 +#define ILI9341_CA_LEN 3 + +#define ILI9341_PIXEL_DPI_16_BITS (BIT(6)|BIT(4)) +#define ILI9341_PIXEL_DPI_18_BITS (BIT(6)|BIT(5)) +#define ILI9341_GAMMA_CURVE_1 BIT(0) +#define ILI9341_IF_WE_MODE BIT(0) +#define ILI9341_IF_BIG_ENDIAN 0x00 +#define ILI9341_IF_DM_RGB BIT(2) +#define ILI9341_IF_DM_INTERNAL 0x00 +#define ILI9341_IF_DM_VSYNC BIT(3) +#define ILI9341_IF_RM_RGB BIT(1) +#define ILI9341_IF_RIM_RGB 0x00 + +#define ILI9341_COLUMN_ADDR 0x00ef +#define ILI9341_PAGE_ADDR 0x013f + +#define ILI9341_RGB_EPL BIT(0) +#define ILI9341_RGB_DPL BIT(1) +#define ILI9341_RGB_HSPL BIT(2) +#define ILI9341_RGB_VSPL BIT(3) +#define ILI9341_RGB_DE_MODE BIT(6) +#define ILI9341_RGB_DISP_PATH_MEM BIT(7) + +#define ILI9341_DBI_VCOMH_4P6V 0x23 +#define ILI9341_DBI_PWR_2_DEFAULT 0x10 +#define ILI9341_DBI_PRC_NORMAL 0x20 +#define ILI9341_DBI_VCOM_1_VMH_4P25V 0x3e +#define ILI9341_DBI_VCOM_1_VML_1P5V 0x28 +#define ILI9341_DBI_VCOM_2_DEC_58 0x86 +#define ILI9341_DBI_FRC_DIVA 0x00 +#define ILI9341_DBI_FRC_RTNA 0x1b +#define ILI9341_DBI_EMS_GAS BIT(0) +#define ILI9341_DBI_EMS_DTS BIT(1) +#define ILI9341_DBI_EMS_GON BIT(2) +/** + * ili9341_command - ili9341 command with optional parameter(s) + * @ili: struct ili9341 + * @cmd: Command + * @seq...: Optional parameter(s) + * + * Send command to the controller. + * + * Returns: + * Zero on success, negative error code on failure. + */ +#define ili9341_command(ili, cmd, seq...) \ +({ \ + u8 d[] = { seq }; \ + _ili9341_command(ili, cmd, d, ARRAY_SIZE(d)); \ +}) + +/** + * struct ili9341_config - the system specific ILI9341 configuration + * @max_spi_speed: 10000000 + */ +struct ili9341_config { + u32 max_spi_speed; + + const struct drm_display_mode mode; + /* TODO: need comments for this register */ + u8 ca[ILI9341_CA_LEN]; + /* TODO: need comments for this register */ + u8 power_b[ILI9341_POWER_B_LEN]; + /* TODO: need comments for this register */ + u8 power_seq[ILI9341_POWER_SEQ_LEN]; + /* TODO: need comments for this register */ + u8 dtca[ILI9341_DTCA_LEN]; + /* TODO: need comments for this register */ + u8 dtcb[ILI9341_DTCB_LEN]; + /* TODO: need comments for this register */ + u8 power_a[ILI9341_POWER_A_LEN]; + /* + * Frame Rate Control (In Normal Mode/Full Colors) (B1h) + * + * Formula to calculate frame frequency: + * Frame Rate=fosc/(Clocks per line x Division ratio x + * (Lines+VBP+VFP)) + * + * Sets the division ratio for internal clocks of Normal mode at MCU + * interface. + * + * fosc : internal oscillator frequency + * Clocks per line : RTNA setting + * Division ratio : DIVA setting + * Lines : total driving line number + * VBP : back porch line number + * VFP : front porch line number + * + * RTNA [4:0] Frame Rate (Hz) RTNA [4:0] Frame Rate (Hz) + * 1 0 0 0 0 119 1 1 0 0 0 79 + * 1 0 0 0 1 112 1 1 0 0 1 76 + * 1 0 0 1 0 106 1 1 0 1 0 73 + * 1 0 0 1 1 100 1 1 0 1 1 70(default) + * 1 0 1 0 0 95 1 1 1 0 0 68 + * 1 0 1 0 1 90 1 1 1 0 1 65 + * 1 0 1 1 0 86 1 1 1 0 1 63 + * 1 0 1 1 1 83 1 1 1 1 1 61 + * + * DIVA [1:0] : division ratio for internal clocks when Normal mode. + * + * DIVA [1:0] Division Ratio + * 0 0 fosc + * 0 1 fosc / 2 + * 1 0 fosc / 4 + * 1 1 fosc / 8 + * + */ + u8 frc[ILI9341_FRC_LEN]; + /* TODO: need comments for this register */ + u8 prc; + /* + * B6h DISCTRL (Display Function Control) + * D/CX RDX WRX D17-8 D7 D6 D5 D4 D3 D2 D1 D0 HEX + * Command 0 1 M XX 1 0 1 1 0 1 1 0 B6h + * 1st Parameter 1 1 M XX 0 0 0 0 PTG[1:0] PT[1:0] 0A + * 2nd Parameter 1 1 M XX REV GS SS SM ISC[3:0] 82 + * 3rd Parameter 1 1 M XX 0 0 NL[5:0] 27 + * 4th Parameter 1 1 M XX 0 0 PCDIV[5:0] XX + * + * PTG [1:0]: Set the scan mode in non-display area. + * PTG1 | PTG0 | Gate outputs in | Source outputs in | VCOM output + * non-display area | non-display area | + * 1 0 Interval scan Set with the PT[2:0] bits + * + * PT [1:0]: Determine source/VCOM output in a non-display area in the + * partial display mode. + * 1 0 AGND AGND AGND AGND + * + * REV: Select whether the liquid crystal type is normally white type + * or normally black type. + * REV Liquid crystal type + * 0 Normally black + * 1 Normally white + * + * SS: Select the shift direction of outputs from the source driver. + * SS Source Output Scan Direction + * 0 S1 -> S720 + * 1 S720 -> S1 + * + * GS: Sets the direction of scan by the gate driver in the range + * determined by SCN [4:0] and NL [4:0]. The scan direction + * determined by GS = 0 can be reversed by setting GS = 1. + * + * GS Gate Output Scan Direction + * 0 G1 -> G320 + * 1 G320 -> G1 + */ + u8 dfc_1[ILI9341_DFC_1_LEN]; + /* + * Power Control 1 (C0h) + * VRH [5:0]: Set the GVDD level, which is a reference level for the + * VCOM level and the grayscale voltage level. + * + * VRH[5:0] GVDD VRH[5:0] GVDD + * 0 0 0 0 0 0 Setting prohibited 1 0 0 0 0 0 4.45 V + * 0 0 0 0 0 1 Setting prohibited 1 0 0 0 0 1 4.50 V + * 0 0 0 0 1 0 Setting prohibited 1 0 0 0 1 0 4.55 V + * 0 0 0 0 1 1 3.00 V 1 0 0 0 1 1 4.60 V + * 0 0 0 1 0 0 3.05 V 1 0 0 1 0 0 4.65 V + * 0 0 0 1 0 1 3.10 V 1 0 0 1 0 1 4.70 V + * 0 0 0 1 1 0 3.15 V 1 0 0 1 1 0 4.75 V + * 0 0 0 1 1 1 3.20 V 1 0 0 1 1 1 4.80 V + * 0 0 1 0 0 0 3.25 V 1 0 1 0 0 0 4.85 V + * 0 0 1 0 0 1 3.30 V 1 0 1 0 0 1 4.90 V + * 0 0 1 0 1 0 3.35 V 1 0 1 0 1 0 4.95 V + * 0 0 1 0 1 1 3.40 V 1 0 1 0 1 1 5.00 V + * 0 0 1 1 0 0 3.45 V 1 0 1 1 0 0 5.05 V + * 0 0 1 1 0 1 3.50 V 1 0 1 1 0 1 5.10 V + * 0 0 1 1 1 0 3.55 V 1 0 1 1 1 0 5.15 V + * 0 0 1 1 1 1 3.60 V 1 0 1 1 1 1 5.20 V + * 0 1 0 0 0 0 3.65 V 1 1 0 0 0 0 5.25 V + * 0 1 0 0 0 1 3.70 V 1 1 0 0 0 1 5.30 V + * 0 1 0 0 1 0 3.75 V 1 1 0 0 1 0 5.35 V + * 0 1 0 0 1 1 3.80 V 1 1 0 0 1 1 5.40 V + * 0 1 0 1 0 0 3.85 V 1 1 0 1 0 0 5.45 V + * 0 1 0 1 0 1 3.90 V 1 1 0 1 0 1 5.50 V + * 0 1 0 1 1 0 3.95 V 1 1 0 1 1 0 5.55 V + * 0 1 0 1 1 1 4.00 V 1 1 0 1 1 1 5.60 V + * 0 1 1 0 0 0 4.05 V 1 1 1 0 0 0 5.65 V + * 0 1 1 0 0 1 4.10 V 1 1 1 0 0 1 5.70 V + * 0 1 1 0 1 0 4.15 V 1 1 1 0 1 0 5.75 V + * 0 1 1 0 1 1 4.20 V 1 1 1 0 1 1 5.80 V + * 0 1 1 1 0 0 4.25 V 1 1 1 1 0 0 5.85 V + * 0 1 1 1 0 1 4.30 V 1 1 1 1 0 1 5.90 V + * 0 1 1 1 1 0 4.35 V 1 1 1 1 1 0 5.95 V + * 0 1 1 1 1 1 4.40 V 1 1 1 1 1 1 6.00 V + */ + u8 power_1; + /* + * Power Control 2 (C1h) + * + * BT [2:0]: Sets the factor used in the step-up circuits. + * Select the optimal step-up factor for the operating voltage. To + * reduce power consumption, set a smaller factor. + * + * BT[2:0] AVDD VGH VGL + * 0 0 0 VCI x 2 VCI x 7 VCI x 4 + * 0 0 1 VCI x 3 + * 0 1 0 VCI x 6 VCI x 4 + * 0 1 1 VCI x 3 + * + */ + u8 power_2; + /* + * VCOM Control 1(C5h) + * + * VMH [6:0] : Set the VCOMH voltage + * + * VMH[6:0] VCOMH(V) VMH[6:0] VCOMH(V) VMH[6:0] VCOMH(V) VMH[6:0] VCOMH + * 0000000 2.700 0100000 3.500 1000000 4.300 1100000 5.100 + * 0000001 2.725 0100001 3.525 1000001 4.325 1100001 5.125 + * 0000010 2.750 0100010 3.550 1000010 4.350 1100010 5.150 + * 0000011 2.775 0100011 3.575 1000011 4.375 1100011 5.175 + * 0000100 2.800 0100100 3.600 1000100 4.400 1100100 5.200 + * 0000101 2.825 0100101 3.625 1000101 4.425 1100101 5.225 + * 0000110 2.850 0100110 3.650 1000110 4.450 1100110 5.250 + * 0000111 2.875 0100111 3.675 1000111 4.475 1100111 5.275 + * 0001000 2.900 0101000 3.700 1001000 4.500 1101000 5.300 + * 0001001 2.925 0101001 3.725 1001001 4.525 1101001 5.325 + * 0001010 2.950 0101010 3.750 1001010 4.550 1101010 5.350 + * 0001011 2.975 0101011 3.775 1001011 4.575 1101011 5.375 + * 0001100 3.000 0101100 3.800 1001100 4.600 1101100 5.400 + * 0001101 3.025 0101101 3.825 1001101 4.625 1101101 5.425 + * 0001110 3.050 0101110 3.850 1001110 4.650 1101110 5.450 + * 0001111 3.075 0101111 3.875 1001111 4.675 1101111 5.475 + * 0010000 3.100 0110000 3.900 1010000 4.700 1110000 5.500 + * 0010001 3.125 0110001 3.925 1010001 4.725 1110001 5.525 + * 0010010 3.150 0110010 3.950 1010010 4.750 1110010 5.550 + * 0010011 3.175 0110011 3.975 1010011 4.775 1110011 5.575 + * 0010100 3.200 0110100 4.000 1010100 4.800 1110100 5.600 + * 0010101 3.225 0110101 4.025 1010101 4.825 1110101 5.625 + * 0010110 3.250 0110110 4.050 1010110 4.850 1110110 5.650 + * 0010111 3.275 0110111 4.075 1010111 4.875 1110111 5.675 + * 0011000 3.300 0111000 4.100 1011000 4.900 1111000 5.700 + * 0011001 3.325 0111001 4.125 1011001 4.925 1111001 5.725 + * 0011010 3.350 0111010 4.150 1011010 4.950 1111010 5.750 + * 0011011 3.375 0111011 4.175 1011011 4.975 1111011 5.775 + * 0011100 3.400 0111100 4.200 1011100 5.000 1111100 5.800 + * 0011101 3.425 0111101 4.225 1011101 5.025 1111101 5.825 + * 0011110 3.450 0111110 4.250 1011110 5.050 1111110 5.850 + * 0011111 3.475 0111111 4.275 1011111 5.075 1111111 5.875 + * + * VML[6:0] : Set the VCOML voltage + * + * VML[6:0] VCOML(V) VML[6:0] VCOML(V) VML[6:0] VCOML(V) VML[6:0] VCOML + * 0000000 -2.500 0100000 -1.700 1000000 -0.900 1100000 -0.100 + * 0000001 -2.475 0100001 -1.675 1000001 -0.875 1100001 -0.075 + * 0000010 -2.450 0100010 -1.650 1000010 -0.850 1100010 -0.050 + * 0000011 -2.425 0100011 -1.625 1000011 -0.825 1100011 -0.025 + * 0000100 -2.400 0100100 -1.600 1000100 -0.800 1100100 0 + * 0000101 -2.375 0100101 -1.575 1000101 -0.775 1100101 Reserved + * 0000110 -2.350 0100110 -1.550 1000110 -0.750 1100110 Reserved + * 0000111 -2.325 0100111 -1.525 1000111 -0.725 1100111 Reserved + * 0001000 -2.300 0101000 -1.500 1001000 -0.700 1101000 Reserved + * 0001001 -2.275 0101001 -1.475 1001001 -0.675 1101001 Reserved + * 0001010 -2.250 0101010 -1.450 1001010 -0.650 1101010 Reserved + * 0001011 -2.225 0101011 -1.425 1001011 -0.625 1101011 Reserved + * 0001100 -2.200 0101100 -1.400 1001100 -0.600 1101100 Reserved + * 0001101 -2.175 0101101 -1.375 1001101 -0.575 1101101 Reserved + * 0001110 -2.150 0101110 -1.350 1001110 -0.550 1101110 Reserved + * 0001111 -2.125 0101111 -1.325 1001111 -0.525 1101111 Reserved + * 0010000 -2.100 0110000 -1.300 1010000 -0.500 1110000 Reserved + * 0010001 -2.075 0110001 -1.275 1010001 -0.475 1110001 Reserved + * 0010010 -2.050 0110010 -1.250 1010010 -0.450 1110010 Reserved + * 0010011 -2.025 0110011 -1.225 1010011 -0.425 1110011 Reserved + * 0010100 -2.000 0110100 -1.200 1010100 -0.400 1110100 Reserved + * 0010101 -1.975 0110101 -1.175 1010101 -0.375 1110101 Reserved + * 0010110 -1.950 0110110 -1.150 1010110 -0.350 1110110 Reserved + * 0010111 -1.925 0110111 -1.125 1010111 -0.325 1110111 Reserved + * 0011000 -1.900 0111000 -1.100 1011000 -0.300 1111000 Reserved + * 0011001 -1.875 0111001 -1.075 1011001 -0.275 1111001 Reserved + * 0011010 -1.850 0111010 -1.050 1011010 -0.250 1111010 Reserved + * 0011011 -1.825 0111011 -1.025 1011011 -0.225 1111011 Reserved + * 0011100 -1.800 0111100 -1.000 1011100 -0.200 1111100 Reserved + * 0011101 -1.775 0111101 -0.975 1011101 -0.175 1111101 Reserved + * 0011110 -1.750 0111110 -0.950 1011110 -0.150 1111110 Reserved + * 0011111 -1.725 0111111 -0.925 1011111 -0.125 1111111 Reserved + */ + u8 vcom_1[ILI9341_VCOM_1_LEN]; + /* + * VCOM Control 2(C7h) + * + * C7h VMCTRL1 (VCOM Control 1) + * D/CX RDX WRX D17-8 D7 D6 D5 D4 D3 D2 D1 D0 HEX + * Command 0 1 M XX 1 1 0 0 0 1 1 1 C7h + * Parameter 1 1 M XX nVM VMF[6:0] C0 + * + * nVM: nVM equals to “0” after power on reset and VCOM offset + * equals to program MTP value. When nVM set to “1”, setting + * of VMF [6:0] becomes valid and VCOMH/VCOML can be adjusted. + * + * VMF [6:0]: Set the VCOM offset voltage. + */ + u8 vcom_2; + /* + * Memory Access Control (36h) + * + * 36h MADCTL (Memory Access Control) + * D/CX RDX WRX D17-8 D7 D6 D5 D4 D3 D2 D1 D0 HEX + * Command 0 1 M XX 0 0 1 1 0 1 1 0 36h + * Parameter 1 1 M XX MY MX MV ML BGR MH 0 0 00 + * + * This command defines read/write scanning direction of frame memory. + * This command makes no change on the other driver status. + * + * Bit Name Description + * MY Row Address Order + * MX Column Address Order + * MV Row / Column Exchange These 3 bits control MCU to memory + * write/read direction. + * ML Vertical Refresh Order LCD vertical refresh direction control. + * BGR RGB-BGR Order Color selector switch control + * (0=RGB color filter panel, 1=BGR + * color filter panel) + * MH Horizontal Refresh ORDER LCD horizontal refreshing + * direction control. + * + * Note: When BGR bit is changed, the new setting is active + * immediately without update the content in Frame Memory again. + * + */ + u8 address_mode; + /* TODO: need comments for this register */ + u8 g3amma_en; + /* RGB Interface Signal Control (B0h) + * + * B0h IFMODE (Interface Mode Control) + * D/CX RDX WRX D17-8 D7 D6 D5 D4 D3 D2 D1 D0 HEX + * Command 0 1 M XX 1 0 1 1 0 0 0 0 B0h + * Parameter 1 1 M XX ByPass_MODE RCM[1] RCM[0] 0 VSPL HSPL DPL EPL 40 + * + * Sets the operation status of the display interface. The setting + * becomes effective as soon as the command is received. + * EPL: DE polarity (“0”= High enable for RGB interface, “1”= Low + * enable for RGB interface) + * + * DPL: DOTCLK polarity set (“0”= data fetched at the rising time, + * “1”= data fetched at the falling time) + * + * HSPL: HSYNC polarity (“0”= Low level sync clock, “1”= High + * level sync clock) + * + * VSPL: VSYNC polarity (“0”= Low level sync clock, “1”= High + * level sync clock) + * + * RCM [1:0]: RGB interface selection (refer to the RGB interface + * section). + * + * ByPass_MODE: Select display data path whether Memory or Direct to + * Shift register when RGB Interface is used. + * + * ByPass_MODE Display Data Path + * 0 Direct to Shift Register (default) + * 1 Memory + */ + u8 rgb_interface; + /* refer to dfc_1 */ + u8 dfc_2[ILI9341_DFC_2_LEN]; + /* + * Column Address Set (2Ah) + * This command is used to define area of frame memory where MCU can + * access. This command makes no change on the + * other driver status. The values of SC [15:0] and EC [15:0] are + * referred when RAMWR command comes. Each value + * represents one column line in the Frame Memory. + */ + u8 column_addr[ILI9341_COLUMN_ADDR_LEN]; + /* + * Page Address Set (2Bh) + * This command is used to define area of frame memory where MCU can + * access. This command makes no change on the + * other driver status. The values of SP [15:0] and EP [15:0] are + * referred when RAMWR command comes. Each value + * represents one Page line in the Frame Memory. + */ + u8 page_addr[ILI9341_PAGE_ADDR_LEN]; + /* + * Interface Control (F6h) + * + * F6h IFCTL (16bits Data Format Selection) + * D/CX RDX WRX D17-8 D7 D6 D5 D4 D3 D2 D1 D0 HEX + * Command 0 1 M XX 1 1 1 1 0 1 1 0 F6h + * 1stParameter 1 1 M XX MY MX MV + * _EOR _EOR _EOR 0 BGR_EOR 0 0 WE MODE 01 + * 2ndParameter 1 1 M XX 0 0 EPF[1] EPF[0] 0 0 MDT[1] MDT[0] 00 + * 3rdParameter 1 1 M XX 0 0 ENDIAN 0 DM[1] DM[0] RM RIM 00 + * + */ + u8 interface[ILI9341_INTERFACE_LEN]; + /* + * This command sets the pixel format for the RGB image data used by + * the interface. DPI [2:0] is the pixel format select of RGB + * interface and DBI [2:0] is the pixel format of MCU interface. If a + * particular interface, either RGB interface or MCU interface, is + * not used then the corresponding bits in the parameter are ignored. + * The pixel format is shown in the table below. + * + * DPI[2:0] RGB Interface Format DBI[2:0] MCU Interface Format + * 0 0 0 Reserved 0 0 0 Reserved + * 0 0 1 Reserved 0 0 1 Reserved + * 0 1 0 Reserved 0 1 0 Reserved + * 0 1 1 Reserved 0 1 1 Reserved + * 1 0 0 Reserved 1 0 0 Reserved + * 1 0 1 16 bits / pixel 1 0 1 16 bits / pixel + * 1 1 0 18 bits / pixel 1 1 0 18 bits / pixel + * 1 1 1 Reserved 1 1 1 Reserved + * + */ + u8 pixel_format; + /* + * This command is used to select the desired Gamma curve for the + * current display. A maximum of 4 fixed gamma curves can + * be selected. The curve is selected by setting the appropriate bit + * in the parameter as described in the Table: + * + * GC [7:0] Curve Selected + * 01h Gamma curve 1 (G2.2) + * 02h --- + * 04h --- + * 08h --- + */ + u8 gamma_curve; + /* + * Positive Gamma Correction (E0h) + * + * Set the gray scale voltage to adjust the gamma characteristics of + * the TFT panel. + */ + u8 pgamma[ILI9341_PGAMMA_LEN]; + /* + * Negative Gamma Correction (E1h) + * + * Set the gray scale voltage to adjust the gamma characteristics of + * the TFT panel. + */ + u8 ngamma[ILI9341_NGAMMA_LEN]; +}; + +struct ili9341 { + struct device *dev; + const struct ili9341_config *conf; + struct drm_panel panel; + struct gpio_desc *reset_gpio; + struct gpio_desc *dc_gpio; + u32 max_spi_speed; + struct regulator *vcc; +}; + +/* + * The Stm32f429-disco board has a panel ili9341 connected to ltdc controller + */ +static const struct ili9341_config ili9341_stm32f429_disco_data = { + .max_spi_speed = 10000000, + .mode = { + .clock = 6100, + .hdisplay = 240, + .hsync_start = 240 + 10,/* hfp 10 */ + .hsync_end = 240 + 10 + 10,/* hsync 10 */ + .htotal = 240 + 10 + 10 + 20,/* hbp 20 */ + .vdisplay = 320, + .vsync_start = 320 + 4,/* vfp 4 */ + .vsync_end = 320 + 4 + 2,/* vsync 2 */ + .vtotal = 320 + 4 + 2 + 2,/* vbp 2 */ + .vrefresh = 60, + .flags = 0, + .width_mm = 65, + .height_mm = 50, + .type = DRM_MODE_TYPE_DRIVER | DRM_MODE_TYPE_PREFERRED, + }, + /* TODO: need comments for this register */ + .ca = {0xc3, 0x08, 0x50}, + /* TODO: need comments for this register */ + .power_b = {0x00, 0xc1, 0x30}, + /* TODO: need comments for this register */ + .power_seq = {0x64, 0x03, 0x12, 0x81}, + /* TODO: need comments for this register */ + .dtca = {0x85, 0x00, 0x78}, + /* TODO: need comments for this register */ + .power_a = {0x39, 0x2c, 0x00, 0x34, 0x02}, + /* TODO: need comments for this register */ + .prc = 0x20, + /* TODO: need comments for this register */ + .dtcb = {0x00, 0x00}, + /* 0x00 fosc, 0x1b 70hz */ + .frc = {0x00, 0x1b}, + /* 0x0a Interval scan, AGND AGND AGND AGND + * 0xa2 Normally white, G1 -> G320, S720 -> S1, + * Scan Cycle 5 frames,85ms + */ + .dfc_1 = {0x0a, 0xa2}, + /* 0x10 3.65v */ + .power_1 = 0x10, + /* 0x10 AVDD=vci*2, VGH=vci*7, VGL=-vci*4 */ + .power_2 = 0x10, + /* 0x45 VCOMH 4.425v, 0x15 VCOML -1.975*/ + .vcom_1 = {0x45, 0x15}, + /* 0x90 offset voltage, VMH-48, VML-48 */ + .vcom_2 = 0x90, + /* 0xc8 Row Address Order, Column Address Order + * BGR 1 + */ + .address_mode = 0xc8, + .g3amma_en = 0x00, + /* 0xc2 + * Display Data Path: Memory + * RGB: DE mode + * DOTCLK polarity set (data fetched at the falling time) + */ + .rgb_interface = ILI9341_RGB_DISP_PATH_MEM | + ILI9341_RGB_DE_MODE | + ILI9341_RGB_DPL, + /* + * 0x0a + * Gate outputs in non-display area: Interval scan + * Determine source/VCOM output in a non-display area in the partial + * display mode: AGND AGND AGND AGND + * + * 0xa7 + * Scan Cycle: 15 frames + * fFLM = 60Hz: 255ms + * Liquid crystal type: Normally white + * Gate Output Scan Direction: G1 -> G320 + * Source Output Scan Direction: S720 -> S1 + * + * 0x27 + * LCD Driver Line: 320 lines + * + * 0x04 + * PCDIV: 4 + */ + .dfc_2 = {0x0a, 0xa7, 0x27, 0x04}, + /* column address: 240 */ + .column_addr = {0x00, 0x00, (ILI9341_COLUMN_ADDR >> 4) & 0xff, + ILI9341_COLUMN_ADDR & 0xff}, + /* page address: 320 */ + .page_addr = {0x00, 0x00, (ILI9341_PAGE_ADDR >> 4) & 0xff, + ILI9341_PAGE_ADDR & 0xff}, + /* Memory write control: When the transfer number of data exceeds + * (EC-SC+1)*(EP-SP+1), the column and page number will be + * reset, and the exceeding data will be written into the following + * column and page. + * Display Operation Mode: RGB Interface Mode + * Interface for RAM Access: RGB interface + * 16- bit RGB interface (1 transfer/pixel) + */ + .interface = {ILI9341_IF_WE_MODE, 0x00, + ILI9341_IF_DM_RGB | ILI9341_IF_RM_RGB}, + /* DPI: 16 bits / pixel */ + .pixel_format = ILI9341_PIXEL_DPI_16_BITS, + /* Curve Selected: Gamma curve 1 (G2.2) */ + .gamma_curve = ILI9341_GAMMA_CURVE_1, + .pgamma = {0x0f, 0x29, 0x24, 0x0c, 0x0e, + 0x09, 0x4e, 0x78, 0x3c, 0x09, + 0x13, 0x05, 0x17, 0x11, 0x00}, + .ngamma = {0x00, 0x16, 0x1b, 0x04, 0x11, + 0x07, 0x31, 0x33, 0x42, 0x05, + 0x0c, 0x0a, 0x28, 0x2f, 0x0f}, +}; +static inline struct ili9341 *panel_to_ili9341(struct drm_panel *panel) +{ + return container_of(panel, struct ili9341, panel); +} + +int ili9341_spi_transfer(struct spi_device *spi, u32 speed_hz, + u8 bpw, const void *buf, size_t len) +{ + size_t max_chunk = spi_max_transfer_size(spi); + struct spi_transfer tr = { + .bits_per_word = bpw, + .speed_hz = speed_hz, + .len = len, + }; + struct spi_message m; + size_t chunk; + int ret; + + spi_message_init_with_transfers(&m, &tr, 1); + + while (len) { + chunk = min(len, max_chunk); + + tr.tx_buf = buf; + tr.len = chunk; + buf += chunk; + len -= chunk; + + ret = spi_sync(spi, &m); + if (ret) { + dev_err(&spi->dev, "spi_sync error: %d\n", ret); + return ret; + } + } + return 0; +} + +static int _ili9341_command(struct ili9341 *ili, u8 cmd, const void *data, + size_t count) +{ + struct spi_device *spi = to_spi_device(ili->dev); + int ret = 0; + + gpiod_set_value_cansleep(ili->dc_gpio, 0); + + ret = ili9341_spi_transfer(spi, ili->max_spi_speed, 8, + (const void *)&cmd, 1); + if (ret || data == NULL || count == 0) + return ret; + + gpiod_set_value_cansleep(ili->dc_gpio, 1); + + return ili9341_spi_transfer(spi, ili->max_spi_speed, 8, + data, count); +} + +static int ili9341_dpi_init(struct ili9341 *ili) +{ + int ret; + ret = _ili9341_command(ili, 0xca, + ili->conf->ca, + ILI9341_CA_LEN); + if (ret) + return ret; + + ret = _ili9341_command(ili, ILI9341_POWERB, + ili->conf->power_b, + ILI9341_POWER_B_LEN); + if (ret) + return ret; + + ret = _ili9341_command(ili, ILI9341_POWER_SEQ, + ili->conf->power_seq, + ILI9341_POWER_SEQ_LEN); + if (ret) + return ret; + + ret = _ili9341_command(ili, ILI9341_DTCA, + ili->conf->dtca, + ILI9341_DTCA_LEN); + if (ret) + return ret; + + ret = _ili9341_command(ili, ILI9341_POWERA, + ili->conf->power_a, + ILI9341_POWER_A_LEN); + if (ret) + return ret; + + ret = _ili9341_command(ili, ILI9341_PRC, + &ili->conf->prc, + 1); + if (ret) + return ret; + + ret = _ili9341_command(ili, ILI9341_DTCB, + ili->conf->dtcb, + ILI9341_DTCB_LEN); + if (ret) + return ret; + + ret = _ili9341_command(ili, ILI9341_FRC, + ili->conf->frc, + ILI9341_FRC_LEN); + if (ret) + return ret; + + ret = _ili9341_command(ili, ILI9341_DFC, + ili->conf->dfc_1, + ILI9341_DFC_1_LEN); + if (ret) + return ret; + + ret = _ili9341_command(ili, ILI9341_POWER1, + &ili->conf->power_1, + 1); + if (ret) + return ret; + + ret = _ili9341_command(ili, ILI9341_POWER2, + &ili->conf->power_2, + 1); + if (ret) + return ret; + + ret = _ili9341_command(ili, ILI9341_VCOM1, + ili->conf->vcom_1, + ILI9341_VCOM_1_LEN); + if (ret) + return ret; + + ret = _ili9341_command(ili, ILI9341_VCOM2, + &ili->conf->vcom_2, + 1); + if (ret) + return ret; + + ret = _ili9341_command(ili, MIPI_DCS_SET_ADDRESS_MODE, + &ili->conf->address_mode, + 1); + if (ret) + return ret; + + ret = _ili9341_command(ili, ILI9341_3GAMMA_EN, + &ili->conf->g3amma_en, + 1); + if (ret) + return ret; + + ret = _ili9341_command(ili, ILI9341_RGB_INTERFACE, + &ili->conf->rgb_interface, + 1); + if (ret) + return ret; + + ret = _ili9341_command(ili, ILI9341_DFC, + ili->conf->dfc_2, + ILI9341_DFC_2_LEN); + if (ret) + return ret; + + ret = _ili9341_command(ili, MIPI_DCS_SET_COLUMN_ADDRESS, + ili->conf->column_addr, + ILI9341_COLUMN_ADDR_LEN); + if (ret) + return ret; + + ret = _ili9341_command(ili, MIPI_DCS_SET_PAGE_ADDRESS, + ili->conf->page_addr, + ILI9341_PAGE_ADDR_LEN); + if (ret) + return ret; + + ret = _ili9341_command(ili, ILI9341_INTERFACE, + ili->conf->interface, + ILI9341_INTERFACE_LEN); + if (ret) + return ret; + + ret = _ili9341_command(ili, MIPI_DCS_SET_PIXEL_FORMAT, + &ili->conf->pixel_format, + 1); + if (ret) + return ret; + + ret = ili9341_command(ili, MIPI_DCS_WRITE_MEMORY_START); + if (ret) + return ret; + + msleep(200); + ret = _ili9341_command(ili, MIPI_DCS_SET_GAMMA_CURVE, + &ili->conf->gamma_curve, + 1); + if (ret) + return ret; + + ret = _ili9341_command(ili, ILI9341_PGAMMA, + ili->conf->pgamma, + ILI9341_PGAMMA_LEN); + if (ret) + return ret; + + ret = _ili9341_command(ili, ILI9341_NGAMMA, + ili->conf->ngamma, + ILI9341_NGAMMA_LEN); + if (ret) + return ret; + + ret = ili9341_command(ili, MIPI_DCS_EXIT_SLEEP_MODE); + if (ret) + return ret; + + msleep(200); + ret = ili9341_command(ili, MIPI_DCS_SET_DISPLAY_ON); + if (ret) + return ret; + + ret = ili9341_command(ili, MIPI_DCS_WRITE_MEMORY_START); + if (ret) + return ret; + + + dev_info(ili->dev, "initialized display rgb interface\n"); + + return 0; +} + +static int ili9341_dpi_power_on(struct ili9341 *ili) +{ + int ret = 0; + + /* Assert RESET */ + gpiod_set_value(ili->reset_gpio, 1); + + /* Enable power */ + if (!IS_ERR(ili->vcc)) { + ret = regulator_enable(ili->vcc); + if (ret < 0) { + dev_err(ili->dev, "unable to enable vcc\n"); + return ret; + } + } + msleep(20); + + /* De-assert RESET */ + gpiod_set_value(ili->reset_gpio, 0); + msleep(10); + + return 0; +} + +static int ili9341_dpi_power_off(struct ili9341 *ili) +{ + /* Assert RESET */ + gpiod_set_value(ili->reset_gpio, 1); + + /* Disable power */ + if (!IS_ERR(ili->vcc)) + return regulator_disable(ili->vcc); + + return 0; +} + +static int ili9341_dpi_disable(struct drm_panel *panel) +{ + struct ili9341 *ili = panel_to_ili9341(panel); + + ili9341_command(ili, MIPI_DCS_SET_DISPLAY_OFF); + + return 0; +} + +static int ili9341_dpi_unprepare(struct drm_panel *panel) +{ + struct ili9341 *ili = panel_to_ili9341(panel); + + return ili9341_dpi_power_off(ili); +} + +static int ili9341_dpi_prepare(struct drm_panel *panel) +{ + struct ili9341 *ili = panel_to_ili9341(panel); + int ret; + + ret = ili9341_dpi_power_on(ili); + if (ret < 0) + return ret; + + ret = ili9341_dpi_init(ili); + if (ret < 0) + ili9341_dpi_unprepare(panel); + + return ret; +} + +static int ili9341_dpi_enable(struct drm_panel *panel) +{ + struct ili9341 *ili = panel_to_ili9341(panel); + + ili9341_command(ili, MIPI_DCS_SET_DISPLAY_ON); + + return 0; +} + +static int ili9341_dpi_get_modes(struct drm_panel *panel, + struct drm_connector *connector) +{ + struct ili9341 *ili = panel_to_ili9341(panel); + struct drm_device *drm = connector->dev; + struct drm_display_mode *mode; + struct drm_display_info *info; + + info = &connector->display_info; + info->width_mm = ili->conf->mode.width_mm; + info->height_mm = ili->conf->mode.height_mm; + + if (ili->conf->rgb_interface & ILI9341_RGB_DPL) + info->bus_flags |= DRM_BUS_FLAG_PIXDATA_DRIVE_POSEDGE; + else + info->bus_flags |= DRM_BUS_FLAG_PIXDATA_DRIVE_NEGEDGE; + + if (ili->conf->rgb_interface & ILI9341_RGB_EPL) + info->bus_flags |= DRM_BUS_FLAG_DE_HIGH; + else + info->bus_flags |= DRM_BUS_FLAG_DE_LOW; + + mode = drm_mode_duplicate(drm, &ili->conf->mode); + if (!mode) { + DRM_ERROR("bad mode or failed to add mode\n"); + return -EINVAL; + } + drm_mode_set_name(mode); + + /* Set up the polarity */ + if (ili->conf->rgb_interface & ILI9341_RGB_HSPL) + mode->flags |= DRM_MODE_FLAG_PHSYNC; + else + mode->flags |= DRM_MODE_FLAG_NHSYNC; + + if (ili->conf->rgb_interface & ILI9341_RGB_VSPL) + mode->flags |= DRM_MODE_FLAG_PVSYNC; + else + mode->flags |= DRM_MODE_FLAG_NVSYNC; + + drm_mode_probed_add(connector, mode); + + return 1; /* Number of modes */ +} + +static const struct drm_panel_funcs ili9341_dpi_funcs = { + .disable = ili9341_dpi_disable, + .unprepare = ili9341_dpi_unprepare, + .prepare = ili9341_dpi_prepare, + .enable = ili9341_dpi_enable, + .get_modes = ili9341_dpi_get_modes, +}; + +static int ili9341_dpi_probe(struct spi_device *spi) +{ + int ret; + struct device *dev = &spi->dev; + struct ili9341 *ili; + + ili = devm_kzalloc(dev, sizeof(struct ili9341), GFP_KERNEL); + if (!ili) + return -ENOMEM; + + spi_set_drvdata(spi, ili); + + ili->dev = dev; + /* + * Every new incarnation of this display must have a unique + * data entry for the system in this driver. + */ + ili->conf = of_device_get_match_data(dev); + if (!ili->conf) { + dev_err(dev, "missing device configuration\n"); + return -ENODEV; + } + + ili->vcc = devm_regulator_get_optional(dev, "vcc"); + if (IS_ERR(ili->vcc)) + dev_err(dev, "get optional vcc failed\n"); + + ili->reset_gpio = devm_gpiod_get_optional(dev, "reset", + GPIOD_OUT_HIGH); + if (IS_ERR(ili->reset_gpio)) { + dev_err(dev, "failed to get RESET GPIO\n"); + return PTR_ERR(ili->reset_gpio); + } + + ili->dc_gpio = devm_gpiod_get(dev, "dc", GPIOD_OUT_LOW); + if (IS_ERR(ili->dc_gpio)) { + dev_err(dev, "failed to get DC GPIO\n"); + return PTR_ERR(ili->dc_gpio); + } + + spi->bits_per_word = 8; + ret = spi_setup(spi); + if (ret < 0) { + dev_err(dev, "spi setup failed.\n"); + return ret; + } + + ili->max_spi_speed = ili->conf->max_spi_speed; + + drm_panel_init(&ili->panel, dev, &ili9341_dpi_funcs, + DRM_MODE_CONNECTOR_DPI); + + return drm_panel_add(&ili->panel); +} + + + +static void ili9341_dbi_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, MIPI_DCS_SET_DISPLAY_OFF); + + mipi_dbi_command(dbi, ILI9341_POWERB, 0x00, 0xc1, 0x30); + mipi_dbi_command(dbi, ILI9341_POWER_SEQ, 0x64, 0x03, 0x12, 0x81); + mipi_dbi_command(dbi, ILI9341_DTCA, 0x85, 0x00, 0x78); + mipi_dbi_command(dbi, ILI9341_POWERA, 0x39, 0x2c, 0x00, 0x34, 0x02); + mipi_dbi_command(dbi, ILI9341_PRC, ILI9341_DBI_PRC_NORMAL); + mipi_dbi_command(dbi, ILI9341_DTCB, 0x00, 0x00); + + /* Power Control */ + mipi_dbi_command(dbi, ILI9341_POWER1, ILI9341_DBI_VCOMH_4P6V); + mipi_dbi_command(dbi, ILI9341_POWER2, ILI9341_DBI_PWR_2_DEFAULT); + /* VCOM */ + mipi_dbi_command(dbi, ILI9341_VCOM1, ILI9341_DBI_VCOM_1_VMH_4P25V, + ILI9341_DBI_VCOM_1_VML_1P5V); + mipi_dbi_command(dbi, ILI9341_VCOM2, ILI9341_DBI_VCOM_2_DEC_58); + + /* Memory Access Control */ + mipi_dbi_command(dbi, MIPI_DCS_SET_PIXEL_FORMAT, + MIPI_DCS_PIXEL_FMT_16BIT); + + /* Frame Rate */ + mipi_dbi_command(dbi, ILI9341_FRC, ILI9341_DBI_FRC_DIVA & 0x03, + ILI9341_DBI_FRC_RTNA & 0x1f); + + /* Gamma */ + mipi_dbi_command(dbi, ILI9341_3GAMMA_EN, 0x00); + mipi_dbi_command(dbi, MIPI_DCS_SET_GAMMA_CURVE, ILI9341_GAMMA_CURVE_1); + mipi_dbi_command(dbi, ILI9341_PGAMMA, + 0x0f, 0x31, 0x2b, 0x0c, 0x0e, 0x08, 0x4e, 0xf1, + 0x37, 0x07, 0x10, 0x03, 0x0e, 0x09, 0x00); + mipi_dbi_command(dbi, ILI9341_NGAMMA, + 0x00, 0x0e, 0x14, 0x03, 0x11, 0x07, 0x31, 0xc1, + 0x48, 0x08, 0x0f, 0x0c, 0x31, 0x36, 0x0f); + + /* DDRAM */ + mipi_dbi_command(dbi, ILI9341_ETMOD, ILI9341_DBI_EMS_GAS | + ILI9341_DBI_EMS_DTS | + ILI9341_DBI_EMS_GON); + + /* Display */ + mipi_dbi_command(dbi, ILI9341_DFC, 0x08, 0x82, 0x27, 0x00); + mipi_dbi_command(dbi, MIPI_DCS_EXIT_SLEEP_MODE); + msleep(100); + + mipi_dbi_command(dbi, MIPI_DCS_SET_DISPLAY_ON); + msleep(100); + +out_enable: + switch (dbidev->rotation) { + default: + addr_mode = ILI9341_MADCTL_MX; + break; + case 90: + addr_mode = ILI9341_MADCTL_MV; + break; + case 180: + addr_mode = ILI9341_MADCTL_MY; + break; + case 270: + addr_mode = ILI9341_MADCTL_MV | ILI9341_MADCTL_MY | + ILI9341_MADCTL_MX; + break; + } + addr_mode |= ILI9341_MADCTL_BGR; + mipi_dbi_command(dbi, MIPI_DCS_SET_ADDRESS_MODE, addr_mode); + mipi_dbi_enable_flush(dbidev, crtc_state, plane_state); + DRM_DEBUG_KMS("initialized display serial interface\n"); +out_exit: + drm_dev_exit(idx); +} + +static const struct drm_simple_display_pipe_funcs ili9341_dbi_funcs = { + .enable = ili9341_dbi_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 ili9341_dbi_mode = { + DRM_SIMPLE_MODE(240, 320, 37, 49), +}; + +DEFINE_DRM_GEM_CMA_FOPS(ili9341_dbi_fops); + +static struct drm_driver ili9341_dbi_driver = { + .driver_features = DRIVER_GEM | DRIVER_MODESET | DRIVER_ATOMIC, + .fops = &ili9341_dbi_fops, + .release = mipi_dbi_release, + DRM_GEM_CMA_VMAP_DRIVER_OPS, + .debugfs_init = mipi_dbi_debugfs_init, + .name = "ili9341", + .desc = "Ilitek ILI9341", + .date = "20180514", + .major = 1, + .minor = 0, +}; +static int ili9341_dbi_probe(struct spi_device *spi) +{ + struct mipi_dbi_dev *dbidev; + struct drm_device *drm; + struct mipi_dbi *dbi; + struct gpio_desc *dc; + struct device *dev = &spi->dev; + 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, &ili9341_dbi_driver); + if (ret) { + kfree(dbidev); + return ret; + } + + drm_mode_config_init(drm); + + dbi->reset = devm_gpiod_get_optional(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_optional(dev, "dc", GPIOD_OUT_LOW); + if (IS_ERR(dc)) { + DRM_DEV_ERROR(dev, "Failed to get gpio 'dc'\n"); + return PTR_ERR(dc); + } + + dbidev->regulator = devm_regulator_get_optional(dev, "vcc"); + if (IS_ERR(dbidev->regulator)) + dev_err(dev, "get optional vcc failed\n"); + + 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; + + ret = mipi_dbi_dev_init(dbidev, &ili9341_dbi_funcs, + &ili9341_dbi_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 ili9341_probe(struct spi_device *spi) +{ + const struct spi_device_id *id = spi_get_device_id(spi); + + if (!strcmp(id->name, "sf-tc240t-9370-t")) + return ili9341_dpi_probe(spi); + else if (!strcmp(id->name, "yx240qv29")) + return ili9341_dbi_probe(spi); + + return -1; +} + +static int ili9341_remove(struct spi_device *spi) +{ + const struct spi_device_id *id = spi_get_device_id(spi); + struct ili9341 *ili = spi_get_drvdata(spi); + struct drm_device *drm = spi_get_drvdata(spi); + + if (!strcmp(id->name, "sf-tc240t-9370-t")) { + ili9341_dpi_power_off(ili); + drm_panel_remove(&ili->panel); + } else if (!strcmp(id->name, "yx240qv29")) { + drm_dev_unplug(drm); + drm_atomic_helper_shutdown(drm); + } + return 0; +} + +static void ili9341_shutdown(struct spi_device *spi) +{ + const struct spi_device_id *id = spi_get_device_id(spi); + + if (!strcmp(id->name, "yx240qv29")) + drm_atomic_helper_shutdown(spi_get_drvdata(spi)); +} + +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", + .data = NULL, + }, +}; +MODULE_DEVICE_TABLE(of, ili9341_of_match); + +static const struct spi_device_id ili9341_id[] = { + { "yx240qv29", 0 }, + { "sf-tc240t-9370-t", 0 }, + { } +}; +MODULE_DEVICE_TABLE(spi, ili9341_id); + +static struct spi_driver ili9341_driver = { + .probe = ili9341_probe, + .remove = ili9341_remove, + .shutdown = ili9341_shutdown, + .id_table = ili9341_id, + .driver = { + .name = "panel-ilitek-ili9341", + .of_match_table = ili9341_of_match, + }, +}; +module_spi_driver(ili9341_driver); + +MODULE_AUTHOR("Dillon Min <dillon.minfei@gmail.com>"); +MODULE_DESCRIPTION("ILI9341 LCD panel driver"); +MODULE_LICENSE("GPL v2"); -- 2.7.4 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH v5 7/8] spi: stm32: Add 'SPI_SIMPLEX_RX', 'SPI_3WIRE_RX' support for stm32f4 2020-05-25 3:40 ` [PATCH v5 0/8] Enable ili9341 and l3gd20 on stm32f429-disco dillon.minfei ` (5 preceding siblings ...) 2020-05-25 3:41 ` [PATCH v5 6/8] drm/panel: Add ilitek ili9341 panel driver dillon.minfei @ 2020-05-25 3:41 ` dillon.minfei 2020-05-25 3:41 ` [PATCH v5 8/8] spi: flags 'SPI_CONTROLLER_MUST_RX' and 'SPI_CONTROLLER_MUST_TX' can't be coexit with 'SPI_3WIRE' mode dillon.minfei 2020-07-21 9:19 ` [PATCH v5 0/8] Enable ili9341 and l3gd20 on stm32f429-disco Alexandre Torgue 8 siblings, 0 replies; 15+ messages in thread From: dillon.minfei @ 2020-05-25 3:41 UTC (permalink / raw) To: robh+dt, p.zabel, mcoquelin.stm32, alexandre.torgue, thierry.reding, sam, airlied, daniel, mturquette, sboyd Cc: devicetree, linux-arm-kernel, linux-kernel, linux-spi, linux-stm32, dri-devel, linux-clk, dillon min From: dillon min <dillon.minfei@gmail.com> in l3gd20 driver startup, there is a setup failed error return from stm32 spi driver " [ 2.687630] st-gyro-spi spi0.0: supply vdd not found, using dummy regulator [ 2.696869] st-gyro-spi spi0.0: supply vddio not found, using dummy regulator [ 2.706707] spi_stm32 40015000.spi: SPI transfer setup failed [ 2.713741] st-gyro-spi spi0.0: SPI transfer failed: -22 [ 2.721096] spi_master spi0: failed to transfer one message from queue [ 2.729268] iio iio:device0: failed to read Who-Am-I register. [ 2.737504] st-gyro-spi: probe of spi0.0 failed with error -22 " after debug into spi-stm32 driver, st-gyro-spi split two steps to read l3gd20 id first: send command to l3gd20 with read id command in tx_buf, rx_buf is null. second: read id with tx_buf is null, rx_buf not null. so, for second step, stm32 driver recongise this process as 'SPI_SIMPLE_RX' from stm32_spi_communication_type(), but there is no related process for this type in stm32f4_spi_set_mode(), then we get error from stm32_spi_transfer_one_setup(). we can use two method to fix this bug. 1, use stm32 spi's "In unidirectional receive-only mode (BIDIMODE=0 and RXONLY=1)". but as our code running in sdram, the read latency is too large to get so many receive overrun error in interrupts handler. 2, use stm32 spi's "In full-duplex (BIDIMODE=0 and RXONLY=0)", as tx_buf is null, so add flag 'SPI_MASTER_MUST_TX' to spi master. Change since V4: 1 remove dummy data sent out by stm32 spi driver 2 add flag 'SPI_MASTER_MUST_TX' to spi master Signed-off-by: dillon min <dillon.minfei@gmail.com> --- Hi Mark, This changes add 'SPI_MASTER_MUST_TX' for stm32 spi controller thanks. drivers/spi/spi-stm32.c | 19 +++++++++++++++---- 1 file changed, 15 insertions(+), 4 deletions(-) diff --git a/drivers/spi/spi-stm32.c b/drivers/spi/spi-stm32.c index 44ac6eb3..4c643df 100644 --- a/drivers/spi/spi-stm32.c +++ b/drivers/spi/spi-stm32.c @@ -811,7 +811,9 @@ static irqreturn_t stm32f4_spi_irq_event(int irq, void *dev_id) mask |= STM32F4_SPI_SR_TXE; } - if (!spi->cur_usedma && spi->cur_comm == SPI_FULL_DUPLEX) { + if (!spi->cur_usedma && (spi->cur_comm == SPI_FULL_DUPLEX || + spi->cur_comm == SPI_SIMPLEX_RX || + spi->cur_comm == SPI_3WIRE_RX)) { /* TXE flag is set and is handled when RXNE flag occurs */ sr &= ~STM32F4_SPI_SR_TXE; mask |= STM32F4_SPI_SR_RXNE | STM32F4_SPI_SR_OVR; @@ -850,7 +852,7 @@ static irqreturn_t stm32f4_spi_irq_event(int irq, void *dev_id) stm32f4_spi_read_rx(spi); if (spi->rx_len == 0) end = true; - else /* Load data for discontinuous mode */ + else if (spi->tx_buf)/* Load data for discontinuous mode */ stm32f4_spi_write_tx(spi); } @@ -1151,7 +1153,9 @@ static int stm32f4_spi_transfer_one_irq(struct stm32_spi *spi) /* Enable the interrupts relative to the current communication mode */ if (spi->cur_comm == SPI_SIMPLEX_TX || spi->cur_comm == SPI_3WIRE_TX) { cr2 |= STM32F4_SPI_CR2_TXEIE; - } else if (spi->cur_comm == SPI_FULL_DUPLEX) { + } else if (spi->cur_comm == SPI_FULL_DUPLEX || + spi->cur_comm == SPI_SIMPLEX_RX || + spi->cur_comm == SPI_3WIRE_RX) { /* In transmit-only mode, the OVR flag is set in the SR register * since the received data are never read. Therefore set OVR * interrupt only when rx buffer is available. @@ -1462,10 +1466,16 @@ static int stm32f4_spi_set_mode(struct stm32_spi *spi, unsigned int comm_type) stm32_spi_set_bits(spi, STM32F4_SPI_CR1, STM32F4_SPI_CR1_BIDIMODE | STM32F4_SPI_CR1_BIDIOE); - } else if (comm_type == SPI_FULL_DUPLEX) { + } else if (comm_type == SPI_FULL_DUPLEX || + comm_type == SPI_SIMPLEX_RX) { stm32_spi_clr_bits(spi, STM32F4_SPI_CR1, STM32F4_SPI_CR1_BIDIMODE | STM32F4_SPI_CR1_BIDIOE); + } else if (comm_type == SPI_3WIRE_RX) { + stm32_spi_set_bits(spi, STM32F4_SPI_CR1, + STM32F4_SPI_CR1_BIDIMODE); + stm32_spi_clr_bits(spi, STM32F4_SPI_CR1, + STM32F4_SPI_CR1_BIDIOE); } else { return -EINVAL; } @@ -1906,6 +1916,7 @@ static int stm32_spi_probe(struct platform_device *pdev) master->prepare_message = stm32_spi_prepare_msg; master->transfer_one = stm32_spi_transfer_one; master->unprepare_message = stm32_spi_unprepare_msg; + master->flags = SPI_MASTER_MUST_TX; spi->dma_tx = dma_request_chan(spi->dev, "tx"); if (IS_ERR(spi->dma_tx)) { -- 2.7.4 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH v5 8/8] spi: flags 'SPI_CONTROLLER_MUST_RX' and 'SPI_CONTROLLER_MUST_TX' can't be coexit with 'SPI_3WIRE' mode 2020-05-25 3:40 ` [PATCH v5 0/8] Enable ili9341 and l3gd20 on stm32f429-disco dillon.minfei ` (6 preceding siblings ...) 2020-05-25 3:41 ` [PATCH v5 7/8] spi: stm32: Add 'SPI_SIMPLEX_RX', 'SPI_3WIRE_RX' support for stm32f4 dillon.minfei @ 2020-05-25 3:41 ` dillon.minfei 2020-07-21 9:19 ` [PATCH v5 0/8] Enable ili9341 and l3gd20 on stm32f429-disco Alexandre Torgue 8 siblings, 0 replies; 15+ messages in thread From: dillon.minfei @ 2020-05-25 3:41 UTC (permalink / raw) To: robh+dt, p.zabel, mcoquelin.stm32, alexandre.torgue, thierry.reding, sam, airlied, daniel, mturquette, sboyd Cc: devicetree, linux-arm-kernel, linux-kernel, linux-spi, linux-stm32, dri-devel, linux-clk, dillon min From: dillon min <dillon.minfei@gmail.com> since chip spi driver need get the transfer direction by 'tx_buf' and 'rx_buf' of 'struct spi_transfer' in 'SPI_3WIRE' mode. so, we need bypass 'SPI_CONTROLLER_MUST_RX' and 'SPI_CONTROLLER_MUST_TX' feature in 'SPI_3WIRE' mode Signed-off-by: dillon min <dillon.minfei@gmail.com> --- drivers/spi/spi.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/spi/spi.c b/drivers/spi/spi.c index c92c894..f884411 100644 --- a/drivers/spi/spi.c +++ b/drivers/spi/spi.c @@ -1023,7 +1023,8 @@ static int spi_map_msg(struct spi_controller *ctlr, struct spi_message *msg) void *tmp; unsigned int max_tx, max_rx; - if (ctlr->flags & (SPI_CONTROLLER_MUST_RX | SPI_CONTROLLER_MUST_TX)) { + if ((ctlr->flags & (SPI_CONTROLLER_MUST_RX | SPI_CONTROLLER_MUST_TX)) + && !(msg->spi->mode & SPI_3WIRE)) { max_tx = 0; max_rx = 0; -- 2.7.4 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH v5 0/8] Enable ili9341 and l3gd20 on stm32f429-disco 2020-05-25 3:40 ` [PATCH v5 0/8] Enable ili9341 and l3gd20 on stm32f429-disco dillon.minfei ` (7 preceding siblings ...) 2020-05-25 3:41 ` [PATCH v5 8/8] spi: flags 'SPI_CONTROLLER_MUST_RX' and 'SPI_CONTROLLER_MUST_TX' can't be coexit with 'SPI_3WIRE' mode dillon.minfei @ 2020-07-21 9:19 ` Alexandre Torgue 2020-07-21 10:39 ` dillon min 8 siblings, 1 reply; 15+ messages in thread From: Alexandre Torgue @ 2020-07-21 9:19 UTC (permalink / raw) To: dillon.minfei, robh+dt, p.zabel, mcoquelin.stm32, thierry.reding, sam, airlied, daniel, mturquette, sboyd Cc: devicetree, linux-arm-kernel, linux-kernel, linux-spi, linux-stm32, dri-devel, linux-clk Hi Dillon On 5/25/20 5:40 AM, dillon.minfei@gmail.com wrote: > From: dillon min <dillon.minfei@gmail.com> > > V5's update based on Mark Brown's suggestion, use 'SPI_MASTER_MUST_RX' > for SPI_SIMPLEX_RX mode on stm32 spi controller. > > V5: > 1 instead of add send dummy data out under SIMPLEX_RX mode, > add flags 'SPI_CONTROLLER_MUST_TX' for stm32 spi driver > 2 bypass 'SPI_CONTROLLER_MUST_TX' and 'SPI_CONTROLLER_MUST_RX' under > 'SPI_3WIRE' mode > Concerning DT patches, they look goods for me. However I'll merge them when drivers parts will be merged. regards Alex > V4: > According to alexandre torgue's suggestion, combine ili9341 and > l3gd20's modification on stm32f429-disco board to one patchset. > > Changes: > > ili9341: > > 1 update ili9341 panel driver according to Linus's suggestion > 2 drop V1's No.5 patch, sumbit new changes for clk-stm32f4 > 3 merge l3gd20's change to this patchset > > V3: > 1 merge original tiny/ili9341.c driver to panel/panel-ilitek-ili9341.c > to support serial spi & parallel rgb interface in one driver. > 2 update ilitek,ili9341.yaml dts binding documentation. > 3 update stm32f429-disco dts binding > > V2: > 1 verify ilitek,ili9341.yaml with make O=../linux-stm32 > dt_binding_check > DT_SCHEMA_FILES=Documentation/devicetree/bindings/display/panel/ > ilitek,ili9341.yaml > > V1: > 1 add ili9341 drm panel driver > 2 add ltdc, spi5 controller for stm32f429-disco > 3 add ltdc, spi5 pin map for stm32f429-disco > 4 add docs about ili9341 > 5 fix ltdc driver loading hang in clk set rate bug > > > L3gd20: > V3: > 1 merge stm32f429-disco dtbs binding with ili9341 part > > V2: > 1 insert blank line at stm32f420-disco.dts line 143 > 2 add more description for l3gd20 in commit message > > V1: > 1 enable spi5 controller on stm32f429-disco (dts) > 2 add spi5 pinmap for stm32f429-disco (dts) > 3 add SPI_SIMPLEX_RX, SPI_3WIRE_RX support for stm32f4 > > > dillon min (8): > ARM: dts: stm32: Add dma config for spi5 > ARM: dts: stm32: Add pin map for ltdc & spi5 on stm32f429-disco board > ARM: dts: stm32: enable ltdc binding with ili9341, gyro l3gd20 on > stm32429-disco board > dt-bindings: display: panel: Add ilitek ili9341 panel bindings > clk: stm32: Fix stm32f429's ltdc driver hang in set clock rate, > fix duplicated ltdc clock register to 'clk_core' case ltdc's clock > turn off by clk_disable_unused() > drm/panel: Add ilitek ili9341 panel driver > spi: stm32: Add 'SPI_SIMPLEX_RX', 'SPI_3WIRE_RX' support for stm32f4 > spi: flags 'SPI_CONTROLLER_MUST_RX' and 'SPI_CONTROLLER_MUST_TX' can't > be coexit with 'SPI_3WIRE' mode > > .../bindings/display/panel/ilitek,ili9341.yaml | 69 ++ > arch/arm/boot/dts/stm32f4-pinctrl.dtsi | 67 + > arch/arm/boot/dts/stm32f429-disco.dts | 48 + > arch/arm/boot/dts/stm32f429.dtsi | 3 + > drivers/clk/clk-stm32f4.c | 7 +- > drivers/gpu/drm/panel/Kconfig | 12 + > drivers/gpu/drm/panel/Makefile | 1 + > drivers/gpu/drm/panel/panel-ilitek-ili9341.c | 1301 ++++++++++++++++++++ > drivers/spi/spi-stm32.c | 19 +- > drivers/spi/spi.c | 3 +- > 10 files changed, 1521 insertions(+), 9 deletions(-) > create mode 100644 Documentation/devicetree/bindings/display/panel/ilitek,ili9341.yaml > create mode 100644 drivers/gpu/drm/panel/panel-ilitek-ili9341.c > ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v5 0/8] Enable ili9341 and l3gd20 on stm32f429-disco 2020-07-21 9:19 ` [PATCH v5 0/8] Enable ili9341 and l3gd20 on stm32f429-disco Alexandre Torgue @ 2020-07-21 10:39 ` dillon min 2020-07-21 11:54 ` Alexandre Torgue 0 siblings, 1 reply; 15+ messages in thread From: dillon min @ 2020-07-21 10:39 UTC (permalink / raw) To: Alexandre Torgue Cc: Rob Herring, p.zabel, Maxime Coquelin, thierry.reding, Sam Ravnborg, Dave Airlie, Daniel Vetter, Michael Turquette, Stephen Boyd, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, Linux ARM, Linux Kernel Mailing List, linux-spi, linux-stm32, open list:DRM PANEL DRIVERS, linux-clk Hi Alexandre, On Tue, Jul 21, 2020 at 5:19 PM Alexandre Torgue <alexandre.torgue@st.com> wrote: > > Hi Dillon > > On 5/25/20 5:40 AM, dillon.minfei@gmail.com wrote: > > From: dillon min <dillon.minfei@gmail.com> > > > > V5's update based on Mark Brown's suggestion, use 'SPI_MASTER_MUST_RX' > > for SPI_SIMPLEX_RX mode on stm32 spi controller. > > > > V5: > > 1 instead of add send dummy data out under SIMPLEX_RX mode, > > add flags 'SPI_CONTROLLER_MUST_TX' for stm32 spi driver > > 2 bypass 'SPI_CONTROLLER_MUST_TX' and 'SPI_CONTROLLER_MUST_RX' under > > 'SPI_3WIRE' mode > > > > Concerning DT patches, they look goods for me. However I'll merge them > when drivers parts will be merged. > > regards > Alex This patchset status is : dts: ARM: dts: stm32: Add dma config for spi5 ARM: dts: stm32: Add pin map for ltdc & spi5 on stm32f429-disco board ARM: dts: stm32: enable ltdc binding with ili9341, gyro l3gd20 on stm32429-disco board panel-bindings: Reviewed-by: Rob Herring, need more reviewing to merge dt-bindings: display: panel: Add ilitek ili9341 panel bindings clk: Acked-by: Stephen Boyd , need more reviewing to merge clk: stm32: Fix stm32f429's ltdc driver hang in set clock rate, fix duplicated ltdc clock register to 'clk_core' case ltdc's clock turn off by clk_disable_unused() drm-panel: Reviewed-by: Linus Walleij . need more reviewing to merge drm/panel: Add ilitek ili9341 panel driver spi-driver: Acked-by: Mark Brown ,merged into mainline spi: stm32: Add 'SPI_SIMPLEX_RX', 'SPI_3WIRE_RX' support for stm32f4 spi: flags 'SPI_CONTROLLER_MUST_RX' and 'SPI_CONTROLLER_MUST_TX' can't becoexit with 'SPI_3WIRE' mode For ili9341 drivers, it seems to need more time to review, or might be dropped. since the drm maintainer might create a new interface to support DPI,DBI,DSI in one driver. Currently it's under discussion. I will rework the ili9341 driver after that. But, we can use the existing "drivers/gpu/drm/tiny/ili9341.c" driver to support ili9341 on stm32f429-disco board only using the spi bus. except the display refresh performance is not as good as the ltdc interface. I can submit a new patch just with dts changed this week for it, if you need. Thanks, Dillon. > > > > > V4: > > According to alexandre torgue's suggestion, combine ili9341 and > > l3gd20's modification on stm32f429-disco board to one patchset. > > > > Changes: > > > > ili9341: > > > > 1 update ili9341 panel driver according to Linus's suggestion > > 2 drop V1's No.5 patch, sumbit new changes for clk-stm32f4 > > 3 merge l3gd20's change to this patchset > > > > V3: > > 1 merge original tiny/ili9341.c driver to panel/panel-ilitek-ili9341.c > > to support serial spi & parallel rgb interface in one driver. > > 2 update ilitek,ili9341.yaml dts binding documentation. > > 3 update stm32f429-disco dts binding > > > > V2: > > 1 verify ilitek,ili9341.yaml with make O=../linux-stm32 > > dt_binding_check > > DT_SCHEMA_FILES=Documentation/devicetree/bindings/display/panel/ > > ilitek,ili9341.yaml > > > > V1: > > 1 add ili9341 drm panel driver > > 2 add ltdc, spi5 controller for stm32f429-disco > > 3 add ltdc, spi5 pin map for stm32f429-disco > > 4 add docs about ili9341 > > 5 fix ltdc driver loading hang in clk set rate bug > > > > > > L3gd20: > > V3: > > 1 merge stm32f429-disco dtbs binding with ili9341 part > > > > V2: > > 1 insert blank line at stm32f420-disco.dts line 143 > > 2 add more description for l3gd20 in commit message > > > > V1: > > 1 enable spi5 controller on stm32f429-disco (dts) > > 2 add spi5 pinmap for stm32f429-disco (dts) > > 3 add SPI_SIMPLEX_RX, SPI_3WIRE_RX support for stm32f4 > > > > > > dillon min (8): > > ARM: dts: stm32: Add dma config for spi5 > > ARM: dts: stm32: Add pin map for ltdc & spi5 on stm32f429-disco board > > ARM: dts: stm32: enable ltdc binding with ili9341, gyro l3gd20 on > > stm32429-disco board > > dt-bindings: display: panel: Add ilitek ili9341 panel bindings > > clk: stm32: Fix stm32f429's ltdc driver hang in set clock rate, > > fix duplicated ltdc clock register to 'clk_core' case ltdc's clock > > turn off by clk_disable_unused() > > drm/panel: Add ilitek ili9341 panel driver > > spi: stm32: Add 'SPI_SIMPLEX_RX', 'SPI_3WIRE_RX' support for stm32f4 > > spi: flags 'SPI_CONTROLLER_MUST_RX' and 'SPI_CONTROLLER_MUST_TX' can't > > be coexit with 'SPI_3WIRE' mode > > > > .../bindings/display/panel/ilitek,ili9341.yaml | 69 ++ > > arch/arm/boot/dts/stm32f4-pinctrl.dtsi | 67 + > > arch/arm/boot/dts/stm32f429-disco.dts | 48 + > > arch/arm/boot/dts/stm32f429.dtsi | 3 + > > drivers/clk/clk-stm32f4.c | 7 +- > > drivers/gpu/drm/panel/Kconfig | 12 + > > drivers/gpu/drm/panel/Makefile | 1 + > > drivers/gpu/drm/panel/panel-ilitek-ili9341.c | 1301 ++++++++++++++++++++ > > drivers/spi/spi-stm32.c | 19 +- > > drivers/spi/spi.c | 3 +- > > 10 files changed, 1521 insertions(+), 9 deletions(-) > > create mode 100644 Documentation/devicetree/bindings/display/panel/ilitek,ili9341.yaml > > create mode 100644 drivers/gpu/drm/panel/panel-ilitek-ili9341.c > > ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v5 0/8] Enable ili9341 and l3gd20 on stm32f429-disco 2020-07-21 10:39 ` dillon min @ 2020-07-21 11:54 ` Alexandre Torgue 2020-07-21 12:55 ` dillon min 0 siblings, 1 reply; 15+ messages in thread From: Alexandre Torgue @ 2020-07-21 11:54 UTC (permalink / raw) To: dillon min Cc: Rob Herring, p.zabel, Maxime Coquelin, thierry.reding, Sam Ravnborg, Dave Airlie, Daniel Vetter, Michael Turquette, Stephen Boyd, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, Linux ARM, Linux Kernel Mailing List, linux-spi, linux-stm32, open list:DRM PANEL DRIVERS, linux-clk On 7/21/20 12:39 PM, dillon min wrote: > Hi Alexandre, > > On Tue, Jul 21, 2020 at 5:19 PM Alexandre Torgue > <alexandre.torgue@st.com> wrote: >> >> Hi Dillon >> >> On 5/25/20 5:40 AM, dillon.minfei@gmail.com wrote: >>> From: dillon min <dillon.minfei@gmail.com> >>> >>> V5's update based on Mark Brown's suggestion, use 'SPI_MASTER_MUST_RX' >>> for SPI_SIMPLEX_RX mode on stm32 spi controller. >>> >>> V5: >>> 1 instead of add send dummy data out under SIMPLEX_RX mode, >>> add flags 'SPI_CONTROLLER_MUST_TX' for stm32 spi driver >>> 2 bypass 'SPI_CONTROLLER_MUST_TX' and 'SPI_CONTROLLER_MUST_RX' under >>> 'SPI_3WIRE' mode >>> >> >> Concerning DT patches, they look goods for me. However I'll merge them >> when drivers parts will be merged. >> >> regards >> Alex > This patchset status is : > dts: > ARM: dts: stm32: Add dma config for spi5 > ARM: dts: stm32: Add pin map for ltdc & spi5 on stm32f429-disco board > ARM: dts: stm32: enable ltdc binding with ili9341, gyro l3gd20 on > stm32429-disco board > > panel-bindings: Reviewed-by: Rob Herring, need more reviewing to merge > dt-bindings: display: panel: Add ilitek ili9341 panel bindings > > clk: Acked-by: Stephen Boyd , need more reviewing to merge > clk: stm32: Fix stm32f429's ltdc driver hang in set clock rate, > fix duplicated ltdc clock register to 'clk_core' case ltdc's clock > turn off by clk_disable_unused() > > drm-panel: Reviewed-by: Linus Walleij . need more reviewing to merge > drm/panel: Add ilitek ili9341 panel driver > > spi-driver: Acked-by: Mark Brown ,merged into mainline > spi: stm32: Add 'SPI_SIMPLEX_RX', 'SPI_3WIRE_RX' support for stm32f4 > spi: flags 'SPI_CONTROLLER_MUST_RX' and 'SPI_CONTROLLER_MUST_TX' can't > becoexit with 'SPI_3WIRE' mode > > For ili9341 drivers, it seems to need more time to review, or might be dropped. > since the drm maintainer might create a new interface to support > DPI,DBI,DSI in one driver. > Currently it's under discussion. I will rework the ili9341 driver after that. > > But, we can use the existing "drivers/gpu/drm/tiny/ili9341.c" driver to support > ili9341 on stm32f429-disco board only using the spi bus. except the > display refresh performance is not as good as the ltdc interface. > > I can submit a new patch just with dts changed this week for it, if you need. > Thanks for this status. I'm close to send my PR for v5.9. There is no needs to have an intermediate version (Except you absolutely want it). Thanks alex > Thanks, > Dillon. >> >> >> >>> V4: >>> According to alexandre torgue's suggestion, combine ili9341 and >>> l3gd20's modification on stm32f429-disco board to one patchset. >>> >>> Changes: >>> >>> ili9341: >>> >>> 1 update ili9341 panel driver according to Linus's suggestion >>> 2 drop V1's No.5 patch, sumbit new changes for clk-stm32f4 >>> 3 merge l3gd20's change to this patchset >>> >>> V3: >>> 1 merge original tiny/ili9341.c driver to panel/panel-ilitek-ili9341.c >>> to support serial spi & parallel rgb interface in one driver. >>> 2 update ilitek,ili9341.yaml dts binding documentation. >>> 3 update stm32f429-disco dts binding >>> >>> V2: >>> 1 verify ilitek,ili9341.yaml with make O=../linux-stm32 >>> dt_binding_check >>> DT_SCHEMA_FILES=Documentation/devicetree/bindings/display/panel/ >>> ilitek,ili9341.yaml >>> >>> V1: >>> 1 add ili9341 drm panel driver >>> 2 add ltdc, spi5 controller for stm32f429-disco >>> 3 add ltdc, spi5 pin map for stm32f429-disco >>> 4 add docs about ili9341 >>> 5 fix ltdc driver loading hang in clk set rate bug >>> >>> >>> L3gd20: >>> V3: >>> 1 merge stm32f429-disco dtbs binding with ili9341 part >>> >>> V2: >>> 1 insert blank line at stm32f420-disco.dts line 143 >>> 2 add more description for l3gd20 in commit message >>> >>> V1: >>> 1 enable spi5 controller on stm32f429-disco (dts) >>> 2 add spi5 pinmap for stm32f429-disco (dts) >>> 3 add SPI_SIMPLEX_RX, SPI_3WIRE_RX support for stm32f4 >>> >>> >>> dillon min (8): >>> ARM: dts: stm32: Add dma config for spi5 >>> ARM: dts: stm32: Add pin map for ltdc & spi5 on stm32f429-disco board >>> ARM: dts: stm32: enable ltdc binding with ili9341, gyro l3gd20 on >>> stm32429-disco board >>> dt-bindings: display: panel: Add ilitek ili9341 panel bindings >>> clk: stm32: Fix stm32f429's ltdc driver hang in set clock rate, >>> fix duplicated ltdc clock register to 'clk_core' case ltdc's clock >>> turn off by clk_disable_unused() >>> drm/panel: Add ilitek ili9341 panel driver >>> spi: stm32: Add 'SPI_SIMPLEX_RX', 'SPI_3WIRE_RX' support for stm32f4 >>> spi: flags 'SPI_CONTROLLER_MUST_RX' and 'SPI_CONTROLLER_MUST_TX' can't >>> be coexit with 'SPI_3WIRE' mode >>> >>> .../bindings/display/panel/ilitek,ili9341.yaml | 69 ++ >>> arch/arm/boot/dts/stm32f4-pinctrl.dtsi | 67 + >>> arch/arm/boot/dts/stm32f429-disco.dts | 48 + >>> arch/arm/boot/dts/stm32f429.dtsi | 3 + >>> drivers/clk/clk-stm32f4.c | 7 +- >>> drivers/gpu/drm/panel/Kconfig | 12 + >>> drivers/gpu/drm/panel/Makefile | 1 + >>> drivers/gpu/drm/panel/panel-ilitek-ili9341.c | 1301 ++++++++++++++++++++ >>> drivers/spi/spi-stm32.c | 19 +- >>> drivers/spi/spi.c | 3 +- >>> 10 files changed, 1521 insertions(+), 9 deletions(-) >>> create mode 100644 Documentation/devicetree/bindings/display/panel/ilitek,ili9341.yaml >>> create mode 100644 drivers/gpu/drm/panel/panel-ilitek-ili9341.c >>> ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v5 0/8] Enable ili9341 and l3gd20 on stm32f429-disco 2020-07-21 11:54 ` Alexandre Torgue @ 2020-07-21 12:55 ` dillon min 2020-07-21 12:58 ` Alexandre Torgue 0 siblings, 1 reply; 15+ messages in thread From: dillon min @ 2020-07-21 12:55 UTC (permalink / raw) To: Alexandre Torgue Cc: Rob Herring, p.zabel, Maxime Coquelin, thierry.reding, Sam Ravnborg, Dave Airlie, Daniel Vetter, Michael Turquette, Stephen Boyd, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, Linux ARM, Linux Kernel Mailing List, linux-spi, linux-stm32, open list:DRM PANEL DRIVERS, linux-clk Hi, Alexandre, On Tue, Jul 21, 2020 at 7:54 PM Alexandre Torgue <alexandre.torgue@st.com> wrote: > > > > On 7/21/20 12:39 PM, dillon min wrote: > > Hi Alexandre, > > > > On Tue, Jul 21, 2020 at 5:19 PM Alexandre Torgue > > <alexandre.torgue@st.com> wrote: > >> > >> Hi Dillon > >> > >> On 5/25/20 5:40 AM, dillon.minfei@gmail.com wrote: > >>> From: dillon min <dillon.minfei@gmail.com> > >>> > >>> V5's update based on Mark Brown's suggestion, use 'SPI_MASTER_MUST_RX' > >>> for SPI_SIMPLEX_RX mode on stm32 spi controller. > >>> > >>> V5: > >>> 1 instead of add send dummy data out under SIMPLEX_RX mode, > >>> add flags 'SPI_CONTROLLER_MUST_TX' for stm32 spi driver > >>> 2 bypass 'SPI_CONTROLLER_MUST_TX' and 'SPI_CONTROLLER_MUST_RX' under > >>> 'SPI_3WIRE' mode > >>> > >> > >> Concerning DT patches, they look goods for me. However I'll merge them > >> when drivers parts will be merged. > >> > >> regards > >> Alex > > This patchset status is : > > dts: > > ARM: dts: stm32: Add dma config for spi5 > > ARM: dts: stm32: Add pin map for ltdc & spi5 on stm32f429-disco board > > ARM: dts: stm32: enable ltdc binding with ili9341, gyro l3gd20 on > > stm32429-disco board > > > > panel-bindings: Reviewed-by: Rob Herring, need more reviewing to merge > > dt-bindings: display: panel: Add ilitek ili9341 panel bindings > > > > clk: Acked-by: Stephen Boyd , need more reviewing to merge > > clk: stm32: Fix stm32f429's ltdc driver hang in set clock rate, > > fix duplicated ltdc clock register to 'clk_core' case ltdc's clock > > turn off by clk_disable_unused() > > > > drm-panel: Reviewed-by: Linus Walleij . need more reviewing to merge > > drm/panel: Add ilitek ili9341 panel driver > > > > spi-driver: Acked-by: Mark Brown ,merged into mainline > > spi: stm32: Add 'SPI_SIMPLEX_RX', 'SPI_3WIRE_RX' support for stm32f4 > > spi: flags 'SPI_CONTROLLER_MUST_RX' and 'SPI_CONTROLLER_MUST_TX' can't > > becoexit with 'SPI_3WIRE' mode > > > > For ili9341 drivers, it seems to need more time to review, or might be dropped. > > since the drm maintainer might create a new interface to support > > DPI,DBI,DSI in one driver. > > Currently it's under discussion. I will rework the ili9341 driver after that. > > > > But, we can use the existing "drivers/gpu/drm/tiny/ili9341.c" driver to support > > ili9341 on stm32f429-disco board only using the spi bus. except the > > display refresh performance is not as good as the ltdc interface. > > > > I can submit a new patch just with dts changed this week for it, if you need. > > > > > Thanks for this status. I'm close to send my PR for v5.9. There is no > needs to have an intermediate version (Except you absolutely want it). > > Thanks > alex Ok, I prefer to use the ltdc interface to drive ili9341 either. Thanks, Dillon. > > > Thanks, > > Dillon. > >> > >> > >> > >>> V4: > >>> According to alexandre torgue's suggestion, combine ili9341 and > >>> l3gd20's modification on stm32f429-disco board to one patchset. > >>> > >>> Changes: > >>> > >>> ili9341: > >>> > >>> 1 update ili9341 panel driver according to Linus's suggestion > >>> 2 drop V1's No.5 patch, sumbit new changes for clk-stm32f4 > >>> 3 merge l3gd20's change to this patchset > >>> > >>> V3: > >>> 1 merge original tiny/ili9341.c driver to panel/panel-ilitek-ili9341.c > >>> to support serial spi & parallel rgb interface in one driver. > >>> 2 update ilitek,ili9341.yaml dts binding documentation. > >>> 3 update stm32f429-disco dts binding > >>> > >>> V2: > >>> 1 verify ilitek,ili9341.yaml with make O=../linux-stm32 > >>> dt_binding_check > >>> DT_SCHEMA_FILES=Documentation/devicetree/bindings/display/panel/ > >>> ilitek,ili9341.yaml > >>> > >>> V1: > >>> 1 add ili9341 drm panel driver > >>> 2 add ltdc, spi5 controller for stm32f429-disco > >>> 3 add ltdc, spi5 pin map for stm32f429-disco > >>> 4 add docs about ili9341 > >>> 5 fix ltdc driver loading hang in clk set rate bug > >>> > >>> > >>> L3gd20: > >>> V3: > >>> 1 merge stm32f429-disco dtbs binding with ili9341 part > >>> > >>> V2: > >>> 1 insert blank line at stm32f420-disco.dts line 143 > >>> 2 add more description for l3gd20 in commit message > >>> > >>> V1: > >>> 1 enable spi5 controller on stm32f429-disco (dts) > >>> 2 add spi5 pinmap for stm32f429-disco (dts) > >>> 3 add SPI_SIMPLEX_RX, SPI_3WIRE_RX support for stm32f4 > >>> > >>> > >>> dillon min (8): > >>> ARM: dts: stm32: Add dma config for spi5 > >>> ARM: dts: stm32: Add pin map for ltdc & spi5 on stm32f429-disco board > >>> ARM: dts: stm32: enable ltdc binding with ili9341, gyro l3gd20 on > >>> stm32429-disco board > >>> dt-bindings: display: panel: Add ilitek ili9341 panel bindings > >>> clk: stm32: Fix stm32f429's ltdc driver hang in set clock rate, > >>> fix duplicated ltdc clock register to 'clk_core' case ltdc's clock > >>> turn off by clk_disable_unused() > >>> drm/panel: Add ilitek ili9341 panel driver > >>> spi: stm32: Add 'SPI_SIMPLEX_RX', 'SPI_3WIRE_RX' support for stm32f4 > >>> spi: flags 'SPI_CONTROLLER_MUST_RX' and 'SPI_CONTROLLER_MUST_TX' can't > >>> be coexit with 'SPI_3WIRE' mode > >>> > >>> .../bindings/display/panel/ilitek,ili9341.yaml | 69 ++ > >>> arch/arm/boot/dts/stm32f4-pinctrl.dtsi | 67 + > >>> arch/arm/boot/dts/stm32f429-disco.dts | 48 + > >>> arch/arm/boot/dts/stm32f429.dtsi | 3 + > >>> drivers/clk/clk-stm32f4.c | 7 +- > >>> drivers/gpu/drm/panel/Kconfig | 12 + > >>> drivers/gpu/drm/panel/Makefile | 1 + > >>> drivers/gpu/drm/panel/panel-ilitek-ili9341.c | 1301 ++++++++++++++++++++ > >>> drivers/spi/spi-stm32.c | 19 +- > >>> drivers/spi/spi.c | 3 +- > >>> 10 files changed, 1521 insertions(+), 9 deletions(-) > >>> create mode 100644 Documentation/devicetree/bindings/display/panel/ilitek,ili9341.yaml > >>> create mode 100644 drivers/gpu/drm/panel/panel-ilitek-ili9341.c > >>> ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v5 0/8] Enable ili9341 and l3gd20 on stm32f429-disco 2020-07-21 12:55 ` dillon min @ 2020-07-21 12:58 ` Alexandre Torgue 0 siblings, 0 replies; 15+ messages in thread From: Alexandre Torgue @ 2020-07-21 12:58 UTC (permalink / raw) To: dillon min Cc: Rob Herring, p.zabel, Maxime Coquelin, thierry.reding, Sam Ravnborg, Dave Airlie, Daniel Vetter, Michael Turquette, Stephen Boyd, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, Linux ARM, Linux Kernel Mailing List, linux-spi, linux-stm32, open list:DRM PANEL DRIVERS, linux-clk On 7/21/20 2:55 PM, dillon min wrote: > Hi, Alexandre, > > > On Tue, Jul 21, 2020 at 7:54 PM Alexandre Torgue > <alexandre.torgue@st.com> wrote: >> >> >> >> On 7/21/20 12:39 PM, dillon min wrote: >>> Hi Alexandre, >>> >>> On Tue, Jul 21, 2020 at 5:19 PM Alexandre Torgue >>> <alexandre.torgue@st.com> wrote: >>>> >>>> Hi Dillon >>>> >>>> On 5/25/20 5:40 AM, dillon.minfei@gmail.com wrote: >>>>> From: dillon min <dillon.minfei@gmail.com> >>>>> >>>>> V5's update based on Mark Brown's suggestion, use 'SPI_MASTER_MUST_RX' >>>>> for SPI_SIMPLEX_RX mode on stm32 spi controller. >>>>> >>>>> V5: >>>>> 1 instead of add send dummy data out under SIMPLEX_RX mode, >>>>> add flags 'SPI_CONTROLLER_MUST_TX' for stm32 spi driver >>>>> 2 bypass 'SPI_CONTROLLER_MUST_TX' and 'SPI_CONTROLLER_MUST_RX' under >>>>> 'SPI_3WIRE' mode >>>>> >>>> >>>> Concerning DT patches, they look goods for me. However I'll merge them >>>> when drivers parts will be merged. >>>> >>>> regards >>>> Alex >>> This patchset status is : >>> dts: >>> ARM: dts: stm32: Add dma config for spi5 >>> ARM: dts: stm32: Add pin map for ltdc & spi5 on stm32f429-disco board >>> ARM: dts: stm32: enable ltdc binding with ili9341, gyro l3gd20 on >>> stm32429-disco board >>> >>> panel-bindings: Reviewed-by: Rob Herring, need more reviewing to merge >>> dt-bindings: display: panel: Add ilitek ili9341 panel bindings >>> >>> clk: Acked-by: Stephen Boyd , need more reviewing to merge >>> clk: stm32: Fix stm32f429's ltdc driver hang in set clock rate, >>> fix duplicated ltdc clock register to 'clk_core' case ltdc's clock >>> turn off by clk_disable_unused() >>> >>> drm-panel: Reviewed-by: Linus Walleij . need more reviewing to merge >>> drm/panel: Add ilitek ili9341 panel driver >>> >>> spi-driver: Acked-by: Mark Brown ,merged into mainline >>> spi: stm32: Add 'SPI_SIMPLEX_RX', 'SPI_3WIRE_RX' support for stm32f4 >>> spi: flags 'SPI_CONTROLLER_MUST_RX' and 'SPI_CONTROLLER_MUST_TX' can't >>> becoexit with 'SPI_3WIRE' mode >>> >>> For ili9341 drivers, it seems to need more time to review, or might be dropped. >>> since the drm maintainer might create a new interface to support >>> DPI,DBI,DSI in one driver. >>> Currently it's under discussion. I will rework the ili9341 driver after that. >>> >>> But, we can use the existing "drivers/gpu/drm/tiny/ili9341.c" driver to support >>> ili9341 on stm32f429-disco board only using the spi bus. except the >>> display refresh performance is not as good as the ltdc interface. >>> >>> I can submit a new patch just with dts changed this week for it, if you need. >>> >> >> >> Thanks for this status. I'm close to send my PR for v5.9. There is no >> needs to have an intermediate version (Except you absolutely want it). >> >> Thanks >> alex > Ok, I prefer to use the ltdc interface to drive ili9341 either. Ok, so let's wait for the full review of this series. Cheers Alex > > Thanks, > Dillon. >> >>> Thanks, >>> Dillon. >>>> >>>> >>>> >>>>> V4: >>>>> According to alexandre torgue's suggestion, combine ili9341 and >>>>> l3gd20's modification on stm32f429-disco board to one patchset. >>>>> >>>>> Changes: >>>>> >>>>> ili9341: >>>>> >>>>> 1 update ili9341 panel driver according to Linus's suggestion >>>>> 2 drop V1's No.5 patch, sumbit new changes for clk-stm32f4 >>>>> 3 merge l3gd20's change to this patchset >>>>> >>>>> V3: >>>>> 1 merge original tiny/ili9341.c driver to panel/panel-ilitek-ili9341.c >>>>> to support serial spi & parallel rgb interface in one driver. >>>>> 2 update ilitek,ili9341.yaml dts binding documentation. >>>>> 3 update stm32f429-disco dts binding >>>>> >>>>> V2: >>>>> 1 verify ilitek,ili9341.yaml with make O=../linux-stm32 >>>>> dt_binding_check >>>>> DT_SCHEMA_FILES=Documentation/devicetree/bindings/display/panel/ >>>>> ilitek,ili9341.yaml >>>>> >>>>> V1: >>>>> 1 add ili9341 drm panel driver >>>>> 2 add ltdc, spi5 controller for stm32f429-disco >>>>> 3 add ltdc, spi5 pin map for stm32f429-disco >>>>> 4 add docs about ili9341 >>>>> 5 fix ltdc driver loading hang in clk set rate bug >>>>> >>>>> >>>>> L3gd20: >>>>> V3: >>>>> 1 merge stm32f429-disco dtbs binding with ili9341 part >>>>> >>>>> V2: >>>>> 1 insert blank line at stm32f420-disco.dts line 143 >>>>> 2 add more description for l3gd20 in commit message >>>>> >>>>> V1: >>>>> 1 enable spi5 controller on stm32f429-disco (dts) >>>>> 2 add spi5 pinmap for stm32f429-disco (dts) >>>>> 3 add SPI_SIMPLEX_RX, SPI_3WIRE_RX support for stm32f4 >>>>> >>>>> >>>>> dillon min (8): >>>>> ARM: dts: stm32: Add dma config for spi5 >>>>> ARM: dts: stm32: Add pin map for ltdc & spi5 on stm32f429-disco board >>>>> ARM: dts: stm32: enable ltdc binding with ili9341, gyro l3gd20 on >>>>> stm32429-disco board >>>>> dt-bindings: display: panel: Add ilitek ili9341 panel bindings >>>>> clk: stm32: Fix stm32f429's ltdc driver hang in set clock rate, >>>>> fix duplicated ltdc clock register to 'clk_core' case ltdc's clock >>>>> turn off by clk_disable_unused() >>>>> drm/panel: Add ilitek ili9341 panel driver >>>>> spi: stm32: Add 'SPI_SIMPLEX_RX', 'SPI_3WIRE_RX' support for stm32f4 >>>>> spi: flags 'SPI_CONTROLLER_MUST_RX' and 'SPI_CONTROLLER_MUST_TX' can't >>>>> be coexit with 'SPI_3WIRE' mode >>>>> >>>>> .../bindings/display/panel/ilitek,ili9341.yaml | 69 ++ >>>>> arch/arm/boot/dts/stm32f4-pinctrl.dtsi | 67 + >>>>> arch/arm/boot/dts/stm32f429-disco.dts | 48 + >>>>> arch/arm/boot/dts/stm32f429.dtsi | 3 + >>>>> drivers/clk/clk-stm32f4.c | 7 +- >>>>> drivers/gpu/drm/panel/Kconfig | 12 + >>>>> drivers/gpu/drm/panel/Makefile | 1 + >>>>> drivers/gpu/drm/panel/panel-ilitek-ili9341.c | 1301 ++++++++++++++++++++ >>>>> drivers/spi/spi-stm32.c | 19 +- >>>>> drivers/spi/spi.c | 3 +- >>>>> 10 files changed, 1521 insertions(+), 9 deletions(-) >>>>> create mode 100644 Documentation/devicetree/bindings/display/panel/ilitek,ili9341.yaml >>>>> create mode 100644 drivers/gpu/drm/panel/panel-ilitek-ili9341.c >>>>> ^ permalink raw reply [flat|nested] 15+ messages in thread
* [PATCH v5 0/8] Enable ili9341 and l3gd20 on stm32f429-disco @ 2020-05-25 3:45 dillon.minfei 2020-05-25 3:45 ` [PATCH v5 8/8] spi: flags 'SPI_CONTROLLER_MUST_RX' and 'SPI_CONTROLLER_MUST_TX' can't be coexit with 'SPI_3WIRE' mode dillon.minfei 0 siblings, 1 reply; 15+ messages in thread From: dillon.minfei @ 2020-05-25 3:45 UTC (permalink / raw) To: linus.walleij, broonie Cc: devicetree, linux-arm-kernel, linux-kernel, linux-spi, linux-stm32, dri-devel, linux-clk, dillon min From: dillon min <dillon.minfei@gmail.com> V5's update based on Mark Brown's suggestion, use 'SPI_MASTER_MUST_RX' for SPI_SIMPLEX_RX mode on stm32 spi controller. V5: 1 instead of add send dummy data out under SIMPLEX_RX mode, add flags 'SPI_CONTROLLER_MUST_TX' for stm32 spi driver 2 bypass 'SPI_CONTROLLER_MUST_TX' and 'SPI_CONTROLLER_MUST_RX' under 'SPI_3WIRE' mode V4: According to alexandre torgue's suggestion, combine ili9341 and l3gd20's modification on stm32f429-disco board to one patchset. Changes: ili9341: 1 update ili9341 panel driver according to Linus's suggestion 2 drop V1's No.5 patch, sumbit new changes for clk-stm32f4 3 merge l3gd20's change to this patchset V3: 1 merge original tiny/ili9341.c driver to panel/panel-ilitek-ili9341.c to support serial spi & parallel rgb interface in one driver. 2 update ilitek,ili9341.yaml dts binding documentation. 3 update stm32f429-disco dts binding V2: 1 verify ilitek,ili9341.yaml with make O=../linux-stm32 dt_binding_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/display/panel/ ilitek,ili9341.yaml V1: 1 add ili9341 drm panel driver 2 add ltdc, spi5 controller for stm32f429-disco 3 add ltdc, spi5 pin map for stm32f429-disco 4 add docs about ili9341 5 fix ltdc driver loading hang in clk set rate bug L3gd20: V3: 1 merge stm32f429-disco dtbs binding with ili9341 part V2: 1 insert blank line at stm32f420-disco.dts line 143 2 add more description for l3gd20 in commit message V1: 1 enable spi5 controller on stm32f429-disco (dts) 2 add spi5 pinmap for stm32f429-disco (dts) 3 add SPI_SIMPLEX_RX, SPI_3WIRE_RX support for stm32f4 dillon min (8): ARM: dts: stm32: Add dma config for spi5 ARM: dts: stm32: Add pin map for ltdc & spi5 on stm32f429-disco board ARM: dts: stm32: enable ltdc binding with ili9341, gyro l3gd20 on stm32429-disco board dt-bindings: display: panel: Add ilitek ili9341 panel bindings clk: stm32: Fix stm32f429's ltdc driver hang in set clock rate, fix duplicated ltdc clock register to 'clk_core' case ltdc's clock turn off by clk_disable_unused() drm/panel: Add ilitek ili9341 panel driver spi: stm32: Add 'SPI_SIMPLEX_RX', 'SPI_3WIRE_RX' support for stm32f4 spi: flags 'SPI_CONTROLLER_MUST_RX' and 'SPI_CONTROLLER_MUST_TX' can't be coexit with 'SPI_3WIRE' mode .../bindings/display/panel/ilitek,ili9341.yaml | 69 ++ arch/arm/boot/dts/stm32f4-pinctrl.dtsi | 67 + arch/arm/boot/dts/stm32f429-disco.dts | 48 + arch/arm/boot/dts/stm32f429.dtsi | 3 + drivers/clk/clk-stm32f4.c | 7 +- drivers/gpu/drm/panel/Kconfig | 12 + drivers/gpu/drm/panel/Makefile | 1 + drivers/gpu/drm/panel/panel-ilitek-ili9341.c | 1301 ++++++++++++++++++++ drivers/spi/spi-stm32.c | 19 +- drivers/spi/spi.c | 3 +- 10 files changed, 1521 insertions(+), 9 deletions(-) create mode 100644 Documentation/devicetree/bindings/display/panel/ilitek,ili9341.yaml create mode 100644 drivers/gpu/drm/panel/panel-ilitek-ili9341.c -- 2.7.4 ^ permalink raw reply [flat|nested] 15+ messages in thread
* [PATCH v5 8/8] spi: flags 'SPI_CONTROLLER_MUST_RX' and 'SPI_CONTROLLER_MUST_TX' can't be coexit with 'SPI_3WIRE' mode 2020-05-25 3:45 dillon.minfei @ 2020-05-25 3:45 ` dillon.minfei 0 siblings, 0 replies; 15+ messages in thread From: dillon.minfei @ 2020-05-25 3:45 UTC (permalink / raw) To: linus.walleij, broonie Cc: devicetree, linux-arm-kernel, linux-kernel, linux-spi, linux-stm32, dri-devel, linux-clk, dillon min From: dillon min <dillon.minfei@gmail.com> since chip spi driver need get the transfer direction by 'tx_buf' and 'rx_buf' of 'struct spi_transfer' in 'SPI_3WIRE' mode. so, we need bypass 'SPI_CONTROLLER_MUST_RX' and 'SPI_CONTROLLER_MUST_TX' feature in 'SPI_3WIRE' mode Signed-off-by: dillon min <dillon.minfei@gmail.com> --- drivers/spi/spi.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/spi/spi.c b/drivers/spi/spi.c index c92c894..f884411 100644 --- a/drivers/spi/spi.c +++ b/drivers/spi/spi.c @@ -1023,7 +1023,8 @@ static int spi_map_msg(struct spi_controller *ctlr, struct spi_message *msg) void *tmp; unsigned int max_tx, max_rx; - if (ctlr->flags & (SPI_CONTROLLER_MUST_RX | SPI_CONTROLLER_MUST_TX)) { + if ((ctlr->flags & (SPI_CONTROLLER_MUST_RX | SPI_CONTROLLER_MUST_TX)) + && !(msg->spi->mode & SPI_3WIRE)) { max_tx = 0; max_rx = 0; -- 2.7.4 ^ permalink raw reply related [flat|nested] 15+ messages in thread
end of thread, other threads:[~2020-07-21 12:59 UTC | newest] Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- [not found] <broonie@kernel.org> 2020-05-25 3:40 ` [PATCH v5 0/8] Enable ili9341 and l3gd20 on stm32f429-disco dillon.minfei 2020-05-25 3:40 ` [PATCH v5 1/8] ARM: dts: stm32: Add dma config for spi5 dillon.minfei 2020-05-25 3:40 ` [PATCH v5 2/8] ARM: dts: stm32: Add pin map for ltdc & spi5 on stm32f429-disco board dillon.minfei 2020-05-25 3:40 ` [PATCH v5 3/8] ARM: dts: stm32: enable ltdc binding with ili9341, gyro l3gd20 on stm32429-disco board dillon.minfei 2020-05-25 3:40 ` [PATCH v5 4/8] dt-bindings: display: panel: Add ilitek ili9341 panel bindings dillon.minfei 2020-05-25 3:40 ` [PATCH v5 5/8] clk: stm32: Fix stm32f429's ltdc driver hang in set clock rate, fix duplicated ltdc clock register to 'clk_core' case ltdc's clock turn off by clk_disable_unused() dillon.minfei 2020-05-25 3:41 ` [PATCH v5 6/8] drm/panel: Add ilitek ili9341 panel driver dillon.minfei 2020-05-25 3:41 ` [PATCH v5 7/8] spi: stm32: Add 'SPI_SIMPLEX_RX', 'SPI_3WIRE_RX' support for stm32f4 dillon.minfei 2020-05-25 3:41 ` [PATCH v5 8/8] spi: flags 'SPI_CONTROLLER_MUST_RX' and 'SPI_CONTROLLER_MUST_TX' can't be coexit with 'SPI_3WIRE' mode dillon.minfei 2020-07-21 9:19 ` [PATCH v5 0/8] Enable ili9341 and l3gd20 on stm32f429-disco Alexandre Torgue 2020-07-21 10:39 ` dillon min 2020-07-21 11:54 ` Alexandre Torgue 2020-07-21 12:55 ` dillon min 2020-07-21 12:58 ` Alexandre Torgue 2020-05-25 3:45 dillon.minfei 2020-05-25 3:45 ` [PATCH v5 8/8] spi: flags 'SPI_CONTROLLER_MUST_RX' and 'SPI_CONTROLLER_MUST_TX' can't be coexit with 'SPI_3WIRE' mode dillon.minfei
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).