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.*
> ---
> Â.../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*
> +Â Â Â Â Â Â Â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";
>Â Â};
> +
> +<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>;
> +Â Â Â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 = <®_3p3v>;
> +Â Â Â Â Â Â Âvdd = <®_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 = <<dc_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 = <®_3p3v>;
> +Â Â Â Â Â Â Âvcc = <®_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>;
>