<html> <head> <meta content="text/html; charset=utf-8" http-equiv="Content-Type"> </head> <body bgcolor="#FFFFFF" text="#000000"> <div class="moz-cite-prefix">On 2016年07月25日 23:21, Thierry Reding wrote: </div> <blockquote cite="mid:20160725152125.GS21170@ulmo.ba.sec" type="cite"> <pre wrap="">On Wed, Jul 20, 2016 at 11:18:50AM +0800, Mark Yao wrote: </pre> <blockquote type="cite"> <pre wrap="">Allow user add display timing on device tree with simple-panel-dsi or simple-panel. Cc: Thierry Reding <a class="moz-txt-link-rfc2396E" href="mailto:thierry.reding@gmail.com"><thierry.reding@gmail.com></a> Cc: Rob Herring <a class="moz-txt-link-rfc2396E" href="mailto:robh+dt@kernel.org"><robh+dt@kernel.org></a> Cc: Mark Rutland <a class="moz-txt-link-rfc2396E" href="mailto:mark.rutland@arm.com"><mark.rutland@arm.com></a> Signed-off-by: Mark Yao <a class="moz-txt-link-rfc2396E" href="mailto:mark.yao@rock-chips.com"><mark.yao@rock-chips.com></a> --- .../bindings/display/panel/simple-panel.txt | 48 ++++++++++++++++++++++ 1 file changed, 48 insertions(+) </pre> </blockquote> <pre wrap=""> Sorry, not going to happen. Read this for an explanation of why not: <a class="moz-txt-link-freetext" href="https://sietch-tagr.blogspot.de/2016/04/display-panels-are-not-special.html">https://sietch-tagr.blogspot.de/2016/04/display-panels-are-not-special.html</a> Thierry </pre> </blockquote> Hi Thierry The blog actually not persuade me why can't use display timing on device tree. 1, Binding panel as a simple string on device tree seems simple on device tree, but it's complex on kernel code, and kernel code would became bigger and bigger. 2, Our customer always ask me, where is the display timing? They only find a simple panel string on device tree, need search the kernel code to find the actually timing. They are used to find all device info on device tree, but panel timing info is not, <meta http-equiv="content-type" content="text/html; charset=utf-8"> this would confuse them. They don't want to know how code work, just want a easier interface. <meta http-equiv="content-type" content="text/html; charset=utf-8"> 3, I think device tree not only can use for kernel, other module also can use it. on our project, we use uboot + kernel, the uboot support fdt, that function can parsing device tree. So if describe the display timing on device tree, both uboot and kernel can share same display timing, not need to describe twice, it would save work and not easy to make mistake. 4, For differentiation product, we face many different panel, every once in a while, need to add a new panel, we can't convert all the panel <meta http-equiv="content-type" content="text/html; charset=utf-8"> , code the panel on kernel seems too bad, and the kernel image became bigger and bigger. <meta http-equiv="content-type" content="text/html; charset=utf-8"> <meta http-equiv="content-type" content="text/html; charset=utf-8"> Generally, Our customer don't want to do any modify on kernel, they just modify device tree to bring up their device. Describe the panel timing on device tree, would make customer easy to use and reuse it. Thanks. <pre class="moz-signature" cols="72">-- Ｍark Yao</pre> </body> </html>