<html>
<head>
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<br>
<div class="moz-cite-prefix">On 11/13/2015 07:38 AM, Rob Herring
wrote:<br>
</div>
<blockquote cite="mid:20151112233807.GA9899@rob-hp-laptop"
type="cite">
<pre wrap="">On Thu, Nov 12, 2015 at 09:27:21AM +0800, Yakir Yang wrote:
</pre>
<blockquote type="cite">
<pre wrap="">Hi Rob,
On 11/12/2015 07:10 AM, Rob Herring wrote:
</pre>
<blockquote type="cite">
<pre wrap="">On Fri, Oct 30, 2015 at 09:09:15AM +0800, Yakir Yang wrote:
</pre>
<blockquote type="cite">
<pre wrap="">Some edp screen do not have hpd signal, so we can't just return
failed when hpd plug in detect failed.
This is an hardware property, so we need add a devicetree property
"analogix,need-force-hpd" to indicate this sutiation.
Tested-by: Heiko Stuebner <a class="moz-txt-link-rfc2396E" href="mailto:heiko@sntech.de"><heiko@sntech.de></a>
Tested-by: Javier Martinez Canillas <a class="moz-txt-link-rfc2396E" href="mailto:javier@osg.samsung.com"><javier@osg.samsung.com></a>
Signed-off-by: Yakir Yang <a class="moz-txt-link-rfc2396E" href="mailto:ykk@rock-chips.com"><ykk@rock-chips.com></a>
---
</pre>
</blockquote>
<pre wrap="">I didn't find this one in v10. Did it get dropped?
</pre>
</blockquote>
<pre wrap="">
You are in my 'to' list, but I haven't send the whole v10 out,
most of them don't need update :)
</pre>
</blockquote>
<pre wrap="">
Okay, it's just gmail's inability to follow threading...
</pre>
<blockquote type="cite">
<pre wrap="">This series should be:
---- [v8 0/17 ...]
|---- [v8 1/17 ...]
|---- [v8 2/17 ...]
|---- [v8 3/17 ...]
|---- [...]
|---- [v8 15/17 ...]
|---- [v8 16/17 ...]
|---- [v8 17/17 ...]
|
|---- [v9 10/17 ...]
|---- [v9 15/17 ...]
|---- We are here
|---- [v9 09/17 ...]
|---- [v10 09/17 ...]
|---- Received an acked from you
</pre>
</blockquote>
<pre wrap="">
</pre>
<blockquote type="cite">
<blockquote type="cite">
<blockquote type="cite">
<pre wrap="">diff --git a/Documentation/devicetree/bindings/display/bridge/analogix_dp.txt b/Documentation/devicetree/bindings/display/bridge/analogix_dp.txt
index 7659a7a..74f0e80 100644
--- a/Documentation/devicetree/bindings/display/bridge/analogix_dp.txt
+++ b/Documentation/devicetree/bindings/display/bridge/analogix_dp.txt
@@ -22,6 +22,9 @@ Required properties for dp-controller:
from general PHY binding: Should be "dp".
Optional properties for dp-controller:
+ -analogix,need-force-hpd:
+ Indicate driver need force hpd when hpd detect failed, this
+ is used for some eDP screen which don't have hpd signal.
</pre>
</blockquote>
<pre wrap="">This should be a generic property.
</pre>
</blockquote>
<pre wrap="">
This property would only need in some no-hpd case, it would be dropped if
panel keep the hotplug signal, that's why I thought it should be optional.
</pre>
</blockquote>
<pre wrap="">
I agree it is optional. I just mean drop the analogix and put in a
common binding doc (i.e. create a display/bridge/common.txt). Needing to
ignore HPD is probably a common problem.</pre>
</blockquote>
<br>
Oops, okay,
<meta http-equiv="content-type" content="text/html;
charset=windows-1252">
<span style="color: rgb(51, 51, 51); font-family: arial; font-size:
18px; font-style: normal; font-variant: normal; font-weight:
normal; letter-spacing: normal; line-height: 20px; orphans: auto;
text-align: start; text-indent: 0px; text-transform: none;
white-space: normal; widows: 1; word-spacing: 0px;
-webkit-text-stroke-width: 0px; display: inline !important; float:
none; background-color: rgb(255, 255, 255);">I misunderstand what
you mean. I thought you want me to put the<br>
"analogix,need-force-hpd" to required property, not an optional
property :)<br>
<br>
I thought this should be a common property of connect, so the
common document<br>
in bridge is not very good, I should be in display/connector/, but
I don't know what<br>
else could I collect for that common document, would like to share
more on this point <br>
<br>
- Yakir<br>
<br>
</span>
<blockquote cite="mid:20151112233807.GA9899@rob-hp-laptop"
type="cite">
<pre wrap="">
</pre>
<blockquote type="cite">
<pre wrap="">I thought if we put this a property to generic property, then we must need
to define it in normal device node, not sure whether it is right :)
</pre>
</blockquote>
<pre wrap="">
Sorry, I don't follow.
Rob
</pre>
</blockquote>
<br>
</body>
</html>