<div dir="ltr">Hi, Alexandre Torgue, Philippe Schenker<div><br></div><div>ok, just working on that now, thanks.</div><div><br></div><div>best regards.</div><div><br></div><div>dillon,</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Alexandre Torgue <<a href="mailto:alexandre.torgue@st.com">alexandre.torgue@st.com</a>> 于2020年5月7日周四 下午5:42写道：<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Dillon<br>
<br>
On 5/7/20 11:16 AM, <a href="mailto:dillon.minfei@gmail.com" target="_blank">dillon.minfei@gmail.com</a> wrote:<br>
> From: dillon min <<a href="mailto:dillon.minfei@gmail.com" target="_blank">dillon.minfei@gmail.com</a>><br>
> <br>
> Enable the stmpe811 touch screen on stm32429-disco board.<br>
> <br>
> Signed-off-by: dillon min <<a href="mailto:dillon.minfei@gmail.com" target="_blank">dillon.minfei@gmail.com</a>><br>
> ---<br>
<br>
You have to add version to your patches. When you send updates of your <br>
patches, it is better to resend the whole series by adding [PATCH v2] <br>
... instead of [PATCH] (you can do it easily using git format-patch and <br>
subject-prefix). Then in your cover letter you have to explain changes <br>
between 2 series versions.<br>
<br>
So please resend the whole series by adding V2 prefix, update <br>
cover-letter explaining changes, and add a title to your cover-letter <br>
please.<br>
<br>
regards<br>
alex<br>
<br>
<br>
<br>
>   arch/arm/boot/dts/stm32f429-disco.dts | 47 +++++++++++++++++++++++++++++++++++<br>
>   1 file changed, 47 insertions(+)<br>
> <br>
> diff --git a/arch/arm/boot/dts/stm32f429-disco.dts b/arch/arm/boot/dts/stm32f429-disco.dts<br>
> index 30c0f67..fad1ec1 100644<br>
> --- a/arch/arm/boot/dts/stm32f429-disco.dts<br>
> +++ b/arch/arm/boot/dts/stm32f429-disco.dts<br>
> @@ -49,6 +49,8 @@<br>
>   #include "stm32f429.dtsi"<br>
>   #include "stm32f429-pinctrl.dtsi"<br>
>   #include <dt-bindings/input/input.h><br>
> +#include <dt-bindings/interrupt-controller/irq.h><br>
> +#include <dt-bindings/gpio/gpio.h><br>
>   <br>
>   / {<br>
>       model = "STMicroelectronics STM32F429i-DISCO board";<br>
> @@ -127,3 +129,48 @@<br>
>       pinctrl-names = "default";<br>
>       status = "okay";<br>
>   };<br>
> +<br>
> +&i2c3 {<br>
> +     pinctrl-names = "default";<br>
> +     pinctrl-0 = <&i2c3_pins>;<br>
> +     clock-frequency = <100000>;<br>
> +     status = "okay";<br>
> +<br>
> +     stmpe811@41 {<br>
> +             compatible = "st,stmpe811";<br>
> +             reg = <0x41>;<br>
> +             interrupts = <15 IRQ_TYPE_EDGE_FALLING>;<br>
> +             interrupt-parent = <&gpioa>;<br>
> +             /* 3.25 MHz ADC clock speed */<br>
> +             st,adc-freq = <1>;<br>
> +             /* 12-bit ADC */<br>
> +             st,mod-12b = <1>;<br>
> +             /* internal ADC reference */<br>
> +             st,ref-sel = <0>;<br>
> +             /* ADC converstion time: 80 clocks */<br>
> +             st,sample-time = <4>;<br>
> +<br>
> +             stmpe_touchscreen {<br>
> +                     compatible = "st,stmpe-ts";<br>
> +                     /* 8 sample average control */<br>
> +                     st,ave-ctrl = <3>;<br>
> +                     /* 7 length fractional part in z */<br>
> +                     st,fraction-z = <7>;<br>
> +                     /*<br>
> +                      * 50 mA typical 80 mA max touchscreen drivers<br>
> +                      * current limit value<br>
> +                      */<br>
> +                     st,i-drive = <1>;<br>
> +                     /* 1 ms panel driver settling time */<br>
> +                     st,settling = <3>;<br>
> +                     /* 5 ms touch detect interrupt delay */<br>
> +                     st,touch-det-delay = <5>;<br>
> +             };<br>
> +<br>
> +             stmpe_adc {<br>
> +                     compatible = "st,stmpe-adc";<br>
> +                     /* forbid to use ADC channels 3-0 (touch) */<br>
> +                     st,norequest-mask = <0x0F>;<br>
> +             };<br>
> +     };<br>
> +};<br>
> <br>
</blockquote></div>