<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body>
<div>On Mon, 2024-10-21 at 14:10 +0530, Naladala, Ramanaidu wrote:</div>
<blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex">
<p>Hi Jouni,<br>
</p>
<div class="moz-cite-prefix">On 10/21/2024 1:03 PM, Jouni Högander wrote:<br>
</div>
<div><br>
</div>
<blockquote type="cite" cite="mid:20241021073349.1222331-1-jouni.hogander@intel.com" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex">
<pre>We don't want to check vbt.psr.enable on DP Panel Replay as it is targeted</pre>
<pre>for eDP panel usage only.</pre>
<pre><br></pre>
<pre>Signed-off-by: Jouni Högander <a class="moz-txt-link-rfc2396E" href="mailto:jouni.hogander@intel.com"><jouni.hogander@intel.com></a></pre>
<pre>---</pre>
<pre> drivers/gpu/drm/i915/display/intel_psr.c | 4 +++-</pre>
<pre> 1 file changed, 3 insertions(+), 1 deletion(-)</pre>
<pre><br></pre>
<pre>diff --git a/drivers/gpu/drm/i915/display/intel_psr.c b/drivers/gpu/drm/i915/display/intel_psr.c</pre>
<pre>index 3b20325b3f6a..4176163ec19a 100644</pre>
<pre>--- a/drivers/gpu/drm/i915/display/intel_psr.c</pre>
<pre>+++ b/drivers/gpu/drm/i915/display/intel_psr.c</pre>
<pre>@@ -233,7 +233,9 @@ static bool psr_global_enabled(struct intel_dp *intel_dp)</pre>
<pre> switch (intel_dp->psr.debug & I915_PSR_DEBUG_MODE_MASK) {</pre>
<pre> case I915_PSR_DEBUG_DEFAULT:</pre>
<pre> if (display->params.enable_psr == -1)</pre>
<pre>- return connector->panel.vbt.psr.enable;</pre>
<pre>+ return intel_dp_is_edp(intel_dp) ?</pre>
<pre>+ connector->panel.vbt.psr.enable :</pre>
<pre>+ true;</pre>
</blockquote>
<p class="MsoPlainText">LGTM,</p>
<div><span style="font-size:12.0pt;font-family:"Aptos",sans-serif;mso-ascii-theme-font:
minor-latin;mso-fareast-font-family:Aptos;mso-fareast-theme-font:minor-latin;
mso-hansi-theme-font:minor-latin;mso-bidi-font-family:"Times New Roman";
mso-bidi-theme-font:minor-bidi;mso-ansi-language:EN-US;mso-fareast-language:
EN-US;mso-bidi-language:AR-SA">Reviewed-by:
Naladala Ramanaidu <<a href="mailto:ramanaidu.naladala@intel.com" class="moz-txt-link-freetext">ramanaidu.naladala@intel.com</a>></span>
<br>
</div>
</blockquote>
<div><br>
</div>
<div>Thank you Ramanaidu for checking my patch. This is now pushed to drm-intel-next.</div>
<div><br>
</div>
<div>BR,</div>
<div><br>
</div>
<div>Jouni Högander</div>
<div><br>
</div>
<blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex">
<blockquote type="cite" cite="mid:20241021073349.1222331-1-jouni.hogander@intel.com" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex">
<pre> return display->params.enable_psr;</pre>
<pre> case I915_PSR_DEBUG_DISABLE:</pre>
<pre> return false;</pre>
</blockquote>
<div></div>
</blockquote>
<div><br>
</div>
<div><span></span></div>
</body>
</html>