<div dir="ltr">Hi,neil:  <div> Thank you for your reply, it's not that the polarity of reset is different. The state of this rst needs to be high during a certain period of time (when hid touch communicate,). I have tried to set the default property to high in DT, but from the log and waveform, the rseet single  is set to low by boe_panel_add before hid touch communication.As shown in the picture, rst needs to be high before hid ready. Datasheet: <a href="https://github.com/HimaxSoftware/Doc/tree/main/Himax_Chipset_Power_Sequence" rel="noreferrer" target="_blank">https://github.com/HimaxSoftware/Doc/tree/main/Himax_Chipset_Power_Sequence</a></div><div><img src="cid:ii_lhyjyabr0" alt="image.png" width="562" height="410"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, May 22, 2023 at 3:24 PM <<a href="mailto:neil.armstrong@linaro.org">neil.armstrong@linaro.org</a>> wrote:<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,<br>
<br>
<br>
On 19/05/2023 10:01, Cong Yang wrote:<br>
> The previous patch is not based on drm-misc-next, resend this series.<br>
> Support Starry-himax83102-j02 and Starry-ili9882t TDDI MIPI-DSI panel,<br>
> set the default high for RST at boe_panel_add and add lp11_before_reset flag.<br>
<br>
If the reset gpio polarity is different, please change it in the DT by using<br>
a different gpio flag instead of changing the driver.<br>
<br>
However if the logic is different and reset must never be asserted to low,<br>
the the bindings + driver to make the reset line optional and set a gpio-hog<br>
in DT to keep it at a safe level.<br>
<br>
Neil<br>
<br>
<br>
> Cong Yang (2):<br>
>    drm/panel: Support for Starry-himax83102-j02 TDDI MIPI-DSI panel<br>
>    drm/panel: Support for Starry-ili9882t TDDI MIPI-DSI panel<br>
> <br>
>   .../gpu/drm/panel/panel-boe-tv101wum-nl6.c    | 474 +++++++++++++++++-<br>
>   1 file changed, 473 insertions(+), 1 deletion(-)<br>
> <br>
<br>
</blockquote></div>