Re: [PATCH 1/4] add dts node for drm panel driver ili9341 add dts i2c3 for stmpe touch add dts spi5 for gyro & ili9341

From: Alexandre Torgue
Date: Thu Apr 30 2020 - 07:46:08 EST




On 4/30/20 12:26 PM, dillon min wrote:
Hi Alexandre,

Alexandre Torgue <alexandre.torgue@xxxxxx <mailto:alexandre.torgue@xxxxxx>> ä2020å4æ30æåå äå5:57åéï

Hi

On 4/30/20 11:43 AM, dillon.minfei@xxxxxxxxx
<mailto:dillon.minfei@xxxxxxxxx> wrote:
> From: dillon min <dillon.minfei@xxxxxxxxx
<mailto:dillon.minfei@xxxxxxxxx>>
>
> Signed-off-by: dillon min <dillon.minfei@xxxxxxxxx
<mailto:dillon.minfei@xxxxxxxxx>>

Commit title should be ARM: dts: stm32: bla bla on stm32f429 and please
a commit message.


*okay, thanks for your quicky response, this is my first kernel pull request, i will resubmit all patchsets following the history submits style who was did.*

ok no pb. In general way you have to execute ./scripts/checkpatch.pl to check some style errors before sending. Use ./scripts/get_maintainer.pl to see to who you have to send the patch. When you have several patches in a series, you need to have a cover-letter which explains the aim of your series. To have an idea of commit format just use git log on the file changed to see how previous accepted commi have been written.

More details here: Documentation/process/submitting-patches.rst


> ---
> Â.../bindings/display/panel/ilitek,ili9341.txt   | 42 +++++++++++
> Âarch/arm/boot/dts/stm32f4-pinctrl.dtsi      Â| 79
+++++++++++++++++++
> Âarch/arm/boot/dts/stm32f429-disco.dts       | 88
++++++++++++++++++++++
> Âarch/arm/boot/dts/stm32f429.dtsi         Â| 12 +++
>Â Â4 files changed, 221 insertions(+)
>Â Âcreate mode 100644
Documentation/devicetree/bindings/display/panel/ilitek,ili9341.txt
>
> diff --git
a/Documentation/devicetree/bindings/display/panel/ilitek,ili9341.txt
b/Documentation/devicetree/bindings/display/panel/ilitek,ili9341.txt

This binding description should be in a separate patch and you have to
write in YAML format.

*okay, will do it later. *


> new file mode 100644
> index 0000000..f5a4e55
> --- /dev/null
> +++
b/Documentation/devicetree/bindings/display/panel/ilitek,ili9341.txt
> @@ -0,0 +1,42 @@
> +Ilitek ILI9341 TFT panel driver with SPI control bus
> +
> +This is a driver for 240x320 TFT panels, accepting a rgb input
> +streams that get adapted and scaled to the panel.
> +
> +Required properties:
> +Â - compatible: "stm32f429-disco,ltdc-panel", "ilitek,ili9341"
> +Â Â (full system-specific compatible is always required to look
up configuration)
> +Â - reg: address of the panel on the SPI bus
> +
> +Optional properties:
> +Â - reset-gpios: a GPIO spec for the reset pin, see gpio/gpio.txt
> +Â - dc-gpios: a GPIO spec for the dc pin, see gpio/gpio.txt
> +
> +Â The following optional properties only apply to RGB input mode:
> +
> +Â - pixelclk-active: see display/panel/display-timing.txt
> +Â - de-active: see display/panel/display-timing.txt
> +Â - hsync-active: see display/panel/display-timing.txt
> +Â - vsync-active: see display/panel/display-timing.txt
> +
> +The panel must obey the rules for a SPI slave device as specified in
> +spi/spi-bus.txt
> +
> +The device node can contain one 'port' child node with one child
> +'endpoint' node, according to the bindings defined in
> +media/video-interfaces.txt. This node should describe panel's
video bus.
> +
> +Example:
> +
> +panel: display@0 {
> +Â Â Âcompatible = "stm32f429-disco,ltdc-panel", "ilitek,ili9341";
> +Â Â Âreg = <0>;
> +Â Â Âspi-3wire;
> +Â Â Âspi-max-frequency = <10000000>;
> +Â Â Âdc-gpios = <&gpiod 13 0>;
> +Â Â Âport {
> +Â Â Â Â Â Â Âpanel_in: endpoint {
> +Â Â Â Â Â Â Â Â Â Â Âremote-endpoint = <&display_out>;
> +Â Â Â Â Â Â Â};
> +Â Â Â};
> +};
> diff --git a/arch/arm/boot/dts/stm32f4-pinctrl.dtsi
b/arch/arm/boot/dts/stm32f4-pinctrl.dtsi
> index 392fa14..45b68f4 100644
> --- a/arch/arm/boot/dts/stm32f4-pinctrl.dtsi
> +++ b/arch/arm/boot/dts/stm32f4-pinctrl.dtsi
> @@ -316,6 +316,85 @@
>Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â};
>Â Â Â Â Â Â Â Â Â Â Â Â};
>
> +Â Â Â Â Â Â Â Â Â Â Â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>;
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â};
> +Â Â Â Â Â Â Â Â Â Â Â};
> +
> +Â Â Â Â Â Â Â Â Â Â Âi2c3_pins: i2c3-0 {
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âpins {
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âpinmux = <STM32_PINMUX('C',
9, AF4)>,
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â/* I2C3_SDA */
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â <STM32_PINMUX('A',
8, AF4)>;
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â/* I2C3_SCL */
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âbias-disable;
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âdrive-open-drain;
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âslew-rate = <3>;
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â};
> +Â Â Â Â Â Â Â Â Â Â Â};
> +
> +Â Â Â Â Â Â Â Â Â Â Â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 */
> diff --git a/arch/arm/boot/dts/stm32f429-disco.dts
b/arch/arm/boot/dts/stm32f429-disco.dts
> index 30c0f67..55eed05 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";
> @@ -98,6 +100,14 @@
>Â Â Â Â Â Â Â Âregulator-name = "vcc5_host1";
>Â Â Â Â Â Â Â Âregulator-always-on;
>Â Â Â Â};
> +
> +Â Â Âreg_3p3v: regulator-3p3v {

3p3 ?


*yes, the mfd stmpe driver need vcc & vioÂregulator, st sensor core driver need vddio & vdd*

I talked about the name. Somthing vdd_3v3 would be better. See how it is done in other stm32fx platform


> +Â Â Â Â Â Â Âcompatible = "regulator-fixed";
> +Â Â Â Â Â Â Âregulator-name = "3P3V";
> +Â Â Â Â Â Â Âregulator-min-microvolt = <3300000>;
> +Â Â Â Â Â Â Âregulator-max-microvolt = <3300000>;
> +Â Â Â Â Â Â Âregulator-always-on;
> +Â Â Â};
>Â Â};
>
>Â Â&clk_hse {
> @@ -127,3 +137,81 @@
>Â Â Â Âpinctrl-names = "default";
>Â Â Â Âstatus = "okay";
>Â Â};
> +
> +&ltdc {
> +Â Â Âstatus = "okay";
> +Â Â Âpinctrl-0 = <&ltdc_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>;
> +Â Â Âdmas = <&dma2 3 2 0x400 0x0>,
> +Â Â Â Â Â Â <&dma2 4 2 0x400 0x0>;
> +Â Â Âdma-names = "rx", "tx";
> +Â Â Â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>;
> +Â Â Â Â Â Â Âvddio = <&reg_3p3v>;
> +Â Â Â Â Â Â Âvdd = <&reg_3p3v>;
> +Â Â Â Â Â Â Âstatus = "okay";
> +Â Â Â};
> +Â Â Âdisplay: display@1{
> +Â Â Â Â Â Â Â/* Connect panel-ilitek-9341 to ltdc */
> +Â Â Â Â Â Â Âcompatible = "stm32f429-disco,ltdc-panel",
"ilitek,ili9341";
> +Â Â Â Â Â Â Âreg = <1>;
> +Â Â Â Â Â Â Âspi-3wire;
> +Â Â Â Â Â Â Âspi-max-frequency = <10000000>;
> +Â Â Â Â Â Â Âdc-gpios = <&gpiod 13 0>;
> +Â Â Â Â Â Â Âport {
> +Â Â Â Â Â Â Â Â Â Â Âpanel_in_rgb: endpoint {
> +Â Â Â Â Â Â Â Â Â Â Âremote-endpoint = <&ltdc_out_rgb>;
> +Â Â Â Â Â Â Â Â Â Â Â};
> +Â Â Â Â Â Â Â};
> +Â Â Â};
> +};
> +
> +&i2c3 {

Nodes are have to be ordered

*sorry, a little confused, i2c-3 is used by stmpe811 driver*


> +Â Â Âpinctrl-names = "default";
> +Â Â Âpinctrl-0 = <&i2c3_pins>;
> +Â Â Âstatus = "okay";
> +
> +Â Â Âtouch: stmpe811@41 {
> +Â Â Â Â Â Â Âcompatible = "st,stmpe811";
> +Â Â Â Â Â Â Âreg = <0x41>;
> +Â Â Â Â Â Â Âinterrupts = <15 IRQ_TYPE_EDGE_FALLING>;
> +Â Â Â Â Â Â Âinterrupt-parent = <&gpioa>;
> +Â Â Â Â Â Â Âvio = <&reg_3p3v>;
> +Â Â Â Â Â Â Âvcc = <&reg_3p3v>;
> +Â Â Â Â Â Â Âstatus = "okay";
> +
> +Â Â Â Â Â Â Âstmpe_touchscreen {
> +Â Â Â Â Â Â Â Â Â Â Âcompatible = "st,stmpe-ts";
> +Â Â Â Â Â Â Â Â Â Â Âst,sample-time = <4>;
> +Â Â Â Â Â Â Â Â Â Â Âst,mod-12b = <1>;
> +Â Â Â Â Â Â Â Â Â Â Âst,ref-sel = <0>;
> +Â Â Â Â Â Â Â Â Â Â Âst,adc-freq = <1>;
> +Â Â Â Â Â Â Â Â Â Â Âst,ave-ctrl = <1>;
> +Â Â Â Â Â Â Â Â Â Â Âst,touch-det-delay = <2>;
> +Â Â Â Â Â Â Â Â Â Â Âst,settling = <2>;
> +Â Â Â Â Â Â Â Â Â Â Âst,fraction-z = <7>;
> +Â Â Â Â Â Â Â Â Â Â Âst,i-drive = <1>;
> +Â Â Â Â Â Â Â};
> +Â Â Â};
> +};
> diff --git a/arch/arm/boot/dts/stm32f429.dtsi
b/arch/arm/boot/dts/stm32f429.dtsi
> index d777069..257b843 100644
> --- a/arch/arm/boot/dts/stm32f429.dtsi
> +++ b/arch/arm/boot/dts/stm32f429.dtsi
> @@ -402,6 +402,18 @@

Split Soc updates in a separate patch

*okay, will do it later*

>Â Â Â Â Â Â Â Â Â Â Â Âstatus = "disabled";
>Â Â Â Â Â Â Â Â};
>
> +Â Â Â Â Â Â Âi2c3: i2c@40005c00 {
> +Â Â Â Â Â Â Â Â Â Â Âcompatible = "st,stm32f4-i2c";
> +Â Â Â Â Â Â Â Â Â Â Âreg = <0x40005c00 0x400>;
> +Â Â Â Â Â Â Â Â Â Â Âinterrupts = <72>,
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â <73>;
> +Â Â Â Â Â Â Â Â Â Â Âresets = <&rcc STM32F4_APB1_RESET(I2C3)>;
> +Â Â Â Â Â Â Â Â Â Â Âclocks = <&rcc 0 STM32F4_APB1_CLOCK(I2C3)>;
> +Â Â Â Â Â Â Â Â Â Â Â#address-cells = <1>;
> +Â Â Â Â Â Â Â Â Â Â Â#size-cells = <0>;
> +Â Â Â Â Â Â Â Â Â Â Âstatus = "disabled";
> +Â Â Â Â Â Â Â};
> +
>Â Â Â Â Â Â Â Âdac: dac@40007400 {
>Â Â Â Â Â Â Â Â Â Â Â Âcompatible = "st,stm32f4-dac-core";
>Â Â Â Â Â Â Â Â Â Â Â Âreg = <0x40007400 0x400>;
>