<div dir="ltr"><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, May 22, 2019 at 12:24 PM Hoegeun Kwon <<a href="mailto:hoegeun.kwon@samsung.com">hoegeun.kwon@samsung.com</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">
  
    
  
  <div bgcolor="#FFFFFF">
    <div class="gmail-m_6435963005183802845moz-cite-prefix">On 4/10/19 1:24 AM, Eric Anholt wrote:<br>
    </div>
    <blockquote type="cite">
      <pre class="gmail-m_6435963005183802845moz-quote-pre">Hoegeun Kwon <a class="gmail-m_6435963005183802845moz-txt-link-rfc2396E" href="mailto:hoegeun.kwon@samsung.com" target="_blank"><hoegeun.kwon@samsung.com></a> writes:

</pre>
      <blockquote type="cite">
        <pre class="gmail-m_6435963005183802845moz-quote-pre">On 4/2/19 2:48 AM, Eric Anholt wrote:
</pre>
        <blockquote type="cite">
          <pre class="gmail-m_6435963005183802845moz-quote-pre">Hoegeun Kwon <a class="gmail-m_6435963005183802845moz-txt-link-rfc2396E" href="mailto:hoegeun.kwon@samsung.com" target="_blank"><hoegeun.kwon@samsung.com></a> writes:

</pre>
          <blockquote type="cite">
            <pre class="gmail-m_6435963005183802845moz-quote-pre">There is a problem when often dpms goes from off to on. pm idle is not
in sync and the problem occurs. Modify pm_runtime_put from
asynchronous to synchronous.
</pre>
          </blockquote>
          <pre class="gmail-m_6435963005183802845moz-quote-pre">Why would we need the power domain to go to off before we try to come
back?  Any idea?  Also, please specify what "the problem" is.
</pre>
        </blockquote>
        <pre class="gmail-m_6435963005183802845moz-quote-pre">Hi Eric,


First thank you for your review.

There is a problem failed to runtime PM enable on DSI when often dpms
</pre>
      </blockquote>
      <pre class="gmail-m_6435963005183802845moz-quote-pre">What do you mean by "failed to runtime PM enable"?  The
pm_runtime_enable() returned an error?  Have you investigated the source
that error, if so?
</pre>
    </blockquote>
    <pre>I'm sorry for the late reply.

The pm_runtime_enable() is not returned because return value is void.

The problem is that if the error log is not output
and the DPMS off on is repeated, the screen will stop.

As a result of debugging the problem, I think that synchronization
is a problem caused by dsi pm_suspend and resume.

So when we entered the pm runtime idle state, if used with sync,
the problem does not occur.</pre></div></blockquote><div><br></div><div>Sounds like not enough hystersis between disable/enable, i.e. some msleep missing somewhere. That's at least my guess, we've had a bunch of these with delayed runtime suspend of various things in i915.</div><div>-Daniel<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div bgcolor="#FFFFFF"><pre>
Best regards,
Hoegeun


</pre>
  <table id="gmail-m_6435963005183802845bannersignimg"><tbody><tr><td><p> </p>
</td></tr></tbody></table><table id="gmail-m_6435963005183802845confidentialsignimg"><tbody><tr><td><p><img style="border: 0px solid; width: 520px; height: 144px; display: inline-block;" src="cid:16adf4ffab5c204bfcc1"> </p>
</td></tr></tbody></table></div>

<img src="http://ext.samsung.net/mail/ext/v1/external/status/update?userid=hoegeun.kwon&do=bWFpbElEPTIwMTkwNTIyMTAyNDQzZXBjYXMxcDMyMTJlODU3ZTA3NTljZDA2ZmU1MDRkMjU0MmVjNmU5YSZyZWNpcGllbnRBZGRyZXNzPWRhbmllbEBmZndsbC5jaA__" style="display: none;" width="0" height="0" border="0"></blockquote></div><br clear="all"><br>-- <br><div dir="ltr" class="gmail_signature">Daniel Vetter<br>Software Engineer, Intel Corporation<br>+41 (0) 79 365 57 48 - <a href="http://blog.ffwll.ch" target="_blank">http://blog.ffwll.ch</a></div></div>