<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<meta name="Generator" content="Microsoft Word 15 (filtered medium)">
<!--[if !mso]><style>v\:* {behavior:url(#default#VML);}
o\:* {behavior:url(#default#VML);}
w\:* {behavior:url(#default#VML);}
.shape {behavior:url(#default#VML);}
</style><![endif]--><style><!--
/* Font Definitions */
@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoPlainText, li.MsoPlainText, div.MsoPlainText
{mso-style-priority:99;
mso-style-link:"Plain Text Char";
margin:0cm;
font-size:11.0pt;
font-family:"Calibri",sans-serif;}
span.PlainTextChar
{mso-style-name:"Plain Text Char";
mso-style-priority:99;
mso-style-link:"Plain Text";
font-family:"Calibri",sans-serif;}
.MsoChpDefault
{mso-style-type:export-only;
font-size:11.0pt;
mso-ligatures:none;}
@page WordSection1
{size:612.0pt 792.0pt;
margin:72.0pt 72.0pt 72.0pt 72.0pt;}
div.WordSection1
{page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang="EN-US" link="#0563C1" vlink="#954F72" style="word-wrap:break-word">
<div class="WordSection1">
<p class="MsoPlainText">Hi Jani,<o:p></o:p></p>
<p class="MsoPlainText">AUX_RD interval can have maximum value of 256ms. usleep_range() is recommended to be used for 10uS-20ms range as per Documentation/timers/timers-howto.rst<o:p></o:p></p>
<p class="MsoPlainText"><o:p> </o:p></p>
<p class="MsoPlainText"><img width="510" height="170" style="width:5.3125in;height:1.7708in" id="Picture_x0020_1" src="cid:image001.png@01DB0539.9751B230"><o:p></o:p></p>
<p class="MsoPlainText"><o:p> </o:p></p>
<p class="MsoPlainText">So, switched to fsleep() which takes care of calling respective functions according to the delay.<o:p></o:p></p>
<p class="MsoPlainText"><o:p> </o:p></p>
<p class="MsoPlainText">Regards<o:p></o:p></p>
<p class="MsoPlainText">Srikanth<o:p></o:p></p>
<p class="MsoPlainText"><o:p> </o:p></p>
<p class="MsoPlainText">-----Original Message-----<br>
From: Murthy, Arun R <arun.r.murthy@intel.com> <br>
Sent: Thursday, September 12, 2024 3:34 PM<br>
To: Jani Nikula <jani.nikula@linux.intel.com>; intel-xe@lists.freedesktop.org; intel-gfx@lists.freedesktop.org<br>
Cc: Srikanth V, NagaVenkata <nagavenkata.srikanth.v@intel.com><br>
Subject: RE: [PATCH 1/3] drm/i915/dp: use fsleep instead of usleep_rage for LT</p>
<p class="MsoPlainText"><o:p> </o:p></p>
<p class="MsoPlainText">> -----Original Message-----<o:p></o:p></p>
<p class="MsoPlainText">> From: Jani Nikula <<a href="mailto:jani.nikula@linux.intel.com"><span style="color:windowtext;text-decoration:none">jani.nikula@linux.intel.com</span></a>><o:p></o:p></p>
<p class="MsoPlainText">> Sent: Thursday, September 12, 2024 2:32 PM<o:p></o:p></p>
<p class="MsoPlainText">> To: Murthy, Arun R <<a href="mailto:arun.r.murthy@intel.com"><span style="color:windowtext;text-decoration:none">arun.r.murthy@intel.com</span></a>>;
<o:p></o:p></p>
<p class="MsoPlainText">> <a href="mailto:intel-xe@lists.freedesktop.org"><span style="color:windowtext;text-decoration:none">intel-xe@lists.freedesktop.org</span></a>;
<a href="mailto:intel-gfx@lists.freedesktop.org"><span style="color:windowtext;text-decoration:none">intel-gfx@lists.freedesktop.org</span></a><o:p></o:p></p>
<p class="MsoPlainText">> Cc: Murthy, Arun R <<a href="mailto:arun.r.murthy@intel.com"><span style="color:windowtext;text-decoration:none">arun.r.murthy@intel.com</span></a>>; Srikanth V, NagaVenkata
<o:p></o:p></p>
<p class="MsoPlainText">> <<a href="mailto:nagavenkata.srikanth.v@intel.com"><span style="color:windowtext;text-decoration:none">nagavenkata.srikanth.v@intel.com</span></a>><o:p></o:p></p>
<p class="MsoPlainText">> Subject: Re: [PATCH 1/3] drm/i915/dp: use fsleep instead of
<o:p></o:p></p>
<p class="MsoPlainText">> usleep_rage for LT<o:p></o:p></p>
<p class="MsoPlainText">> <o:p></o:p></p>
<p class="MsoPlainText">> On Thu, 12 Sep 2024, Arun R Murthy <<a href="mailto:arun.r.murthy@intel.com"><span style="color:windowtext;text-decoration:none">arun.r.murthy@intel.com</span></a>> wrote:<o:p></o:p></p>
<p class="MsoPlainText">> > Aux RD Interval value depends on the value read from the dpcd
<o:p></o:p></p>
<p class="MsoPlainText">> > register which is updated from the sink device use flseep thereby we
<o:p></o:p></p>
<p class="MsoPlainText">> > adhere to the Documentation/timers/timers-howto.rst<o:p></o:p></p>
<p class="MsoPlainText">> <o:p></o:p></p>
<p class="MsoPlainText">> Please explain why instead of just referencing a file.<o:p></o:p></p>
<p class="MsoPlainText"><o:p> </o:p></p>
<p class="MsoPlainText">Sleeping for < 10us use udelay, for 10us to 20ms use usleep_range and for > 10ms use msleep.<o:p></o:p></p>
<p class="MsoPlainText">fsleep() will call the particular api based on the above condition.<o:p></o:p></p>
<p class="MsoPlainText"><o:p> </o:p></p>
<p class="MsoPlainText">Thanks and Regards,<o:p></o:p></p>
<p class="MsoPlainText">Arun R Murthy<o:p></o:p></p>
<p class="MsoPlainText">--------------------<o:p></o:p></p>
</div>
</body>
</html>