<div dir="ltr"><div dir="ltr">Hi Linus,<div><br></div><div>Thanks for your detailed reply.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, 18 Jul 2021 at 08:17, Linus Walleij <<a href="mailto:linus.walleij@linaro.org">linus.walleij@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 Dillon,<br>
<br>
thanks for your patch!<br>
<br>
On Fri, Jul 16, 2021 at 12:20 PM <<a href="mailto:dillon.minfei@gmail.com" target="_blank">dillon.minfei@gmail.com</a>> wrote:<br>
<br>
> From: Dillon Min <<a href="mailto:dillon.minfei@gmail.com" target="_blank">dillon.minfei@gmail.com</a>><br>
><br>
> Add documentation for "ilitek,ili9341" panel.<br>
><br>
> Signed-off-by: Dillon Min <<a href="mailto:dillon.minfei@gmail.com" target="_blank">dillon.minfei@gmail.com</a>><br>
<br>
> +  dc-gpios:<br>
> +    maxItems: 1<br>
> +    description: Display data/command selection (D/CX)<br>
<br>
This is a DBI feature so mention in the description that this is a<br>
DBI panel.<br></blockquote><div><br></div><div>Okay, I will add the DBI panel to v2.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> +  spi-3wire: true<br>
> +<br>
> +  spi-max-frequency:<br>
> +    const: 10000000<br>
> +<br>
> +  port: true<br>
> +<br>
> +additionalProperties: false<br>
<br>
Please add regulator supplies for the power lines, it's fine<br>
not to implement code handling them in the driver but they<br>
should be in the bindings.<br>
<br>
For the ili9341 it should be<br>
<br>
  vci-supply:<br>
    description: Analog voltage supply (2.5 .. 3.3V)<br>
<br>
  vddi-supply:<br>
    description: Voltage supply for interface logic (1.65 .. 3.3 V)<br>
<br>
  vddi-led-supply:<br>
    description: Voltage supply for the LED driver (1.65 .. 3.3 V)<br>
<br></blockquote><div><br></div><div>Thanks so much, I will add it to v2.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Yours,<br>
Linus Walleij<br>
</blockquote></div></div>